[richard.henderson%linaro.org@localhost: Re: [PATCH v3 0/6] qemu/bswap: Use compiler __builtin_bswap()]

[Reinoud Zandijk <reinoud%NetBSD.org@localhost>]

Hi folks,

will this create an issue worthy of reporting to qemu?

With regards,
Reinoud

----- Forwarded message from Richard Henderson <richard.henderson%linaro.org@localhost> -----

Date: Thu, 12 Jan 2023 15:05:01 -1000
From: Richard Henderson <richard.henderson%linaro.org@localhost>
To: Philippe Mathieu-Daudé <philmd%linaro.org@localhost>, qemu-devel%nongnu.org@localhost
Cc: Marc-André Lureau <marcandre.lureau%redhat.com@localhost>, Ryo ONODERA <ryoon%netbsd.org@localhost>, "Daniel P. Berrangé" <berrange%redhat.com@localhost>, Thomas Huth <thuth%redhat.com@localhost>,
	Ira Weiny <ira.weiny%intel.com@localhost>, Paolo Bonzini <pbonzini%redhat.com@localhost>, Kamil Rytarowski <kamil%netbsd.org@localhost>, Reinoud Zandijk <reinoud%netbsd.org@localhost>, Jonathan
	Cameron <Jonathan.Cameron%huawei.com@localhost>
Subject: Re: [PATCH v3 0/6] qemu/bswap: Use compiler __builtin_bswap()
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.4.2

On 1/11/23 08:31, Philippe Mathieu-Daudé wrote:
> Implement Richard's suggestion to use __builtin_bswap().
> 
> Convert to __builtin_bswap() one patch per OS to simplify
> maintainers review.
> 
> Since v2:
> - Rebased adapting ./configure changes to meson
> 
> Since v1:
> - Remove the Haiku/BSD ifdef'ry (Peter)
> - Include the Haiku VM image from Alexander
> 
> Philippe Mathieu-Daudé (6):
>    qemu/bswap: Replace bswapXX() by compiler __builtin_bswap()
>    qemu/bswap: Replace bswapXXs() by compiler __builtin_bswap()
>    qemu/bswap: Remove <byteswap.h> dependency
>    qemu/bswap: Use compiler __builtin_bswap() on Haiku
>    qemu/bswap: Use compiler __builtin_bswap() on FreeBSD
>    qemu/bswap: Use compiler __builtin_bswap() on NetBSD

If this passes on all the odd OS's, great.
Failure on some oddball is what blocked my patch set years ago.


r~

----- End forwarded message -----