| Message ID | 20220303115911.20962-1-slp@redhat.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Enable vhost-user to be used on BSD systems | expand |
On Thu, Mar 03, 2022 at 12:59:07PM +0100, Sergio Lopez wrote: > Since QEMU is already able to emulate ioeventfd using pipefd, we're already > pretty close to supporting vhost-user on non-Linux systems. > > This two patches bridge the gap by: > > 1. Adding a new event_notifier_get_wfd() to return wfd on the places where > the peer is expected to write to the notifier. > > 2. Modifying the build system to it allows enabling vhost-user on BSD. > > v1->v2: > - Drop: "Allow returning EventNotifier's wfd" (Alex Williamson) > - Add: "event_notifier: add event_notifier_get_wfd()" (Alex Williamson) > - Add: "vhost: use wfd on functions setting vring call fd" > - Rename: "Allow building vhost-user in BSD" to "configure, meson: allow > enabling vhost-user on all POSIX systems" > - Instead of making possible enabling vhost-user on Linux and BSD systems, > allow enabling it on all non-Windows platforms. (Paolo Bonzini) > > v2->v3: > - Add a section to docs/interop/vhost-user.rst explaining how vhost-user > is supported on non-Linux platforms. (Stefan Hajnoczi) > > Sergio Lopez (4): > event_notifier: add event_notifier_get_wfd() > vhost: use wfd on functions setting vring call fd > configure, meson: allow enabling vhost-user on all POSIX systems > docs: vhost-user: add subsection for non-Linux platforms > > configure | 4 ++-- > docs/interop/vhost-user.rst | 18 ++++++++++++++++++ > hw/virtio/vhost.c | 6 +++--- > include/qemu/event_notifier.h | 1 + > meson.build | 2 +- > util/event_notifier-posix.c | 5 +++++ > 6 files changed, 30 insertions(+), 6 deletions(-) > > -- > 2.35.1 > > I posted comments on the vhost-user.rst patch. Otherwise: Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
On Thu, Mar 03, 2022 at 12:59:07PM +0100, Sergio Lopez wrote: > Since QEMU is already able to emulate ioeventfd using pipefd, we're already > pretty close to supporting vhost-user on non-Linux systems. > > This two patches bridge the gap by: > > 1. Adding a new event_notifier_get_wfd() to return wfd on the places where > the peer is expected to write to the notifier. > > 2. Modifying the build system to it allows enabling vhost-user on BSD. > > v1->v2: > - Drop: "Allow returning EventNotifier's wfd" (Alex Williamson) > - Add: "event_notifier: add event_notifier_get_wfd()" (Alex Williamson) > - Add: "vhost: use wfd on functions setting vring call fd" > - Rename: "Allow building vhost-user in BSD" to "configure, meson: allow > enabling vhost-user on all POSIX systems" > - Instead of making possible enabling vhost-user on Linux and BSD systems, > allow enabling it on all non-Windows platforms. (Paolo Bonzini) I picked 1,2. Waiting on updated doc patch to apply 3,4. > v2->v3: > - Add a section to docs/interop/vhost-user.rst explaining how vhost-user > is supported on non-Linux platforms. (Stefan Hajnoczi) > > Sergio Lopez (4): > event_notifier: add event_notifier_get_wfd() > vhost: use wfd on functions setting vring call fd > configure, meson: allow enabling vhost-user on all POSIX systems > docs: vhost-user: add subsection for non-Linux platforms > > configure | 4 ++-- > docs/interop/vhost-user.rst | 18 ++++++++++++++++++ > hw/virtio/vhost.c | 6 +++--- > include/qemu/event_notifier.h | 1 + > meson.build | 2 +- > util/event_notifier-posix.c | 5 +++++ > 6 files changed, 30 insertions(+), 6 deletions(-) > > -- > 2.35.1 >
Since QEMU is already able to emulate ioeventfd using pipefd, we're already pretty close to supporting vhost-user on non-Linux systems. This two patches bridge the gap by: 1. Adding a new event_notifier_get_wfd() to return wfd on the places where the peer is expected to write to the notifier. 2. Modifying the build system to it allows enabling vhost-user on BSD. v1->v2: - Drop: "Allow returning EventNotifier's wfd" (Alex Williamson) - Add: "event_notifier: add event_notifier_get_wfd()" (Alex Williamson) - Add: "vhost: use wfd on functions setting vring call fd" - Rename: "Allow building vhost-user in BSD" to "configure, meson: allow enabling vhost-user on all POSIX systems" - Instead of making possible enabling vhost-user on Linux and BSD systems, allow enabling it on all non-Windows platforms. (Paolo Bonzini) v2->v3: - Add a section to docs/interop/vhost-user.rst explaining how vhost-user is supported on non-Linux platforms. (Stefan Hajnoczi) Sergio Lopez (4): event_notifier: add event_notifier_get_wfd() vhost: use wfd on functions setting vring call fd configure, meson: allow enabling vhost-user on all POSIX systems docs: vhost-user: add subsection for non-Linux platforms configure | 4 ++-- docs/interop/vhost-user.rst | 18 ++++++++++++++++++ hw/virtio/vhost.c | 6 +++--- include/qemu/event_notifier.h | 1 + meson.build | 2 +- util/event_notifier-posix.c | 5 +++++ 6 files changed, 30 insertions(+), 6 deletions(-)