Message-ID: <c8fd3125-b61c-8182-d7af-0f7a85eba345@intel.com>
Date:   Thu, 6 Jan 2022 15:49:57 -0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Firefox/91.0 Thunderbird/91.4.0
Subject: Re: [PATCH v2 4/5] selftests/resctrl: Change default limited time to
 120 seconds for resctrl_tests
Content-Language: en-US
To:     Shaopeng Tan <tan.shaopeng@jp.fujitsu.com>,
        Fenghua Yu <fenghua.yu@intel.com>,
        Shuah Khan <shuah@kernel.org>
CC:     <linux-kernel@vger.kernel.org>, <linux-kselftest@vger.kernel.org>
References: <20211213100154.180599-1-tan.shaopeng@jp.fujitsu.com>
 <20211213100154.180599-5-tan.shaopeng@jp.fujitsu.com>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20211213100154.180599-5-tan.shaopeng@jp.fujitsu.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?enVWQWQ5ZkxxN0FDRlpLOExhREsvQVdnbk0yZGVEUEhFQzdyYmQ5Sm5FZldk?=
 =?utf-8?B?WG5mQm5xM1FXa0RFK3Z2bWMwNGVHRzlLOUpHbVNYN3VZWGdoclFQeFFFWC9P?=
 =?utf-8?B?dTkyZUlsdTV0Y0wyUE55cWt3bCtVSFVmOGxJUUozaWJxVjd0QkNNSDJpNllB?=
 =?utf-8?B?ZzVGN1FnY1NkL0Jzd1E2dWY0VmovRGszT1k4SnRxV2RXN1JHRmZWekJWaXpI?=
 =?utf-8?B?SktyVGtDWU0yU0FWREkwZXJKOGhhWjV6REJMdEZkUzFiQlRrQlN2M1c3UzU4?=
 =?utf-8?B?ME1iaGlDSU5LS1ZxU1doMDhQbVdldng0ZmlwMEdtUzdxeEJoZzZTUXRvKzFF?=
 =?utf-8?B?bkhTWXdqWEhyZ3Fwd3BXYTliYkxHUllsMEFuOFVlbHBOM3ZaREZRa09SUjdq?=
 =?utf-8?B?R0Q2MFFudnVPc1Q1VTBWUFhlaFN1Z3FJbncrSlZJS2RPY2U2VkhEc2xSZ2t1?=
 =?utf-8?B?UXlZMzMwaUFIdHRJZzBWTnVucGNhelhXakhxS25mVlNvOHlSZWZPbVU0R3pE?=
 =?utf-8?B?aWt6ZC9qZ2x6Ynh3dFllRUI5OGRmWlUzdnVEVDZVOFpNeEJvZC9CMWlrSW5r?=
 =?utf-8?B?dWRmbnRjMUp4ODlOYTFENDRHcGRHWm1Ydll4QU1tL0JERlFmWkMxMUFlUGFO?=
 =?utf-8?B?YTNtdXJhV3puVmQ3Ym13K1pBWDNlejdFUXkwQjVMZXVUY2w3K25JY1A3bEJa?=
 =?utf-8?B?RVVTdFh0dldGclJNVEtvbkQraCtIS3M2dE1IZ0Vqb2ZBUXJ1c0JVbUt4UVNw?=
 =?utf-8?B?djFRbDBTVmduZFVQR1h2dHp0WTB5Z1FuRmIwSEU4a0M3aVZYMmIwd2NlQUpx?=
 =?utf-8?B?c0hFVEdFdEVHQnBxOE9uK1hueWx0MzNxK29nUTc0M3Z2MkY4alFQTzdldndS?=
 =?utf-8?B?WTZESlRoU1Z1SloyRUNHMERvdTNGdXdqdWwvMTZ1QWJjUktybXJ5bnBVQi9E?=
 =?utf-8?B?VDhQRkwxU3BBbWdzTEtTb3AzQ0QvN3AyV01PcGJSRHZYcUM3SjVNOEZ1ZTNS?=
 =?utf-8?B?ekNERmZtcURFZ3RacURsdXhrYVh0S3Q3WWdhRGdYQ0p0UGd0akhCcHN1UmNy?=
 =?utf-8?B?SDRuMnEyZ0Q4Y3JWWUZ2KzJ5blMvbjFZd3MyL2duNlVJWHF4Q1pleTdKSi94?=
 =?utf-8?B?MUlOTUduMDFFRnNEREtyZEs5NUNuNkg5N0R0ZCtORkZDMDllVkVzV1F1VGp5?=
 =?utf-8?B?aGl5ajdpTTFYcXRKSnZIazd1LzZ0L1MyRXcvcnNrazRXbGdjK1JGTWNldXhT?=
 =?utf-8?B?dUdXQ05OczZRWllGSEl2b2ZISGo0V1JXYXpSU040ZXZoc2w4a3k2ek5zYlUr?=
 =?utf-8?B?dU1HUkh6UEdWT0NlMm85WEM5SzVOUmJGMjdac1d5RWpTRytCUDUveHZ1OU05?=
 =?utf-8?B?aVAwY2VnKytTaWNSL2ZWVmd2WFFKU1k3K0VwSEJ0SU9GdlFMSW1GK2hyVWUy?=
 =?utf-8?B?NDBFVklDZHByQ0xzMFFBR0JyOWtWd3R6emE5MTFWWDNtRGVqM3hzSm5SdGF1?=
 =?utf-8?B?akNwTGpmS243OUxscjlQemJXQ2JLY3JsUVFPUEU0SVh3VHk3QXVsNncreXZ1?=
 =?utf-8?B?S0xyWjJNRUljVU5tRUliZVJybnRsRUdIYlBSOWZXWnlHUEJ2ZzZGbGhpRGpo?=
 =?utf-8?B?emFSM2lRNTBwWWoySUFId0FVNFFlK3VJbG9jMUZDSVdYTnVRQXlBdGc4dWEv?=
 =?utf-8?B?ZjNqWnpaREFlTHpvN1locW9zQy9yWitmUDNZQ3ArMFo0dVJZV1hmOFBrbGFU?=
 =?utf-8?B?REtZV2UweGl3WnhtMlBvbVB6ZFBmVHRDUkN4dzBwdHc0bmJwNnFKZFFQWHV5?=
 =?utf-8?B?bVpjanRJdXNOZnZsTTNKMGNXZm1yRTlHclk2OFJuUkIvVnRzdml5dEgySjdU?=
 =?utf-8?B?KzlnZDA5QmcyV1BLZjArU29iODkyOGUxQUhJVWhINHJFbVluTWtMa0RoY09T?=
 =?utf-8?B?TG5sMXAzcUhaa3hOK2JFVTFIUTdxM2h4dXRlaWRic0dVSERLS1BtLythTFpM?=
 =?utf-8?B?bHhsVzI0UEI0NHVTUzZDYTRTbzllNkVCYjBwUW1hYnEvYXNoV3ZEYjVabm9o?=
 =?utf-8?B?VEp2WndSdWNkMlp0TE5JMFNPbGN4NlExZENjbThjbUpMNnhndnZRdUFBR2hH?=
 =?utf-8?B?ZGxIS1VLT2JTMFJTS096bHFOYW9DQnp1Sk9TRzVvVUoyUk5FYzlaY0NmcjdX?=
 =?utf-8?B?a0gxV0NHSVg0REpKeFp4aEcxYUdFc0EwbEdGTUErbDhTekYwQjNrem9tQjJu?=
 =?utf-8?Q?IrPsvoxD7DLNsqtHWbybw8oCTYH9PqYFoqDsfJKc9A=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: b1c6fced-dddb-4e1b-6f91-08d9d16f4112
X-MS-Exchange-CrossTenant-AuthSource: BN0PR11MB5744.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jan 2022 23:50:01.6062
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: VbBQf5wgG25R5jLOzRxcH9CaFW+xbP5hQx3I4A+iQvAJZ7rrDwQ+uTgY9/STS1aasxSu3NBkOVNrJrl0LZMp5ZSzv2pLYycQe4OL05JM6/4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR11MB1969
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Shaopeng Tan,

On 12/13/2021 2:01 AM, Shaopeng Tan wrote:
> This commit changed the default limited time(45s) to 120 seconds for

(This commit)

> resctrl_tests by adding "setting" file.  Since 45 seconds may not be
> enough on some machine environments.

This sounds very imprecise when you say "may not be enough". From what I
understand you have already encountered systems where this is not enough
so you have established that 45 seconds is too short. The next question is,
how was 120 seconds determined and how do you know it will be enough?

> 
> Signed-off-by: Shaopeng Tan <tan.shaopeng@jp.fujitsu.com>
> ---
>  tools/testing/selftests/resctrl/settings | 1 +
>  1 file changed, 1 insertion(+)
>  create mode 100644 tools/testing/selftests/resctrl/settings
> 
> diff --git a/tools/testing/selftests/resctrl/settings b/tools/testing/selftests/resctrl/settings
> new file mode 100644
> index 000000000000..6091b45d226b
> --- /dev/null
> +++ b/tools/testing/selftests/resctrl/settings
> @@ -0,0 +1 @@
> +timeout=120

Reinette