Re: [RFC v2 PATCH 0/7] x86/resctrl: Miscellaneous resctrl features

[Moger, Babu]

Gentle ping. Any comments on this!

Thanks

Babu

On 2/2/23 15:46, Babu Moger wrote:
> These series adds support few minor features.
> 1. Support assigning multiple tasks to control/mon groups in one command.
> 2. Add debug mount option for resctrl interface.
> 3. Add RMID and CLOSID in resctrl interface when mounted with debug option.
> 4. While doing these above changes, found that rftype flags needed some cleanup.
>    They were named inconsistently. Re-arranged them much more cleanly now.
>    Hope it can help future additions.
>
> Code is built on top of tip branch x86/cache.
>
> ---
> v2: Changes since v1
>   a. Removed the changes to add the task's threads automatically. It required
>      book keeping to handle the failures and gets complicated. Removed that change
>      for now.
>   b. Added -o debug option to mount in debug mode(comment from Fenghua)
>   c. Added debug files rmid and closid. Stephane wanted to rename them more
>      generic to accommodate ARM. It kind of loses meaning if is renamed differently.
>      Kept it same for now. Will change if he feels strong about it. 
>
> Babu Moger (7):
>       x86/resctrl: Add multiple tasks to the resctrl group at once
>       x86/resctrl: Remove few unnecessary rftype flags
>       x86/resctrl: Rename rftype flags for consistency
>       x86/resctrl: Re-arrange RFTYPE flags based on hierarchy
>       x86/resctrl: Introduce -o debug mount option
>       x86/resctrl: Display the RMID and COSID for resctrl groups
>       x86/resctrl: Add debug files when mounted with debug option
>
>
>  Documentation/x86/resctrl.rst          |  28 ++++-
>  arch/x86/kernel/cpu/resctrl/core.c     |   8 +-
>  arch/x86/kernel/cpu/resctrl/internal.h |  68 +++++++++--
>  arch/x86/kernel/cpu/resctrl/rdtgroup.c | 154 +++++++++++++++++++++----
>  4 files changed, 215 insertions(+), 43 deletions(-)
>
> --
>
-- 
Thanks
Babu Moger