ping On Sun, Nov 12, 2023 at 9:03 PM Hyman Huang <yong.hu...@smartx.com> wrote:
> The patchset "Fix the virtio features negotiation flaw" fix a > vhost-user negotiation flaw: > c9bdc449f9 vhost-user: Fix the virtio features negotiation flaw > bebcac052a vhost-user: Refactor the chr_closed_bh > 937b7d96e4 vhost-user: Refactor vhost acked features saving > > While the test case remain unmerged, the detail reference: > > https://lore.kernel.org/qemu-devel/cover.1667232396.git.huang...@chinatelecom.cn/ > > Since Michael pointed out that the info virtio makes sense to query > the negotiation feature, this patchset uses the x-query-virtio-status > to retrieve the features instead of exporting netdev capabilities and > information as we did in the previous patchset to aid in confirming > the negotiation's validity. > > To do that, we first introduce an "show-bits" argument for > x-query-virtio-status such that the feature bits can be used > directly, and then implement the test case for negotiated features > check. As we post, the code is divided into two patches. > > Please review, thanks, > Yong > > Hyman Huang (2): > qapi/virtio: introduce the "show-bits" argument for > x-query-virtio-status > vhost-user-test: Add negotiated features check > > hw/virtio/virtio-hmp-cmds.c | 2 +- > hw/virtio/virtio-qmp.c | 21 ++++++- > qapi/virtio.json | 49 ++++++++++++++++- > tests/qtest/vhost-user-test.c | 100 ++++++++++++++++++++++++++++++++++ > 4 files changed, 167 insertions(+), 5 deletions(-) > > -- > 2.39.1 > > -- Best regards