Re: [Qemu-devel] [PATCH v2 3/5] libvhost-user: quit when no more data received

[Jens Freimann]

On Thu, Sep 21, 2017 at 01:31:37PM +0000, Jens Freimann wrote:
> On Wed, Sep 20, 2017 at 04:14:33PM +0000, Marc-Andr?? Lureau wrote:
> > On Wed, Sep 20, 2017 at 5:09 PM, Jens Freimann <address@hidden> wrote:
> > > On Tue, Sep 19, 2017 at 04:46:24PM +0000, Marc-Andr?? Lureau wrote:
> > > > On Tue, Aug 8, 2017 at 10:52 PM Jens Freimann <address@hidden>
> > > > wrote:
> > We want to treat disconnect as a panic condition imho, that the
> > library user is free to implement in different way (abort() clean
> > exit, reconnect etc).
>
> Ok, that wasn't obvious to me. Thanks for clarifying! So I was 
> "fixing" the wrong part. On a disconnect vubr_panic() in
> vhost-user-bridge.c is called and is supposed to do the right thing.
>
> Currently it will print an error message "PANIC: Unhandled request:

Actually it is "Error while recvmsg" because in vu_message_read()
vu_panic is called if rc from recvmsg is <= 0. Followed by "Error
while dispatching" printed by vhost-user-bridge. 

regards,
Jens