On Fri, Feb 02, 2024 at 10:32:14PM +0800, Hyman Huang wrote:
> Sorry for the late post of version 3. The modifications are as follows:
>
> v3:
> - Rebase on master
> - Use the refined commit message furnished by Markus for [PATCH v2 1/2]
> - Drop the [PATCH v2 2/2]
> - Add [PATCH v3 2/3] to declare the decoding functions to static
> - Add [PATCH v3 3/3] to Define VhostDeviceProtocols and
> VirtioDeviceFeatures as plain C types
>
> Since Markus inspired all of the alterations above, we would like to
> thank him for his contribution to this series.
>
> Please review,
> Yong
Markus do you think you can take a look at this?
> v2:
> - Changing the hmp_virtio_dump_xxx function signatures to implement
> the bitmap decoding, suggested by Philippe.
>
> 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 (3):
> qmp: Switch x-query-virtio-status back to numeric encoding
> virtio: Declare the decoding functions to static
> qapi: Define VhostDeviceProtocols and VirtioDeviceFeatures as plain C
> types
>
> hw/virtio/meson.build | 3 +-
> hw/virtio/virtio-hmp-cmds.c | 702 +++++++++++++++++++++++++++++++++++-
> hw/virtio/virtio-qmp.c | 684 +----------------------------------
> hw/virtio/virtio-qmp.h | 3 -
> qapi/virtio.json | 231 +-----------
> 5 files changed, 723 insertions(+), 900 deletions(-)
>
> --
> 2.31.1