[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v2 28/33] migration: return incoming task tag for
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [RFC v2 28/33] migration: return incoming task tag for sockets |
Date: |
Thu, 28 Sep 2017 11:12:55 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Fri, Sep 22, 2017 at 09:11:50PM +0100, Dr. David Alan Gilbert wrote:
[...]
> > -void tcp_start_incoming_migration(const char *host_port, Error **errp)
> > +guint tcp_start_incoming_migration(const char *host_port, Error **errp)
> > {
> > Error *err = NULL;
> > SocketAddress *saddr = tcp_build_address(host_port, &err);
> > + guint tag;
> > +
> > if (!err) {
> > - socket_start_incoming_migration(saddr, &err);
> > + tag = socket_start_incoming_migration(saddr, &err);
> > }
>
> I'd be tempted to initialise that tag = 0 for the case where
> there's an error; but OK.
Yeh, I think it worths a touch-up.
>
> Reviewed-by: Dr. David Alan Gilbert <address@hidden>
Will take the r-b after fixing above. Thanks!
--
Peter Xu