From: kaf24@firebug.cl.cam.ac.uk <kaf24@firebug.cl.cam.ac.uk>
Date: Thu, 25 Aug 2005 15:51:30 +0000 (+0000)
Subject: 1. Remove workaround patch
X-Git-Tag: archive/raspbian/4.8.0-1+rpi1~1^2~16844^2~4
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=2a17786484b7424607dba0432d7166a1de105e7c;p=xen.git

1. Remove workaround patch
2. Update scripts/vif-bridge to not call 'brctl delif'

When a domU is shutdown/destroyed and the netif is destroyed, the
notify_call_chain triggered from unregister_netdevice() will trigger the
bridge event handler and which will call the proper code to remove the
device from the bridge.

I can't see any reason why brtcl delif should be called when taking out
a domain if the call chain will delete the interface from the bridge
when the vif is destroyed automatically.

Signed-off-by: Ryan Harper <ryanh@us.ibm.com>
---

diff --git a/patches/linux-2.6.12/workaround_double_br_del_if.patch b/patches/linux-2.6.12/workaround_double_br_del_if.patch
deleted file mode 100644
index 8c43e19628..0000000000
--- a/patches/linux-2.6.12/workaround_double_br_del_if.patch
+++ /dev/null
@@ -1,11 +0,0 @@
---- linux-2.6.12/net/bridge/br_if.c	2005-06-17 14:48:29.000000000 -0500
-+++ linux-2.6.12-xen0-smp/net/bridge/br_if.c	2005-08-18 15:17:27.302615846 -0500
-@@ -382,7 +382,7 @@
- {
- 	struct net_bridge_port *p = dev->br_port;
- 	
--	if (!p || p->br != br) 
-+	if (!p || p->br != br || p->state == BR_STATE_DISABLED)
- 		return -EINVAL;
- 
- 	br_sysfs_removeif(p);
diff --git a/tools/examples/vif-bridge b/tools/examples/vif-bridge
index 1c5bc161f9..f27a4b03ee 100755
--- a/tools/examples/vif-bridge
+++ b/tools/examples/vif-bridge
@@ -74,8 +74,10 @@ if [ "${bridge}" == "null" ] ; then
     exit
 fi
 
-# Add/remove vif to/from bridge.
-brctl ${brcmd} ${bridge} ${vif}
+# Add vif to bridge. vifs are auto-removed from bridge.
+if [ "${brcmd}" == "addif" ] ; then
+    brctl ${brcmd} ${bridge} ${vif}
+fi
 ifconfig ${vif} $OP
 
 if [ ${ip} ] ; then