mbox series

[0/7] vhost: FD manager improvements

Message ID 20240229122502.2572343-1-maxime.coquelin@redhat.com (mailing list archive)
Headers
Series vhost: FD manager improvements |

Message

Maxime Coquelin Feb. 29, 2024, 12:24 p.m. UTC 
  This series aims at improving the Vhost FD manager.

First patch is a fix necessary to have VDUSE devices
destroy to work. I expect it to be taken into v24.03
release.

The rest of the series are various improvements to the
FD manager that can wait v24.07 release.

Maxime Coquelin (7):
  vhost: fix VDUSE device destruction failure
  vhost: rename polling mutex
  vhost: make use of FD manager init function
  vhost: hide synchronization within FD manager
  vhost: improve fdset initialization
  vhost: convert fdset sync to eventfd
  vhost: improve FD manager logging

 lib/vhost/fd_man.c      | 313 +++++++++++++++++------
 lib/vhost/fd_man.c.orig | 538 ++++++++++++++++++++++++++++++++++++++++
 lib/vhost/fd_man.h      |  41 +--
 lib/vhost/socket.c      |  37 +--
 lib/vhost/vduse.c       |  51 +---
 5 files changed, 800 insertions(+), 180 deletions(-)
 create mode 100644 lib/vhost/fd_man.c.orig

Comments

David Marchand March 4, 2024, 10:35 a.m. UTC | #1 
On Thu, Feb 29, 2024 at 1:25 PM Maxime Coquelin
<maxime.coquelin@redhat.com> wrote:
>
> This series aims at improving the Vhost FD manager.
>
> First patch is a fix necessary to have VDUSE devices
> destroy to work. I expect it to be taken into v24.03
> release.
>
> The rest of the series are various improvements to the
> FD manager that can wait v24.07 release.
>
> Maxime Coquelin (7):
>   vhost: fix VDUSE device destruction failure
>   vhost: rename polling mutex
>   vhost: make use of FD manager init function
>   vhost: hide synchronization within FD manager
>   vhost: improve fdset initialization
>   vhost: convert fdset sync to eventfd
>   vhost: improve FD manager logging
>
>  lib/vhost/fd_man.c      | 313 +++++++++++++++++------
>  lib/vhost/fd_man.c.orig | 538 ++++++++++++++++++++++++++++++++++++++++
>  lib/vhost/fd_man.h      |  41 +--
>  lib/vhost/socket.c      |  37 +--
>  lib/vhost/vduse.c       |  51 +---
>  5 files changed, 800 insertions(+), 180 deletions(-)
>  create mode 100644 lib/vhost/fd_man.c.orig

I marked all but the first patch as Deferred.
I'll send a new revision of the fix to address the deadlock revealed by CI.