Shawn,
I am thinking of committing this, if you disagree please say something
in the next couple of days
On Tue, 19 Dec 2023, Simone Ballarin wrote:
> From: Maria Celeste Cesario <maria.celeste.cesa...@bugseng.com>
>
> The xen sources contain violations of MISRA C:2012 Rule 11.8 whose
> headline states:
> "A conversion shall not remove any const, volatile or _Atomic qualification
> from the type pointed to by a pointer".
>
> Fix violation by adding missing const qualifier in cast.
>
> Signed-off-by: Maria Celeste Cesario <maria.celeste.cesa...@bugseng.com>
> Signed-off-by: Simone Ballarin <simone.balla...@bugseng.com>
> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org>
> ---
> Adaptation requested by the community to make the code more consistent.
> ---
> xen/arch/ppc/include/asm/atomic.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/xen/arch/ppc/include/asm/atomic.h
> b/xen/arch/ppc/include/asm/atomic.h
> index 64168aa3f1..fe778579fb 100644
> --- a/xen/arch/ppc/include/asm/atomic.h
> +++ b/xen/arch/ppc/include/asm/atomic.h
> @@ -16,7 +16,7 @@
>
> static inline int atomic_read(const atomic_t *v)
> {
> - return *(volatile int *)&v->counter;
> + return *(const volatile int *)&v->counter;
> }
>
> static inline int _atomic_read(atomic_t v)
> --
> 2.40.0
>
