From: Markus Armbruster <armbru@redhat.com>
To: qemu-devel@nongnu.org
Cc: afaerber@suse.de, stefanha@redhat.com, ehabkost@redhat.com
Subject: [Qemu-devel] [PATCH 0/7] Fix device introspection regressions
Date: Fri, 18 Sep 2015 14:00:33 +0200 [thread overview]
Message-ID: <1442577640-11612-1-git-send-email-armbru@redhat.com> (raw)
QMP command device-list-properties regressed in 2.1: it can crash or
leave dangling pointers behind.
-device FOO,help regressed in 2.2: it no longer works for
non-pluggable devices. I tried to fix that some time ago[*], but my
fix failed review. This is my second, more comprehensive try.
PATCH 1,2 are preliminaries, PATCH 3 adds tests to demonstrate the
bugs, PATCH 4-6 fix them to a degree (see PATCH 5 for limitations),
and PATCH 7 cleans up.
[*] [PATCH] qdev: Make -device FOO,help help again when FOO is not
pluggable
https://lists.gnu.org/archive/html/qemu-devel/2015-03/msg03459.html
Message-Id: <1426527232-15044-1-git-send-email-armbru@redhat.com>
Markus Armbruster (7):
libqtest: Clean up unused QTestState member sigact_old
libqtest: New hmp() & friends
device-introspect-test: New, covering device introspection
qmp: Fix device-list-properties not to crash for abstract device
qdev: Protect device-list-properties against broken devices
Revert "qdev: Use qdev_get_device_class() for -device <type>,help"
tests: Simplify how qom-test is run
hw/arm/allwinner-a10.c | 2 +
hw/arm/digic.c | 2 +
hw/arm/fsl-imx25.c | 2 +
hw/arm/fsl-imx31.c | 2 +
hw/arm/xlnx-zynqmp.c | 2 +
hw/pci-host/versatile.c | 11 ++++
hw/pcmcia/pxa2xx.c | 9 ++++
hw/s390x/event-facility.c | 3 ++
hw/s390x/sclp.c | 3 ++
include/hw/qdev-core.h | 13 +++++
qdev-monitor.c | 9 ++--
qmp.c | 11 ++++
qom/cpu.c | 2 +
target-i386/cpu.c | 2 +
target-ppc/kvm.c | 4 ++
tests/Makefile | 16 +++---
tests/device-introspect-test.c | 117 +++++++++++++++++++++++++++++++++++++++++
tests/drive_del-test.c | 22 +++-----
tests/ide-test.c | 8 +--
tests/libqtest.c | 36 ++++++++++++-
tests/libqtest.h | 33 ++++++++++++
21 files changed, 276 insertions(+), 33 deletions(-)
create mode 100644 tests/device-introspect-test.c
--
2.4.3
next reply other threads:[~2015-09-18 12:00 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-18 12:00 Markus Armbruster [this message]
2015-09-18 12:00 ` [Qemu-devel] [PATCH 1/7] libqtest: Clean up unused QTestState member sigact_old Markus Armbruster
2015-09-18 15:36 ` Eric Blake
2015-09-18 12:00 ` [Qemu-devel] [PATCH 2/7] libqtest: New hmp() & friends Markus Armbruster
2015-09-18 15:47 ` Eric Blake
2015-09-21 5:59 ` Markus Armbruster
2015-09-18 12:00 ` [Qemu-devel] [PATCH 3/7] device-introspect-test: New, covering device introspection Markus Armbruster
2015-09-18 15:55 ` Eric Blake
2015-09-21 6:05 ` Markus Armbruster
2015-09-18 12:00 ` [Qemu-devel] [PATCH 4/7] qmp: Fix device-list-properties not to crash for abstract device Markus Armbruster
2015-09-18 15:58 ` Eric Blake
2015-09-18 12:00 ` [Qemu-devel] [PATCH 5/7] qdev: Protect device-list-properties against broken devices Markus Armbruster
2015-09-18 12:38 ` Christian Borntraeger
2015-09-21 8:30 ` David Hildenbrand
2015-09-21 15:38 ` Eduardo Habkost
2015-09-22 8:02 ` David Hildenbrand
2015-09-22 8:07 ` Markus Armbruster
2015-09-18 16:09 ` Eric Blake
2015-09-21 6:08 ` Markus Armbruster
2015-09-18 16:36 ` Eduardo Habkost
2015-09-21 6:09 ` Markus Armbruster
2015-09-21 15:13 ` Eduardo Habkost
2015-09-18 18:42 ` Thomas Huth
2015-09-18 19:32 ` Eduardo Habkost
2015-09-21 6:14 ` Markus Armbruster
2015-09-21 15:20 ` Eduardo Habkost
2015-09-21 15:48 ` Thomas Huth
2015-09-21 16:39 ` Markus Armbruster
2015-09-21 17:22 ` Thomas Huth
2015-09-21 18:19 ` Eduardo Habkost
2015-09-18 12:00 ` [Qemu-devel] [PATCH 6/7] Revert "qdev: Use qdev_get_device_class() for -device <type>, help" Markus Armbruster
2015-09-18 16:13 ` Eric Blake
2015-09-18 12:00 ` [Qemu-devel] [PATCH 7/7] tests: Simplify how qom-test is run Markus Armbruster
2015-09-18 12:53 ` Andreas Färber
2015-09-18 14:24 ` Markus Armbruster
2015-09-18 15:28 ` Andreas Färber
2015-09-21 6:15 ` Markus Armbruster
2015-09-23 13:57 ` Markus Armbruster
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1442577640-11612-1-git-send-email-armbru@redhat.com \
--to=armbru@redhat.com \
--cc=afaerber@suse.de \
--cc=ehabkost@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).