Re: [PATCH 5/5] ARM/cache: Drop legacy __read_mostly/__ro_after_init definitions

[Julien Grall <julien@xxxxxxx>]

Hi Andrew,

On 05/09/2024 23:08, Andrew Cooper wrote:
> These are no longer needed.
>
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
Acked-by: Julien Grall <jgrall@xxxxxxxxxx>

Cheers,

> ---
> CC: Jan Beulich <JBeulich@xxxxxxxx>
> CC: Roger Pau Monné <roger.pau@xxxxxxxxxx>
> CC: Stefano Stabellini <sstabellini@xxxxxxxxxx>
> CC: Julien Grall <julien@xxxxxxx>
> CC: Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
> CC: Bertrand Marquis <bertrand.marquis@xxxxxxx>
> CC: Michal Orzel <michal.orzel@xxxxxxx>
> ---
>   xen/arch/arm/include/asm/cache.h | 3 ---
>   xen/include/xen/cache.h          | 5 -----
>   2 files changed, 8 deletions(-)
>
> diff --git a/xen/arch/arm/include/asm/cache.h b/xen/arch/arm/include/asm/cache.h
> index 029b2896fb3e..b75c970b6806 100644
> --- a/xen/arch/arm/include/asm/cache.h
> +++ b/xen/arch/arm/include/asm/cache.h
> @@ -6,9 +6,6 @@
>   #define L1_CACHE_SHIFT  (CONFIG_ARM_L1_CACHE_SHIFT)
>   #define L1_CACHE_BYTES  (1 << L1_CACHE_SHIFT)
>   
> -/* TODO: Phase out the use of this via cache.h */
> -#define __read_mostly __section(".data.read_mostly")
> -
>   #endif
>   /*
>    * Local variables:
> diff --git a/xen/include/xen/cache.h b/xen/include/xen/cache.h
> index 90816ba05b1a..6ee174efa439 100644
> --- a/xen/include/xen/cache.h
> +++ b/xen/include/xen/cache.h
> @@ -15,9 +15,4 @@
>   #define __cacheline_aligned __attribute__((__aligned__(SMP_CACHE_BYTES)))
>   #endif
>   
> -#if defined(CONFIG_ARM)
> -/* TODO: Phase out the use of this via cache.h */
> -#define __ro_after_init __section(".data.ro_after_init")
> -#endif
> -
>   #endif /* __LINUX_CACHE_H */
--
Julien Grall