Hi Tony, On 10/20/2023 2:30 PM, Tony Luck wrote: > The rdt_domain structure is used for both control and monitor features. > It is about to be split into separate structures for these two usages > because the scope for control and monitoring features for a resource > will be different for future resources. > > To allow for common code that scans a list of domains looking for a > specific domain id, move the "list" and "id" fields into their own > structure within the rdt_domain structure. The motivation for this split is not clear to me. Here the motivation is to support the code that needs to traverse both lists. Later (patch #4) the motivation is that what remains should be "just the fields required for control and monitoring respectively". The comment "common header for different domain types" also makes me think this new structure should contain all common members? The reason why the motivation needs to be clear in this regard is because there is a common field, cpu_mask, that did not make it into the header. Should it? > Add a "type" field to the header to be used later so that callers looking > up a domain can be assured that they found one of the expected type. Please move this addition to when it is used. Reinette
