-----Message d'origine-----
De : Aaron Tomlin [off-list ref]
Envoyé : lundi 7 février 2022 19:02
À : Christophe Leroy [off-list ref]
Cc : Aaron Tomlin [off-list ref]; Luis Chamberlain
[off-list ref]; Michal Suchanek [off-list ref]; cl@linux.com;
pmladek@suse.com; mbenes@suse.cz; akpm@linux-foundation.org;
jeyu@kernel.org; linux-kernel@vger.kernel.org; linux-modules@vger.kernel.org;
live-patching@vger.kernel.org; ghalat@redhat.com; allen.lkml@gmail.com;
void@manifault.com; joe@perches.com
Objet : Re: [RFC PATCH v4 00/13] module: core code clean up

On Mon 2022-02-07 17:17 +0000, Christophe Leroy wrote:

quoted

Yes and that's the purpose of the patch I proposed at
https://patchwork.kernel.org/project/linux-

modules/patch/203348805c9ac9851d8939d15cb9802ef047b5e2.1643919758.gi
t.christophe.leroy@csgroup.eu/

I see.

quoted

Allthough I need to find out what's the problem reported by the robot.

I'll have a look too.

quoted

As suggested by Luis, this fix should go once all ongoing work is done.
But it would be nice if you could just remove patch 5 from you series,
otherwise we would have to revert it later.

Perhaps it might be easier if I keep the patch within the series; once
merged into module-next, by Luis, you can rebase and then add the "Fixes:"
tag to resolve the issue, no?

I don't think it is easier.

If we do that it means we'll move some code from main.c to arch_strict_rwx.c with your series, then move it back to main.c and remove arch_strict_rwx.c when we do the fix. That's not good for history tracking because the code we move back and forth will appear as new code in main.c whereas it's code that has been there for years.

As we know the code will be back in main.c at the end, I looks easier to me to not move it at all by not applying your patch 5.

Thanks
Christophe