Re: [Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906

2016-09-06 Thread Benjamin Herrenschmidt
On Wed, 2016-09-07 at 07:52 +1000, Benjamin Herrenschmidt wrote: > On Tue, 2016-09-06 at 23:09 +0200, Thomas Huth wrote: > > > > The bad commit is: "ppc: Speed up load/store multiple" > > > > There are two "#if defined(HOST_WORDS_BIGENDIAN)" sections in this > > patch > > which are both bad: The

Re: [Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906

2016-09-06 Thread David Gibson
On Tue, Sep 06, 2016 at 11:26:34PM +0200, Thomas Huth wrote: > On 06.09.2016 22:55, Thomas Huth wrote: > > On 06.09.2016 22:23, Paolo Bonzini wrote: > >> > >> > >> On 06/09/2016 21:35, Thomas Huth wrote: > >>> D'oh! I'll have a look... > >>> > > I also see warnings on non-KVM hosts during 'make

Re: [Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906

2016-09-06 Thread Thomas Huth
On 06.09.2016 16:04, Peter Maydell wrote: > On 6 September 2016 at 04:39, David Gibson > wrote: >> The following changes since commit e87d397e5ef66276ccc49b829527d605ca07d0ad: >> >> Open 2.8 development tree (2016-09-05 11:38:54 +0100) >> >> are available in the git repository at: >> >> git:/

Re: [Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906

2016-09-06 Thread Benjamin Herrenschmidt
On Tue, 2016-09-06 at 23:09 +0200, Thomas Huth wrote: > The bad commit is: "ppc: Speed up load/store multiple" > > There are two "#if defined(HOST_WORDS_BIGENDIAN)" sections in this patch > which are both bad: The memcpy tries to copy 32-bit values into 64-bit > registers, which of course does not

Re: [Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906

2016-09-06 Thread Paolo Bonzini
On 06/09/2016 21:35, Thomas Huth wrote: > D'oh! I'll have a look... > >> > I also see warnings on non-KVM hosts during 'make check' which seem >> > to be produced by this new test: >> > >> > TEST: tests/boot-serial-test... (pid=2836) >> > /i386/boot-serial/isapc: >> > warning: TCG doesn't sup

Re: [Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906

2016-09-06 Thread Thomas Huth
On 06.09.2016 22:55, Thomas Huth wrote: > On 06.09.2016 22:23, Paolo Bonzini wrote: >> >> >> On 06/09/2016 21:35, Thomas Huth wrote: >>> D'oh! I'll have a look... >>> > I also see warnings on non-KVM hosts during 'make check' which seem > to be produced by this new test: > > TEST: t

Re: [Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906

2016-09-06 Thread Thomas Huth
On 06.09.2016 16:04, Peter Maydell wrote: > On 6 September 2016 at 04:39, David Gibson > wrote: >> The following changes since commit e87d397e5ef66276ccc49b829527d605ca07d0ad: >> >> Open 2.8 development tree (2016-09-05 11:38:54 +0100) >> >> are available in the git repository at: >> >> git:/

Re: [Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906

2016-09-06 Thread Thomas Huth
On 06.09.2016 22:23, Paolo Bonzini wrote: > > > On 06/09/2016 21:35, Thomas Huth wrote: >> D'oh! I'll have a look... >> I also see warnings on non-KVM hosts during 'make check' which seem to be produced by this new test: TEST: tests/boot-serial-test... (pid=2836) /i386/

Re: [Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906

2016-09-06 Thread Peter Maydell
On 6 September 2016 at 04:39, David Gibson wrote: > The following changes since commit e87d397e5ef66276ccc49b829527d605ca07d0ad: > > Open 2.8 development tree (2016-09-05 11:38:54 +0100) > > are available in the git repository at: > > git://github.com/dgibson/qemu.git tags/ppc-for-2.8-20160906

Re: [Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906

2016-09-06 Thread no-reply
Hi, Your series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906 Type: series Message-id: 1473133396-18940-1-git-send-email-da...@gibson.dropbear.id.au === TEST SCRIPT BEGIN === #!/bin/bash BASE=base

Re: [Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906

2016-09-05 Thread no-reply
Hi, Your series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906 Type: series Message-id: 1473133253-17598-1-git-send-email-da...@gibson.dropbear.id.au === TEST SCRIPT BEGIN === #!/bin/bash BASE=base

[Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906

2016-09-05 Thread David Gibson
The following changes since commit e87d397e5ef66276ccc49b829527d605ca07d0ad: Open 2.8 development tree (2016-09-05 11:38:54 +0100) are available in the git repository at: git://github.com/dgibson/qemu.git tags/ppc-for-2.8-20160906 for you to fetch changes up to 2ccbe3e29adb7c016bc5525d38d2a

[Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906

2016-09-05 Thread David Gibson
The following changes since commit e87d397e5ef66276ccc49b829527d605ca07d0ad: Open 2.8 development tree (2016-09-05 11:38:54 +0100) are available in the git repository at: git://github.com/dgibson/qemu.git tags/ppc-for-2.8-20160906 for you to fetch changes up to 2ccbe3e29adb7c016bc5525d38d2a