Re: [PATCH v5 04/15] sparc: simplify module_alloc()

From: Sam Ravnborg
Date: Tue Apr 23 2024 - 12:41:49 EST


Hi Mike,
On Mon, Apr 22, 2024 at 12:44:25PM +0300, Mike Rapoport wrote:
> From: "Mike Rapoport (IBM)" <rppt@xxxxxxxxxx>
>
> Define MODULES_VADDR and MODULES_END as VMALLOC_START and VMALLOC_END
> for 32-bit and reduce module_alloc() to
>
> __vmalloc_node_range(size, 1, MODULES_VADDR, MODULES_END, ...)
>
> as with the new defines the allocations becames identical for both 32
> and 64 bits.
>
> While on it, drop unsed include of <linux/jump_label.h>
>
> Suggested-by: Sam Ravnborg <sam@xxxxxxxxxxxx>
> Signed-off-by: Mike Rapoport (IBM) <rppt@xxxxxxxxxx>

Looks good.
Reviewed-by: Sam Ravnborg <sam@xxxxxxxxxxxx>