[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 16/24] vhost user: add rarp sending after liv
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH v7 16/24] vhost user: add rarp sending after live migration for legacy guest |
Date: |
Fri, 2 Oct 2015 09:55:01 -0400 (EDT) |
----- Original Message -----
> On Thu, Oct 01, 2015 at 07:24:00PM +0200, address@hidden wrote:
> > diff --git a/hw/net/vhost_net.c b/hw/net/vhost_net.c
> > index 840f443..da66b64 100644
> > --- a/hw/net/vhost_net.c
> > +++ b/hw/net/vhost_net.c
> > @@ -388,6 +388,18 @@ void vhost_net_cleanup(struct vhost_net *net)
> > g_free(net);
> > }
> >
> > +int vhost_net_notify_migration_done(struct vhost_net *net, char* mac_addr)
> > +{
> > + const VhostOps *vhost_ops = net->dev.vhost_ops;
> > + int r = -1;
> > +
> > + if (vhost_ops->vhost_migration_done) {
> > + r = vhost_ops->vhost_migration_done(&net->dev, mac_addr);
> > + }
> > +
> > + return r;
> > +}
> > +
> > bool vhost_net_virtqueue_pending(VHostNetState *net, int idx)
> > {
> > return vhost_virtqueue_pending(&net->dev, idx);
> > @@ -479,6 +491,11 @@ void vhost_net_virtqueue_mask(VHostNetState *net,
> > VirtIODevice *dev,
> > {
> > }
> >
> > +int vhost_net_notify_migration_done(struct vhost_net *net)
> > +{
> > + return -1;
> > +}
> > +
> > VHostNetState *get_vhost_net(NetClientState *nc)
> > {
> > return 0;
>
> This signature does not fit the one above.
> How was this tested?
>
Good question, I totally missed that. It has been there since Thibaut v6 series.
I guess we all compile with CONFIG_VHOST_NET, and adding the missing char*
mac_addr is enough to fix this.
- [Qemu-devel] [PATCH v7 13/24] vhost-user: document migration log, (continued)
[Qemu-devel] [PATCH v7 15/24] vhost user: add support of live migration, marcandre . lureau, 2015/10/08
[Qemu-devel] [PATCH v7 18/24] vhost: add migration block if memfd failed, marcandre . lureau, 2015/10/08
[Qemu-devel] [PATCH v7 16/24] vhost user: add rarp sending after live migration for legacy guest, marcandre . lureau, 2015/10/08
[Qemu-devel] [PATCH v7 24/24] vhost-user-test: check ownership during migration, marcandre . lureau, 2015/10/08
[Qemu-devel] [PATCH v7 01/24] configure: probe for memfd, marcandre . lureau, 2015/10/08
[Qemu-devel] [PATCH v7 07/24] vhost: add vhost_set_log_base op, marcandre . lureau, 2015/10/08
[Qemu-devel] [PATCH v7 17/24] vhost-user: use an enum helper for features mask, marcandre . lureau, 2015/10/08
[Qemu-devel] [PATCH v7 20/24] vhost-user-test: remove useless static check, marcandre . lureau, 2015/10/08
[Qemu-devel] [PATCH v7 04/24] util: add memfd helpers, marcandre . lureau, 2015/10/08
Re: [Qemu-devel] [PATCH v7 00/24] vhost-user: add migration support, Michael S. Tsirkin, 2015/10/08