Received-SPF: pass (google.com: domain of linux-kernel+bounces-208916-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID: <399b6497-2c76-4f16-8e21-27d209e214d2@intel.com>
Date: Mon, 10 Jun 2024 14:28:09 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v2] selftests/resctrl: Fix noncont_cat_run_test for AMD
To: <babu.moger@amd.com>, =?UTF-8?Q?Ilpo_J=C3=A4rvinen?=
	<ilpo.jarvinen@linux.intel.com>
CC: <fenghua.yu@intel.com>, <shuah@kernel.org>, LKML
	<linux-kernel@vger.kernel.org>, <linux-kselftest@vger.kernel.org>,
	=?UTF-8?Q?Maciej_Wiecz=C3=B3r-Retman?= <maciej.wieczor-retman@intel.com>,
	<peternewman@google.com>, <eranian@google.com>
References: <3a6c9dd9dc6bda6e2582db049bfe853cd836139f.1717622080.git.babu.moger@amd.com>
 <7679d70a0ea939db13ae9dac20de56644460d6df.1718035091.git.babu.moger@amd.com>
 <91052cec-071b-7cc7-5f85-4fa29ec2aea4@linux.intel.com>
 <0e153d27-fa1d-4c47-8d4e-a2004c991ffc@amd.com>
From: Reinette Chatre <reinette.chatre@intel.com>
Content-Language: en-US
In-Reply-To: <0e153d27-fa1d-4c47-8d4e-a2004c991ffc@amd.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?NmZWcjBnRXRaV2pKQytVNU1BSzJMNENBMG9xV2YwN3BjdzR2bmVRd08xV2k3?=
 =?utf-8?B?ZUhqOUVxRE5rZGhRWExwTEdpUStRVGVvTGM2cmxma2Zub3lkcXpuMEkvMGxH?=
 =?utf-8?B?ZUNmM1FJcUhWcm9Fb1cxQWxOWU5oaWhRY2ZPUHpEMW5JM0IvK291SnM4MGx1?=
 =?utf-8?B?Y0hUeVBkZU9sN253aGFoM1piOE9nNkdEVVRYRDFLK1dHZVFJdEM1NGplL1M4?=
 =?utf-8?B?RVRjSWJqMmZ6bVBmSTAyUWhaWVpxQVFFN1pTWUJhVmZldjNKaEt5RE1CZWxH?=
 =?utf-8?B?OHRvRHVRNlJQTVo3ODRUellrQjhHbS9XVG1NTytxbjdnVXRrNkt6bzBJdFFO?=
 =?utf-8?B?d2crT1ozbWt0Yy8vTDl6MFRRWXNWcUZmUE1oWmwwMzZxVmJ4b1BHcUpsZlhM?=
 =?utf-8?B?UElWdjZnV01GTmk4UUdseFhXOEFVUW9SRWRyQWRtb0lHd0JlRzFxOUNFQXBJ?=
 =?utf-8?B?TWFjU3hPbXBCVENyY25ONTliaVIzcXc1N09DNENrM2lmZ3VjRkkzOHkvTGpi?=
 =?utf-8?B?MlVON2hWekhGZXBCakdBZkZWejdNeXdTdHBGeHJmaVZMaWU3NGxrV0RYcVhE?=
 =?utf-8?B?THM1NTEvbzN0RXl6a3ZDWnJGNmNBZ2FIRXA0UWhHVXpnV3lFWWRyZ2RKVzF1?=
 =?utf-8?B?K2F5UkxGbVA4L3BmS2FrdnUrN29MenZRZU83TlRQMHE4cEphUkd3ZGE0Y21x?=
 =?utf-8?B?eGd5bkJndEZzMWlmbmJ5LzNwL3dTMXR3eEowYVJZT2lodnVLRGRJN1NnS2hj?=
 =?utf-8?B?TkUycFdENGR1dXJHUnJmWVhWdHF5R3BqbmhpUGdZNHBWcTBIY2h6N2kvWngv?=
 =?utf-8?B?dmVyUGlxa0l4aGpPZldnM1pJWkdMVkZBa0YwQWM3c2pRTU5MK2pOSGtwSEFI?=
 =?utf-8?B?SkZyeEZsWUIvOFRubmtjS3phVDRXUTRHL2NJOWNIQ0NzOU5WVnNRTUZPYm4v?=
 =?utf-8?B?RjNFeU5mSGE2Q0s5TWovdDBrMlI1TDVTeFF6Q1ByQzI2OTZSSEZxVnhmS0Y5?=
 =?utf-8?B?RWFlTWNMcjZlSEtrdWpWMDVDdDlXS2JqNU91c2NJNkszUmZhcDJ6RVpmL29J?=
 =?utf-8?B?VUhOT3ZJczZ1aExicEdhSjJlRVY4ZTFRcHU3TTJrVDVDZ05IcVM2MXF5bm1C?=
 =?utf-8?B?YUpmNndvY3VVd3BlMnJxZTV4SzZBVXA0OVNzanVLa0NRSGIwTThlOGk1ZTR4?=
 =?utf-8?B?SktyYVNJOHV1d2RpN244MTd6YWJmVWg2L1JxSHJTZTZMWHlXMU1VR1hzSW1H?=
 =?utf-8?B?ejlkT3NIUjAzR2QzMUZ6UU5xL2xGc3dnZjlGYVE1em41V1BYamNBWTZQeGc5?=
 =?utf-8?B?NG1DUGF4UkIvck9oWFptMVE0VVZJUk1mSTdzcE1rSlZQSk43eVVoRlV0MkVC?=
 =?utf-8?B?aGxHWjBDNmh3N0VMbEwzMWtVUTJMZUJxc08ya20xdm1mb1RYTFRQam52eW5K?=
 =?utf-8?B?VzQ5VUZTQnIyRXhpZWRXdTZmeFVBSzhZNnBDZlYrMlJHaGJhY0JxVEdEZ0p4?=
 =?utf-8?B?dUtuNHU0TG9iWUQvZDhJQ3FtL3VmSWpvY3FZbk9CeGgvaU9tOU5GSVgvbDZO?=
 =?utf-8?B?N1dvK0NHOHNkTndzbm9YcjJXMU4rL0h0OEVNbEZ5akx2a2YzNVFWZ1VYdjR0?=
 =?utf-8?B?eXNJNW01blBtckFVUVF1QmNlNTQyZ3VxZFNndnloaERsWU83aXMrZmZZMmYy?=
 =?utf-8?B?djJTeWhKVGxTc0kzaDk4TXVSOUY2TCs1TWFHRVhFdEJjSWJHblNBT3Y5SUFs?=
 =?utf-8?B?U2RJZFlXTDdVNkFSeXIxQUVLK2svWXFwamJwbkpXa01iS0JGZjlqRnhjRzVU?=
 =?utf-8?B?a0FZaGVWT2NKdWFYRlNTeTBKcU5RVUJxUFIrVVhaWll3YUFPakNSenBzeURw?=
 =?utf-8?B?UXl6ZDZmdTRGODJaM0k1NnFzam5pRTI2UFJJRE5Mc2hVcm8wT2JqaW5yQ1Zl?=
 =?utf-8?B?aUVpUnBEY1JYWXBQR0l4elNUaXpFSUlkTUpVbnk3ZU55SWRhQ2NvWWo3VVdD?=
 =?utf-8?B?bXU4aU5tMkVQS1IyWUJtQk1mR3lPV1JqNEVKcncxOGlrUmR2SWErTEt0aXVm?=
 =?utf-8?B?MkY3dU1QdVZlWXNoQ09iYjFKU0lUVW1aVVd5VitXL0pGUmw2YWdMSnlPaFJZ?=
 =?utf-8?B?K2JLbnF3WEtrV2F0ZUIwY3Y0NjJ2U1Z0TlFOanhhNDl0NmVaRHo3OWVuV0tp?=
 =?utf-8?B?N0E9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 0fc63526-d1d9-4cce-22aa-08dc89943b26
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Jun 2024 21:28:12.2116
 (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: kFw5a219WfmLp1/3Jm+iWBbtAUAXcPdJUfdy9/p4mWW5idQ312PvuKP4kStEh8w78ICF/RtLvCEX/q+5GfIt9Rig2lsstcNmhcbxxc9ovtA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CYYPR11MB8307
X-OriginatorOrg: intel.com

Hi Babu,

On 6/10/24 10:51 AM, Moger, Babu wrote:
> Hi
> 
> On 6/10/24 11:20, Ilpo Järvinen wrote:
>> On Mon, 10 Jun 2024, Babu Moger wrote:
>>
>>> The selftest noncont_cat_run_test fails on AMD with the warnings. Reason
>>
>> noncont_cat_run_test()
> 
> I want to mention the test here. not function. How about this?
> 
> "The selftest non-contiguous CBM test fails on AMD with the warnings."
> 

This does not take Ilpo's feedback regarding "the warnings" into account.
I agree with Ilpo (keeping his comment below) that "with the warnings"
sounds incomplete. Also, it is not necessary to mention both "selftest"
and "test" in "The selftest non-contiguous CBM test".

You can change it to something like:
	The non-contiguous CBM test fails on AMD with:
		/* insert error message here */

	AMD always supports non-contiguous CBM but does not report
	it via CPUID.

	Fix the non-contiguous CBM test to use CPUID to discover
	non-contiguous CBM support only on Intel.

>>
>> (In general, use () when refering to a function, same thing in the
>> shortlog).
>>
>> "the warnings" sounds like I should know about what warning it fails with
>> but there's no previous context which tells that information. I suggest
>> you either use "a warning" or quote the warning it fails with into the
>> commit message.
>>
>>> is, AMD supports non contiguous CBM masks but does not report it via CPUID.

The "M" in CBM is for "mask" so  there is no need to write "mask" after CBM.

Reinette