Message-ID: <003b0346-e525-cdd3-57e5-96389ccbc05b@intel.com>
Date:   Thu, 16 Mar 2023 08:57:13 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Firefox/102.0 Thunderbird/102.8.0
Subject: Re: [PATCH v2 8/9] selftests/resctrl: Use remount_resctrlfs()
 consistently with boolean
Content-Language: en-US
To:     =?UTF-8?Q?Ilpo_J=c3=a4rvinen?= <ilpo.jarvinen@linux.intel.com>,
        <linux-kselftest@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        Fenghua Yu <fenghua.yu@intel.com>,
        Shuah Khan <shuah@kernel.org>
References: <20230215130605.31583-1-ilpo.jarvinen@linux.intel.com>
 <20230215130605.31583-9-ilpo.jarvinen@linux.intel.com>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20230215130605.31583-9-ilpo.jarvinen@linux.intel.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?aUpWenFtKzFYMWtMR2NBZUt1aHRwLytrcFlyL2J6bjJ5ZnNHejRLd2J3Wko2?=
 =?utf-8?B?cWs5OURhbDUrOW1IZmt6Q1dKVTgyMUt0RjBvbDZJUXVGQ3BTenlYQVkzV010?=
 =?utf-8?B?SnZIMktuNitiWDlvUXJLSE5YVVFyblF3QXpKVnRpTjV1K2lLUTVlNk1MWk5D?=
 =?utf-8?B?ZFR1UzVOV0hSWFdvVmt0MURCSXZoMGNIZEpQMHZhMUxlNlJpeThzTCt1TU8y?=
 =?utf-8?B?TjFaWGUzR0Z1U3RoVUs2VC9ZV3JLKy9tdC9iUWEvSk55eWRKaklOM09EMWVz?=
 =?utf-8?B?d00vU3lORDFCbmFqV0NDQ3MxUTRoc0l3VTZLQ0lYZzMrU1g4Sjk5L29VM0xy?=
 =?utf-8?B?TndoaWpYRkEzWGtielU1aTRHQ3RXanRWaUQwVFpmV3p2Z0xSWjFxOUhnR3JQ?=
 =?utf-8?B?cS9Fdk43TE1KSGNITDBDR25qQUF2TStrSzVueTRjRmVsL1M5bGJvc2ZqaWxm?=
 =?utf-8?B?VFh5OUEzdVFINTFrSWlKL3dxZ283ZFpoMGpVN0N1VWJjbFptNFBkSnRheUZr?=
 =?utf-8?B?a2paKzZEbjhVUElTL3N4anJiMm0vZk90K05tWXM3eU90SWZmWkUyT0VBODhp?=
 =?utf-8?B?TzlidEZrT2JYSXA4QWFublJzU3pqUDRnNTlPckVZdEN1M1REU3c2aTk0ZWtx?=
 =?utf-8?B?VWwvZmd1MDF3TW1uQlJLS3NJZXFPVlFoaVNEL3pTY0JNdXZSNnhQd0w2RjFN?=
 =?utf-8?B?MkZLTDZDTzgrMTN5VW1TTmFMczFzeTFIeW5DcEFaUDhZTkhOOFdjcGI2dXdM?=
 =?utf-8?B?bVlmWEFYQitvemNRbzdHcFEzNEhaT2RoOXRSUTd0cFh6QTM2UWk1VHNFNWV1?=
 =?utf-8?B?TTE3alQrZ0pZdDNMSXpBVFRvQU53VkhuVmhmdVdickFGTEphN25kUzZkSmRM?=
 =?utf-8?B?Tmg4d05aS2RCR296bUNWZWEyVVVTblRSazhyN0ZBZXJ4ZnFRclBsUkp5QmFH?=
 =?utf-8?B?UXBwSmUrMDBzNHM5UkJSd1FReko2aFJNZWVPNGE3TVdlSDMvSlhwVGMwdVdz?=
 =?utf-8?B?dVJIanhjTmtFMGl6N2wzVE8zekxqb25wN1Y4ZURaWGNzZ3VEVXUvUldvL0Iz?=
 =?utf-8?B?cHBlZ1hyaEpJLy9TeXkrblpiNlVxUTlSemljbkx6WEJXSmN4S041bUgzcHRM?=
 =?utf-8?B?dWx4ZFVDYUR2bVYwRnNRZS90Umpzb0RiTTFvaGdLNGtrTVdLeE1HdFp4OUVq?=
 =?utf-8?B?ZjdNS2ZGMkI5eVVPSC8zQk1HcWVBQVJWVmRPU25keE0xMnVRamU5enVRWmIv?=
 =?utf-8?B?NlZQZWFLMkp3a3dtbnZldjBDWFYwb1J5STdNYWpNMGw1bFpEcWNSYmRGMkxI?=
 =?utf-8?B?RHM4am1YOUZjRElob1NBbWIzeEhxcW13NjNJYk5VTHpPRlM3NDdOMDFJZVI1?=
 =?utf-8?B?U0E4UGVpUXRUOGdKYWFLT3RSTFpBR1A5OEpCdnpuYTUwQ2Z6T3VIKy9VT01r?=
 =?utf-8?B?TGVseGlJQzFYa2JMNkdHL1JITTNWbzFvMjRBb0lxamNNa2ZhdHBGeWU2OWN0?=
 =?utf-8?B?Y0VpQnZNVmR5UzlXNWJlc01aZnBKR1YzRy9OK0VrbTh6aU0vNUNpVmZDKy9h?=
 =?utf-8?B?Q1RDcnlhVkZNYWlxQTgzcHpCRk5weXVtTTcxNE8vb2ZjRVdjTGsxQXg2L0NM?=
 =?utf-8?B?UGVacmN3V00zVDZqQVozNVMzMTJvZDB2a3RId0hvWGxJZGFqNkxVMldMNlk4?=
 =?utf-8?B?cUZicEdOb2diMnA2OHdTdlYrR0ExNTdjL0ZNb2dTc1puUWJvYjUxb1EvWlIx?=
 =?utf-8?B?eVEyRjF4NmZTMzhZTmZBZW1hV3lXSXVKOWdsNXVVZUw0WTkyWXJzelhKMmpQ?=
 =?utf-8?B?M3VtZkFHRVBoWUl0RGwzcUxUVmFma3U0L3BTM3ZCazlaTHFxNWtCN0U1ODBS?=
 =?utf-8?B?dmhUdVlDY1dsYlFObmhqeW5zbE1oRVBYajVEQWtWZmh5UFAxc2pJbklTK0Vv?=
 =?utf-8?B?NEJMTERkMnNFRnRTNFVpM2REc0xWN1c2eVUzdXJtcXVHcDJ6N09VUHVUYzZV?=
 =?utf-8?B?NldnQ0YzMjFJY0JoYkh0K0llNUhhZVZLZmVpa0tSNUFmVjljUkdWamFoMGJi?=
 =?utf-8?B?RURSalYyZjBGdklYaFN1ZkYyaWtGc0xtM1Y1WGZQYXhuTUhZcXErSHpOai84?=
 =?utf-8?B?NU1TN1FWRi9YMlFCYWdOandxTC9WbVJKczlRWDlkdUwvNkVZeCt5dTVOL0dx?=
 =?utf-8?B?Wmc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 070c5fa6-b10e-47ea-10ee-08db26371d28
X-MS-Exchange-CrossTenant-AuthSource: CY4PR11MB1862.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Mar 2023 15:57:16.0091
 (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: CAUDcPpIlBffXHQg2eSUeIjhu+KzqU0H6tC1m0V0S2RupxRU8BK/RFDb6ti7UTqSy/0j+DyzVQ8fGEFxIX7NcP0ArcBj6Mn5mBKk3BeIFUk=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB7997
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Ilpo,

On 2/15/2023 5:06 AM, Ilpo Järvinen wrote:
> remount_resctrlfs() accepts a boolean value as an argument. Some tests
> pass 0/1 and some tests pass true/false.
> 
> Make all the callers of remount_resctrlfs() use true/false so that the
> parameter usage is consistent across tests.
> 
> Co-developed-by: Fenghua Yu <fenghua.yu@intel.com>
> Signed-off-by: Fenghua Yu <fenghua.yu@intel.com>
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
> ---

Thank you

Reviewed-by: Reinette Chatre <reinette.chatre@intel.com>

Reinette