On 18.06.2025 11:13, Oleksii Kurochko wrote: > Instead of declaring __ro_after_init_{start,end} in each architecture's > asm/setup.h, move these declarations to the common header xen/sections.h. > > This centralizes the declarations and reduces duplication across > architectures. > > No functional change intended. > > Signed-off-by: Oleksii Kurochko <oleksii.kuroc...@gmail.com>
It can probably go in as-is, hence Acked-by: Jan Beulich <jbeul...@suse.com> but ... > --- a/xen/include/xen/sections.h > +++ b/xen/include/xen/sections.h > @@ -5,6 +5,9 @@ > > #include <xen/compiler.h> > > +/* SAF-0-safe */ > +extern const char __ro_after_init_start[], __ro_after_init_end[]; > + > /* SAF-0-safe */ > extern char __init_begin[], __init_end[]; > #define is_init_section(p) ({ \ ... I generally dislike this pattern of adding to the front, as if the addition was more important than what's there already. But maybe that's indeed just me ... Jan