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

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

Oops, wrong list, sorry. Although its supposed to compile on NetBSD 9.3 I
still wonder if this really is a good idea.


On Wed, Jan 18, 2023 at 01:28:12PM +0100, Reinoud Zandijk wrote:
> 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 -----
>