From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Jackson Subject: Re: [PATCH] fix: domains do not get created when using vifname variable for bridged interfaces Date: Mon, 7 Feb 2011 16:11:48 +0000 Message-ID: <19792.6596.96339.883733@mariner.uk.xensource.com> References: <201102051726.30996.pittipatti@web.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <201102051726.30996.pittipatti@web.de> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Patrick Scharrenberg Cc: Anthony PERARD , xen-devel@lists.xensource.com, Stefano Stabellini List-Id: xen-devel@lists.xenproject.org Patrick Scharrenberg writes ("[Xen-devel] [PATCH] fix: domains do not get created when using vifname variable for bridged interfaces"): > In the two scripts vif-bridge and vif-route the variable containing > the right interface-name, after an interface was renamed using > "ifname", is $vif. Otherwise hotplug can't handle renamed > interfaces and prevents xm from creating domains. This seems to be partly reverting 21922:0232bc7c9544 "tools/hotplug, Use udev rules instead of qemu script to setup the bridge." I can't exactly follow the logic of 21922 but it seems to me that the bug is that in the case of an interface of type vif which is later renamed (by the code around line 75 of vif-common.sh in xen-unstable), the shell variable dev is not changed. The change you are proposing would also an effect in the case of interfaces of type tap and I'm not convinced that in this case it would do the right thing. Anthony, Stefano: as the authors of 21922 would you care to comment ? Ian.