Hi Reinette, On 7/30/25 14:47, Reinette Chatre wrote: > Hi Babu, > > On 7/25/25 11:29 AM, Babu Moger wrote: >> From: Tony Luck <tony.luck@xxxxxxxxx> >> >> There are currently only three monitor events, all associated with >> the RDT_RESOURCE_L3 resource. Growing support for additional events >> will be easier with some restructuring to have a single point in >> file system code where all attributes of all events are defined. >> >> Place all event descriptions into an array mon_event_all[]. Doing >> this has the beneficial side effect of removing the need for >> rdt_resource::evt_list. >> >> Add resctrl_event_id::QOS_FIRST_EVENT for a lower bound on range >> checks for event ids and as the starting index to scan mon_event_all[]. >> >> Drop the code that builds evt_list and change the two places where >> the list is scanned to scan mon_event_all[] instead using a new >> helper macro for_each_mon_event(). >> >> Architecture code now informs file system code which events are >> available with resctrl_enable_mon_event(). >> >> Signed-off-by: Tony Luck <tony.luck@xxxxxxxxx> >> Reviewed-by: Fenghua Yu <fenghuay@xxxxxxxxxx> >> Reviewed-by: Reinette Chatre <reinette.chatre@xxxxxxxxx> > > Please add your "Signed-off-by" to the commit tags of the first four patches. > When you do, ensure it follows the expected ordering > per "Ordering of commit tags" in Documentation/process/maintainer-tip.rst. > Sure. It appears I am patch handler in this case. Seems like this is a correct order. Signed-off-by: Tony Luck <tony.luck@xxxxxxxxx> Signed-off-by: Babu Moger <babu.moger@xxxxxxx> Reviewed-by: Fenghua Yu <fenghuay@xxxxxxxxxx> Reviewed-by: Reinette Chatre <reinette.chatre@xxxxxxxxx> -- Thanks Babu Moger
