This new changelog has structure of "context, solution, problem". > > Additionally, when "mbm_event" counter assignment mode is enabled, a This seems to be primary motivation since passing struct rdtgroup will simplify the code (when I consider the original patch, not what this new changelog implies) ... but if this change is indeed to the arch API as the context suggest then passing the individual members is the right thing to do because arch code should not access struct rdtgroup. > counter ID is required to read the event. The counter ID is obtained > through mbm_cntr_get(), which expects a struct rdtgroup pointer." This is even stranger. mbm_cntr_get() is private to resctrl fs while the new changelog describes how the arch functions resctrl_arch_reset_rmid() and resctrl_arch_rmid_read() need struct rdtgroup to call mbm_cntr_get()? Reinette
