Re: [PATCH v5 2/6] vmalloc: Add text_alloc() and text_free()

From: Jarkko Sakkinen
Date: Fri Jul 24 2020 - 22:21:06 EST


On Fri, Jul 24, 2020 at 01:22:43PM +0300, Mike Rapoport wrote:
> On Fri, Jul 24, 2020 at 08:05:49AM +0300, Jarkko Sakkinen wrote:
> > Introduce functions for allocating memory for dynamic trampolines, such
> > as kprobes. An arch can promote the availability of these functions with
> > CONFIG_ARCH_HAS_TEXT_ALLOC.
>
> As it was pointed out at the discussion on the previous version [1],
> text_alloc() alone won't necessarily suit other architectures.
>
> I don't see a point in defining a "generic" interface that apriory could
> not be imeplemented by several architectures.
>
> [1] https://lore.kernel.org/lkml/20200714094625.1443261-1-jarkko.sakkinen@xxxxxxxxxxxxxxx/

These changes do actually acknowledge the feedback [1][2][3]. They do not
interfere with module_alloc() and are fully optional.

[1] https://lore.kernel.org/linux-riscv/20200714102826.GB4756@willie-the-truck/
[2] https://lore.kernel.org/linux-riscv/20200714164245.GE1551@xxxxxxxxxxxxxxxxxxxxx/
[3] https://lore.kernel.org/linux-riscv/20200714135651.GA27819@linux-8ccs/

/Jarkko