Re: [PATCH 7/7] tests/fw_cfg: Run the tests on big-endian targets

2019-10-06 Thread Thomas Huth
- Original Message - > Sent: Friday, October 4, 2019 10:05:09 AM > On 04/10/2019 00:54, Philippe Mathieu-Daudé wrote: [...] > > diff --git a/tests/Makefile.include b/tests/Makefile.include > > index 3543451ed3..322bdb36ff 100644 > > --- a/tests/Makefile.include > > +++ b/tests/Makefile.incl

Re: [PATCH 7/7] tests/fw_cfg: Run the tests on big-endian targets

2019-10-04 Thread Philippe Mathieu-Daudé
On 10/4/19 11:14 AM, Laurent Vivier wrote: On 04/10/2019 11:03, Philippe Mathieu-Daudé wrote: On 10/4/19 10:59 AM, Laurent Vivier wrote: On 04/10/2019 10:53, Philippe Mathieu-Daudé wrote: On 10/4/19 10:05 AM, Laurent Vivier wrote: On 04/10/2019 00:54, Philippe Mathieu-Daudé wrote: We have be

Re: [PATCH 7/7] tests/fw_cfg: Run the tests on big-endian targets

2019-10-04 Thread Laurent Vivier
On 04/10/2019 11:03, Philippe Mathieu-Daudé wrote: > On 10/4/19 10:59 AM, Laurent Vivier wrote: >> On 04/10/2019 10:53, Philippe Mathieu-Daudé wrote: >>> On 10/4/19 10:05 AM, Laurent Vivier wrote: On 04/10/2019 00:54, Philippe Mathieu-Daudé wrote: > We have been restricting our fw_cfg test

Re: [PATCH 7/7] tests/fw_cfg: Run the tests on big-endian targets

2019-10-04 Thread Philippe Mathieu-Daudé
On 10/4/19 10:59 AM, Laurent Vivier wrote: On 04/10/2019 10:53, Philippe Mathieu-Daudé wrote: On 10/4/19 10:05 AM, Laurent Vivier wrote: On 04/10/2019 00:54, Philippe Mathieu-Daudé wrote: We have been restricting our fw_cfg tests to the PC machine, which is a little-endian architecture. The fw

Re: [PATCH 7/7] tests/fw_cfg: Run the tests on big-endian targets

2019-10-04 Thread Laurent Vivier
On 04/10/2019 10:53, Philippe Mathieu-Daudé wrote: > On 10/4/19 10:05 AM, Laurent Vivier wrote: >> On 04/10/2019 00:54, Philippe Mathieu-Daudé wrote: >>> We have been restricting our fw_cfg tests to the PC machine, >>> which is a little-endian architecture. >>> The fw_cfg device is also used on the

Re: [PATCH 7/7] tests/fw_cfg: Run the tests on big-endian targets

2019-10-04 Thread Philippe Mathieu-Daudé
On 10/4/19 10:05 AM, Laurent Vivier wrote: On 04/10/2019 00:54, Philippe Mathieu-Daudé wrote: We have been restricting our fw_cfg tests to the PC machine, which is a little-endian architecture. The fw_cfg device is also used on the SPARC and PowerPC architectures, which can run in big-endian con

Re: [PATCH 7/7] tests/fw_cfg: Run the tests on big-endian targets

2019-10-04 Thread Laurent Vivier
On 04/10/2019 00:54, Philippe Mathieu-Daudé wrote: > We have been restricting our fw_cfg tests to the PC machine, > which is a little-endian architecture. > The fw_cfg device is also used on the SPARC and PowerPC > architectures, which can run in big-endian configuration. > > Since we want to be s

[PATCH 7/7] tests/fw_cfg: Run the tests on big-endian targets

2019-10-03 Thread Philippe Mathieu-Daudé
We have been restricting our fw_cfg tests to the PC machine, which is a little-endian architecture. The fw_cfg device is also used on the SPARC and PowerPC architectures, which can run in big-endian configuration. Since we want to be sure our device does not regress regardless the endianess used,