[v1] util/event_notifier: more strict API

[PATCH 0/4] util/event_notifier: more strict API

Vladimir Sementsov-Ogievskiy posted 4 patches 2 weeks, 1 day ago

Diff against v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260911180854.123969-1-vsementsov@yandex-team.ru

Maintainers: Elena Ufimtseva <elena.ufimtseva@oracle.com>, Jagannathan Raman <jag.raman@oracle.com>, "Michael S. Tsirkin" <mst@redhat.com>, Stefano Garzarella <sgarzare@redhat.com>, "Eugenio Pérez" <eperezma@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>

There is a newer version of this series

hw/remote/iohub.c                  |  2 --
hw/virtio/vhost-shadow-virtqueue.c | 32 +++++++++++++++++-------------
include/qemu/event_notifier.h      |  1 +
util/event_notifier-posix.c        |  5 +++--
4 files changed, 22 insertions(+), 18 deletions(-)

Expand all Fold all

[PATCH 0/4] util/event_notifier: more strict API

Posted by Vladimir Sementsov-Ogievskiy 2 weeks, 1 day ago

Some callers explicitly set fd to -1, to mark event_notifier
"uninitialized". In the same time, we have .uninitialized
field in the notifier itself. Let's cleanup the mess.

Vladimir Sementsov-Ogievskiy (4):
  util/event_notifier: return -1 for uninitialized notifiers
  remote/iohub: stop  set event notifiers to -1
  vhost: stop set event notifier to -1
  util/event_notifier: assert that init_fd receives a valid fd

 hw/remote/iohub.c                  |  2 --
 hw/virtio/vhost-shadow-virtqueue.c | 32 +++++++++++++++++-------------
 include/qemu/event_notifier.h      |  1 +
 util/event_notifier-posix.c        |  5 +++--
 4 files changed, 22 insertions(+), 18 deletions(-)

-- 
2.43.0