Re: [PATCH v9 17/26] x86/resctrl: Implement resctrl_arch_config_cntr() to assign a counter with ABMC

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Reinette,

On 11/22/2024 3:52 PM, Reinette Chatre wrote:
Hi Babu,

On 11/22/24 10:54 AM, Moger, Babu wrote:
Hi Reinette,

On 11/21/2024 2:18 PM, Reinette Chatre wrote:
Hi Babu,

On 11/19/24 12:12 PM, Moger, Babu wrote:
Hi Reinette,

On 11/15/24 18:44, Reinette Chatre wrote:
Hi Babu,

On 10/29/24 4:21 PM, Babu Moger wrote:
The ABMC feature provides an option to the user to assign a hardware
counter to an RMID, event pair and monitor the bandwidth as long as it is
assigned. The assigned RMID will be tracked by the hardware until the user
unassigns it manually.

Counters are configured by writing to L3_QOS_ABMC_CFG MSR and
specifying the counter id, bandwidth source, and bandwidth types.

needs imperative tone

How about this?

Configure the counters by writing to the L3_QOS_ABMC_CFG MSR and
specifying the counter ID, bandwidth source, and bandwidth types.


ok with me. Exactly what ChatGPT suggests.

Hmm. ):


Please do note that that first paragraph informs reader that
a counter is assigned by user to "an RMID, event pair" while the hardware is configured with
"the counter ID, bandwidth source, and bandwidth types". There thus does not seem
to be a clear connection between what user assigns and what is programmed to hardware.


Adding RMID in the text might help.

Configure the counters by writing to the L3_QOS_ABMC_CFG MSR and specifying the counter ID, RMID, bandwidth source, and bandwidth types.


Isn't the bandwidth source and the RMID the same thing? How about something like:
"Configure the counters by writing to the L3_QOS_ABMC_CFG MSR and specifying
  the counter ID, bandwidth source (RMID), and bandwidth event configuration."

Please feel free to improve.

Looks good. Thanks
--
- Babu Moger





[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux