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

Re: [Xen-devel] Trying to adapt Olaf's vif MTU patch for EL5.6's 2.6.18-

To: Teck Choon Giam <giamteckchoon@xxxxxxxxx>
Subject: Re: [Xen-devel] Trying to adapt Olaf's vif MTU patch for EL5.6's 2.6.18-238.9 kernel - patch in-line.
From: Digimer <linux@xxxxxxxxxxx>
Date: Sun, 17 Apr 2011 11:41:41 -0400
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Sun, 17 Apr 2011 08:42:36 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <BANLkTikGwJ_DuRMa9nN6H+g336bZb+0mAg@xxxxxxxxxxxxxx>
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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <4DAA4602.8080701@xxxxxxxxxxx> <BANLkTikGwJ_DuRMa9nN6H+g336bZb+0mAg@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.15) Gecko/20110307 Fedora/3.1.9-0.39.b3pre.fc14 Thunderbird/3.1.9
Morning,

  I've been able to test this patch and it works, provided that the
following patch is applied to /etc/xen/scripts/vif-bridge:

====
--- vif-bridge.orig     2011-04-17 01:58:49.000000000 -0400
+++ vif-bridge  2011-04-17 11:20:14.000000000 -0400
@@ -56,6 +56,11 @@
 case "$command" in
     online)
        setup_bridge_port "$vif"
+        mtu="`ip link show $bridge | awk '/mtu/ { print $5 }'`"
+        if [ -n "$mtu" ] && [ "$mtu" -gt 0 ]
+        then
+                ip link set $vif mtu $mtu || :
+        fi
        add_to_bridge "$bridge" "$vif"
         ;;
====

  Please note, this differs slightly from Olaf's patch found here:
http://lists.xensource.com/archives/html/xen-devel/2011-02/msg00351.html

  The only difference is that, in Xen 3.0.3/EL5.6, the variable name is
'$vif', not '$dev'. I'm trying to create a proper patch for the
xen-3.0.3 but am having trouble with source rpm. I'll post back with a
proper patch once I get it working.

  This is related to bz 697021
(https://bugzilla.redhat.com/show_bug.cgi?id=697021).

-- 
Digimer
E-Mail: digimer@xxxxxxxxxxx
AN!Whitepapers: http://alteeve.com
Node Assassin:  http://nodeassassin.org

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel