From: Yosuke Iwamatsu <y-iwamatsu@ab.jp.nec.com>
To: Masroor Vettuparambil <Masroor.Vettuparambil@neterion.com>
Cc: xen-devel@lists.xensource.com
Subject: incompatibility of netfront driver with bonding module
Date: Thu, 03 Apr 2008 15:29:32 +0900 [thread overview]
Message-ID: <47F4794C.2040707@ab.jp.nec.com> (raw)
In-Reply-To: <78C9135A3D2ECE4B8162EBDCE82CAD77034D3337@nekter>
[-- Attachment #1: Type: text/plain, Size: 1032 bytes --]
Hi,
Masroor Vettuparambil wrote:
> 1. Normally, bonding will inherit the mac from the first slave and
> assign it to all the other slaves added later. So the mac of vif will be
> updated. But during migration, the mac of vif is getting updated from
> xenstore(/vm/). So how about having a set_mac_address entry in netfront
> that update the xenstore?
> So I need help, especially on #1. Is it ok to update the xenstore
> /vm/ keys from domU? how to do this?
I tried a bit to find out the way to update /vm/ keys from frontend,
but didn't succeed. Either way, I don't think it is a good idea to
update the xenstore key of the vif mac address, because the original
mac will be lost and we won't be able to reset it e.g. after the guest
reboot.
So the attached patch adds set mac_address() support to netfront.
It doesn't touch xenstore at all, but just preserves the modified mac
address in netfront_info structure and keeps using it after migration.
Thanks,
-----------------------
Yosuke Iwamatsu
NEC Corporation
[-- Attachment #2: netfront_set_mac.patch --]
[-- Type: all/allfiles, Size: 1946 bytes --]
[-- Attachment #3: Type: text/plain, Size: 138 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
next prev parent reply other threads:[~2008-04-03 6:29 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-31 15:40 incompatibility of netfront driver with bonding module Masroor Vettuparambil
2008-04-03 6:29 ` Yosuke Iwamatsu [this message]
2008-04-04 10:25 ` Masroor Vettuparambil
2008-04-07 7:05 ` Yosuke Iwamatsu
2008-04-10 10:41 ` Masroor Vettuparambil
2008-04-10 12:05 ` [PATCH] " Yosuke Iwamatsu
2008-04-10 12:24 ` Masroor Vettuparambil
2008-04-10 12:30 ` Yosuke Iwamatsu
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=47F4794C.2040707@ab.jp.nec.com \
--to=y-iwamatsu@ab.jp.nec.com \
--cc=Masroor.Vettuparambil@neterion.com \
--cc=xen-devel@lists.xensource.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.