Received-SPF: pass (google.com: domain of linux-kernel+bounces-173809-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <b8a87fb1-838f-4337-8940-8eb1c5328a2b@intel.com>
Date: Wed, 8 May 2024 13:41:58 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [RFC PATCH v3 00/17] x86/resctrl : Support AMD Assignable
 Bandwidth Monitoring Counters (ABMC)
To: <babu.moger@amd.com>, <corbet@lwn.net>, <fenghua.yu@intel.com>,
	<tglx@linutronix.de>, <mingo@redhat.com>, <bp@alien8.de>,
	<dave.hansen@linux.intel.com>
CC: <x86@kernel.org>, <hpa@zytor.com>, <paulmck@kernel.org>,
	<rdunlap@infradead.org>, <tj@kernel.org>, <peterz@infradead.org>,
	<yanjiewtw@gmail.com>, <kim.phillips@amd.com>, <lukas.bulwahn@gmail.com>,
	<seanjc@google.com>, <jmattson@google.com>, <leitao@debian.org>,
	<jpoimboe@kernel.org>, <rick.p.edgecombe@intel.com>,
	<kirill.shutemov@linux.intel.com>, <jithu.joseph@intel.com>,
	<kai.huang@intel.com>, <kan.liang@linux.intel.com>,
	<daniel.sneddon@linux.intel.com>, <pbonzini@redhat.com>,
	<sandipan.das@amd.com>, <ilpo.jarvinen@linux.intel.com>,
	<peternewman@google.com>, <maciej.wieczor-retman@intel.com>,
	<linux-doc@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<eranian@google.com>, <james.morse@arm.com>
References: <cover.1711674410.git.babu.moger@amd.com>
 <48b595cc-5ffe-4507-bffd-335a60fdaab9@intel.com>
 <2016b830-64c7-43bd-8116-bdfd239221e3@amd.com>
 <ea75801f-e56a-486d-85a4-85c57bce0c81@intel.com>
 <7e92200e-d68c-4dc4-85c3-7192a23f8cbc@amd.com>
Content-Language: en-US
From: Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <7e92200e-d68c-4dc4-85c3-7192a23f8cbc@amd.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?QVdQOVoyaXpOeU1tb3FFRml6NjFBZzRpUjI0TDRsdDVDcU1OWHU0V3l2OWFM?=
 =?utf-8?B?WjhxYmhqWVpOOW1yTW9KcDc3OWUrQ21scnl4S1ZGWjFoNmMzVGo1QkdMZGd1?=
 =?utf-8?B?cnU4QjlROUxCajFHeFBBYkgySzUwZjhZdk1VSWVxQ2U5N0ZydTJVUVJ5RC9k?=
 =?utf-8?B?U3JrNXJZRDErYytQelAwdUR1Y2Z1aWxRUHJ1WDBKZzlzMlp4Sk9tTDFtazVS?=
 =?utf-8?B?UVN6OXpGbHRWYjhwMFdLMzI3dWhua2dOQ3RyWlFWejZlcmF1UEtFd1RwUXBo?=
 =?utf-8?B?RG5oUmVxdnlWditYTTA3ME5SNWdtRHVUOXZsTDVYb1lSRHJpdWx4YThFQjdn?=
 =?utf-8?B?ekJwU2c5TGhuRGp4Ymg2TGIrcFF0MU1WY2JubUtGajdjREJVSlR0b3oxenVo?=
 =?utf-8?B?UzI3aUorQVF0bCtpZ1Z3ZllUTDN5UUJTcG5CcUdRTXo4YUFUanU0ZVRDaDFW?=
 =?utf-8?B?a2lTR2tjdSs3K01oODVGaXRuTmxjTENlRzNMc1RiQ1ZYZm0vbnpkVTBjNkha?=
 =?utf-8?B?c3NsbjExMnZPWFdtS3JYcmV0M3pXa0pXSDhjZFBxVGJOV3RqbDl4bEVCdGhK?=
 =?utf-8?B?T2FNdXE0R2xxVHJyTEQvVXhRSEdvcEhKNWtDMndZbVVaUm1iczgzcEllTE1I?=
 =?utf-8?B?VlpmQUlkTlRlODBkTWRBWnBPUEs5blBxN01MMit1MHZsRXdyMitUVmUrdkE4?=
 =?utf-8?B?ZExnMTk3bnZUUjM0TVRra0g0cWJZcWVpSVA4cW52OE9xS2YyN1V4RkxhWTlE?=
 =?utf-8?B?ZlFVOURjeC9vVnhtUm9ZK2JvcWhHOS9YY0V1VTUyM0dtZWRyOGowWjdKc3Rq?=
 =?utf-8?B?aVpCSmJscVc3NFhqd3lHYTVGYVpIVFVSMUFNdUs3Y2d4MEgwVWhNKy83S2Vq?=
 =?utf-8?B?dS9yYndGVnB0bWpNbSs1Zi9KZzNYcnV0dFppazJXbnBVYkZYdFZBemJCRnFJ?=
 =?utf-8?B?UW12R0hIREt4RE04bVE2TmtkdWFUb0hjNDFUeHgvVUVoWmZ6by9TZkI5NUtI?=
 =?utf-8?B?V0FYK1VmVURxMmd4TmwrRnE4S2xrY3JoQUF1Q3dhZkdIQ3RkeVBSK3VkeW9k?=
 =?utf-8?B?bG1uOWtWMng2aU44cmdZem8rM3FaajM2ZFNObVRtWUovWjhUOHU5YmFnVFVU?=
 =?utf-8?B?c3dVUGM0ZWtJTUdRaXcrMnFKTzJWWGNYMWRBUFQ4WXhiSmI3aGZRL0x4bmpN?=
 =?utf-8?B?ZGpYVjBLUmpVUkpOeTFFcERjN1FsN0I4TU9vUndnUytmeEt1KzVQSjRsaHpJ?=
 =?utf-8?B?M0p3YTIrVGp6dlhuT2pQb01DM0NvUzNReDVMVVVIcU1tZVlzeVczMkZxMG1L?=
 =?utf-8?B?YzJPQVc2enJjMUlTWHhjZFU0cHlMMkRLUjhMV2twd1ZJUE1HYWdXSHNrUWhj?=
 =?utf-8?B?SkZLcURKZHhHcGUzR0RTcGhzZVBiSk96RXdVblRuMlIxL2pvbGNscm9TSUNk?=
 =?utf-8?B?bjMzK28vNzRpUzFMQ3M2TkN5eHZLb0REZVBaS0NDMGszZWRjNHhvWG96aG9K?=
 =?utf-8?B?KzVXVkRUbzZyK2hNMFFldThKS1NOSEdBUTRXbCsvKzZ5WXMzV3BESytIZytj?=
 =?utf-8?B?NjRzMVhyS0lvRXA2R0tNNFBDcGc1SGFQSnF6MFMwUkhJZ3I1a090WG40cTJE?=
 =?utf-8?B?aElOSUpXSC85Y2FrR2JucEpPendxOEZwM2JUQWphSURmUHFXblFWdGtic1BF?=
 =?utf-8?B?d0lyck1BR3M2UlcwbUs5M2hMYnJyQnpiVERnOURnK3FhUDdsVk1vQ3Ruamls?=
 =?utf-8?B?aEFPYWJHU3hOakFtUHh0Sm5oWlVNOUxWVFNhMTExTDl6SFg1UkFrOG1hNTNX?=
 =?utf-8?B?ajcvMkdna2dRSHBWaU96RTE2S1NBbTRrRHVTNnQ4akdBUDlqK01wYmEreW9r?=
 =?utf-8?B?NndkZWhDSU42WkxQVG9kVHZwY2YzZlJDaCt3UGhvWXBhc2hGQVlzaHZpU25T?=
 =?utf-8?B?V2NiNnVDdWJ3VDJxNkR4TFRSNGZWYUNmUEFqRDZSZDA5L2x0WFJyUHBDRkc3?=
 =?utf-8?B?dVdIdEFSNEVuK1NGSUhwdjkwK0FFRDl6bS9QSGVqcENuQURZRUZGMng3N2JZ?=
 =?utf-8?B?OFZxZXpnSUpoTU82U21XelNIemhzZGJabjNubnpDcGl6bXozYm0rbUtmdmxn?=
 =?utf-8?B?b3ZJcHAvZ0V6SWliQXFFMW1oOVZ4WkhTUlZ0NWVQTmJ2NWtDR0FUN0JsQUFQ?=
 =?utf-8?B?Wnc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 382ccb05-b27a-4fdc-cf29-08dc6f9f5072
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 May 2024 20:42:02.1541
 (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: TBG3iCpwpiP8zrRpucAghNS3u+Ip+nQYyQq87mdEQ5ytm+lpO60PI0MOobR81uhe4KzDXlJ1nvZBVFIbJmS1R6sexbL80g0oFlK17qoAsjI=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR11MB6248
X-OriginatorOrg: intel.com

Hi Babu,

On 5/8/2024 1:07 PM, Moger, Babu wrote:
> Hi Reinette,
> 
> On 5/7/24 15:26, Reinette Chatre wrote:
>> Hi Babu,
>>
>> On 5/6/2024 10:18 AM, Moger, Babu wrote:
>>> On 5/3/24 18:24, Reinette Chatre wrote:
>>>> On 3/28/2024 6:06 PM, Babu Moger wrote:
>>>>
>>>>> a. Check if ABMC support is available
>>>>> 	#mount -t resctrl resctrl /sys/fs/resctrl/
>>>>>
>>>>> 	#cat /sys/fs/resctrl/info/L3_MON/mbm_assign
>>>>> 	[abmc] 
>>>>> 	legacy_mbm
>>>>>
>>>>> 	Linux kernel detected ABMC feature and it is enabled.
>>>>
>>>> Please note that this adds the "abmc" feature to the resctrl
>>>> *filesystem* that supports more architectures than just AMD. Calling the
>>>> resctrl filesystem feature "abmc" means that (a) AMD needs to be ok with
>>>> other architectures calling their features that are
>>>> similar-but-maybe-not-identical-to-AMD-ABMC "abmc", or (b) this needs
>>>> a new generic name.
>>>
>>> It should not a problem if other architecture calling abmc for similar
>>> feature. But generic name is always better if there is a suggestion.
>>
>> "should not a problem" does not instill confidence that AMD is
>> actually ok with this.
> 
> The feature "ABMC" has been used in the public document already to refer
> this feature.
> https://www.amd.com/content/dam/amd/en/documents/processor-tech-docs/programmer-references/24594.pdf

It is clear to me that Assignable Bandwidth Monitoring Counters (ABMC) is the
name of the AMD feature. The question is whether users can use the 
same name to interact with "similar but maybe not identical" features from other
architectures, which is what this series enables.

> If there comes a conflict then we can change it to amd_abmc. Didn't see
> any conflict at this pint.

How do you envision this? The resctrl filesystem interface is intended to be
architecture neutral so it is not obvious to me how "amd_abmc" is expected
to look? Why would it be necessary to have different architecture specific names
for a similar feature from different architectures that users interact with in
the same way? Sounds to me as though this just needs a new non-AMD marketing name. 

Reinette