Hi Reinette, On 2/10/25 12:08, Reinette Chatre wrote: > Hi Babu, > > On 2/7/25 10:52 AM, Moger, Babu wrote: >> Hi Reinette, >> >> On 2/7/2025 11:18 AM, Moger, Babu wrote: >>> Does this look ok? Just added domain in the text. >>> >>> "The number of monitoring counters available in each domain for assignment when the system supports mbm_cntr_assign mode. >>> :: >>> # cat /sys/fs/resctrl/info/L3_MON/num_mbm_cntrs >>> 32 >>> >>> The resctrl file system supports tracking up to two memory bandwidth >>> events per monitoring group: mbm_total_bytes and/or mbm_local_bytes. >>> Up to two counters can be assigned per monitoring group, one for each >>> memory bandwidth event in each domain. More monitoring groups can be tracked by assigning one counter per monitoring group. However, doing so limits memory bandwidth tracking to a single memory bandwidth event per >>> monitoring group." >> >> Revised again: >> >> "The number of monitoring counters available in each domain for assignment when the system supports mbm_cntr_assign mode. For example, on a system with 32 monitoring counters: > > I think we need to be careful with "available" since all these counters > may not be available. That is why "available_mbm_cntrs" exist. > > How about something like (please feel free to improve): > "The maximum number of monitoring counters (total of available and assigned counters) > in each domain when the system supports mbm_cntr_assign mode." Sure. > Could you please make the "For example" a new paragraph (this follows existing style in the > docs). It could also be made more specific, for example, > > "For example, on a system with 32 monitoring counters in each domain:" Yes. > >> :: >> # cat /sys/fs/resctrl/info/L3_MON/num_mbm_cntrs >> 32 >> > > The rest of the documentation seems like a repeat of what can be found in > the "mbm_assign_mode" section right above it. It does not look as though > any information will be lost by dropping the text below? Sure. > >> The resctrl file system supports tracking up to two memory bandwidth >> events per monitoring group: mbm_total_bytes and/or mbm_local_bytes. >> Up to two counters can be assigned per monitoring group, one for each >> memory bandwidth event in each domain. More monitoring groups can be tracked by assigning one counter per monitoring group. However, doing so limits memory bandwidth tracking to a single memory bandwidth event per >> monitoring group." >> >> Thanks >> Babu > > Reinette > -- Thanks Babu Moger