Re: [PATCH v5 7/8] execmem: add support for cache of large ROX pages

From: Luis Chamberlain
Date: Mon Oct 14 2024 - 15:16:36 EST


On Sun, Oct 13, 2024 at 08:26:26PM -0700, Andrew Morton wrote:
> On Sun, 13 Oct 2024 11:43:41 +0300 Mike Rapoport <rppt@xxxxxxxxxx> wrote:
>
> > > > The idea is to keep everything together and have execmem_info describe all
> > > > that architecture needs.
> > >
> > > But why? That's pretty different from our normal style of arch hooks,
> > > and introduces an indirect call in a security sensitive area.
> >
> > Will change to __weak hook.
> >
>
> Thanks, I'll drop the v1 series;
>
> The todos which I collected are:
>
> https://lkml.kernel.org/r/CAPhsuW66etfdU3Fvk0KsELXcgWD6_TkBFjJ-BTHQu5OejDsP2w@xxxxxxxxxxxxxx
> https://lkml.kernel.org/r/Zwd6vH0rz0PVedLI@xxxxxxxxxxxxx
> https://lkml.kernel.org/r/ZwjXz0dz-RldVNx0@xxxxxxxxxxxxx
> https://lkml.kernel.org/r/202410111408.8fe6f604-lkp@xxxxxxxxx

BTW Andrew I'd like to pick this up through the modules tree, and while
at it, also beat it up with some more testing as we're expanding also
with the modversions stuff for Rust modules.

Luis