Hi Reinette, On 5/22/25 18:05, Reinette Chatre wrote: > Hi Babu, > > On 5/15/25 3:52 PM, Babu Moger wrote: >> The mbm_cntr_assign mode requires a cntr_id to read event data. The > > cntr_id -> "counter ID" > Sure. >> cntr_id is retrieved via mbm_cntr_get, which takes a struct rdtgroup as > > cntr_id -> "counter ID" > Sure. > mbm_cntr_get -> mbm_cntr_get() > Sure. >> a parameter. >> >> Passing the full rdtgroup also provides access to closid and rmid, both of > > closid -> CLOSID > rmid -> RMID > Sure. >> which are necessary to read monitoring events. >> >> Refactor the code to pass the entire struct rdtgroup instead of individual > > "the entire" -> "a pointer to" > Sure. >> members in preparation for this requirement. >> >> Suggested-by: Reinette Chatre <reinette.chatre@xxxxxxxxx> >> Signed-off-by: Babu Moger <babu.moger@xxxxxxx> >> --- > Patch looks good.-- Thanks Babu Moger
