Re: [PATCH v7 07/31] x86,fs/resctrl: Refactor domain_remove_cpu_mon() ready for new domain types

[Reinette Chatre <reinette.chatre@intel.com>]

Hi Tony,

On 7/11/25 4:53 PM, Tony Luck wrote:
> Historically all monitoring events have been associated with the L3
> resource. This will change when support for telemetry events is added.

This is not "history" but the current state. How about:
"All monitoring events are associated with the L3 resource." 

Drop the "This will change when support for telemetry events is added." from the
first paragraph since it does not contribute to the context but can be used
in problem statement later. 

> 
> The RDT_RESOURCE_L3 resource carries a lot of state in the domain
> structures which needs to be dealt with when a domain is taken offline
> by removing the last CPU in the domain.

Above can be part of the context.

<insert problem statement here> 

> Refactor domain_remove_cpu_mon() so all the L3 processing is separated
> from general actions of clearing the CPU bit in the mask and removing
> directories from mon_data.

"directories from mon_data" -> "sub-directories from the mon_data directory"

> 
> resctrl_offline_mon_domain() needs to remove domain specific

"needs" -> "continues"

> directories and files from the "mon_data" directories, but can skip the
> L3 resource specific cleanup when called for other resource types.
> 
> Signed-off-by: Tony Luck <tony.luck@intel.com>
> ---

Patch looks good.

Reinette