tools/hotplug: set mtu from bridge for tap interface

With changeset 22885 support was added for setting the MTU in the vif-bridge
script for when a vif interface was set to 'online'.  The was not done for the
'add' operation.  The 'add' operation was added to the script for when tap
devices were specified (c/s 21944). With the setting of the MTU for the
'online' case was there a reason for omitting the 'add'?

This patch sets the MTU for both 'online' and 'add' in the vif-bridge script.

Signed-off-by: Charles Arnold <carnold@suse.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
This commit is contained in:
Charles Arnold authored and Ian Campbell committed 2013-09-17 15:26:27 +01:00
1 parent c5e9596cd0
commit f3f5f1927f
2 files changed
+12 -5

No files matched your search

+2 -5
View File
@@ -81,11 +81,7 @@ fi
case "$command" in
online)
setup_virtual_bridge_port "$dev"
mtu="`ip link show $bridge | awk '/mtu/ { print $5 }'`"
if [ -n "$mtu" ] && [ "$mtu" -gt 0 ]
then
ip link set $dev mtu $mtu || :
fi
set_mtu $bridge $dev
add_to_bridge "$bridge" "$dev"
;;
@@ -96,6 +92,7 @@ case "$command" in
add)
setup_virtual_bridge_port "$dev"
set_mtu $bridge $dev
add_to_bridge "$bridge" "$dev"
;;
esac
+10
View File
@@ -132,3 +132,13 @@ add_to_bridge () {
ip link set ${dev} up
}
# Usage: set_mtu bridge dev
set_mtu () {
local bridge=$1
local dev=$2
mtu="`ip link show ${bridge}| awk '/mtu/ { print $5 }'`"
if [ -n "$mtu" ] && [ "$mtu" -gt 0 ]
then
ip link set ${dev} mtu $mtu || :
fi
}
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.