Re: [Qemu-devel] [PULL 00/11] Ide patches

2017-09-20 Thread Mark Cave-Ayland
On 20/09/17 18:55, John Snow wrote: > Guh. From which distro does your GCC 4.7 hail? > > Regardless, I suppose I will revert to Eric's workaround, though I like > the way it reads an awful lot less. Thanks John - it's just a standard Debian Wheezy installation on amd64. ATB, Mark.

Re: [Qemu-devel] [PULL 00/11] Ide patches

2017-09-20 Thread John Snow
On 09/20/2017 01:02 PM, Mark Cave-Ayland wrote: > On 18/09/17 19:14, Peter Maydell wrote: > >> On 18 September 2017 at 19:00, Peter Maydell >> wrote: >>> On 18 September 2017 at 18:55, John Snow wrote: On 09/16/2017 10:34 AM, Peter Maydell

Re: [Qemu-devel] [PULL 00/11] Ide patches

2017-09-20 Thread Mark Cave-Ayland
On 18/09/17 19:14, Peter Maydell wrote: > On 18 September 2017 at 19:00, Peter Maydell wrote: >> On 18 September 2017 at 18:55, John Snow wrote: >>> On 09/16/2017 10:34 AM, Peter Maydell wrote: Hi; I'm afraid this doesn't build with clang:

Re: [Qemu-devel] [PULL 00/11] Ide patches

2017-09-18 Thread Peter Maydell
On 18 September 2017 at 19:00, Peter Maydell wrote: > On 18 September 2017 at 18:55, John Snow wrote: >> On 09/16/2017 10:34 AM, Peter Maydell wrote: >>> Hi; I'm afraid this doesn't build with clang: >>> >>>

Re: [Qemu-devel] [PULL 00/11] Ide patches

2017-09-18 Thread Peter Maydell
On 18 September 2017 at 18:55, John Snow wrote: > On 09/16/2017 10:34 AM, Peter Maydell wrote: >> Hi; I'm afraid this doesn't build with clang: >> >> /home/petmay01/linaro/qemu-for-merges/hw/ide/core.c:70:15: error: >> comparison of unsigned enum expression >= 0 is always true

Re: [Qemu-devel] [PULL 00/11] Ide patches

2017-09-18 Thread John Snow
On 09/16/2017 10:34 AM, Peter Maydell wrote: > On 16 September 2017 at 02:03, John Snow wrote: >> The following changes since commit 5faf2d376af3cb4eb92da44c2580e08d39832caa: >> >> Merge remote-tracking branch 'remotes/huth/tags/check-20170915' into >> staging (2017-09-15

Re: [Qemu-devel] [PULL 00/11] Ide patches

2017-09-18 Thread Eric Blake
On 09/16/2017 09:34 AM, Peter Maydell wrote: > Hi; I'm afraid this doesn't build with clang: > > /home/petmay01/linaro/qemu-for-merges/hw/ide/core.c:70:15: error: > comparison of unsigned enum expression >= 0 is always true > [-Werror,-Wtautological-compare] > if (enval >= 0 && enval <

Re: [Qemu-devel] [PULL 00/11] Ide patches

2017-09-16 Thread Peter Maydell
On 16 September 2017 at 02:03, John Snow wrote: > The following changes since commit 5faf2d376af3cb4eb92da44c2580e08d39832caa: > > Merge remote-tracking branch 'remotes/huth/tags/check-20170915' into > staging (2017-09-15 20:29:44 +0100) > > are available in the git

Re: [Qemu-devel] [PULL 00/11] Ide patches

2017-09-15 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PULL 00/11] Ide patches Message-id: 20170916010330.10435-1-js...@redhat.com Type: series === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1 total=$(git log --oneline $BASE

[Qemu-devel] [PULL 00/11] Ide patches

2017-09-15 Thread John Snow
The following changes since commit 5faf2d376af3cb4eb92da44c2580e08d39832caa: Merge remote-tracking branch 'remotes/huth/tags/check-20170915' into staging (2017-09-15 20:29:44 +0100) are available in the git repository at: https://github.com/jnsnow/qemu.git tags/ide-pull-request for you to

Re: [Qemu-devel] [PULL 00/11] Ide patches

2016-02-11 Thread Peter Maydell
On 10 February 2016 at 19:37, John Snow wrote: > The following changes since commit c9f19dff101e2c2cf3fa3967eceec2833e845e40: > > Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into > staging (2016-02-09 19:34:46 +) > > are available in the git

[Qemu-devel] [PULL 00/11] Ide patches

2016-02-10 Thread John Snow
The following changes since commit c9f19dff101e2c2cf3fa3967eceec2833e845e40: Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging (2016-02-09 19:34:46 +) are available in the git repository at: https://github.com/jnsnow/qemu.git tags/ide-pull-request for you

[Qemu-devel] [PULL 00/11] Ide patches

2016-02-10 Thread John Snow
The following changes since commit c9f19dff101e2c2cf3fa3967eceec2833e845e40: Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging (2016-02-09 19:34:46 +) are available in the git repository at: https://github.com/jnsnow/qemu.git tags/ide-pull-request for you

Re: [Qemu-devel] [PULL 00/11] Ide patches

2016-01-11 Thread Peter Maydell
On 9 January 2016 at 00:51, John Snow wrote: > The following changes since commit 38a762fec63fd5c035aae29ba9a77d357e21e4a7: > > Merge remote-tracking branch > 'remotes/berrange/tags/pull-crypto-fixes-2015-12-23-1' into staging > (2015-12-23 13:53:32 +) > > are available

Re: [Qemu-devel] [PULL 00/11] Ide patches

2016-01-11 Thread Peter Maydell
On 11 January 2016 at 17:18, John Snow wrote: > On 01/11/2016 06:18 AM, Peter Maydell wrote: >> On 9 January 2016 at 00:51, John Snow wrote: >> This kind of thing: >> > > "This kind of thing" as one might say while holding up a rotting fish > with just two

Re: [Qemu-devel] [PULL 00/11] Ide patches

2016-01-11 Thread John Snow
On 01/11/2016 06:18 AM, Peter Maydell wrote: > On 9 January 2016 at 00:51, John Snow wrote: >> The following changes since commit 38a762fec63fd5c035aae29ba9a77d357e21e4a7: >> >> Merge remote-tracking branch >> 'remotes/berrange/tags/pull-crypto-fixes-2015-12-23-1' into

[Qemu-devel] [PULL 00/11] Ide patches

2016-01-08 Thread John Snow
The following changes since commit 38a762fec63fd5c035aae29ba9a77d357e21e4a7: Merge remote-tracking branch 'remotes/berrange/tags/pull-crypto-fixes-2015-12-23-1' into staging (2015-12-23 13:53:32 +) are available in the git repository at: https://github.com/jnsnow/qemu.git

[Qemu-devel] [PULL 00/11] Ide patches

2015-09-18 Thread John Snow
The following changes since commit 16a1b6e97c2a2919fd296db4bea2f9da2ad3cc4d: target-cris: update CPU state save/load to use VMStateDescription (2015-09-17 14:31:38 +0100) are available in the git repository at: https://github.com/jnsnow/qemu.git tags/ide-pull-request for you to fetch

Re: [Qemu-devel] [PULL 00/11] Ide patches

2015-09-18 Thread Peter Maydell
On 18 September 2015 at 16:04, John Snow wrote: > The following changes since commit 16a1b6e97c2a2919fd296db4bea2f9da2ad3cc4d: > > target-cris: update CPU state save/load to use VMStateDescription > (2015-09-17 14:31:38 +0100) > > are available in the git repository at: > >