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 <[email protected]>
It can probably go in as-is, hence
Acked-by: Jan Beulich <[email protected]>
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
