[PATCH v5 0/5] qapi: Add detection for the 'savevm' fix for blockdev

Markus Armbruster posted 5 patches 4 years, 6 months ago
Test asan passed
Test checkpatch passed
Test FreeBSD passed
Test docker-mingw@fedora passed
Test docker-clang@ubuntu passed
Test docker-quick@centos7 passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20191018081454.21369-1-armbru@redhat.com
Maintainers: Eric Blake <eblake@redhat.com>, Markus Armbruster <armbru@redhat.com>, Michael Roth <mdroth@linux.vnet.ibm.com>
docs/devel/qapi-code-gen.txt            | 10 ++--
tests/qapi-schema/doc-good.texi         | 22 ++++++++
qapi/introspect.json                    |  6 +-
qapi/misc.json                          |  9 ++-
tests/test-qmp-cmds.c                   | 24 ++++++++
scripts/qapi/commands.py                |  3 +-
scripts/qapi/doc.py                     |  4 +-
scripts/qapi/expr.py                    | 35 +++++++-----
scripts/qapi/introspect.py              |  7 ++-
scripts/qapi/schema.py                  | 22 ++++++--
tests/qapi-schema/doc-good.json         | 17 +++++-
tests/qapi-schema/doc-good.out          |  9 ++-
tests/qapi-schema/event-case.out        |  2 +-
tests/qapi-schema/indented-expr.out     |  4 +-
tests/qapi-schema/qapi-schema-test.json | 18 ++++++
tests/qapi-schema/qapi-schema-test.out  | 75 ++++++++++++++++---------
tests/qapi-schema/test-qapi.py          | 20 ++++---
17 files changed, 219 insertions(+), 68 deletions(-)
[PATCH v5 0/5] qapi: Add detection for the 'savevm' fix for blockdev
Posted by Markus Armbruster 4 years, 6 months ago
Add 'features' field in the schema for commands and add a feature flag
to advertise that the fix for savevm [1] is present.

[1] https://lists.gnu.org/archive/html/qemu-devel/2019-09/msg03487.html

Based-on: <20191018074345.24034-1-armbru@redhat.com>

v5:
* PATCH 2:
  - qapi-code-gen.txt grammar updated
  - Doc generation for boxed commands fixed
  - Commit message tweaked
* PATCH 3:
  - Command names in qapi-schema-test.json tweaked
  - Trivial pycodestyle-3 fix
* PATCH 4: New
* PATCH 5: Whitespace tweaked

v4:
* PATCH 1: New
* PATCH 2: Factor out check_features()
* PATCH 3: Factor out _print_features(), drop duplicated test
* PATCH 4
  - Shorten savevm-blockdev-monitor-nodes to just savevm-monitor-nodes
  - Tweak commit message and documentation

Markus Armbruster (2):
  tests/qapi-schema: Tidy up test output indentation
  tests/qapi-schema: Cover feature documentation comments

Peter Krempa (3):
  qapi: Add feature flags to commands
  tests: qapi: Test 'features' of commands
  qapi: Allow introspecting fix for savevm's cooperation with blockdev

 docs/devel/qapi-code-gen.txt            | 10 ++--
 tests/qapi-schema/doc-good.texi         | 22 ++++++++
 qapi/introspect.json                    |  6 +-
 qapi/misc.json                          |  9 ++-
 tests/test-qmp-cmds.c                   | 24 ++++++++
 scripts/qapi/commands.py                |  3 +-
 scripts/qapi/doc.py                     |  4 +-
 scripts/qapi/expr.py                    | 35 +++++++-----
 scripts/qapi/introspect.py              |  7 ++-
 scripts/qapi/schema.py                  | 22 ++++++--
 tests/qapi-schema/doc-good.json         | 17 +++++-
 tests/qapi-schema/doc-good.out          |  9 ++-
 tests/qapi-schema/event-case.out        |  2 +-
 tests/qapi-schema/indented-expr.out     |  4 +-
 tests/qapi-schema/qapi-schema-test.json | 18 ++++++
 tests/qapi-schema/qapi-schema-test.out  | 75 ++++++++++++++++---------
 tests/qapi-schema/test-qapi.py          | 20 ++++---
 17 files changed, 219 insertions(+), 68 deletions(-)

-- 
2.21.0