[PATCH 0/3] xen/arm: Some clean-up found with -Wconversion and -Warith-conversion

Julien Grall posted 3 patches 8 months, 2 weeks ago
Only 2 patches received!
xen/arch/arm/domain.c           |  3 +-
xen/arch/arm/include/asm/mmio.h |  6 ++--
xen/arch/arm/include/asm/vgic.h | 17 ++++++-----
xen/arch/arm/io.c               |  2 +-
xen/arch/arm/vgic-v2.c          | 14 +++++++--
xen/arch/arm/vgic-v3.c          |  2 +-
xen/arch/arm/vgic.c             | 23 ++++++++-------
xen/arch/arm/vgic/vgic-init.c   |  2 +-
xen/include/public/arch-arm.h   | 52 ++++++++++++++++-----------------
9 files changed, 68 insertions(+), 53 deletions(-)
[PATCH 0/3] xen/arm: Some clean-up found with -Wconversion and -Warith-conversion
Posted by Julien Grall 8 months, 2 weeks ago
From: Julien Grall <jgrall@amazon.com>

Hi all,

This is a small series to fix some of the issues found while playing
with -Wconversion and -Warith-conversion.

There are a lot more but the bulk are in
 - bitmap
 - cpumask
 - nodemask
 - bitops/atomics
 - find_*

Some are not too difficult to address but other there are even
prototype conflicts between arm x86.

Cheers,

Julien Grall (3):
  xen/arm: vmmio: The number of entries cannot be negative
  xen/arm: vgic: Use 'unsigned int' rather than 'int' whenever it is
    possible
  xen/public: arch-arm: All PSR_* defines should be unsigned

 xen/arch/arm/domain.c           |  3 +-
 xen/arch/arm/include/asm/mmio.h |  6 ++--
 xen/arch/arm/include/asm/vgic.h | 17 ++++++-----
 xen/arch/arm/io.c               |  2 +-
 xen/arch/arm/vgic-v2.c          | 14 +++++++--
 xen/arch/arm/vgic-v3.c          |  2 +-
 xen/arch/arm/vgic.c             | 23 ++++++++-------
 xen/arch/arm/vgic/vgic-init.c   |  2 +-
 xen/include/public/arch-arm.h   | 52 ++++++++++++++++-----------------
 9 files changed, 68 insertions(+), 53 deletions(-)

-- 
2.40.1
Re: [PATCH 0/3] xen/arm: Some clean-up found with -Wconversion and -Warith-conversion
Posted by Julien Grall 8 months, 1 week ago
Hi,

On 17/08/2023 22:43, Julien Grall wrote:
> From: Julien Grall <jgrall@amazon.com>
> 
> Hi all,
> 
> This is a small series to fix some of the issues found while playing
> with -Wconversion and -Warith-conversion.
> 
> There are a lot more but the bulk are in
>   - bitmap
>   - cpumask
>   - nodemask
>   - bitops/atomics
>   - find_*
> 
> Some are not too difficult to address but other there are even
> prototype conflicts between arm x86.
> 
> Cheers,
> 
> Julien Grall (3):
>    xen/arm: vmmio: The number of entries cannot be negative
>    xen/arm: vgic: Use 'unsigned int' rather than 'int' whenever it is
>      possible
>    xen/public: arch-arm: All PSR_* defines should be unsigned

I have committed the series.

Cheers,

-- 
Julien Grall