[v3] hw/virtio: Minor housekeeping patches

[PATCH v3 0/3] hw/virtio: Minor housekeeping patches

Philippe Mathieu-Daudé posted 3 patches 2 years, 7 months ago

Diff against v1 v2
Download series mbox

Test checkpatch passed

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

include/hw/virtio/virtio.h |  7 +++++++
hw/virtio/virtio.c         | 32 +++++++++++++++-----------------
2 files changed, 22 insertions(+), 17 deletions(-)

Expand all Fold all

[PATCH v3 0/3] hw/virtio: Minor housekeeping patches

Posted by Philippe Mathieu-Daudé 2 years, 7 months ago

Hi,

This series contains few patches I gathered while tooking notes
trying to understand issues #300-#302.

Since v2:
- Rebased on top of 88afdc92b64 ("Merge 'remotes/mst/tags/for_upstream' into staging")

Since v1:
- Added virtqueue_flush comment (Stefano)
- Call RCU_READ_LOCK_GUARD in virtqueue_packed_drop_all (Stefano)

Philippe Mathieu-Daudé (3):
  hw/virtio: Comment virtqueue_flush() must be called with RCU read lock
  hw/virtio: Acquire RCU read lock in virtqueue_packed_drop_all()
  hw/virtio: Have virtqueue_get_avail_bytes() pass caches arg to callees

 include/hw/virtio/virtio.h |  7 +++++++
 hw/virtio/virtio.c         | 32 +++++++++++++++-----------------
 2 files changed, 22 insertions(+), 17 deletions(-)

-- 
2.31.1

Re: [PATCH v3 0/3] hw/virtio: Minor housekeeping patches

Posted by Stefano Garzarella 2 years, 7 months ago

On Mon, Sep 06, 2021 at 12:43:15PM +0200, Philippe Mathieu-Daudé wrote:
>Hi,
>
>This series contains few patches I gathered while tooking notes
>trying to understand issues #300-#302.
>
>Since v2:
>- Rebased on top of 88afdc92b64 ("Merge 'remotes/mst/tags/for_upstream' into staging")
>
>Since v1:
>- Added virtqueue_flush comment (Stefano)
>- Call RCU_READ_LOCK_GUARD in virtqueue_packed_drop_all (Stefano)
>
>Philippe Mathieu-Daudé (3):
>  hw/virtio: Comment virtqueue_flush() must be called with RCU read lock
>  hw/virtio: Acquire RCU read lock in virtqueue_packed_drop_all()
>  hw/virtio: Have virtqueue_get_avail_bytes() pass caches arg to callees
>
> include/hw/virtio/virtio.h |  7 +++++++
> hw/virtio/virtio.c         | 32 +++++++++++++++-----------------
> 2 files changed, 22 insertions(+), 17 deletions(-)
>
>-- 
>2.31.1
>
>

Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>