> > +static int get_domain_id_from_scope(int cpu, enum resctrl_scope scope) > > +{ > > + switch (scope) { > > + case RESCTRL_L2_CACHE: > > + case RESCTRL_L3_CACHE: > > + return get_cpu_cacheinfo_id(cpu, scope); > > + default: > > + break; > > + } > > + > > + return -EINVAL; > > +} > > This function is meanigfull when you introduce node scope RESCTRL_NODE. > > Can you please move this to patch 5? Code has been this way since v5. I'll note your opinion and if others agree I'll move it. But I think this is a reasonable part of the first step moving away from purely cache scope for domains. -Tony