Re: [PATCH net-next v2 1/2] net/iucv: Avoid explicit cpumask var allocation on stack

From: Alexandra Winter
Date: Tue Apr 02 2024 - 08:44:12 EST




On 31.03.24 07:34, Dawei Li wrote:
> For CONFIG_CPUMASK_OFFSTACK=y kernel, explicit allocation of cpumask
> variable on stack is not recommended since it can cause potential stack
> overflow.
>
> Instead, kernel code should always use *cpumask_var API(s) to allocate
> cpumask var in config-neutral way, leaving allocation strategy to
> CONFIG_CPUMASK_OFFSTACK.
>
> Use *cpumask_var API(s) to address it.
>
> Signed-off-by: Dawei Li <dawei.li@xxxxxxxxxxxx>
> ---

LGTM;
Thank you Eric for the comment and solution with the 'static' keyword.
Reviewed-by: Alexandra Winter <wintera@xxxxxxxxxxxxx>