From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Wolf Subject: Re: [Qemu-devel] [PATCH 1/4] Use getaddrinfo for migration Date: Fri, 24 Feb 2012 10:34:00 +0100 Message-ID: <4F475988.10603@redhat.com> References: <20120210062608.13397.43361.stgit@dhcp-8-167.nay.redhat.com> <20120210062700.13397.6305.stgit@dhcp-8-167.nay.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Cc: aliguori@us.ibm.com, kvm@vger.kernel.org, quintela@redhat.com, jasowang@redhat.com, qemu-devel@nongnu.org, laine@redhat.com To: Amos Kong Return-path: Received: from mx1.redhat.com ([209.132.183.28]:6575 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756164Ab2BXJag (ORCPT ); Fri, 24 Feb 2012 04:30:36 -0500 In-Reply-To: <20120210062700.13397.6305.stgit@dhcp-8-167.nay.redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: Am 10.02.2012 07:27, schrieb Amos Kong: > This allows us to use ipv4/ipv6 for migration addresses. > Once there, it also uses /etc/services names (it came free). > > Signed-off-by: Juan Quintela > Signed-off-by: Amos Kong > --- > migration-tcp.c | 60 ++++++++----------------------- > net.c | 108 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ > qemu_socket.h | 3 ++ > 3 files changed, 127 insertions(+), 44 deletions(-) > @@ -157,28 +141,16 @@ out2: > > int tcp_start_incoming_migration(const char *host_port) > { > - struct sockaddr_in addr; > - int val; > + int ret; > int s; > > DPRINTF("Attempting to start an incoming migration\n"); > > - if (parse_host_port(&addr, host_port) < 0) { > - fprintf(stderr, "invalid host/port combination: %s\n", host_port); > - return -EINVAL; > - } Oh, and this case doesn't print an error message any more now. Kevin