linux-kernel - Re: [RFC PATCH v3 00/17] x86/resctrl : Support AMD Assignable Bandwidth Monitoring Counters (ABMC)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <b8a87fb1-838f-4337-8940-8eb1c5328a2b@intel.com>
Date: Wed, 8 May 2024 13:41:58 -0700
From: Reinette Chatre <reinette.chatre@...el.com>
To: <babu.moger@....com>, <corbet@....net>, <fenghua.yu@...el.com>,
	<tglx@...utronix.de>, <mingo@...hat.com>, <bp@...en8.de>,
	<dave.hansen@...ux.intel.com>
CC: <x86@...nel.org>, <hpa@...or.com>, <paulmck@...nel.org>,
	<rdunlap@...radead.org>, <tj@...nel.org>, <peterz@...radead.org>,
	<yanjiewtw@...il.com>, <kim.phillips@....com>, <lukas.bulwahn@...il.com>,
	<seanjc@...gle.com>, <jmattson@...gle.com>, <leitao@...ian.org>,
	<jpoimboe@...nel.org>, <rick.p.edgecombe@...el.com>,
	<kirill.shutemov@...ux.intel.com>, <jithu.joseph@...el.com>,
	<kai.huang@...el.com>, <kan.liang@...ux.intel.com>,
	<daniel.sneddon@...ux.intel.com>, <pbonzini@...hat.com>,
	<sandipan.das@....com>, <ilpo.jarvinen@...ux.intel.com>,
	<peternewman@...gle.com>, <maciej.wieczor-retman@...el.com>,
	<linux-doc@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
	<eranian@...gle.com>, <james.morse@....com>
Subject: Re: [RFC PATCH v3 00/17] x86/resctrl : Support AMD Assignable
 Bandwidth Monitoring Counters (ABMC)

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