All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Roth <mdroth@linux.vnet.ibm.com>
To: Eric Blake <eblake@redhat.com>
Cc: aliguori@us.ibm.com, quintela@redhat.com, jasowang@redhat.com,
	qemu-devel@nongnu.org, owasserm@redhat.com,
	Amos Kong <akong@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v12 0/4] support to migrate with IPv6 address
Date: Thu, 10 May 2012 13:12:51 -0500	[thread overview]
Message-ID: <20120510181251.GE17353@illuin> (raw)
In-Reply-To: <4FABFB0C.7010801@redhat.com>

On Thu, May 10, 2012 at 11:29:48AM -0600, Eric Blake wrote:
> On 05/10/2012 10:27 AM, Amos Kong wrote:
> > Those patches updated help functions in qemu-socket.c,
> > and used them in migrate-tcp.c to supporting IPv6 migration.
> > 
> 
> > ---
> > 
> > Amos Kong (4):
> >       qerror: add five qerror strings
> >       sockets: change inet_connect() to support nonblock socket
> >       sockets: use error class to pass listen error
> >       use inet_listen()/inet_connect() to support ipv6 migration
> 
> I'm trying to understand the scope of this series, and how it might
> impact what libvirt needs to support for migration in an IPv6
> environment.  Is the point of this patch that IPv6 migration was
> previously not possible, and is now possible using [IP6addr]:port notation?

Looks like Amos went offline: but yes, in a nutshell.

addr parsing now relies on qemu-sockets.c:inet_parse(), which has supported
[ip6addr]:port for a while, as opposed to net.c:parse_host_port(), which
didn't.

Additional Error-handling was added to
qemu-sockets.c:inet_connect()/inet_listen(), along with non-blocking support
for inet_connect(), to subsume the ad-hoc client/server setup in
migration-tcp.c

> 
> -- 
> Eric Blake   eblake@redhat.com    +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
> 

  reply	other threads:[~2012-05-10 18:13 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-10 16:27 [Qemu-devel] [PATCH v12 0/4] support to migrate with IPv6 address Amos Kong
2012-05-10 16:28 ` [Qemu-devel] [PATCH v12 1/4] qerror: add five qerror strings Amos Kong
2012-05-10 17:21   ` Michael Roth
2012-05-10 16:28 ` [Qemu-devel] [PATCH v12 2/4] sockets: change inet_connect() to support nonblock socket Amos Kong
2012-05-10 16:28 ` [Qemu-devel] [PATCH v12 3/4] sockets: use error class to pass listen error Amos Kong
2012-05-10 16:28 ` [Qemu-devel] [PATCH v12 4/4] use inet_listen()/inet_connect() to support ipv6 migration Amos Kong
2012-05-10 17:29 ` [Qemu-devel] [PATCH v12 0/4] support to migrate with IPv6 address Eric Blake
2012-05-10 18:12   ` Michael Roth [this message]
2012-05-11  6:46     ` Amos Kong
2012-05-11 12:46       ` Eric Blake
2012-05-14 15:04 ` Anthony Liguori

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120510181251.GE17353@illuin \
    --to=mdroth@linux.vnet.ibm.com \
    --cc=akong@redhat.com \
    --cc=aliguori@us.ibm.com \
    --cc=eblake@redhat.com \
    --cc=jasowang@redhat.com \
    --cc=owasserm@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.