From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55256) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZkYn2-0007wZ-RE for qemu-devel@nongnu.org; Fri, 09 Oct 2015 10:36:58 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZkYmw-0003Rf-E3 for qemu-devel@nongnu.org; Fri, 09 Oct 2015 10:36:56 -0400 Received: from mx1.redhat.com ([209.132.183.28]:55640) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZkYmw-0003RB-6e for qemu-devel@nongnu.org; Fri, 09 Oct 2015 10:36:50 -0400 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (Postfix) with ESMTPS id E3546C0C388E for ; Fri, 9 Oct 2015 14:36:49 +0000 (UTC) Received: from blackfin.pond.sub.org (ovpn-116-33.ams2.redhat.com [10.36.116.33]) by int-mx10.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id t99EalFv010181 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO) for ; Fri, 9 Oct 2015 10:36:49 -0400 From: Markus Armbruster Date: Fri, 9 Oct 2015 16:36:35 +0200 Message-Id: <1444401407-7849-1-git-send-email-armbru@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] [PULL v3 00/12] Fix device introspection regressions List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org 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-5 fix several bugs involved in the regressions, PATCH 6-8 are libqtest preliminaries, PATCH 9 adds tests to demonstrate the remaining bugs, PATCH 10-11 fix them to a degree (see PATCH 11 for limitations), and PATCH 12 cleans up. v3: * PATCH 01: new v2: * PATCH 01: new * PATCH 08,10: Drop all changes for devices "sclp" and "s390-sclp-event-facility", because they've been fixed in master The following changes since commit b37686f7e84b22cfaf7fd01ac5133f2617cc30= 27: Merge remote-tracking branch 'remotes/stefanha/tags/tracing-pull-reques= t' into staging (2015-10-09 12:18:14 +0100) are available in the git repository at: git://repo.or.cz/qemu/armbru.git tags/pull-monitor-2015-10-09 for you to fetch changes up to 33fe96833015cf15f4c0aa5bf8d34f60526e0732: Revert "qdev: Use qdev_get_device_class() for -device ,help" (201= 5-10-09 15:25:57 +0200) ---------------------------------------------------------------- Fix device introspection regressions ---------------------------------------------------------------- Markus Armbruster (9): update-linux-headers: Rename SW_MAX to SW_MAX_ virtio-input: Fix device introspection on non-Linux hosts tests: Fix how qom-test is run 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 ,help" Paolo Bonzini (3): memory: allow destroying a non-empty MemoryRegion hw: do not pass NULL to memory_region_init from instance_init macio: move DBDMA_init from instance_init to realize hw/arm/allwinner-a10.c | 6 ++ hw/arm/digic.c | 6 ++ hw/arm/fsl-imx25.c | 6 ++ hw/arm/fsl-imx31.c | 6 ++ hw/arm/pxa2xx.c | 2 +- hw/arm/xlnx-zynqmp.c | 6 ++ hw/display/cg3.c | 4 +- hw/display/tcx.c | 2 +- hw/input/Makefile.objs | 2 +- hw/misc/arm_integrator_debug.c | 2 +- hw/misc/macio/cuda.c | 2 +- hw/misc/macio/macio.c | 14 ++-- hw/pci-host/versatile.c | 11 +++ hw/pcmcia/pxa2xx.c | 6 +- hw/virtio/virtio-pci.c | 20 +++--- hw/virtio/virtio-pci.h | 4 ++ include/hw/qdev-core.h | 13 ++++ include/standard-headers/linux/input.h | 4 +- memory.c | 17 ++++- qdev-monitor.c | 9 ++- qmp.c | 11 +++ scripts/update-linux-headers.sh | 1 + target-alpha/cpu.c | 7 ++ target-arm/cpu.c | 11 +++ target-cris/cpu.c | 7 ++ target-i386/cpu.c | 8 +++ target-lm32/cpu.c | 7 ++ target-m68k/cpu.c | 7 ++ target-microblaze/cpu.c | 6 ++ target-mips/cpu.c | 7 ++ target-moxie/cpu.c | 7 ++ target-openrisc/cpu.c | 7 ++ target-ppc/kvm.c | 4 ++ target-s390x/cpu.c | 7 ++ target-sh4/cpu.c | 7 ++ target-sparc/cpu.c | 7 ++ target-tilegx/cpu.c | 7 ++ target-tricore/cpu.c | 6 ++ target-unicore32/cpu.c | 7 ++ target-xtensa/cpu.c | 7 ++ tests/Makefile | 20 +++--- tests/device-introspect-test.c | 124 +++++++++++++++++++++++++++= ++++++ tests/drive_del-test.c | 22 ++---- tests/ide-test.c | 8 +-- tests/libqtest.c | 38 +++++++++- tests/libqtest.h | 33 +++++++++ 46 files changed, 463 insertions(+), 62 deletions(-) create mode 100644 tests/device-introspect-test.c --=20 2.4.3