WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[Xen-devel] [PATCH] >1 IP address on eth0 causes hotplug script failure

To: xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-devel] [PATCH] >1 IP address on eth0 causes hotplug script failure
From: James Dykman <dykman@xxxxxxxxxx>
Date: Wed, 22 Feb 2006 15:47:38 -0500
Delivery-date: Wed, 22 Feb 2006 20:47:59 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
Importance: Normal
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Sensitivity:
Adding a second IP address onto eth0 causes the vif-* hotplug scripts to 
fail: ip_of() is returning a LIST of IP addresses, one per line.
The vif-scripts attempt to use the list where single addresses ought to 
be, with predictable results. This patch changes ip_of() to return
the first address in the list, which seems to be the first one configured.

This came up while testing the patch for the checksum errors on IPSEC 
tunnels. I added an IP address for the tunnel on eth0 and
domUs wouldn't start anymore. 

Since this changes vif-common.sh, I ran xm-test using -bridge, -route, and 
-nat scripts, each with 1 and 4 IP addresses on eth0.

Signed-off-by: Jim Dykman <dykman@xxxxxxxxxx>

diff -r 57e6d7218427 tools/examples/vif-common.sh
--- a/tools/examples/vif-common.sh      Fri Feb  3 18:45:14 2006
+++ b/tools/examples/vif-common.sh      Wed Feb  8 16:08:56 2006
@@ -125,7 +125,7 @@
 #
 function ip_of()
 {
-  ip addr show "$1" | awk "/^.*inet.*$1\$/{print \$2}" | sed 's,/.*,,'
+  ip addr show "$1" | awk "/^.*inet.*$1\$/{print \$2}" | sed 's,/.*,,' | 
head -1
 }




Jim

Attachment: vifcommon.patch
Description: Binary data

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
<Prev in Thread] Current Thread [Next in Thread>