| Message ID | 20230530071941.8954-1-sgarzare@redhat.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | block/blkio: support fd passing for virtio-blk-vhost-vdpa driver | expand |
On 5/30/23 2:19 AM, Stefano Garzarella wrote: > v5: > - moved `features` to the object level to simplify libvirt code [Jonathon] > - wrapped a line too long in the documentation [Markus] > - added Stefan R-b tags > > v4: https://lore.kernel.org/qemu-devel/20230526150304.158206-1-sgarzare@redhat.com/ > - added patch 02 to allow libvirt to discover we support fdset [Markus] > - modified the commit description of patch 01 > > v3: https://lore.kernel.org/qemu-devel/20230511091527.46620-1-sgarzare@redhat.com/ > - use qemu_open() on `path` to simplify libvirt code [Jonathon] > - remove patch 01 since we are not using monitor_fd_param() anymore > > v2: https://lore.kernel.org/qemu-devel/20230504092843.62493-1-sgarzare@redhat.com/ > - added patch 01 to use monitor_fd_param() in the blkio module > - use monitor_fd_param() to parse the fd like vhost devices [Stefan] > > v1: https://lore.kernel.org/qemu-devel/20230502145050.224615-1-sgarzare@redhat.com/ > > The virtio-blk-vhost-vdpa driver in libblkio 1.3.0 supports the new > 'fd' property. Let's expose this to the user, so the management layer > can pass the file descriptor of an already opened vhost-vdpa character > device. This is useful especially when the device can only be accessed > with certain privileges. > > Stefano Garzarella (2): > block/blkio: use qemu_open() to support fd passing for virtio-blk > qapi: add '@fdset' feature for BlockdevOptionsVirtioBlkVhostVdpa > > meson.build | 4 ++++ > qapi/block-core.json | 6 +++++ > block/blkio.c | 53 ++++++++++++++++++++++++++++++++++++-------- > 3 files changed, 54 insertions(+), 9 deletions(-) > Looks good from my perspective as a libvirt developer. Thanks, Jonathon
On Tue, May 30, 2023 at 09:19:39AM +0200, Stefano Garzarella wrote: > v5: > - moved `features` to the object level to simplify libvirt code [Jonathon] > - wrapped a line too long in the documentation [Markus] > - added Stefan R-b tags > > v4: https://lore.kernel.org/qemu-devel/20230526150304.158206-1-sgarzare@redhat.com/ > - added patch 02 to allow libvirt to discover we support fdset [Markus] > - modified the commit description of patch 01 > > v3: https://lore.kernel.org/qemu-devel/20230511091527.46620-1-sgarzare@redhat.com/ > - use qemu_open() on `path` to simplify libvirt code [Jonathon] > - remove patch 01 since we are not using monitor_fd_param() anymore > > v2: https://lore.kernel.org/qemu-devel/20230504092843.62493-1-sgarzare@redhat.com/ > - added patch 01 to use monitor_fd_param() in the blkio module > - use monitor_fd_param() to parse the fd like vhost devices [Stefan] > > v1: https://lore.kernel.org/qemu-devel/20230502145050.224615-1-sgarzare@redhat.com/ > > The virtio-blk-vhost-vdpa driver in libblkio 1.3.0 supports the new > 'fd' property. Let's expose this to the user, so the management layer > can pass the file descriptor of an already opened vhost-vdpa character > device. This is useful especially when the device can only be accessed > with certain privileges. > > Stefano Garzarella (2): > block/blkio: use qemu_open() to support fd passing for virtio-blk > qapi: add '@fdset' feature for BlockdevOptionsVirtioBlkVhostVdpa > > meson.build | 4 ++++ > qapi/block-core.json | 6 +++++ > block/blkio.c | 53 ++++++++++++++++++++++++++++++++++++-------- > 3 files changed, 54 insertions(+), 9 deletions(-) > > -- > 2.40.1 > Thanks, applied to my block tree: https://gitlab.com/stefanha/qemu/commits/block Stefan