On 03.10.2022 18:21, Marek Marczykowski-Górecki wrote:
> Documentation for credit2_runqueue=all says it should create one queue
> for all pCPUs on the host. But since introduction
> sched_credit2_max_cpus_runqueue, it actually created separate runqueue
> per socket, even if the CPUs count is below
> sched_credit2_max_cpus_runqueue.
> 
> Adjust the condition to skip syblink check in case of
> credit2_runqueue=all.
> 
> Fixes: 8e2aa76dc167 ("xen: credit2: limit the max number of CPUs in a 
> runqueue")
> Signed-off-by: Marek Marczykowski-Górecki <[email protected]>
> Reviewed-by: Juergen Gross <[email protected]>
> ---
> Changes in v2:
>  - fix indentation
>  - adjust doc
> 
> The whole thing is under cpu_runqueue_match() already, so maybe
> cpu_runqueue_siblings_match() isn't needed at all?
> ---
>  docs/misc/xen-command-line.pandoc | 5 +++++
>  xen/common/sched/credit2.c        | 9 +++++++--
>  2 files changed, 12 insertions(+), 2 deletions(-)

George, Dario - any chance of an ack (or reasons not to)?

Jan

Reply via email to