On Fri, Feb 09, 2024 at 09:29:03AM -0600, Moger, Babu wrote: > Hi Tony, > > This patch probably needs to be merged with with patch 7. If it just added RESCTRL_NODE to the enum and the switch() I'd agree (as patch 7 is where RESCTRL_NODE first used). But this patch also adds the sanity checks on scope where it has to be a cache level. Patch 7 is already on the big side (119 lines added to core.c). If you really think this series needs to cut back the number of patches, I could move the sanity check pieces from here to patch 3 (where the enum is introduced) and just the RESCTRL_NODE bits to patch 7. -Tony
