Hi Tony, On 3/19/2024 1:31 PM, Luck, Tony wrote: >> I agree to drop patch #9. >> >> The core support for SNC continue to look good to me (I just had a few nitpicks). >> >> What remains is the user interface that continues to gather opinions [3]. These new >> discussions were prompted by user space needing a way to determine if resctrl supports >> SNC. This started by using the "size" file but thinking about it more user space could >> also look at whether the number of L3 control domains are different from the number >> of L3 monitoring domains? I am adding Maciej for his opinion (please also include him >> in future versions of this series). > > Implementing the revised user interface will make significant changes to the > patches needed to support SNC. Working on them now. > > Minor stuff is the that "size" file won't need to change (because summing across > all SNC domains mean that all of the cache is counted in mon_data/mon_L3_xx/llc_occupancy. > The new mon_data/mon_L3_xx/mon_NODE_xxllc_occupancy files are the ones limited > to 1/snc_ways of L3 capacity. If the "size" file is impacted by this change then I think there is something wrong with the current series because "size" is intended to mirror the schemata file that represents the size in bytes that correspond to the bits from schemata - this monitoring change does not change this relationship. > > Major stuff is that we now need both the L3-scoped domain list as well as the NODE > scoped domain list. So no longer just changing the "mon_scope" field in the L3 > rdt_resource from one value to another. Here may be opportunity to isolate the SNC layout from the customs. For example, the extra layers of files only need to be added when knowing SNC is active and the cache<->node relationships can be determined dynamically on file create and read instead of creating new lists and data structures to try and describe it as part of the architecture. > > I'll maybe have a draft set of patches in a day or two. I was actually hoping to get a discussion started about what folks really want to see instead of you implementing yet another new feature that folks may disagree with. Reinette
