On Mon, Jul 21, 2025 at 04:23:14PM +0900, Akihiko Odaki wrote:
On 2025/07/18 17:52, Paolo Abeni wrote:
Extend the VirtioDeviceFeatures struct with an additional u64
to track unknown features in the 64-127 bit range and decode
the full virtio features spaces for vhost and virtio devices.

Also add entries for the soon-to-be-supported virtio net GSO over
UDP features.

Signed-off-by: Paolo Abeni <pab...@redhat.com>
---
v2 -> v3:
  - unknown-dev-features-dword2 -> unknown-dev-features2
  - _array -> _ex
  - fixed typos in entries description

v1 -> v2:
  - uint128_t -> uint64_t[]
---
 hw/virtio/virtio-hmp-cmds.c |  3 +-
 hw/virtio/virtio-qmp.c      | 89 ++++++++++++++++++++++++++-----------
 hw/virtio/virtio-qmp.h      |  3 +-
 qapi/virtio.json            |  8 +++-
 4 files changed, 73 insertions(+), 30 deletions(-)


[...]

diff --git a/qapi/virtio.json b/qapi/virtio.json
index 9d652fe4a8..f2e2dd6e97 100644
--- a/qapi/virtio.json
+++ b/qapi/virtio.json
@@ -490,14 +490,18 @@
 #     unique features)
 #
 # @unknown-dev-features: Virtio device features bitmap that have not
-#     been decoded
+#     been decoded (bits 0-63)
+#
+# @unknown-dev-features2: Virtio device features bitmap that have not
+#     been decoded (bits 64-127)

This documentation should contain "(since 10.1)" as described in:
docs/devel/qapi-code-gen.rst

Good catch!

BTW 10.1 is already in soft-freeze, so I guess this will land in the next cycle, so I'd suggest to use 10.2

Thanks,
Stefano


Reply via email to