Skip to content

RFC device-crash-test: device_add test mode

Eduardo Habkost qemu-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 RFC Latest6 patches 2 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6device-crash-test: Make whitelist code a bit more flexiblescripts/device-crash-test+3 -20 replies
2/6device-crash-test: Log detailed info on success tooscripts/device-crash-test+29 -320 replies
3/6device-crash-test: Allow checkOneCase() to report multiple resultsscripts/device-crash-test+14 -150 replies
4/6device-crash-test: Exit immediately on fatal failures on quick modescripts/device-crash-test+4 -00 replies
5/6device-crash-test: Basic device_add supportscripts/device-crash-test+33 -50 replies
6/6device-crash-test: Multi-device device_add testscripts/device-crash-test+34 -72 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [Qemu-devel] [RFC 0/6] device-crash-test: device_add test mode

This series implements device_add testing on device-crash-test. The new test mode will be enabled by default in addition to the old test cases, but if only device_add testing is wanted, it can be specified using the '-t' command-line option, e.g.:

  $ ./scripts/device-crash-test -t method=device_add

In comparison to the "-device" test cases, a full test run runs reasonably quickly (17 seconds on my machine for testing all hotpluggable devices on all 72 machine-types in qemu-system-x86_64). I believe it is a good candidate to be eventually included on "make check".

TODO:
* Fix the QEMU crash found using '-t plug_all=1' (see patch 5/5 for details)
* Test non-x86 architectures
* Compare results with Thomas' HMP unit test ("tests: Add a device_add/del HMP test" patch").

Based-on: [email protected] (Subject: [PATCH 0/4] qmp: query-device-type command)

Eduardo Habkost (6):
device-crash-test: Make whitelist code a bit more flexible
device-crash-test: Log detailed info on success too
device-crash-test: Allow checkOneCase() to report multiple results
device-crash-test: Exit immediately on fatal failures on quick mode
device-crash-test: Basic device_add support
device-crash-test: Multi-device device_add test

 scripts/device-crash-test | 132 +++++++++++++++++++++++++++++++++-------------
 1 file changed, 94 insertions(+), 38 deletions(-)
Signature · 1 line
2.13.5