[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qtest: ask endianness of the target in qtest_in
From: |
Greg Kurz |
Subject: |
Re: [Qemu-devel] [PATCH] qtest: ask endianness of the target in qtest_init() |
Date: |
Fri, 7 Oct 2016 12:10:07 +0200 |
On Fri, 7 Oct 2016 10:39:09 +0100
Peter Maydell <address@hidden> wrote:
> On 7 October 2016 at 00:55, David Gibson <address@hidden> wrote:
> > It is an improvement. But I still think if we're relying on the
> > ill-defined "target endianness" we're already doing something wrong.
>
> Target endianness is not ill-defined. It's a clear and constant
> property of the bus the CPU is plugged into. It is a bit weird
> to rely on it in the test code, which is why only the virtio
> tests currently use qtest_big_endian().
>
And to discourage anyone to use it in a test program, maybe it
could even be renamed virtio_big_endian() and put in a virtio
specific header file ? This is how it is done in QEMU.
> thanks
> -- PMM
Cheers.
--
Greg
Re: [Qemu-devel] [PATCH] qtest: ask endianness of the target in qtest_init(), Greg Kurz, 2016/10/06