Re: [PATCH 3/3] apparmor: replace get_zeroed_page() with kzalloc()
From: Paul Moore
Date: Wed May 27 2026 - 19:43:34 EST
On May 20, 2026 "Mike Rapoport (Microsoft)" <rppt@xxxxxxxxxx> wrote:
>
> multi_transaction_new() allocates memory with get_zeroed_page() and uses
> it as struct multi_transaction.
>
> The usage of that structure does not require struct page access and it is
> better to allocate multi_transaction objects with kzalloc() that provides
> better scalability and more debugging possibilities.
>
> Replace use of get_zeroed_page() with kzalloc().
>
> Link: https://lore.kernel.org/all/635405e4-9423-4a25-a6e7-e03c8ea0bcbe@xxxxxxxxxx
> Signed-off-by: Mike Rapoport (Microsoft) <rppt@xxxxxxxxxx>
> ---
> security/apparmor/apparmorfs.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
I'll leave this for John and/or Georgia to review and ultimately decide
to merge, but it looks okay to me.
Reviewed-by: Paul Moore <paul@xxxxxxxxxxxxxx>
--
paul-moore.com