Re: [PATCH v3 08/10] perf/hisi_uncore: Avoid placing cpumask on the stack

From: Jonathan Cameron
Date: Fri Apr 05 2024 - 05:57:12 EST


On Wed, 3 Apr 2024 23:59:48 +0800
Dawei Li <dawei.li@xxxxxxxxxxxx> wrote:

> In general it's preferable to avoid placing cpumasks on the stack, as
> for large values of NR_CPUS these can consume significant amounts of
> stack space and make stack overflows more likely.
>
> Use cpumask_any_and_but() to avoid the need for a temporary cpumask on
> the stack.
>
> Suggested-by: Mark Rutland <mark.rutland@xxxxxxx>
> Reviewed-by: Mark Rutland <mark.rutland@xxxxxxx>
> Signed-off-by: Dawei Li <dawei.li@xxxxxxxxxxxx>
Acked-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>