[v3] virtio: remove unnecessary host_features in ->get_features()

[PATCH RESEND v3 0/2] virtio: remove unnecessary host_features in ->get_features()

Stefan Hajnoczi posted 2 patches 1 year, 5 months ago

Diff against v1 v1 v2 v3
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20221121155020.1915166-1-stefanha@redhat.com

Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Stefan Hajnoczi <stefanha@redhat.com>, Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, Laurent Vivier <lvivier@redhat.com>, Amit Shah <amit@kernel.org>, "Marc-André Lureau" <marcandre.lureau@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Jason Wang <jasowang@redhat.com>, Fam Zheng <fam@euphon.net>, David Hildenbrand <david@redhat.com>

include/hw/virtio/virtio.h  | 7 +++++++
hw/block/virtio-blk.c       | 3 ---
hw/char/virtio-serial-bus.c | 1 -
hw/net/virtio-net.c         | 3 ---
hw/scsi/vhost-scsi-common.c | 3 ---
hw/scsi/virtio-scsi.c       | 4 ----
hw/virtio/virtio-balloon.c  | 2 --
7 files changed, 7 insertions(+), 16 deletions(-)

Expand all Fold all

[PATCH RESEND v3 0/2] virtio: remove unnecessary host_features in ->get_features()

Posted by Stefan Hajnoczi 1 year, 5 months ago

v3 (resend):
- On top of master instead of the qemu.git/staging tree :)
v3:
- Use an informal comment instead of a full doc comment [Cornelia]
v2:
- Document vdv->get_features() callback [Cornelia]

The vdc->get_features() callbacks are a little inconsistent in how they use
vdev->host_features. This is because the function's behavior changed over time.
Clean things up.

Stefan Hajnoczi (2):
  virtio: document ->host_features usage in vdc->get_features() callback
  virtio: remove unnecessary host_features in ->get_features()

 include/hw/virtio/virtio.h  | 7 +++++++
 hw/block/virtio-blk.c       | 3 ---
 hw/char/virtio-serial-bus.c | 1 -
 hw/net/virtio-net.c         | 3 ---
 hw/scsi/vhost-scsi-common.c | 3 ---
 hw/scsi/virtio-scsi.c       | 4 ----
 hw/virtio/virtio-balloon.c  | 2 --
 7 files changed, 7 insertions(+), 16 deletions(-)

-- 
2.38.1