Re: [Qemu-devel] [PULL 00/14] Cleanup qdev legacy properties

2014-02-10 Thread Paolo Bonzini
Il 10/02/2014 19:40, Peter Maydell ha scritto: > Since Peter has not pulled yet, I'll pull or apply into my tree to check > if anything collides. Just to be clear, that means I should not apply this pull request, correct? Yes, Andreas added it to qom-next. Andreas, are you fine with having th

Re: [Qemu-devel] [PULL 00/14] Cleanup qdev legacy properties

2014-02-10 Thread Peter Maydell
On 10 February 2014 14:42, Andreas Färber wrote: > Since Peter has not pulled yet, I'll pull or apply into my tree to check > if anything collides. Just to be clear, that means I should not apply this pull request, correct? thanks -- PMM

Re: [Qemu-devel] [PULL 00/14] Cleanup qdev legacy properties

2014-02-10 Thread Markus Armbruster
Andreas Färber writes: > Am 10.02.2014 10:20, schrieb Markus Armbruster: [...] >> You're right to challenge new uses of QERR_*, but the use you spotted is >> appropriate, since we want consistency with the existing visitors. > > OK, I still don't fully understand the logic why sometimes we should

Re: [Qemu-devel] [PULL 00/14] Cleanup qdev legacy properties

2014-02-10 Thread Andreas Färber
Am 10.02.2014 10:20, schrieb Markus Armbruster: > Andreas Färber writes: > >> Paolo, >> >> Am 08.02.2014 11:01, schrieb Paolo Bonzini: >>> Anthony, Peter, >>> >>> The following changes since commit 0169c511554cb0014a00290b0d3d26c31a49818f: >>> >>> Merge remote-tracking branch 'qemu-kvm/uq/maste

Re: [Qemu-devel] [PULL 00/14] Cleanup qdev legacy properties

2014-02-10 Thread Markus Armbruster
Andreas Färber writes: > Paolo, > > Am 08.02.2014 11:01, schrieb Paolo Bonzini: >> Anthony, Peter, >> >> The following changes since commit 0169c511554cb0014a00290b0d3d26c31a49818f: >> >> Merge remote-tracking branch 'qemu-kvm/uq/master' into staging (2014-01-24 >> 15:52:44 -0800) >> >> are

Re: [Qemu-devel] [PULL 00/14] Cleanup qdev legacy properties

2014-02-09 Thread Paolo Bonzini
Il 08/02/2014 15:24, Andreas Färber ha scritto: I had specifically requested to review and take these through qom-next, like most qdev changes have gone lately. Why are you sending a pull nontheless? In particular Luiz has not yet replied to the QERR issue I pointed out. Because that's not what

Re: [Qemu-devel] [PULL 00/14] Cleanup qdev legacy properties

2014-02-09 Thread Igor Mammedov
On Sat, 08 Feb 2014 15:24:05 +0100 Andreas Färber wrote: > Paolo, > > Am 08.02.2014 11:01, schrieb Paolo Bonzini: > > Anthony, Peter, > > > > The following changes since commit 0169c511554cb0014a00290b0d3d26c31a49818f: > > > > Merge remote-tracking branch 'qemu-kvm/uq/master' into staging >

Re: [Qemu-devel] [PULL 00/14] Cleanup qdev legacy properties

2014-02-08 Thread Andreas Färber
Paolo, Am 08.02.2014 11:01, schrieb Paolo Bonzini: > Anthony, Peter, > > The following changes since commit 0169c511554cb0014a00290b0d3d26c31a49818f: > > Merge remote-tracking branch 'qemu-kvm/uq/master' into staging (2014-01-24 > 15:52:44 -0800) > > are available in the git repository at: >

[Qemu-devel] [PULL 00/14] Cleanup qdev legacy properties

2014-02-08 Thread Paolo Bonzini
Anthony, Peter, The following changes since commit 0169c511554cb0014a00290b0d3d26c31a49818f: Merge remote-tracking branch 'qemu-kvm/uq/master' into staging (2014-01-24 15:52:44 -0800) are available in the git repository at: git://github.com/bonzini/qemu.git qdev-props for you to fetch cha