Re: [Qemu-devel] [PULL 00/30] ppc-for-2.12 queue 20180306

2018-03-07 Thread Daniel P . Berrangé
On Tue, Mar 06, 2018 at 05:56:03PM +, Mark Cave-Ayland wrote: > On 06/03/18 17:47, Thomas Huth wrote: > > > > It seems that the error is being raised when setting the property rather > > > than during realize so I'm not sure what I can do to handle this. Any > > > thoughts? > > > > Does the d

Re: [Qemu-devel] [PULL 00/30] ppc-for-2.12 queue 20180306

2018-03-06 Thread Peter Maydell
On 6 March 2018 at 17:47, Thomas Huth wrote: > Does the device need to be hot-pluggable or even user_creatable at all? > It seems like it is also using serial_hds[] directly, so that is a good > indication that it is *not* user creatable. So maybe the easiest fix is > to simply set > > dc-

Re: [Qemu-devel] [PULL 00/30] ppc-for-2.12 queue 20180306

2018-03-06 Thread Mark Cave-Ayland
On 06/03/18 17:47, Thomas Huth wrote: It seems that the error is being raised when setting the property rather than during realize so I'm not sure what I can do to handle this. Any thoughts? Does the device need to be hot-pluggable or even user_creatable at all? It seems like it is also using

Re: [Qemu-devel] [PULL 00/30] ppc-for-2.12 queue 20180306

2018-03-06 Thread Thomas Huth
On 06.03.2018 18:28, Mark Cave-Ayland wrote: > On 06/03/18 16:48, Thomas Huth wrote: > >> Something in the recent commits introduced a new way to cause unexpected >> aborts of QEMU: >> >> $ ppc64-softmmu/qemu-system-ppc64 -monitor stdio >> QEMU 2.11.50 monitor - type 'help' for more information >>

Re: [Qemu-devel] [PULL 00/30] ppc-for-2.12 queue 20180306

2018-03-06 Thread Mark Cave-Ayland
On 06/03/18 16:48, Thomas Huth wrote: Something in the recent commits introduced a new way to cause unexpected aborts of QEMU: $ ppc64-softmmu/qemu-system-ppc64 -monitor stdio QEMU 2.11.50 monitor - type 'help' for more information (qemu) device_add macio-newworld Unexpected error in qemu_chr_f

Re: [Qemu-devel] [PULL 00/30] ppc-for-2.12 queue 20180306

2018-03-06 Thread Thomas Huth
On 06.03.2018 05:01, David Gibson wrote: > The following changes since commit f2bb2d14c2958f3f5aef456bd2cdb1ff99f4a562: > > Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' > into staging (2018-03-05 16:41:20 +) > > are available in the Git repository at: > > git:

Re: [Qemu-devel] [PULL 00/30] ppc-for-2.12 queue 20180306

2018-03-06 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20180306040154.3669-1-da...@gibson.dropbear.id.au Subject: [Qemu-devel] [PULL 00/30] ppc-for-2.12 queue 20180306 === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1 total

Re: [Qemu-devel] [PULL 00/30] ppc-for-2.12 queue 20180306

2018-03-06 Thread Peter Maydell
On 6 March 2018 at 04:01, David Gibson wrote: > The following changes since commit f2bb2d14c2958f3f5aef456bd2cdb1ff99f4a562: > > Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' > into staging (2018-03-05 16:41:20 +) > > are available in the Git repository at: > > g

Re: [Qemu-devel] [PULL 00/30] ppc-for-2.12 queue 20180306

2018-03-05 Thread David Gibson
On Mon, Mar 05, 2018 at 08:30:40PM -0800, no-re...@patchew.org wrote: > Hi, > > This series seems to have some coding style problems. See output below for > more information: > > Type: series > Message-id: 20180306040154.3669-1-da...@gibson.dropbear.id.au > Subject: [Qemu

[Qemu-devel] [PULL 00/30] ppc-for-2.12 queue 20180306

2018-03-05 Thread David Gibson
The following changes since commit f2bb2d14c2958f3f5aef456bd2cdb1ff99f4a562: Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' into staging (2018-03-05 16:41:20 +) are available in the Git repository at: git://github.com/dgibson/qemu.git tags/ppc-for-2.12-20180306