On 16.05.2025 15:33, Andrew Cooper wrote: > Very few files need the stubs. Move the infrastructure out of > processor.h and config.h into a new stubs.h, and adjust the includes > accordingly. > > Make the per-cpu struct stubs be read mostly; they're unmodified > during the uptime of the CPU, and move them into smpboot.c seeing as > that's where they're allocated and freed. > > No functional change. > > Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com>
Acked-by: Jan Beulich <jbeul...@suse.com> with one possible suggestion: > --- /dev/null > +++ b/xen/arch/x86/include/asm/stubs.h > @@ -0,0 +1,37 @@ > +/* SPDX-License-Identifier: GPL-2.0-only */ > +#ifndef X86_ASM_STUBS_H > +#define X86_ASM_STUBS_H > + > +/* > + * Xen has several per-cpu executable stubs which are written dynamically. This puts it pretty well. Yet in principle there may be further, perhaps entirely different stubs in the future. Hence stubs.h feels a little generic. What about exec-stubs.h? Jan
