Re: [Qemu-devel] [PULL 00/16] KVM, QOM, NBD, build fixes for 2015-05-08

2015-05-12 Thread Daniel P. Berrange
On Fri, May 08, 2015 at 02:29:01PM +0200, Andreas Färber wrote:
 Am 08.05.2015 um 14:07 schrieb Paolo Bonzini:
  The following changes since commit 498147529d1f8e902e6528a0115143b53475791e:
  
Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20150430' into 
  staging (2015-04-30 14:15:56 +0100)
  
  are available in the git repository at:
  
git://github.com/bonzini/qemu.git tags/for-upstream
  
  for you to fetch changes up to d51026b22b97332a95d91acfb6c23cd9b087955c:
  
qemu-nbd: only send a limited number of errno codes on the wire 
  (2015-05-08 13:14:54 +0200)
  
  
  - Daniel's QOM improvements
 
 Once again, objection.

Paolo, I'll re-send a v4 of these QOM related improvements that
incorporate Andreas' feedback.


  - build bugfix from Fam and new configure check from Emilio
  - two improvements to info mtere from Gerd
  - KVM support for memory transaction attributes
  - one more small step towards unlocked MMIO dispatch
  - one piece of the qemu-nbd errno fixes
  - trivial-ish patches from Denis and Thomas

  
  
  Daniel P. Berrange (7):
qom: fix typename of 'policy' enum property in hostmem obj
qom: document user creatable object types in help text
qom: create objects in two phases
qom: add object_new_propv / object_new_proplist constructors
qom: make enum string tables const-correct
qom: add a object_property_add_enum helper method
qom: don't pass string table to object_get_enum method

Regards,
Daniel
-- 
|: http://berrange.com  -o-http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org  -o- http://virt-manager.org :|
|: http://autobuild.org   -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org   -o-   http://live.gnome.org/gtk-vnc :|



Re: [Qemu-devel] [PULL 00/16] KVM, QOM, NBD, build fixes for 2015-05-08

2015-05-08 Thread Andreas Färber
Am 08.05.2015 um 14:07 schrieb Paolo Bonzini:
 The following changes since commit 498147529d1f8e902e6528a0115143b53475791e:
 
   Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20150430' into 
 staging (2015-04-30 14:15:56 +0100)
 
 are available in the git repository at:
 
   git://github.com/bonzini/qemu.git tags/for-upstream
 
 for you to fetch changes up to d51026b22b97332a95d91acfb6c23cd9b087955c:
 
   qemu-nbd: only send a limited number of errno codes on the wire (2015-05-08 
 13:14:54 +0200)
 
 
 - Daniel's QOM improvements

Once again, objection.

Andreas

 - build bugfix from Fam and new configure check from Emilio
 - two improvements to info mtere from Gerd
 - KVM support for memory transaction attributes
 - one more small step towards unlocked MMIO dispatch
 - one piece of the qemu-nbd errno fixes
 - trivial-ish patches from Denis and Thomas
 
 
 Daniel P. Berrange (7):
   qom: fix typename of 'policy' enum property in hostmem obj
   qom: document user creatable object types in help text
   qom: create objects in two phases
   qom: add object_new_propv / object_new_proplist constructors
   qom: make enum string tables const-correct
   qom: add a object_property_add_enum helper method
   qom: don't pass string table to object_get_enum method
 
 Denis V. Lunev (1):
   apic_common: improve readability of apic_reset_common
 
 Emilio G. Cota (1):
   configure: require __thread support
 
 Fam Zheng (1):
   rules.mak: Force CFLAGS for all objects in DSO
 
 Gerd Hoffmann (2):
   mtree: tag  indent a bit better
   mtree: also print disabled regions
 
 Paolo Bonzini (3):
   kvm: add support for memory transaction attributes
   exec: move rcu_read_lock/unlock to address_space_translate callers
   qemu-nbd: only send a limited number of errno codes on the wire
 
 Thomas Huth (1):
   kvm: Silence warning from valgrind
 
  backends/hostmem.c  |  22 ++--
  configure   |  11 ++
  exec.c  |  33 -
  hw/intc/apic_common.c   |   7 +-
  hw/vfio/common.c|   7 +-
  include/exec/memory.h   |   4 +-
  include/hw/qdev-core.h  |   2 +-
  include/qapi/util.h |   2 +-
  include/qapi/visitor-impl.h |   6 +-
  include/qapi/visitor.h  |   2 +-
  include/qom/object.h|  90 -
  include/sysemu/kvm.h|   3 +-
  kvm-all.c   |  35 ++---
  memory.c|  23 ++--
  nbd.c   |  52 
  numa.c  |   2 +-
  qapi/qapi-dealloc-visitor.c |   3 +-
  qapi/qapi-util.c|   2 +-
  qapi/qapi-visit-core.c  |   6 +-
  qemu-options.hx |  70 +++---
  qom/object.c| 144 +++-
  rules.mak   |   4 +-
  scripts/qapi-types.py   |   4 +-
  target-arm/kvm.c|   4 +-
  target-i386/kvm.c   |   4 +-
  target-mips/kvm.c   |   4 +-
  target-ppc/kvm.c|   4 +-
  target-s390x/kvm.c  |   4 +-
  tests/.gitignore|   1 +
  tests/Makefile  |   5 +-
  tests/check-qom-proplist.c  | 310 
 
  translate-all.c |   3 +
  vl.c|  40 +-
  33 files changed, 819 insertions(+), 94 deletions(-)
  create mode 100644 tests/check-qom-proplist.c
 


-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Jennifer Guild, Dilip Upmanyu,
Graham Norton; HRB 21284 (AG Nürnberg)



Re: [Qemu-devel] [PULL 00/16] KVM, QOM, NBD, build fixes for 2015-05-08

2015-05-08 Thread Daniel P. Berrange
On Fri, May 08, 2015 at 02:29:01PM +0200, Andreas Färber wrote:
 Am 08.05.2015 um 14:07 schrieb Paolo Bonzini:
  The following changes since commit 498147529d1f8e902e6528a0115143b53475791e:
  
Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20150430' into 
  staging (2015-04-30 14:15:56 +0100)
  
  are available in the git repository at:
  
git://github.com/bonzini/qemu.git tags/for-upstream
  
  for you to fetch changes up to d51026b22b97332a95d91acfb6c23cd9b087955c:
  
qemu-nbd: only send a limited number of errno codes on the wire 
  (2015-05-08 13:14:54 +0200)
  
  
  - Daniel's QOM improvements
 
 Once again, objection.

If you have any specific technical comments I'm happy to respond to them,
but I've not seen any further negative feedback on the patches since last
time I posted them.

Regards,
Daniel
-- 
|: http://berrange.com  -o-http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org  -o- http://virt-manager.org :|
|: http://autobuild.org   -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org   -o-   http://live.gnome.org/gtk-vnc :|



Re: [Qemu-devel] [PULL 00/16] KVM, QOM, NBD, build fixes for 2015-05-08

2015-05-08 Thread Andreas Färber
Am 08.05.2015 um 14:34 schrieb Daniel P. Berrange:
 On Fri, May 08, 2015 at 02:29:01PM +0200, Andreas Färber wrote:
 Am 08.05.2015 um 14:07 schrieb Paolo Bonzini:
 The following changes since commit 498147529d1f8e902e6528a0115143b53475791e:

   Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20150430' into 
 staging (2015-04-30 14:15:56 +0100)

 are available in the git repository at:

   git://github.com/bonzini/qemu.git tags/for-upstream

 for you to fetch changes up to d51026b22b97332a95d91acfb6c23cd9b087955c:

   qemu-nbd: only send a limited number of errno codes on the wire 
 (2015-05-08 13:14:54 +0200)

 
 - Daniel's QOM improvements

 Once again, objection.
 
 If you have any specific technical comments I'm happy to respond to them,
 but I've not seen any further negative feedback on the patches since last
 time I posted them.

The point is, I am still the QOM maintainer, so I need to review them,
not just not-negatively-comment. I have two QOM patches from Eduardo in
my queue already. And unless I'm totally mistaken, I was only CC'ed on
the very last v3, not any previous versions.

I will take a look at them shortly and, assuming everything is fine and
no conflicts, they can then go through Paolo's queue with my Ab/Rb.

Regards,
Andreas

-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Jennifer Guild, Dilip Upmanyu,
Graham Norton; HRB 21284 (AG Nürnberg)



Re: [Qemu-devel] [PULL 00/16] KVM, QOM, NBD, build fixes for 2015-05-08

2015-05-08 Thread Daniel P. Berrange
On Fri, May 08, 2015 at 02:39:57PM +0200, Andreas Färber wrote:
 Am 08.05.2015 um 14:34 schrieb Daniel P. Berrange:
  On Fri, May 08, 2015 at 02:29:01PM +0200, Andreas Färber wrote:
  Am 08.05.2015 um 14:07 schrieb Paolo Bonzini:
  The following changes since commit 
  498147529d1f8e902e6528a0115143b53475791e:
 
Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20150430' into 
  staging (2015-04-30 14:15:56 +0100)
 
  are available in the git repository at:
 
git://github.com/bonzini/qemu.git tags/for-upstream
 
  for you to fetch changes up to d51026b22b97332a95d91acfb6c23cd9b087955c:
 
qemu-nbd: only send a limited number of errno codes on the wire 
  (2015-05-08 13:14:54 +0200)
 
  
  - Daniel's QOM improvements
 
  Once again, objection.
  
  If you have any specific technical comments I'm happy to respond to them,
  but I've not seen any further negative feedback on the patches since last
  time I posted them.
 
 The point is, I am still the QOM maintainer, so I need to review them,
 not just not-negatively-comment. I have two QOM patches from Eduardo in
 my queue already. And unless I'm totally mistaken, I was only CC'ed on
 the very last v3, not any previous versions.

I did copy you on the v2 a couple of days before, but I had indeed left
you off the v1 - it was part of a much larger RFC series which I hadn't
copied all the subsystem maintainers on.

Regards,
Daniel
-- 
|: http://berrange.com  -o-http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org  -o- http://virt-manager.org :|
|: http://autobuild.org   -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org   -o-   http://live.gnome.org/gtk-vnc :|



[Qemu-devel] [PULL 00/16] KVM, QOM, NBD, build fixes for 2015-05-08

2015-05-08 Thread Paolo Bonzini
The following changes since commit 498147529d1f8e902e6528a0115143b53475791e:

  Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20150430' into 
staging (2015-04-30 14:15:56 +0100)

are available in the git repository at:

  git://github.com/bonzini/qemu.git tags/for-upstream

for you to fetch changes up to d51026b22b97332a95d91acfb6c23cd9b087955c:

  qemu-nbd: only send a limited number of errno codes on the wire (2015-05-08 
13:14:54 +0200)


- Daniel's QOM improvements
- build bugfix from Fam and new configure check from Emilio
- two improvements to info mtere from Gerd
- KVM support for memory transaction attributes
- one more small step towards unlocked MMIO dispatch
- one piece of the qemu-nbd errno fixes
- trivial-ish patches from Denis and Thomas


Daniel P. Berrange (7):
  qom: fix typename of 'policy' enum property in hostmem obj
  qom: document user creatable object types in help text
  qom: create objects in two phases
  qom: add object_new_propv / object_new_proplist constructors
  qom: make enum string tables const-correct
  qom: add a object_property_add_enum helper method
  qom: don't pass string table to object_get_enum method

Denis V. Lunev (1):
  apic_common: improve readability of apic_reset_common

Emilio G. Cota (1):
  configure: require __thread support

Fam Zheng (1):
  rules.mak: Force CFLAGS for all objects in DSO

Gerd Hoffmann (2):
  mtree: tag  indent a bit better
  mtree: also print disabled regions

Paolo Bonzini (3):
  kvm: add support for memory transaction attributes
  exec: move rcu_read_lock/unlock to address_space_translate callers
  qemu-nbd: only send a limited number of errno codes on the wire

Thomas Huth (1):
  kvm: Silence warning from valgrind

 backends/hostmem.c  |  22 ++--
 configure   |  11 ++
 exec.c  |  33 -
 hw/intc/apic_common.c   |   7 +-
 hw/vfio/common.c|   7 +-
 include/exec/memory.h   |   4 +-
 include/hw/qdev-core.h  |   2 +-
 include/qapi/util.h |   2 +-
 include/qapi/visitor-impl.h |   6 +-
 include/qapi/visitor.h  |   2 +-
 include/qom/object.h|  90 -
 include/sysemu/kvm.h|   3 +-
 kvm-all.c   |  35 ++---
 memory.c|  23 ++--
 nbd.c   |  52 
 numa.c  |   2 +-
 qapi/qapi-dealloc-visitor.c |   3 +-
 qapi/qapi-util.c|   2 +-
 qapi/qapi-visit-core.c  |   6 +-
 qemu-options.hx |  70 +++---
 qom/object.c| 144 +++-
 rules.mak   |   4 +-
 scripts/qapi-types.py   |   4 +-
 target-arm/kvm.c|   4 +-
 target-i386/kvm.c   |   4 +-
 target-mips/kvm.c   |   4 +-
 target-ppc/kvm.c|   4 +-
 target-s390x/kvm.c  |   4 +-
 tests/.gitignore|   1 +
 tests/Makefile  |   5 +-
 tests/check-qom-proplist.c  | 310 
 translate-all.c |   3 +
 vl.c|  40 +-
 33 files changed, 819 insertions(+), 94 deletions(-)
 create mode 100644 tests/check-qom-proplist.c
-- 
2.3.5