[Qemu-devel] [PATCH 0/3] qdev: fix the order compat and global properties are applied

Eduardo Habkost posted 3 patches 8 years, 3 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170711004303.3902-1-ehabkost@redhat.com
Test FreeBSD passed
Test checkpatch passed
Test docker passed
Test s390x passed
hw/core/machine.c              | 26 +++-----------------------
hw/core/qdev-properties.c      | 15 ++-------------
tests/test-qdev-global-props.c | 33 +++++++++++++++++++++++++++++++++
3 files changed, 38 insertions(+), 36 deletions(-)
[Qemu-devel] [PATCH 0/3] qdev: fix the order compat and global properties are applied
Posted by Eduardo Habkost 8 years, 3 months ago
Before 2.8 was released, we found a bug in the way global
properties are applied by device code.  Greg Kurz sent a fix[1],
but we decide to include a more conservative workaround[2]
because the 2.8 release was very close.

The plan was to include Greg's patch in 2.9, but we forgot to do
that.  I'm now resending: Greg's original patch; a test case to
detect the original bug; and a revert of the workaround we
included in 2.8.

[1] https://www.mail-archive.com/qemu-devel@nongnu.org/msg416944.html
[2] commit 0bcba41fe379e4c6834adcf1456d9099db31a5b2
    "machine: Convert abstract typename on compat_props to subclass names"

Eduardo Habkost (2):
  test-qdev-global-props: Test global property ordering
  Revert "machine: Convert abstract typename on compat_props to subclass
    names"

Greg Kurz (1):
  qdev: fix the order compat and global properties are applied

 hw/core/machine.c              | 26 +++-----------------------
 hw/core/qdev-properties.c      | 15 ++-------------
 tests/test-qdev-global-props.c | 33 +++++++++++++++++++++++++++++++++
 3 files changed, 38 insertions(+), 36 deletions(-)

-- 
2.9.4