Hi Babu and Tony, On 10/10/24 8:12 AM, Moger, Babu wrote: > > All good points. How about this text: > > "num_mbm_cntrs": > The number of monitoring counters available for assignment when the > architecture supports mbm_cntr_assign mode. > > Resctrl subsystem provides the interface to count maximum of two memory subsystem -> filesystem > bandwidth events per group, from a combination of available total and Is this "from a combination of ..." snippet intended to hint at BMEC? > local events. Keeping the current interface, users can enable a maximum of What is meant by "Keeping the current interface"? Which interface? What will "current" mean when a user reads this documentation? > 2 counters per group. User will also have the option to enable only one "User will also have" is talking about the future. When will this be the case? > counter to the group to maximize the number of groups monitored. > > I think that we need to be careful when making this documentation so specific to the ABMC implementation. We already know that "soft-ABMC" is coming and Peter already shared [1] that with software assignment it will not be possible to assign counters to individual events. The goal of this work is to create a generic interface and this is the documentation for it. If this documentation is created to be specific to the first implementation it will make it difficult to use this same interface to support other implementations. Reinette [1] https://lore.kernel.org/all/CALPaoCi_TBZnULHQpYns+H+30jODZvyQpUHJRDHNwjQzajrD=A@xxxxxxxxxxxxxx/