Re: [Qemu-devel] [PATCH v3 10/26] vhost-user: split vhost_user_read()

[Gerd Hoffmann]

On Tue, Jun 19, 2018 at 11:01:00AM +0200, Marc-André Lureau wrote:
> Hi
> 
> On Tue, Jun 19, 2018 at 8:23 AM, Gerd Hoffmann <address@hidden> wrote:
> >> @@ -237,7 +251,7 @@ static int vhost_user_read(struct vhost_dev *dev, 
> >> VhostUserMsg *msg)
> >>          error_report("Failed to read msg header."
> >>                  " Size %d exceeds the maximum %zu.", msg->hdr.size,
> >>                  VHOST_USER_PAYLOAD_SIZE);
> >> -        goto fail;
> >> +        return -1;
> >
> > Hmm?  Looks like a pointless indirection ...
> 
> Isn't it removed by the patch? :)

Ah, right.  Sorry about the noise.

cheers,
  Gerd