[AMD Official Use Only - General]

-----Original Message-----
From: Reinette Chatre <reinette.chatre@intel.com>
Sent: Wednesday, November 23, 2022 1:06 PM
To: Moger, Babu <Babu.Moger@amd.com>; corbet@lwn.net;
tglx@linutronix.de; mingo@redhat.com; bp@alien8.de
Cc: fenghua.yu@intel.com; dave.hansen@linux.intel.com; x86@kernel.org;
hpa@zytor.com; paulmck@kernel.org; akpm@linux-foundation.org;
quic_neeraju@quicinc.com; rdunlap@infradead.org;
damien.lemoal@opensource.wdc.com; songmuchun@bytedance.com;
peterz@infradead.org; jpoimboe@kernel.org; pbonzini@redhat.com;
chang.seok.bae@intel.com; pawan.kumar.gupta@linux.intel.com;
jmattson@google.com; daniel.sneddon@linux.intel.com; Das1, Sandipan
[off-list ref]; tony.luck@intel.com; james.morse@arm.com;
linux-doc@vger.kernel.org; linux-kernel@vger.kernel.org;
bagasdotme@gmail.com; eranian@google.com
Subject: Re: [PATCH v8 07/13] x86/resctrl: Introduce data structure to support
monitor configuration

Hi Babu,

On 11/23/2022 10:23 AM, Moger, Babu wrote:

quoted

Hi Reinette,

On 11/22/22 18:14, Reinette Chatre wrote:

quoted

Hi Babu,

On 11/4/2022 1:00 PM, Babu Moger wrote:

quoted

Add a new field in mon_evt to support Bandwidth Monitoring Event
Configuration(BMEC) and also update the "mon_features" display.

The sysfs file "mon_features" will display the monitor configuration

sysfs -> resctrl ?

Sure.

quoted

quoted

if supported.

This is not clear. "mon_features" does not display the monitor
configuration, it displays the name of the file that can be used to
see the monitor configuration.

Will change it to:

The sysfs -> resctrl file "mon_features" will display the supported
events and files that can be used to configure those events if monitor
configuration is supported.

I meant that "sysfs" should be replaced by "resctrl".

Ok. Got it.
Thanks
Babu