Re: [PATCH 3/4] kprobes/treewide: Explicitly override alloc/free functions

From: Alexander Gordeev
Date: Sat Apr 13 2024 - 03:26:03 EST


On Tue, Mar 26, 2024 at 04:36:23PM +0000, Mark Rutland wrote:
Hi Mark,

..
> (a) Architectures using the generic kprobes_free_insn_page() and
kprobes_alloc_insn_page()?
> kprobes_free_insn_page() are left as-is. The __weak annotation is
> removed from the generic implementations so that accidental
> overrides/misuse can be detected easily.
>
> (b) Architectures which provide their own kprobes_free_insn_page() are
kprobes_alloc_insn_page()?
> given a matching implementation of kprobes_free_insn_page(), and
> select HAVE_KPROBES_ALLOC.
.
> arch/s390/Kconfig | 3 ++-
> arch/s390/kernel/kprobes.c | 5 +++++

I tried the repo:
git://git.kernel.org/pub/scm/linux/kernel/git/mark/linux.git kprobes/without-modules

For s390:
Acked-by: Alexander Gordeev <agordeev@xxxxxxxxxxxxx>

Thanks!