Re: [PATCH v9 6/9] x86/resctrl: Add "mba_MBps_event" file to ctrl_mon directories
From: Reinette Chatre
Date: Tue Nov 19 2024 - 23:04:06 EST
Hi Tony,
On 11/13/24 4:17 PM, Tony Luck wrote:
> The "mba_MBps" mount option provides an alternate method to
> control memory bandwidth. Instead of specifying allowable
> bandwidth as a percentage of maximum possible, the user
> provides a MiB/s limit value.
>
> Historically the limit was enforced by a feedback loop from
"Historically the limit was enforced" no history needed since
this is still the case at the time of this patch.
> the measure local bandwidth to adjust the memory bandwidth
> allocation controls.
I am not sure what is meant by "a feedback loop from the measure
local bandwidth" (that was copy&pasted to next patch).
>
> In preparation to allow the user to pick the memory bandwidth
> monitoring event used as input to the feedback loop, provide
> a file in each ctrl_mon group directory that shows the event
In the documentation the custom is to use CTRL_MON.
> currently in use.
Much better changelog. I think it will help to add a snippet that
mentions the file is only visible to user space if resctrl
was mounted with mba_MBps, and thus only visible when mba_sc is
enabled, and thus reinforcing that this maintains the contract
that rdtgrp->mba_mbps_event is only accessed when mba_sc is enabled.
>
> Suggested-by: Reinette Chatre <reinette.chatre@xxxxxxxxx>
> Signed-off-by: Tony Luck <tony.luck@xxxxxxxxx>
> ---
Reinette