* [PATCH v2 0/2] Adjust the output of x-query-virtio-status @ 2024-01-04 16:19 Hyman Huang 2024-01-04 16:19 ` [PATCH v2 1/2] qapi/virtio: Keep feature and status bits in the QMP output Hyman Huang 2024-01-04 16:19 ` [PATCH v2 2/2] hmp: Drop unknown feature and status bits Hyman Huang 0 siblings, 2 replies; 7+ messages in thread From: Hyman Huang @ 2024-01-04 16:19 UTC (permalink / raw) To: qemu-devel Cc: Michael S . Tsirkin, Markus Armbruster, Philippe Mathieu-Daudé, Eric Blake, yong.huang v2: - Changing the hmp_virtio_dump_xxx function signatures to implement the bitmap decoding, suggested by Philippe. Please review, thanks, Yong This patchset is derived from the series: https://lore.kernel.org/qemu-devel/cover.1699793550.git.yong.huang@smartx.com/ Please go to the link to see more background information. The following points are what we have done in the patchset: 1. Take the policy of adding human-readable output just in HMP. 2. For the HMP output, display the human-readable information and drop the unknown bits in practice. 3. For the QMP output, remove the descriptive strings and only display bits encoded as numbers. Hyman Huang (2): qapi/virtio: Keep feature and status bits in the QMP output hmp: Drop unknown feature and status bits hw/virtio/virtio-hmp-cmds.c | 42 ++++---- hw/virtio/virtio-qmp.c | 23 ++--- qapi/virtio.json | 192 ++++-------------------------------- 3 files changed, 48 insertions(+), 209 deletions(-) -- 2.39.1 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2 1/2] qapi/virtio: Keep feature and status bits in the QMP output 2024-01-04 16:19 [PATCH v2 0/2] Adjust the output of x-query-virtio-status Hyman Huang @ 2024-01-04 16:19 ` Hyman Huang 2024-01-12 14:10 ` Markus Armbruster 2024-01-04 16:19 ` [PATCH v2 2/2] hmp: Drop unknown feature and status bits Hyman Huang 1 sibling, 1 reply; 7+ messages in thread From: Hyman Huang @ 2024-01-04 16:19 UTC (permalink / raw) To: qemu-devel Cc: Michael S . Tsirkin, Markus Armbruster, Philippe Mathieu-Daudé, Eric Blake, yong.huang Maintain the feature and status bits in the x-query-virtio-status output and, as usual, add human-readable output only in HMP. Applications may find it useful to compare features and status information directly. An upper application, for example, could use the QMP command x-query-virtio-status to retrieve vhost-user net device features and the "ovs-vsctl list interface" command to retrieve interface features (in number format) in order to verify the correctness of the virtio negotiation between guest, QEMU, and OVS-DPDK. The application could then compare the two features directly, without the need for additional feature encoding. Signed-off-by: Hyman Huang <yong.huang@smartx.com> --- hw/virtio/virtio-hmp-cmds.c | 29 ++++-- hw/virtio/virtio-qmp.c | 23 ++--- qapi/virtio.json | 192 ++++-------------------------------- 3 files changed, 48 insertions(+), 196 deletions(-) diff --git a/hw/virtio/virtio-hmp-cmds.c b/hw/virtio/virtio-hmp-cmds.c index 477c97dea2..4fabba4f9c 100644 --- a/hw/virtio/virtio-hmp-cmds.c +++ b/hw/virtio/virtio-hmp-cmds.c @@ -6,6 +6,7 @@ */ #include "qemu/osdep.h" +#include "virtio-qmp.h" #include "monitor/hmp.h" #include "monitor/monitor.h" #include "qapi/qapi-commands-virtio.h" @@ -13,8 +14,10 @@ static void hmp_virtio_dump_protocols(Monitor *mon, - VhostDeviceProtocols *pcol) + uint64_t bitmap) { + VhostDeviceProtocols *pcol = qmp_decode_protocols(bitmap); + strList *pcol_list = pcol->protocols; while (pcol_list) { monitor_printf(mon, "\t%s", pcol_list->value); @@ -31,8 +34,10 @@ static void hmp_virtio_dump_protocols(Monitor *mon, } static void hmp_virtio_dump_status(Monitor *mon, - VirtioDeviceStatus *status) + uint64_t bitmap) { + VirtioDeviceStatus *status = qmp_decode_status(bitmap); + strList *status_list = status->statuses; while (status_list) { monitor_printf(mon, "\t%s", status_list->value); @@ -49,8 +54,12 @@ static void hmp_virtio_dump_status(Monitor *mon, } static void hmp_virtio_dump_features(Monitor *mon, - VirtioDeviceFeatures *features) + uint16_t device_id, + uint64_t bitmap) { + VirtioDeviceFeatures *features = + qmp_decode_features(device_id, bitmap); + strList *transport_list = features->transports; while (transport_list) { monitor_printf(mon, "\t%s", transport_list->value); @@ -147,11 +156,11 @@ void hmp_virtio_status(Monitor *mon, const QDict *qdict) monitor_printf(mon, " status:\n"); hmp_virtio_dump_status(mon, s->status); monitor_printf(mon, " Guest features:\n"); - hmp_virtio_dump_features(mon, s->guest_features); + hmp_virtio_dump_features(mon, s->device_id, s->guest_features); monitor_printf(mon, " Host features:\n"); - hmp_virtio_dump_features(mon, s->host_features); + hmp_virtio_dump_features(mon, s->device_id, s->host_features); monitor_printf(mon, " Backend features:\n"); - hmp_virtio_dump_features(mon, s->backend_features); + hmp_virtio_dump_features(mon, s->device_id, s->backend_features); if (s->vhost_dev) { monitor_printf(mon, " VHost:\n"); @@ -172,11 +181,13 @@ void hmp_virtio_status(Monitor *mon, const QDict *qdict) monitor_printf(mon, " log_size: %"PRId64"\n", s->vhost_dev->log_size); monitor_printf(mon, " Features:\n"); - hmp_virtio_dump_features(mon, s->vhost_dev->features); + hmp_virtio_dump_features(mon, s->device_id, s->vhost_dev->features); monitor_printf(mon, " Acked features:\n"); - hmp_virtio_dump_features(mon, s->vhost_dev->acked_features); + hmp_virtio_dump_features(mon, + s->device_id, s->vhost_dev->acked_features); monitor_printf(mon, " Backend features:\n"); - hmp_virtio_dump_features(mon, s->vhost_dev->backend_features); + hmp_virtio_dump_features(mon, + s->device_id, s->vhost_dev->backend_features); monitor_printf(mon, " Protocol features:\n"); hmp_virtio_dump_protocols(mon, s->vhost_dev->protocol_features); } diff --git a/hw/virtio/virtio-qmp.c b/hw/virtio/virtio-qmp.c index 1dd96ed20f..1660c17653 100644 --- a/hw/virtio/virtio-qmp.c +++ b/hw/virtio/virtio-qmp.c @@ -733,12 +733,9 @@ VirtioStatus *qmp_x_query_virtio_status(const char *path, Error **errp) status->name = g_strdup(vdev->name); status->device_id = vdev->device_id; status->vhost_started = vdev->vhost_started; - status->guest_features = qmp_decode_features(vdev->device_id, - vdev->guest_features); - status->host_features = qmp_decode_features(vdev->device_id, - vdev->host_features); - status->backend_features = qmp_decode_features(vdev->device_id, - vdev->backend_features); + status->guest_features = vdev->guest_features; + status->host_features = vdev->host_features; + status->backend_features = vdev->backend_features; switch (vdev->device_endian) { case VIRTIO_DEVICE_ENDIAN_LITTLE: @@ -753,7 +750,7 @@ VirtioStatus *qmp_x_query_virtio_status(const char *path, Error **errp) } status->num_vqs = virtio_get_num_queues(vdev); - status->status = qmp_decode_status(vdev->status); + status->status = vdev->status; status->isr = vdev->isr; status->queue_sel = vdev->queue_sel; status->vm_running = vdev->vm_running; @@ -775,14 +772,10 @@ VirtioStatus *qmp_x_query_virtio_status(const char *path, Error **errp) status->vhost_dev->n_tmp_sections = hdev->n_tmp_sections; status->vhost_dev->nvqs = hdev->nvqs; status->vhost_dev->vq_index = hdev->vq_index; - status->vhost_dev->features = - qmp_decode_features(vdev->device_id, hdev->features); - status->vhost_dev->acked_features = - qmp_decode_features(vdev->device_id, hdev->acked_features); - status->vhost_dev->backend_features = - qmp_decode_features(vdev->device_id, hdev->backend_features); - status->vhost_dev->protocol_features = - qmp_decode_protocols(hdev->protocol_features); + status->vhost_dev->features = hdev->features; + status->vhost_dev->acked_features = hdev->acked_features; + status->vhost_dev->backend_features = hdev->backend_features; + status->vhost_dev->protocol_features = hdev->protocol_features; status->vhost_dev->max_queues = hdev->max_queues; status->vhost_dev->backend_cap = hdev->backend_cap; status->vhost_dev->log_enabled = hdev->log_enabled; diff --git a/qapi/virtio.json b/qapi/virtio.json index 19c7c36e36..26516fb29c 100644 --- a/qapi/virtio.json +++ b/qapi/virtio.json @@ -102,10 +102,10 @@ 'n-tmp-sections': 'int', 'nvqs': 'uint32', 'vq-index': 'int', - 'features': 'VirtioDeviceFeatures', - 'acked-features': 'VirtioDeviceFeatures', - 'backend-features': 'VirtioDeviceFeatures', - 'protocol-features': 'VhostDeviceProtocols', + 'features': 'uint64', + 'acked-features': 'uint64', + 'backend-features': 'uint64', + 'protocol-features': 'uint64', 'max-queues': 'uint64', 'backend-cap': 'uint64', 'log-enabled': 'bool', @@ -170,11 +170,11 @@ 'device-id': 'uint16', 'vhost-started': 'bool', 'device-endian': 'str', - 'guest-features': 'VirtioDeviceFeatures', - 'host-features': 'VirtioDeviceFeatures', - 'backend-features': 'VirtioDeviceFeatures', + 'guest-features': 'uint64', + 'host-features': 'uint64', + 'backend-features': 'uint64', 'num-vqs': 'int', - 'status': 'VirtioDeviceStatus', + 'status': 'uint8', 'isr': 'uint8', 'queue-sel': 'uint16', 'vm-running': 'bool', @@ -217,41 +217,14 @@ # "name": "virtio-crypto", # "started": true, # "device-id": 20, -# "backend-features": { -# "transports": [], -# "dev-features": [] -# }, +# "backend-features": 0, # "start-on-kick": false, # "isr": 1, # "broken": false, -# "status": { -# "statuses": [ -# "VIRTIO_CONFIG_S_ACKNOWLEDGE: Valid virtio device found", -# "VIRTIO_CONFIG_S_DRIVER: Guest OS compatible with device", -# "VIRTIO_CONFIG_S_FEATURES_OK: Feature negotiation complete", -# "VIRTIO_CONFIG_S_DRIVER_OK: Driver setup and ready" -# ] -# }, +# "status": 15, # "num-vqs": 2, -# "guest-features": { -# "dev-features": [], -# "transports": [ -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields enabled", -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors supported", -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec (legacy)" -# ] -# }, -# "host-features": { -# "unknown-dev-features": 1073741824, -# "dev-features": [], -# "transports": [ -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields enabled", -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors supported", -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec (legacy)", -# "VIRTIO_F_ANY_LAYOUT: Device accepts arbitrary desc. layouts", -# "VIRTIO_F_NOTIFY_ON_EMPTY: Notify when device runs out of avail. descs. on VQ" -# ] -# }, +# "guest-features": 5100273664, +# "host-features": 6325010432, # "use-guest-notifier-mask": true, # "vm-running": true, # "queue-sel": 1, @@ -279,147 +252,22 @@ # "max-queues": 1, # "backend-cap": 2, # "log-size": 0, -# "backend-features": { -# "dev-features": [], -# "transports": [] -# }, +# "backend-features": 0, # "nvqs": 2, -# "protocol-features": { -# "protocols": [] -# }, +# "protocol-features": 0, # "vq-index": 0, # "log-enabled": false, -# "acked-features": { -# "dev-features": [ -# "VIRTIO_NET_F_MRG_RXBUF: Driver can merge receive buffers" -# ], -# "transports": [ -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields enabled", -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors supported", -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec (legacy)" -# ] -# }, -# "features": { -# "dev-features": [ -# "VHOST_F_LOG_ALL: Logging write descriptors supported", -# "VIRTIO_NET_F_MRG_RXBUF: Driver can merge receive buffers" -# ], -# "transports": [ -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields enabled", -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors supported", -# "VIRTIO_F_IOMMU_PLATFORM: Device can be used on IOMMU platform", -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec (legacy)", -# "VIRTIO_F_ANY_LAYOUT: Device accepts arbitrary desc. layouts", -# "VIRTIO_F_NOTIFY_ON_EMPTY: Notify when device runs out of avail. descs. on VQ" -# ] -# } -# }, -# "backend-features": { -# "dev-features": [ -# "VHOST_USER_F_PROTOCOL_FEATURES: Vhost-user protocol features negotiation supported", -# "VIRTIO_NET_F_GSO: Handling GSO-type packets supported", -# "VIRTIO_NET_F_CTRL_MAC_ADDR: MAC address set through control channel", -# "VIRTIO_NET_F_GUEST_ANNOUNCE: Driver sending gratuitous packets supported", -# "VIRTIO_NET_F_CTRL_RX_EXTRA: Extra RX mode control supported", -# "VIRTIO_NET_F_CTRL_VLAN: Control channel VLAN filtering supported", -# "VIRTIO_NET_F_CTRL_RX: Control channel RX mode supported", -# "VIRTIO_NET_F_CTRL_VQ: Control channel available", -# "VIRTIO_NET_F_STATUS: Configuration status field available", -# "VIRTIO_NET_F_MRG_RXBUF: Driver can merge receive buffers", -# "VIRTIO_NET_F_HOST_UFO: Device can receive UFO", -# "VIRTIO_NET_F_HOST_ECN: Device can receive TSO with ECN", -# "VIRTIO_NET_F_HOST_TSO6: Device can receive TSOv6", -# "VIRTIO_NET_F_HOST_TSO4: Device can receive TSOv4", -# "VIRTIO_NET_F_GUEST_UFO: Driver can receive UFO", -# "VIRTIO_NET_F_GUEST_ECN: Driver can receive TSO with ECN", -# "VIRTIO_NET_F_GUEST_TSO6: Driver can receive TSOv6", -# "VIRTIO_NET_F_GUEST_TSO4: Driver can receive TSOv4", -# "VIRTIO_NET_F_MAC: Device has given MAC address", -# "VIRTIO_NET_F_CTRL_GUEST_OFFLOADS: Control channel offloading reconfig. supported", -# "VIRTIO_NET_F_GUEST_CSUM: Driver handling packets with partial checksum supported", -# "VIRTIO_NET_F_CSUM: Device handling packets with partial checksum supported" -# ], -# "transports": [ -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields enabled", -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors supported", -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec (legacy)", -# "VIRTIO_F_ANY_LAYOUT: Device accepts arbitrary desc. layouts", -# "VIRTIO_F_NOTIFY_ON_EMPTY: Notify when device runs out of avail. descs. on VQ" -# ] +# "acked-features": 5100306432, +# "features": 13908344832, # }, +# "backend-features": 6337593319, # "start-on-kick": false, # "isr": 1, # "broken": false, -# "status": { -# "statuses": [ -# "VIRTIO_CONFIG_S_ACKNOWLEDGE: Valid virtio device found", -# "VIRTIO_CONFIG_S_DRIVER: Guest OS compatible with device", -# "VIRTIO_CONFIG_S_FEATURES_OK: Feature negotiation complete", -# "VIRTIO_CONFIG_S_DRIVER_OK: Driver setup and ready" -# ] -# }, +# "status": 15, # "num-vqs": 3, -# "guest-features": { -# "dev-features": [ -# "VIRTIO_NET_F_CTRL_MAC_ADDR: MAC address set through control channel", -# "VIRTIO_NET_F_GUEST_ANNOUNCE: Driver sending gratuitous packets supported", -# "VIRTIO_NET_F_CTRL_VLAN: Control channel VLAN filtering supported", -# "VIRTIO_NET_F_CTRL_RX: Control channel RX mode supported", -# "VIRTIO_NET_F_CTRL_VQ: Control channel available", -# "VIRTIO_NET_F_STATUS: Configuration status field available", -# "VIRTIO_NET_F_MRG_RXBUF: Driver can merge receive buffers", -# "VIRTIO_NET_F_HOST_UFO: Device can receive UFO", -# "VIRTIO_NET_F_HOST_ECN: Device can receive TSO with ECN", -# "VIRTIO_NET_F_HOST_TSO6: Device can receive TSOv6", -# "VIRTIO_NET_F_HOST_TSO4: Device can receive TSOv4", -# "VIRTIO_NET_F_GUEST_UFO: Driver can receive UFO", -# "VIRTIO_NET_F_GUEST_ECN: Driver can receive TSO with ECN", -# "VIRTIO_NET_F_GUEST_TSO6: Driver can receive TSOv6", -# "VIRTIO_NET_F_GUEST_TSO4: Driver can receive TSOv4", -# "VIRTIO_NET_F_MAC: Device has given MAC address", -# "VIRTIO_NET_F_CTRL_GUEST_OFFLOADS: Control channel offloading reconfig. supported", -# "VIRTIO_NET_F_GUEST_CSUM: Driver handling packets with partial checksum supported", -# "VIRTIO_NET_F_CSUM: Device handling packets with partial checksum supported" -# ], -# "transports": [ -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields enabled", -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors supported", -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec (legacy)" -# ] -# }, -# "host-features": { -# "dev-features": [ -# "VHOST_USER_F_PROTOCOL_FEATURES: Vhost-user protocol features negotiation supported", -# "VIRTIO_NET_F_GSO: Handling GSO-type packets supported", -# "VIRTIO_NET_F_CTRL_MAC_ADDR: MAC address set through control channel", -# "VIRTIO_NET_F_GUEST_ANNOUNCE: Driver sending gratuitous packets supported", -# "VIRTIO_NET_F_CTRL_RX_EXTRA: Extra RX mode control supported", -# "VIRTIO_NET_F_CTRL_VLAN: Control channel VLAN filtering supported", -# "VIRTIO_NET_F_CTRL_RX: Control channel RX mode supported", -# "VIRTIO_NET_F_CTRL_VQ: Control channel available", -# "VIRTIO_NET_F_STATUS: Configuration status field available", -# "VIRTIO_NET_F_MRG_RXBUF: Driver can merge receive buffers", -# "VIRTIO_NET_F_HOST_UFO: Device can receive UFO", -# "VIRTIO_NET_F_HOST_ECN: Device can receive TSO with ECN", -# "VIRTIO_NET_F_HOST_TSO6: Device can receive TSOv6", -# "VIRTIO_NET_F_HOST_TSO4: Device can receive TSOv4", -# "VIRTIO_NET_F_GUEST_UFO: Driver can receive UFO", -# "VIRTIO_NET_F_GUEST_ECN: Driver can receive TSO with ECN", -# "VIRTIO_NET_F_GUEST_TSO6: Driver can receive TSOv6", -# "VIRTIO_NET_F_GUEST_TSO4: Driver can receive TSOv4", -# "VIRTIO_NET_F_MAC: Device has given MAC address", -# "VIRTIO_NET_F_CTRL_GUEST_OFFLOADS: Control channel offloading reconfig. supported", -# "VIRTIO_NET_F_GUEST_CSUM: Driver handling packets with partial checksum supported", -# "VIRTIO_NET_F_CSUM: Device handling packets with partial checksum supported" -# ], -# "transports": [ -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields enabled", -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors supported", -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec (legacy)", -# "VIRTIO_F_ANY_LAYOUT: Device accepts arbitrary desc. layouts", -# "VIRTIO_F_NOTIFY_ON_EMPTY: Notify when device runs out of avail. descs. on VQ" -# ] -# }, +# "guest-features": 5111807911, +# "host-features": 6337593319, # "use-guest-notifier-mask": true, # "vm-running": true, # "queue-sel": 2, -- 2.39.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/2] qapi/virtio: Keep feature and status bits in the QMP output 2024-01-04 16:19 ` [PATCH v2 1/2] qapi/virtio: Keep feature and status bits in the QMP output Hyman Huang @ 2024-01-12 14:10 ` Markus Armbruster 2024-01-12 15:02 ` Yong Huang 0 siblings, 1 reply; 7+ messages in thread From: Markus Armbruster @ 2024-01-12 14:10 UTC (permalink / raw) To: Hyman Huang Cc: qemu-devel, Michael S . Tsirkin, Philippe Mathieu-Daudé, Eric Blake Hyman Huang <yong.huang@smartx.com> writes: > Maintain the feature and status bits in the x-query-virtio-status > output and, as usual, add human-readable output only in HMP. > > Applications may find it useful to compare features and status > information directly. An upper application, for example, could > use the QMP command x-query-virtio-status to retrieve vhost-user > net device features and the "ovs-vsctl list interface" command to > retrieve interface features (in number format) in order to verify > the correctness of the virtio negotiation between guest, QEMU, > and OVS-DPDK. The application could then compare the two features > directly, without the need for additional feature encoding. > > Signed-off-by: Hyman Huang <yong.huang@smartx.com> A bit terse. Here's my try: qmp: Switch x-query-virtio-status back to numeric encoding x-query-virtio-status returns several sets of virtio feature and status flags. It goes back to v7.2.0. In the initial commit 90c066cd682 (qmp: add QMP command x-query-virtio-status), we returned them as numbers, using virtio's well-known binary encoding. The next commit f3034ad71fc (qmp: decode feature & status bits in virtio-status) replaced the numbers by objects. The objects represent bits QEMU knows symbolically, and any unknown bits numerically just like before. Commit 8a8287981d1 (hmp: add virtio commands) the matching HMP command "info virtio" (and a few more, which aren't relevant here). The symbolic representation uses lists of strings. The string format is undocumented. The strings look like "WELL_KNOWN_SYMBOL: human readable explanation". This symbolic representation is nice for humans. Machines it can save the trouble of decoding virtio's well-known binary encoding. However, we sometimes want to compare features and status bits without caring for their exact meaning. Say we want to verify the correctness of the virtio negotiation between guest, QEMU, and OVS-DPDK. We can use QMP command x-query-virtio-status to retrieve vhost-user net device features, and the "ovs-vsctl list interface" command to retrieve interface features. Without commit f3034ad71fc, we could then simply compare the numbers. With this commit, we first have to map from the strings back to the numeric encoding. Revert the decoding for QMP, but keep it for HMP. This makes the QMP command easier to use for use cases where we don't need to decode, like the comparison above. For use cases where we need to decode, we replace parsing undocumented strings by decoding virtio's well-known binary encoding. Incompatible change; acceptable because x-query-virtio-status does comes without a stability promise. > --- > hw/virtio/virtio-hmp-cmds.c | 29 ++++-- > hw/virtio/virtio-qmp.c | 23 ++--- > qapi/virtio.json | 192 ++++-------------------------------- > 3 files changed, 48 insertions(+), 196 deletions(-) > > diff --git a/hw/virtio/virtio-hmp-cmds.c b/hw/virtio/virtio-hmp-cmds.c > index 477c97dea2..4fabba4f9c 100644 > --- a/hw/virtio/virtio-hmp-cmds.c > +++ b/hw/virtio/virtio-hmp-cmds.c > @@ -6,6 +6,7 @@ > */ > > #include "qemu/osdep.h" > +#include "virtio-qmp.h" > #include "monitor/hmp.h" > #include "monitor/monitor.h" > #include "qapi/qapi-commands-virtio.h" > @@ -13,8 +14,10 @@ > > > static void hmp_virtio_dump_protocols(Monitor *mon, > - VhostDeviceProtocols *pcol) > + uint64_t bitmap) > { > + VhostDeviceProtocols *pcol = qmp_decode_protocols(bitmap); > + > strList *pcol_list = pcol->protocols; > while (pcol_list) { > monitor_printf(mon, "\t%s", pcol_list->value); > @@ -31,8 +34,10 @@ static void hmp_virtio_dump_protocols(Monitor *mon, > } > > static void hmp_virtio_dump_status(Monitor *mon, > - VirtioDeviceStatus *status) > + uint64_t bitmap) > { > + VirtioDeviceStatus *status = qmp_decode_status(bitmap); > + > strList *status_list = status->statuses; > while (status_list) { > monitor_printf(mon, "\t%s", status_list->value); > @@ -49,8 +54,12 @@ static void hmp_virtio_dump_status(Monitor *mon, > } > > static void hmp_virtio_dump_features(Monitor *mon, > - VirtioDeviceFeatures *features) > + uint16_t device_id, > + uint64_t bitmap) > { > + VirtioDeviceFeatures *features = > + qmp_decode_features(device_id, bitmap); > + > strList *transport_list = features->transports; > while (transport_list) { > monitor_printf(mon, "\t%s", transport_list->value); > @@ -147,11 +156,11 @@ void hmp_virtio_status(Monitor *mon, const QDict *qdict) > monitor_printf(mon, " status:\n"); > hmp_virtio_dump_status(mon, s->status); > monitor_printf(mon, " Guest features:\n"); > - hmp_virtio_dump_features(mon, s->guest_features); > + hmp_virtio_dump_features(mon, s->device_id, s->guest_features); > monitor_printf(mon, " Host features:\n"); > - hmp_virtio_dump_features(mon, s->host_features); > + hmp_virtio_dump_features(mon, s->device_id, s->host_features); > monitor_printf(mon, " Backend features:\n"); > - hmp_virtio_dump_features(mon, s->backend_features); > + hmp_virtio_dump_features(mon, s->device_id, s->backend_features); > > if (s->vhost_dev) { > monitor_printf(mon, " VHost:\n"); > @@ -172,11 +181,13 @@ void hmp_virtio_status(Monitor *mon, const QDict *qdict) > monitor_printf(mon, " log_size: %"PRId64"\n", > s->vhost_dev->log_size); > monitor_printf(mon, " Features:\n"); > - hmp_virtio_dump_features(mon, s->vhost_dev->features); > + hmp_virtio_dump_features(mon, s->device_id, s->vhost_dev->features); > monitor_printf(mon, " Acked features:\n"); > - hmp_virtio_dump_features(mon, s->vhost_dev->acked_features); > + hmp_virtio_dump_features(mon, > + s->device_id, s->vhost_dev->acked_features); > monitor_printf(mon, " Backend features:\n"); > - hmp_virtio_dump_features(mon, s->vhost_dev->backend_features); > + hmp_virtio_dump_features(mon, > + s->device_id, s->vhost_dev->backend_features); > monitor_printf(mon, " Protocol features:\n"); > hmp_virtio_dump_protocols(mon, s->vhost_dev->protocol_features); > } > diff --git a/hw/virtio/virtio-qmp.c b/hw/virtio/virtio-qmp.c > index 1dd96ed20f..1660c17653 100644 > --- a/hw/virtio/virtio-qmp.c > +++ b/hw/virtio/virtio-qmp.c > @@ -733,12 +733,9 @@ VirtioStatus *qmp_x_query_virtio_status(const char *path, Error **errp) > status->name = g_strdup(vdev->name); > status->device_id = vdev->device_id; > status->vhost_started = vdev->vhost_started; > - status->guest_features = qmp_decode_features(vdev->device_id, > - vdev->guest_features); > - status->host_features = qmp_decode_features(vdev->device_id, > - vdev->host_features); > - status->backend_features = qmp_decode_features(vdev->device_id, > - vdev->backend_features); > + status->guest_features = vdev->guest_features; > + status->host_features = vdev->host_features; > + status->backend_features = vdev->backend_features; > > switch (vdev->device_endian) { > case VIRTIO_DEVICE_ENDIAN_LITTLE: > @@ -753,7 +750,7 @@ VirtioStatus *qmp_x_query_virtio_status(const char *path, Error **errp) > } > > status->num_vqs = virtio_get_num_queues(vdev); > - status->status = qmp_decode_status(vdev->status); > + status->status = vdev->status; > status->isr = vdev->isr; > status->queue_sel = vdev->queue_sel; > status->vm_running = vdev->vm_running; > @@ -775,14 +772,10 @@ VirtioStatus *qmp_x_query_virtio_status(const char *path, Error **errp) > status->vhost_dev->n_tmp_sections = hdev->n_tmp_sections; > status->vhost_dev->nvqs = hdev->nvqs; > status->vhost_dev->vq_index = hdev->vq_index; > - status->vhost_dev->features = > - qmp_decode_features(vdev->device_id, hdev->features); > - status->vhost_dev->acked_features = > - qmp_decode_features(vdev->device_id, hdev->acked_features); > - status->vhost_dev->backend_features = > - qmp_decode_features(vdev->device_id, hdev->backend_features); > - status->vhost_dev->protocol_features = > - qmp_decode_protocols(hdev->protocol_features); > + status->vhost_dev->features = hdev->features; > + status->vhost_dev->acked_features = hdev->acked_features; > + status->vhost_dev->backend_features = hdev->backend_features; > + status->vhost_dev->protocol_features = hdev->protocol_features; > status->vhost_dev->max_queues = hdev->max_queues; > status->vhost_dev->backend_cap = hdev->backend_cap; > status->vhost_dev->log_enabled = hdev->log_enabled; > diff --git a/qapi/virtio.json b/qapi/virtio.json > index 19c7c36e36..26516fb29c 100644 > --- a/qapi/virtio.json > +++ b/qapi/virtio.json > @@ -102,10 +102,10 @@ > 'n-tmp-sections': 'int', > 'nvqs': 'uint32', > 'vq-index': 'int', > - 'features': 'VirtioDeviceFeatures', > - 'acked-features': 'VirtioDeviceFeatures', > - 'backend-features': 'VirtioDeviceFeatures', > - 'protocol-features': 'VhostDeviceProtocols', > + 'features': 'uint64', > + 'acked-features': 'uint64', > + 'backend-features': 'uint64', > + 'protocol-features': 'uint64', > 'max-queues': 'uint64', > 'backend-cap': 'uint64', > 'log-enabled': 'bool', > @@ -170,11 +170,11 @@ > 'device-id': 'uint16', > 'vhost-started': 'bool', > 'device-endian': 'str', > - 'guest-features': 'VirtioDeviceFeatures', > - 'host-features': 'VirtioDeviceFeatures', > - 'backend-features': 'VirtioDeviceFeatures', > + 'guest-features': 'uint64', > + 'host-features': 'uint64', > + 'backend-features': 'uint64', > 'num-vqs': 'int', > - 'status': 'VirtioDeviceStatus', > + 'status': 'uint8', > 'isr': 'uint8', > 'queue-sel': 'uint16', > 'vm-running': 'bool', > @@ -217,41 +217,14 @@ > # "name": "virtio-crypto", > # "started": true, > # "device-id": 20, > -# "backend-features": { > -# "transports": [], > -# "dev-features": [] > -# }, > +# "backend-features": 0, > # "start-on-kick": false, > # "isr": 1, > # "broken": false, > -# "status": { > -# "statuses": [ > -# "VIRTIO_CONFIG_S_ACKNOWLEDGE: Valid virtio device found", > -# "VIRTIO_CONFIG_S_DRIVER: Guest OS compatible with device", > -# "VIRTIO_CONFIG_S_FEATURES_OK: Feature negotiation complete", > -# "VIRTIO_CONFIG_S_DRIVER_OK: Driver setup and ready" > -# ] > -# }, > +# "status": 15, > # "num-vqs": 2, > -# "guest-features": { > -# "dev-features": [], > -# "transports": [ > -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields enabled", > -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors supported", > -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec (legacy)" > -# ] > -# }, > -# "host-features": { > -# "unknown-dev-features": 1073741824, > -# "dev-features": [], > -# "transports": [ > -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields enabled", > -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors supported", > -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec (legacy)", > -# "VIRTIO_F_ANY_LAYOUT: Device accepts arbitrary desc. layouts", > -# "VIRTIO_F_NOTIFY_ON_EMPTY: Notify when device runs out of avail. descs. on VQ" > -# ] > -# }, > +# "guest-features": 5100273664, > +# "host-features": 6325010432, > # "use-guest-notifier-mask": true, > # "vm-running": true, > # "queue-sel": 1, > @@ -279,147 +252,22 @@ > # "max-queues": 1, > # "backend-cap": 2, > # "log-size": 0, > -# "backend-features": { > -# "dev-features": [], > -# "transports": [] > -# }, > +# "backend-features": 0, > # "nvqs": 2, > -# "protocol-features": { > -# "protocols": [] > -# }, > +# "protocol-features": 0, > # "vq-index": 0, > # "log-enabled": false, > -# "acked-features": { > -# "dev-features": [ > -# "VIRTIO_NET_F_MRG_RXBUF: Driver can merge receive buffers" > -# ], > -# "transports": [ > -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields enabled", > -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors supported", > -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec (legacy)" > -# ] > -# }, > -# "features": { > -# "dev-features": [ > -# "VHOST_F_LOG_ALL: Logging write descriptors supported", > -# "VIRTIO_NET_F_MRG_RXBUF: Driver can merge receive buffers" > -# ], > -# "transports": [ > -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields enabled", > -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors supported", > -# "VIRTIO_F_IOMMU_PLATFORM: Device can be used on IOMMU platform", > -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec (legacy)", > -# "VIRTIO_F_ANY_LAYOUT: Device accepts arbitrary desc. layouts", > -# "VIRTIO_F_NOTIFY_ON_EMPTY: Notify when device runs out of avail. descs. on VQ" > -# ] > -# } > -# }, > -# "backend-features": { > -# "dev-features": [ > -# "VHOST_USER_F_PROTOCOL_FEATURES: Vhost-user protocol features negotiation supported", > -# "VIRTIO_NET_F_GSO: Handling GSO-type packets supported", > -# "VIRTIO_NET_F_CTRL_MAC_ADDR: MAC address set through control channel", > -# "VIRTIO_NET_F_GUEST_ANNOUNCE: Driver sending gratuitous packets supported", > -# "VIRTIO_NET_F_CTRL_RX_EXTRA: Extra RX mode control supported", > -# "VIRTIO_NET_F_CTRL_VLAN: Control channel VLAN filtering supported", > -# "VIRTIO_NET_F_CTRL_RX: Control channel RX mode supported", > -# "VIRTIO_NET_F_CTRL_VQ: Control channel available", > -# "VIRTIO_NET_F_STATUS: Configuration status field available", > -# "VIRTIO_NET_F_MRG_RXBUF: Driver can merge receive buffers", > -# "VIRTIO_NET_F_HOST_UFO: Device can receive UFO", > -# "VIRTIO_NET_F_HOST_ECN: Device can receive TSO with ECN", > -# "VIRTIO_NET_F_HOST_TSO6: Device can receive TSOv6", > -# "VIRTIO_NET_F_HOST_TSO4: Device can receive TSOv4", > -# "VIRTIO_NET_F_GUEST_UFO: Driver can receive UFO", > -# "VIRTIO_NET_F_GUEST_ECN: Driver can receive TSO with ECN", > -# "VIRTIO_NET_F_GUEST_TSO6: Driver can receive TSOv6", > -# "VIRTIO_NET_F_GUEST_TSO4: Driver can receive TSOv4", > -# "VIRTIO_NET_F_MAC: Device has given MAC address", > -# "VIRTIO_NET_F_CTRL_GUEST_OFFLOADS: Control channel offloading reconfig. supported", > -# "VIRTIO_NET_F_GUEST_CSUM: Driver handling packets with partial checksum supported", > -# "VIRTIO_NET_F_CSUM: Device handling packets with partial checksum supported" > -# ], > -# "transports": [ > -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields enabled", > -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors supported", > -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec (legacy)", > -# "VIRTIO_F_ANY_LAYOUT: Device accepts arbitrary desc. layouts", > -# "VIRTIO_F_NOTIFY_ON_EMPTY: Notify when device runs out of avail. descs. on VQ" > -# ] > +# "acked-features": 5100306432, > +# "features": 13908344832, > # }, > +# "backend-features": 6337593319, > # "start-on-kick": false, > # "isr": 1, > # "broken": false, > -# "status": { > -# "statuses": [ > -# "VIRTIO_CONFIG_S_ACKNOWLEDGE: Valid virtio device found", > -# "VIRTIO_CONFIG_S_DRIVER: Guest OS compatible with device", > -# "VIRTIO_CONFIG_S_FEATURES_OK: Feature negotiation complete", > -# "VIRTIO_CONFIG_S_DRIVER_OK: Driver setup and ready" > -# ] > -# }, > +# "status": 15, > # "num-vqs": 3, > -# "guest-features": { > -# "dev-features": [ > -# "VIRTIO_NET_F_CTRL_MAC_ADDR: MAC address set through control channel", > -# "VIRTIO_NET_F_GUEST_ANNOUNCE: Driver sending gratuitous packets supported", > -# "VIRTIO_NET_F_CTRL_VLAN: Control channel VLAN filtering supported", > -# "VIRTIO_NET_F_CTRL_RX: Control channel RX mode supported", > -# "VIRTIO_NET_F_CTRL_VQ: Control channel available", > -# "VIRTIO_NET_F_STATUS: Configuration status field available", > -# "VIRTIO_NET_F_MRG_RXBUF: Driver can merge receive buffers", > -# "VIRTIO_NET_F_HOST_UFO: Device can receive UFO", > -# "VIRTIO_NET_F_HOST_ECN: Device can receive TSO with ECN", > -# "VIRTIO_NET_F_HOST_TSO6: Device can receive TSOv6", > -# "VIRTIO_NET_F_HOST_TSO4: Device can receive TSOv4", > -# "VIRTIO_NET_F_GUEST_UFO: Driver can receive UFO", > -# "VIRTIO_NET_F_GUEST_ECN: Driver can receive TSO with ECN", > -# "VIRTIO_NET_F_GUEST_TSO6: Driver can receive TSOv6", > -# "VIRTIO_NET_F_GUEST_TSO4: Driver can receive TSOv4", > -# "VIRTIO_NET_F_MAC: Device has given MAC address", > -# "VIRTIO_NET_F_CTRL_GUEST_OFFLOADS: Control channel offloading reconfig. supported", > -# "VIRTIO_NET_F_GUEST_CSUM: Driver handling packets with partial checksum supported", > -# "VIRTIO_NET_F_CSUM: Device handling packets with partial checksum supported" > -# ], > -# "transports": [ > -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields enabled", > -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors supported", > -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec (legacy)" > -# ] > -# }, > -# "host-features": { > -# "dev-features": [ > -# "VHOST_USER_F_PROTOCOL_FEATURES: Vhost-user protocol features negotiation supported", > -# "VIRTIO_NET_F_GSO: Handling GSO-type packets supported", > -# "VIRTIO_NET_F_CTRL_MAC_ADDR: MAC address set through control channel", > -# "VIRTIO_NET_F_GUEST_ANNOUNCE: Driver sending gratuitous packets supported", > -# "VIRTIO_NET_F_CTRL_RX_EXTRA: Extra RX mode control supported", > -# "VIRTIO_NET_F_CTRL_VLAN: Control channel VLAN filtering supported", > -# "VIRTIO_NET_F_CTRL_RX: Control channel RX mode supported", > -# "VIRTIO_NET_F_CTRL_VQ: Control channel available", > -# "VIRTIO_NET_F_STATUS: Configuration status field available", > -# "VIRTIO_NET_F_MRG_RXBUF: Driver can merge receive buffers", > -# "VIRTIO_NET_F_HOST_UFO: Device can receive UFO", > -# "VIRTIO_NET_F_HOST_ECN: Device can receive TSO with ECN", > -# "VIRTIO_NET_F_HOST_TSO6: Device can receive TSOv6", > -# "VIRTIO_NET_F_HOST_TSO4: Device can receive TSOv4", > -# "VIRTIO_NET_F_GUEST_UFO: Driver can receive UFO", > -# "VIRTIO_NET_F_GUEST_ECN: Driver can receive TSO with ECN", > -# "VIRTIO_NET_F_GUEST_TSO6: Driver can receive TSOv6", > -# "VIRTIO_NET_F_GUEST_TSO4: Driver can receive TSOv4", > -# "VIRTIO_NET_F_MAC: Device has given MAC address", > -# "VIRTIO_NET_F_CTRL_GUEST_OFFLOADS: Control channel offloading reconfig. supported", > -# "VIRTIO_NET_F_GUEST_CSUM: Driver handling packets with partial checksum supported", > -# "VIRTIO_NET_F_CSUM: Device handling packets with partial checksum supported" > -# ], > -# "transports": [ > -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields enabled", > -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors supported", > -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec (legacy)", > -# "VIRTIO_F_ANY_LAYOUT: Device accepts arbitrary desc. layouts", > -# "VIRTIO_F_NOTIFY_ON_EMPTY: Notify when device runs out of avail. descs. on VQ" > -# ] > -# }, > +# "guest-features": 5111807911, > +# "host-features": 6337593319, > # "use-guest-notifier-mask": true, > # "vm-running": true, > # "queue-sel": 2, qmp_decode_protocols(), qmp_decode_status(), and qmp_decode_features() are now only used in virtio-hmp-cmds.c. Please move them there, and replace the qmp_ prefix by hmp_ or simply drop it. VhostDeviceProtocols and VirtioDeviceFeatures are then only used in virtio-hmp-cmds.c. Please define them as plain C types there, and drop them from the QAPI schema. You may want to do this as a separate cleanup commit. Use your judgement. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/2] qapi/virtio: Keep feature and status bits in the QMP output 2024-01-12 14:10 ` Markus Armbruster @ 2024-01-12 15:02 ` Yong Huang 0 siblings, 0 replies; 7+ messages in thread From: Yong Huang @ 2024-01-12 15:02 UTC (permalink / raw) To: Markus Armbruster Cc: qemu-devel, Michael S . Tsirkin, Philippe Mathieu-Daudé, Eric Blake [-- Attachment #1: Type: text/plain, Size: 24882 bytes --] On Fri, Jan 12, 2024 at 10:10 PM Markus Armbruster <armbru@redhat.com> wrote: > Hyman Huang <yong.huang@smartx.com> writes: > > > Maintain the feature and status bits in the x-query-virtio-status > > output and, as usual, add human-readable output only in HMP. > > > > Applications may find it useful to compare features and status > > information directly. An upper application, for example, could > > use the QMP command x-query-virtio-status to retrieve vhost-user > > net device features and the "ovs-vsctl list interface" command to > > retrieve interface features (in number format) in order to verify > > the correctness of the virtio negotiation between guest, QEMU, > > and OVS-DPDK. The application could then compare the two features > > directly, without the need for additional feature encoding. > > > > Signed-off-by: Hyman Huang <yong.huang@smartx.com> > > A bit terse. Here's my try: > > qmp: Switch x-query-virtio-status back to numeric encoding > > x-query-virtio-status returns several sets of virtio feature and > status flags. It goes back to v7.2.0. > > In the initial commit 90c066cd682 (qmp: add QMP command > x-query-virtio-status), we returned them as numbers, using virtio's > well-known binary encoding. > > The next commit f3034ad71fc (qmp: decode feature & status bits in > virtio-status) replaced the numbers by objects. The objects represent > bits QEMU knows symbolically, and any unknown bits numerically just > like > before. > > Commit 8a8287981d1 (hmp: add virtio commands) the matching HMP command > "info virtio" (and a few more, which aren't relevant here). > > The symbolic representation uses lists of strings. The string format > is > undocumented. The strings look like "WELL_KNOWN_SYMBOL: human readable > explanation". > > This symbolic representation is nice for humans. Machines it can save > the trouble of decoding virtio's well-known binary encoding. > > However, we sometimes want to compare features and status bits without > caring for their exact meaning. Say we want to verify the correctness > of the virtio negotiation between guest, QEMU, and OVS-DPDK. We can > use > QMP command x-query-virtio-status to retrieve vhost-user net device > features, and the "ovs-vsctl list interface" command to retrieve > interface features. Without commit f3034ad71fc, we could then simply > compare the numbers. With this commit, we first have to map from the > strings back to the numeric encoding. > > Revert the decoding for QMP, but keep it for HMP. > > This makes the QMP command easier to use for use cases where we > don't need to decode, like the comparison above. For use cases > where we need to decode, we replace parsing undocumented strings by > decoding virtio's well-known binary encoding. > > Incompatible change; acceptable because x-query-virtio-status does > comes without a stability promise. > Thanks for the refining, I'll post this commit message in the next version. > > > --- > > hw/virtio/virtio-hmp-cmds.c | 29 ++++-- > > hw/virtio/virtio-qmp.c | 23 ++--- > > qapi/virtio.json | 192 ++++-------------------------------- > > 3 files changed, 48 insertions(+), 196 deletions(-) > > > > diff --git a/hw/virtio/virtio-hmp-cmds.c b/hw/virtio/virtio-hmp-cmds.c > > index 477c97dea2..4fabba4f9c 100644 > > --- a/hw/virtio/virtio-hmp-cmds.c > > +++ b/hw/virtio/virtio-hmp-cmds.c > > @@ -6,6 +6,7 @@ > > */ > > > > #include "qemu/osdep.h" > > +#include "virtio-qmp.h" > > #include "monitor/hmp.h" > > #include "monitor/monitor.h" > > #include "qapi/qapi-commands-virtio.h" > > @@ -13,8 +14,10 @@ > > > > > > static void hmp_virtio_dump_protocols(Monitor *mon, > > - VhostDeviceProtocols *pcol) > > + uint64_t bitmap) > > { > > + VhostDeviceProtocols *pcol = qmp_decode_protocols(bitmap); > > + > > strList *pcol_list = pcol->protocols; > > while (pcol_list) { > > monitor_printf(mon, "\t%s", pcol_list->value); > > @@ -31,8 +34,10 @@ static void hmp_virtio_dump_protocols(Monitor *mon, > > } > > > > static void hmp_virtio_dump_status(Monitor *mon, > > - VirtioDeviceStatus *status) > > + uint64_t bitmap) > > { > > + VirtioDeviceStatus *status = qmp_decode_status(bitmap); > > + > > strList *status_list = status->statuses; > > while (status_list) { > > monitor_printf(mon, "\t%s", status_list->value); > > @@ -49,8 +54,12 @@ static void hmp_virtio_dump_status(Monitor *mon, > > } > > > > static void hmp_virtio_dump_features(Monitor *mon, > > - VirtioDeviceFeatures *features) > > + uint16_t device_id, > > + uint64_t bitmap) > > { > > + VirtioDeviceFeatures *features = > > + qmp_decode_features(device_id, bitmap); > > + > > strList *transport_list = features->transports; > > while (transport_list) { > > monitor_printf(mon, "\t%s", transport_list->value); > > @@ -147,11 +156,11 @@ void hmp_virtio_status(Monitor *mon, const QDict > *qdict) > > monitor_printf(mon, " status:\n"); > > hmp_virtio_dump_status(mon, s->status); > > monitor_printf(mon, " Guest features:\n"); > > - hmp_virtio_dump_features(mon, s->guest_features); > > + hmp_virtio_dump_features(mon, s->device_id, s->guest_features); > > monitor_printf(mon, " Host features:\n"); > > - hmp_virtio_dump_features(mon, s->host_features); > > + hmp_virtio_dump_features(mon, s->device_id, s->host_features); > > monitor_printf(mon, " Backend features:\n"); > > - hmp_virtio_dump_features(mon, s->backend_features); > > + hmp_virtio_dump_features(mon, s->device_id, s->backend_features); > > > > if (s->vhost_dev) { > > monitor_printf(mon, " VHost:\n"); > > @@ -172,11 +181,13 @@ void hmp_virtio_status(Monitor *mon, const QDict > *qdict) > > monitor_printf(mon, " log_size: %"PRId64"\n", > > s->vhost_dev->log_size); > > monitor_printf(mon, " Features:\n"); > > - hmp_virtio_dump_features(mon, s->vhost_dev->features); > > + hmp_virtio_dump_features(mon, s->device_id, > s->vhost_dev->features); > > monitor_printf(mon, " Acked features:\n"); > > - hmp_virtio_dump_features(mon, s->vhost_dev->acked_features); > > + hmp_virtio_dump_features(mon, > > + s->device_id, s->vhost_dev->acked_features); > > monitor_printf(mon, " Backend features:\n"); > > - hmp_virtio_dump_features(mon, s->vhost_dev->backend_features); > > + hmp_virtio_dump_features(mon, > > + s->device_id, s->vhost_dev->backend_features); > > monitor_printf(mon, " Protocol features:\n"); > > hmp_virtio_dump_protocols(mon, s->vhost_dev->protocol_features); > > } > > diff --git a/hw/virtio/virtio-qmp.c b/hw/virtio/virtio-qmp.c > > index 1dd96ed20f..1660c17653 100644 > > --- a/hw/virtio/virtio-qmp.c > > +++ b/hw/virtio/virtio-qmp.c > > @@ -733,12 +733,9 @@ VirtioStatus *qmp_x_query_virtio_status(const char > *path, Error **errp) > > status->name = g_strdup(vdev->name); > > status->device_id = vdev->device_id; > > status->vhost_started = vdev->vhost_started; > > - status->guest_features = qmp_decode_features(vdev->device_id, > > - vdev->guest_features); > > - status->host_features = qmp_decode_features(vdev->device_id, > > - vdev->host_features); > > - status->backend_features = qmp_decode_features(vdev->device_id, > > - > vdev->backend_features); > > + status->guest_features = vdev->guest_features; > > + status->host_features = vdev->host_features; > > + status->backend_features = vdev->backend_features; > > > > switch (vdev->device_endian) { > > case VIRTIO_DEVICE_ENDIAN_LITTLE: > > @@ -753,7 +750,7 @@ VirtioStatus *qmp_x_query_virtio_status(const char > *path, Error **errp) > > } > > > > status->num_vqs = virtio_get_num_queues(vdev); > > - status->status = qmp_decode_status(vdev->status); > > + status->status = vdev->status; > > status->isr = vdev->isr; > > status->queue_sel = vdev->queue_sel; > > status->vm_running = vdev->vm_running; > > @@ -775,14 +772,10 @@ VirtioStatus *qmp_x_query_virtio_status(const char > *path, Error **errp) > > status->vhost_dev->n_tmp_sections = hdev->n_tmp_sections; > > status->vhost_dev->nvqs = hdev->nvqs; > > status->vhost_dev->vq_index = hdev->vq_index; > > - status->vhost_dev->features = > > - qmp_decode_features(vdev->device_id, hdev->features); > > - status->vhost_dev->acked_features = > > - qmp_decode_features(vdev->device_id, hdev->acked_features); > > - status->vhost_dev->backend_features = > > - qmp_decode_features(vdev->device_id, > hdev->backend_features); > > - status->vhost_dev->protocol_features = > > - qmp_decode_protocols(hdev->protocol_features); > > + status->vhost_dev->features = hdev->features; > > + status->vhost_dev->acked_features = hdev->acked_features; > > + status->vhost_dev->backend_features = hdev->backend_features; > > + status->vhost_dev->protocol_features = hdev->protocol_features; > > status->vhost_dev->max_queues = hdev->max_queues; > > status->vhost_dev->backend_cap = hdev->backend_cap; > > status->vhost_dev->log_enabled = hdev->log_enabled; > > diff --git a/qapi/virtio.json b/qapi/virtio.json > > index 19c7c36e36..26516fb29c 100644 > > --- a/qapi/virtio.json > > +++ b/qapi/virtio.json > > @@ -102,10 +102,10 @@ > > 'n-tmp-sections': 'int', > > 'nvqs': 'uint32', > > 'vq-index': 'int', > > - 'features': 'VirtioDeviceFeatures', > > - 'acked-features': 'VirtioDeviceFeatures', > > - 'backend-features': 'VirtioDeviceFeatures', > > - 'protocol-features': 'VhostDeviceProtocols', > > + 'features': 'uint64', > > + 'acked-features': 'uint64', > > + 'backend-features': 'uint64', > > + 'protocol-features': 'uint64', > > 'max-queues': 'uint64', > > 'backend-cap': 'uint64', > > 'log-enabled': 'bool', > > @@ -170,11 +170,11 @@ > > 'device-id': 'uint16', > > 'vhost-started': 'bool', > > 'device-endian': 'str', > > - 'guest-features': 'VirtioDeviceFeatures', > > - 'host-features': 'VirtioDeviceFeatures', > > - 'backend-features': 'VirtioDeviceFeatures', > > + 'guest-features': 'uint64', > > + 'host-features': 'uint64', > > + 'backend-features': 'uint64', > > 'num-vqs': 'int', > > - 'status': 'VirtioDeviceStatus', > > + 'status': 'uint8', > > 'isr': 'uint8', > > 'queue-sel': 'uint16', > > 'vm-running': 'bool', > > @@ -217,41 +217,14 @@ > > # "name": "virtio-crypto", > > # "started": true, > > # "device-id": 20, > > -# "backend-features": { > > -# "transports": [], > > -# "dev-features": [] > > -# }, > > +# "backend-features": 0, > > # "start-on-kick": false, > > # "isr": 1, > > # "broken": false, > > -# "status": { > > -# "statuses": [ > > -# "VIRTIO_CONFIG_S_ACKNOWLEDGE: Valid virtio device > found", > > -# "VIRTIO_CONFIG_S_DRIVER: Guest OS compatible with > device", > > -# "VIRTIO_CONFIG_S_FEATURES_OK: Feature negotiation > complete", > > -# "VIRTIO_CONFIG_S_DRIVER_OK: Driver setup and ready" > > -# ] > > -# }, > > +# "status": 15, > > # "num-vqs": 2, > > -# "guest-features": { > > -# "dev-features": [], > > -# "transports": [ > > -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields > enabled", > > -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors > supported", > > -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec > (legacy)" > > -# ] > > -# }, > > -# "host-features": { > > -# "unknown-dev-features": 1073741824, > > -# "dev-features": [], > > -# "transports": [ > > -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields > enabled", > > -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors > supported", > > -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec > (legacy)", > > -# "VIRTIO_F_ANY_LAYOUT: Device accepts arbitrary desc. > layouts", > > -# "VIRTIO_F_NOTIFY_ON_EMPTY: Notify when device runs > out of avail. descs. on VQ" > > -# ] > > -# }, > > +# "guest-features": 5100273664, > > +# "host-features": 6325010432, > > # "use-guest-notifier-mask": true, > > # "vm-running": true, > > # "queue-sel": 1, > > @@ -279,147 +252,22 @@ > > # "max-queues": 1, > > # "backend-cap": 2, > > # "log-size": 0, > > -# "backend-features": { > > -# "dev-features": [], > > -# "transports": [] > > -# }, > > +# "backend-features": 0, > > # "nvqs": 2, > > -# "protocol-features": { > > -# "protocols": [] > > -# }, > > +# "protocol-features": 0, > > # "vq-index": 0, > > # "log-enabled": false, > > -# "acked-features": { > > -# "dev-features": [ > > -# "VIRTIO_NET_F_MRG_RXBUF: Driver can merge > receive buffers" > > -# ], > > -# "transports": [ > > -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event > fields enabled", > > -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect > descriptors supported", > > -# "VIRTIO_F_VERSION_1: Device compliant for v1 > spec (legacy)" > > -# ] > > -# }, > > -# "features": { > > -# "dev-features": [ > > -# "VHOST_F_LOG_ALL: Logging write descriptors > supported", > > -# "VIRTIO_NET_F_MRG_RXBUF: Driver can merge > receive buffers" > > -# ], > > -# "transports": [ > > -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event > fields enabled", > > -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect > descriptors supported", > > -# "VIRTIO_F_IOMMU_PLATFORM: Device can be used on > IOMMU platform", > > -# "VIRTIO_F_VERSION_1: Device compliant for v1 > spec (legacy)", > > -# "VIRTIO_F_ANY_LAYOUT: Device accepts arbitrary > desc. layouts", > > -# "VIRTIO_F_NOTIFY_ON_EMPTY: Notify when device > runs out of avail. descs. on VQ" > > -# ] > > -# } > > -# }, > > -# "backend-features": { > > -# "dev-features": [ > > -# "VHOST_USER_F_PROTOCOL_FEATURES: Vhost-user protocol > features negotiation supported", > > -# "VIRTIO_NET_F_GSO: Handling GSO-type packets > supported", > > -# "VIRTIO_NET_F_CTRL_MAC_ADDR: MAC address set through > control channel", > > -# "VIRTIO_NET_F_GUEST_ANNOUNCE: Driver sending > gratuitous packets supported", > > -# "VIRTIO_NET_F_CTRL_RX_EXTRA: Extra RX mode control > supported", > > -# "VIRTIO_NET_F_CTRL_VLAN: Control channel VLAN > filtering supported", > > -# "VIRTIO_NET_F_CTRL_RX: Control channel RX mode > supported", > > -# "VIRTIO_NET_F_CTRL_VQ: Control channel available", > > -# "VIRTIO_NET_F_STATUS: Configuration status field > available", > > -# "VIRTIO_NET_F_MRG_RXBUF: Driver can merge receive > buffers", > > -# "VIRTIO_NET_F_HOST_UFO: Device can receive UFO", > > -# "VIRTIO_NET_F_HOST_ECN: Device can receive TSO with > ECN", > > -# "VIRTIO_NET_F_HOST_TSO6: Device can receive TSOv6", > > -# "VIRTIO_NET_F_HOST_TSO4: Device can receive TSOv4", > > -# "VIRTIO_NET_F_GUEST_UFO: Driver can receive UFO", > > -# "VIRTIO_NET_F_GUEST_ECN: Driver can receive TSO with > ECN", > > -# "VIRTIO_NET_F_GUEST_TSO6: Driver can receive TSOv6", > > -# "VIRTIO_NET_F_GUEST_TSO4: Driver can receive TSOv4", > > -# "VIRTIO_NET_F_MAC: Device has given MAC address", > > -# "VIRTIO_NET_F_CTRL_GUEST_OFFLOADS: Control channel > offloading reconfig. supported", > > -# "VIRTIO_NET_F_GUEST_CSUM: Driver handling packets > with partial checksum supported", > > -# "VIRTIO_NET_F_CSUM: Device handling packets with > partial checksum supported" > > -# ], > > -# "transports": [ > > -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields > enabled", > > -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors > supported", > > -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec > (legacy)", > > -# "VIRTIO_F_ANY_LAYOUT: Device accepts arbitrary desc. > layouts", > > -# "VIRTIO_F_NOTIFY_ON_EMPTY: Notify when device runs > out of avail. descs. on VQ" > > -# ] > > +# "acked-features": 5100306432, > > +# "features": 13908344832, > > # }, > > +# "backend-features": 6337593319, > > # "start-on-kick": false, > > # "isr": 1, > > # "broken": false, > > -# "status": { > > -# "statuses": [ > > -# "VIRTIO_CONFIG_S_ACKNOWLEDGE: Valid virtio device > found", > > -# "VIRTIO_CONFIG_S_DRIVER: Guest OS compatible with > device", > > -# "VIRTIO_CONFIG_S_FEATURES_OK: Feature negotiation > complete", > > -# "VIRTIO_CONFIG_S_DRIVER_OK: Driver setup and ready" > > -# ] > > -# }, > > +# "status": 15, > > # "num-vqs": 3, > > -# "guest-features": { > > -# "dev-features": [ > > -# "VIRTIO_NET_F_CTRL_MAC_ADDR: MAC address set through > control channel", > > -# "VIRTIO_NET_F_GUEST_ANNOUNCE: Driver sending > gratuitous packets supported", > > -# "VIRTIO_NET_F_CTRL_VLAN: Control channel VLAN > filtering supported", > > -# "VIRTIO_NET_F_CTRL_RX: Control channel RX mode > supported", > > -# "VIRTIO_NET_F_CTRL_VQ: Control channel available", > > -# "VIRTIO_NET_F_STATUS: Configuration status field > available", > > -# "VIRTIO_NET_F_MRG_RXBUF: Driver can merge receive > buffers", > > -# "VIRTIO_NET_F_HOST_UFO: Device can receive UFO", > > -# "VIRTIO_NET_F_HOST_ECN: Device can receive TSO with > ECN", > > -# "VIRTIO_NET_F_HOST_TSO6: Device can receive TSOv6", > > -# "VIRTIO_NET_F_HOST_TSO4: Device can receive TSOv4", > > -# "VIRTIO_NET_F_GUEST_UFO: Driver can receive UFO", > > -# "VIRTIO_NET_F_GUEST_ECN: Driver can receive TSO with > ECN", > > -# "VIRTIO_NET_F_GUEST_TSO6: Driver can receive TSOv6", > > -# "VIRTIO_NET_F_GUEST_TSO4: Driver can receive TSOv4", > > -# "VIRTIO_NET_F_MAC: Device has given MAC address", > > -# "VIRTIO_NET_F_CTRL_GUEST_OFFLOADS: Control channel > offloading reconfig. supported", > > -# "VIRTIO_NET_F_GUEST_CSUM: Driver handling packets > with partial checksum supported", > > -# "VIRTIO_NET_F_CSUM: Device handling packets with > partial checksum supported" > > -# ], > > -# "transports": [ > > -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields > enabled", > > -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors > supported", > > -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec > (legacy)" > > -# ] > > -# }, > > -# "host-features": { > > -# "dev-features": [ > > -# "VHOST_USER_F_PROTOCOL_FEATURES: Vhost-user protocol > features negotiation supported", > > -# "VIRTIO_NET_F_GSO: Handling GSO-type packets > supported", > > -# "VIRTIO_NET_F_CTRL_MAC_ADDR: MAC address set through > control channel", > > -# "VIRTIO_NET_F_GUEST_ANNOUNCE: Driver sending > gratuitous packets supported", > > -# "VIRTIO_NET_F_CTRL_RX_EXTRA: Extra RX mode control > supported", > > -# "VIRTIO_NET_F_CTRL_VLAN: Control channel VLAN > filtering supported", > > -# "VIRTIO_NET_F_CTRL_RX: Control channel RX mode > supported", > > -# "VIRTIO_NET_F_CTRL_VQ: Control channel available", > > -# "VIRTIO_NET_F_STATUS: Configuration status field > available", > > -# "VIRTIO_NET_F_MRG_RXBUF: Driver can merge receive > buffers", > > -# "VIRTIO_NET_F_HOST_UFO: Device can receive UFO", > > -# "VIRTIO_NET_F_HOST_ECN: Device can receive TSO with > ECN", > > -# "VIRTIO_NET_F_HOST_TSO6: Device can receive TSOv6", > > -# "VIRTIO_NET_F_HOST_TSO4: Device can receive TSOv4", > > -# "VIRTIO_NET_F_GUEST_UFO: Driver can receive UFO", > > -# "VIRTIO_NET_F_GUEST_ECN: Driver can receive TSO with > ECN", > > -# "VIRTIO_NET_F_GUEST_TSO6: Driver can receive TSOv6", > > -# "VIRTIO_NET_F_GUEST_TSO4: Driver can receive TSOv4", > > -# "VIRTIO_NET_F_MAC: Device has given MAC address", > > -# "VIRTIO_NET_F_CTRL_GUEST_OFFLOADS: Control channel > offloading reconfig. supported", > > -# "VIRTIO_NET_F_GUEST_CSUM: Driver handling packets > with partial checksum supported", > > -# "VIRTIO_NET_F_CSUM: Device handling packets with > partial checksum supported" > > -# ], > > -# "transports": [ > > -# "VIRTIO_RING_F_EVENT_IDX: Used & avail. event fields > enabled", > > -# "VIRTIO_RING_F_INDIRECT_DESC: Indirect descriptors > supported", > > -# "VIRTIO_F_VERSION_1: Device compliant for v1 spec > (legacy)", > > -# "VIRTIO_F_ANY_LAYOUT: Device accepts arbitrary desc. > layouts", > > -# "VIRTIO_F_NOTIFY_ON_EMPTY: Notify when device runs > out of avail. descs. on VQ" > > -# ] > > -# }, > > +# "guest-features": 5111807911, > > +# "host-features": 6337593319, > > # "use-guest-notifier-mask": true, > > # "vm-running": true, > > # "queue-sel": 2, > > qmp_decode_protocols(), qmp_decode_status(), and qmp_decode_features() > are now only used in virtio-hmp-cmds.c. Please move them there, and > replace the qmp_ prefix by hmp_ or simply drop it. > > VhostDeviceProtocols and VirtioDeviceFeatures are then only used in > virtio-hmp-cmds.c. Please define them as plain C types there, and drop > them from the QAPI schema. > > You may want to do this as a separate cleanup commit. Use your > judgement. > Get it, I'll try in the next version. > Thanks for the comment. Yong -- Best regards [-- Attachment #2: Type: text/html, Size: 33667 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2 2/2] hmp: Drop unknown feature and status bits 2024-01-04 16:19 [PATCH v2 0/2] Adjust the output of x-query-virtio-status Hyman Huang 2024-01-04 16:19 ` [PATCH v2 1/2] qapi/virtio: Keep feature and status bits in the QMP output Hyman Huang @ 2024-01-04 16:19 ` Hyman Huang 2024-01-12 14:03 ` Markus Armbruster 1 sibling, 1 reply; 7+ messages in thread From: Hyman Huang @ 2024-01-04 16:19 UTC (permalink / raw) To: qemu-devel Cc: Michael S . Tsirkin, Markus Armbruster, Philippe Mathieu-Daudé, Eric Blake, yong.huang The QMP command "x-query-virtio-status" outputs the full feature and status bit information, so there is no need to maintain it in the HMP output; drop it. Signed-off-by: Hyman Huang <yong.huang@smartx.com> --- hw/virtio/virtio-hmp-cmds.c | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/hw/virtio/virtio-hmp-cmds.c b/hw/virtio/virtio-hmp-cmds.c index 4fabba4f9c..ae27968523 100644 --- a/hw/virtio/virtio-hmp-cmds.c +++ b/hw/virtio/virtio-hmp-cmds.c @@ -27,10 +27,6 @@ static void hmp_virtio_dump_protocols(Monitor *mon, } } monitor_printf(mon, "\n"); - if (pcol->has_unknown_protocols) { - monitor_printf(mon, " unknown-protocols(0x%016"PRIx64")\n", - pcol->unknown_protocols); - } } static void hmp_virtio_dump_status(Monitor *mon, @@ -47,10 +43,6 @@ static void hmp_virtio_dump_status(Monitor *mon, } } monitor_printf(mon, "\n"); - if (status->has_unknown_statuses) { - monitor_printf(mon, " unknown-statuses(0x%016"PRIx32")\n", - status->unknown_statuses); - } } static void hmp_virtio_dump_features(Monitor *mon, @@ -81,11 +73,6 @@ static void hmp_virtio_dump_features(Monitor *mon, } monitor_printf(mon, "\n"); } - - if (features->has_unknown_dev_features) { - monitor_printf(mon, " unknown-features(0x%016"PRIx64")\n", - features->unknown_dev_features); - } } void hmp_virtio_query(Monitor *mon, const QDict *qdict) -- 2.39.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2 2/2] hmp: Drop unknown feature and status bits 2024-01-04 16:19 ` [PATCH v2 2/2] hmp: Drop unknown feature and status bits Hyman Huang @ 2024-01-12 14:03 ` Markus Armbruster 2024-01-12 14:58 ` Yong Huang 0 siblings, 1 reply; 7+ messages in thread From: Markus Armbruster @ 2024-01-12 14:03 UTC (permalink / raw) To: Hyman Huang Cc: qemu-devel, Michael S . Tsirkin, Philippe Mathieu-Daudé, Eric Blake Hyman Huang <yong.huang@smartx.com> writes: > The QMP command "x-query-virtio-status" outputs the full > feature and status bit information, so there is no need > to maintain it in the HMP output; drop it. > > Signed-off-by: Hyman Huang <yong.huang@smartx.com> I'm not sure hiding unknown bits from HMP users is a good idea. Sure, you can use QMP to find them, but why would you when HMP gives no clue that its output is incomplete? I'd simply keep them, i.e. drop this patch. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 2/2] hmp: Drop unknown feature and status bits 2024-01-12 14:03 ` Markus Armbruster @ 2024-01-12 14:58 ` Yong Huang 0 siblings, 0 replies; 7+ messages in thread From: Yong Huang @ 2024-01-12 14:58 UTC (permalink / raw) To: Markus Armbruster Cc: qemu-devel, Michael S . Tsirkin, Philippe Mathieu-Daudé, Eric Blake [-- Attachment #1: Type: text/plain, Size: 677 bytes --] On Fri, Jan 12, 2024 at 10:04 PM Markus Armbruster <armbru@redhat.com> wrote: > Hyman Huang <yong.huang@smartx.com> writes: > > > The QMP command "x-query-virtio-status" outputs the full > > feature and status bit information, so there is no need > > to maintain it in the HMP output; drop it. > > > > Signed-off-by: Hyman Huang <yong.huang@smartx.com> > > I'm not sure hiding unknown bits from HMP users is a good idea. Sure, > you can use QMP to find them, but why would you when HMP gives no clue > that its output is incomplete? > > I'd simply keep them, i.e. drop this patch. > > Ok, I'm not insisting on that. Thanks, Yong -- Best regards [-- Attachment #2: Type: text/html, Size: 1771 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2024-01-12 15:05 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-01-04 16:19 [PATCH v2 0/2] Adjust the output of x-query-virtio-status Hyman Huang 2024-01-04 16:19 ` [PATCH v2 1/2] qapi/virtio: Keep feature and status bits in the QMP output Hyman Huang 2024-01-12 14:10 ` Markus Armbruster 2024-01-12 15:02 ` Yong Huang 2024-01-04 16:19 ` [PATCH v2 2/2] hmp: Drop unknown feature and status bits Hyman Huang 2024-01-12 14:03 ` Markus Armbruster 2024-01-12 14:58 ` Yong Huang
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).