Hi Reinette,

I'm posting as a reply to your series to try and keep things together.
These patches should just be taken as suggestions. The first two
patches make the printing of the schema name simpler while keeping it
out of the arch code. The last add support for scalar cache controls
in the size file. I found these sufficient to add support for CMIN/CMAX
on top of MPAM. I'll post these separately but if makes sense to you
feel free to incorporate them in your PoC.

Thanks,

Ben

Ben Horgan (3):
  fs/resctrl: Add schema_name to struct resctrl_ctrl
  fs/resctrl: Use correct schema name in bw_validate() last_cmd_status
    messages
  fs/resctrl: display size for scalar cache controls

 fs/resctrl/ctrlmondata.c |  59 ++++++---------------
 fs/resctrl/internal.h    |   4 +-
 fs/resctrl/pseudo_lock.c |   2 +-
 fs/resctrl/rdtgroup.c    | 110 +++++++++++++++++++++++++--------------
 include/linux/resctrl.h  |   1 +
 5 files changed, 91 insertions(+), 85 deletions(-)

-- 
2.43.0

On Mon, Sep 21, 2026 at 02:56:59PM +0100, Ben Horgan wrote:
> Hi Reinette,
> 
> I'm posting as a reply to your series to try and keep things together.
> These patches should just be taken as suggestions. The first two
> patches make the printing of the schema name simpler while keeping it
> out of the arch code. The last add support for scalar cache controls
> in the size file. I found these sufficient to add support for CMIN/CMAX
> on top of MPAM. I'll post these separately but if makes sense to you
> feel free to incorporate them in your PoC.

Thanks for posting as these help RISC-V too. CBQRI has minimum reserved
bandwidth and a shared bw weight but there is no throttle-based MB
control. This should fix the "Invalid MB value" for CBQRI.

I'll rebase the CBQRI support onto resctrl/controls_rfc_v2 and will try
these patches in Qemu. The CMIN/CMAX modelling on your branch is helpful
for CBQRI scalar controls.

Thanks,
Drew

On 21/09/2026 14:56, Ben Horgan wrote:
> Hi Reinette,
> 
> I'm posting as a reply to your series to try and keep things together.
> These patches should just be taken as suggestions. The first two
> patches make the printing of the schema name simpler while keeping it
> out of the arch code. The last add support for scalar cache controls
> in the size file. I found these sufficient to add support for CMIN/CMAX
> on top of MPAM. I'll post these separately but if makes sense to you
> feel free to incorporate them in your PoC.

I've just put these MPAM changes on a branch here rather than making too much noise on the list.

https://gitlab.arm.com/linux-arm/linux-bh/ mpam_controls_on_generic_schema_poc

Ben

> 
> Thanks,
> 
> Ben
> 
> Ben Horgan (3):
>   fs/resctrl: Add schema_name to struct resctrl_ctrl
>   fs/resctrl: Use correct schema name in bw_validate() last_cmd_status
>     messages
>   fs/resctrl: display size for scalar cache controls
> 
>  fs/resctrl/ctrlmondata.c |  59 ++++++---------------
>  fs/resctrl/internal.h    |   4 +-
>  fs/resctrl/pseudo_lock.c |   2 +-
>  fs/resctrl/rdtgroup.c    | 110 +++++++++++++++++++++++++--------------
>  include/linux/resctrl.h  |   1 +
>  5 files changed, 91 insertions(+), 85 deletions(-)
>