Re: [BUG FIX][PATCH v3 0/3] vhost-user-blk: fix bug on device disconnection during initialization

[Valentin Sinitsyn]

On 01.04.2021 14:21, Denis Plotnikov wrote:
> This is a series fixing a bug in host-user-blk.
More specifically, it's not just a bug but crasher.

Valentine

> Is there any chance for it to be considered for the next rc?
>
> Thanks!
>
> Denis
>
> On 29.03.2021 16:44, Denis Plotnikov wrote:
> > ping!
> >
> > On 25.03.2021 18:12, Denis Plotnikov wrote:
> > > v3:
> > >    * 0003: a new patch added fixing the problem on vm shutdown
> > >      I stumbled on this bug after v2 sending.
> > >    * 0001: gramma fixing (Raphael)
> > >    * 0002: commit message fixing (Raphael)
> > >
> > > v2:
> > >    * split the initial patch into two (Raphael)
> > >    * rename init to realized (Raphael)
> > >    * remove unrelated comment (Raphael)
> > >
> > > When the vhost-user-blk device lose the connection to the daemon during
> > > the initialization phase it kills qemu because of the assert in the code.
> > > The series fixes the bug.
> > >
> > > 0001 is preparation for the fix
> > > 0002 fixes the bug, patch description has the full motivation for the series
> > > 0003 (added in v3) fix bug on vm shutdown
> > >
> > > Denis Plotnikov (3):
> > >    vhost-user-blk: use different event handlers on initialization
> > >    vhost-user-blk: perform immediate cleanup if disconnect on
> > >      initialization
> > >    vhost-user-blk: add immediate cleanup on shutdown
> > >
> > >   hw/block/vhost-user-blk.c | 79 ++++++++++++++++++++++++---------------
> > >   1 file changed, 48 insertions(+), 31 deletions(-)