tech-kern archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: [richard.henderson%linaro.org@localhost: Re: [PATCH v3 0/6] qemu/bswap: Use compiler __builtin_bswap()]
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 -----
>
Home |
Main Index |
Thread Index |
Old Index