Hi Tony, On 2/9/24 13:35, Tony Luck wrote: > On Fri, Feb 09, 2024 at 09:29:16AM -0600, Moger, Babu wrote: >>> >>> +extern unsigned int snc_nodes_per_l3_cache; >> >> I feel this can be part of rdt_resource instead of global. > > Mixed emotions about that. It would be another field that appears > in every instance of rdt_resource, but only used by the RDT_RESOURCE_L3_MON > copy. > That was the comment I got earlier for my patches(search for global). https://lore.kernel.org/lkml/47f870e0-ad1a-4a4d-9d9e-4c05bf431858@xxxxxxxxx/ -- Thanks Babu Moger