Fri, Jul 24, 2015 at 04:08:35PM CEST, jiri(a)resnulli.us wrote:
Fri, Jul 24, 2015 at 12:06:21PM CEST, jtluka(a)redhat.com wrote:
>This patch adds mtu alias to all phase1 virt tests so that user can
>override the default mtu of size 1500. Using higher mtu in these setups
>gives much better throughput on the receiving side.
>
>Test with mtu=1500:
> tx: 13.7592 Gbits/s
> rx: 0.167 Gbits/s
>
>Test with mtu=8000:
> tx: 25.052 Gbits/s
> rx: 7.4019 Gbits/s
>
>Signed-off-by: Jan Tluka <jtluka(a)redhat.com>
>---
> ...tual_bridge_2_vlans_over_active_backup_bond.xml | 1 +
> .../phase1/virtual_bridge_2_vlans_over_bond.py | 23 ++++++++++++++++++++++
> .../phase1/virtual_bridge_vlan_in_guest.py | 11 +++++++++++
> .../phase1/virtual_bridge_vlan_in_guest.xml | 1 +
> .../phase1/virtual_bridge_vlan_in_host.py | 12 +++++++++++
> .../phase1/virtual_bridge_vlan_in_host.xml | 1 +
> 6 files changed, 49 insertions(+)
>
>diff --git
a/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_active_backup_bond.xml
b/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_active_backup_bond.xml
>index 2c9ed11..d4ff566 100644
>---
a/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_active_backup_bond.xml
>+++
b/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_active_backup_bond.xml
>@@ -2,6 +2,7 @@
> <define>
> <alias name="ipv" value="both" />
> <alias name="netperf_duration" value="60" />
>+ <alias name="mtu" value="1500" />
> </define>
> <network>
> <host id="host1">
>diff --git a/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_bond.py
b/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_bond.py
>index 8347eaf..2ecd7f2 100644
>--- a/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_bond.py
>+++ b/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_bond.py
>@@ -26,6 +26,7 @@ offloads = ["gso", "gro", "tso"]
>
> ipv = ctl.get_alias("ipv")
> netperf_duration = ctl.get_alias("netperf_duration")
>+mtu = ctl.get_alias("mtu")
>
> ping_mod = ctl.get_module("IcmpPing",
> options={
>@@ -145,6 +146,28 @@ ping_mod6_bad2 = ctl.get_module("Icmp6Ping",
> "interval" : 0.1
> })
>
>+# configure mtu on all devices
>+h1.run("ip link set dev %s mtu %s" % (h1.get_devname("bond"),
mtu))
Something like:
bond_if.set_mtu(mtu)
perhaps?
Very valid point. Let's discard this patch.
>+h1.run("ip link set dev %s mtu %s" %
(h1.get_devname("tap1"), mtu))
>+h1.run("ip link set dev %s mtu %s" % (h1.get_devname("tap2"),
mtu))
>+h1.run("ip link set dev %s mtu %s" % (h1.get_devname("vlan10"),
mtu))
>+h1.run("ip link set dev %s mtu %s" % (h1.get_devname("vlan20"),
mtu))
>+h1.run("ip link set dev %s mtu %s" % (h1.get_devname("br1"),
mtu))
>+h1.run("ip link set dev %s mtu %s" % (h1.get_devname("br2"),
mtu))
>+
>+h2.run("ip link set dev %s mtu %s" % (h2.get_devname("bond"),
mtu))
>+h2.run("ip link set dev %s mtu %s" % (h2.get_devname("tap1"),
mtu))
>+h2.run("ip link set dev %s mtu %s" % (h2.get_devname("tap2"),
mtu))
>+h2.run("ip link set dev %s mtu %s" % (h2.get_devname("vlan10"),
mtu))
>+h2.run("ip link set dev %s mtu %s" % (h2.get_devname("vlan20"),
mtu))
>+h2.run("ip link set dev %s mtu %s" % (h2.get_devname("br1"),
mtu))
>+h2.run("ip link set dev %s mtu %s" % (h2.get_devname("br2"),
mtu))
>+
>+g1.run("ip link set dev %s mtu %s" % (g1.get_devname("guestnic"),
mtu))
>+g2.run("ip link set dev %s mtu %s" % (g2.get_devname("guestnic"),
mtu))
>+g3.run("ip link set dev %s mtu %s" % (g3.get_devname("guestnic"),
mtu))
>+g4.run("ip link set dev %s mtu %s" % (g4.get_devname("guestnic"),
mtu))
>+
> ctl.wait(15)
>
> for offload in offloads:
>diff --git a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.py
b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.py
>index bc90f5d..78585a1 100644
>--- a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.py
>+++ b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.py
>@@ -20,6 +20,7 @@ offloads = ["gso", "gro", "tso"]
>
> ipv = ctl.get_alias("ipv")
> netperf_duration = ctl.get_alias("netperf_duration")
>+mtu = ctl.get_alias("mtu")
>
> ping_mod = ctl.get_module("IcmpPing",
> options={
>@@ -92,6 +93,16 @@ netperf_cli_udp6 = ctl.get_module("Netperf",
> "-L %s -6" %
h2.get_ip("vlan10", 1)
> })
>
>+# configure mtu
>+h1.run("ip link set dev %s mtu %s" % (h1.get_devname("nic"),
mtu))
>+h1.run("ip link set dev %s mtu %s" % (h1.get_devname("tap"),
mtu))
>+h1.run("ip link set dev %s mtu %s" % (h1.get_devname("br"),
mtu))
>+
>+g1.run("ip link set dev %s mtu %s" % (g1.get_devname("guestnic"),
mtu))
>+g1.run("ip link set dev %s mtu %s" % (g1.get_devname("vlan10"),
mtu))
>+
>+h2.run("ip link set dev %s mtu %s" % (h2.get_devname("nic"),
mtu))
>+
> ctl.wait(15)
>
> for offload in offloads:
>diff --git a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.xml
b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.xml
>index ea49082..f552876 100644
>--- a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.xml
>+++ b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.xml
>@@ -2,6 +2,7 @@
> <define>
> <alias name="ipv" value="both" />
> <alias name="netperf_duration" value="60" />
>+ <alias name="mtu" value="1500" />
> </define>
> <network>
> <host id="host1">
>diff --git a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.py
b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.py
>index 0344641..c8d22c3 100644
>--- a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.py
>+++ b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.py
>@@ -20,6 +20,7 @@ offloads = ["gso", "gro", "tso"]
>
> ipv = ctl.get_alias("ipv")
> netperf_duration = ctl.get_alias("netperf_duration")
>+mtu = ctl.get_alias("mtu")
>
> ping_mod = ctl.get_module("IcmpPing",
> options={
>@@ -92,6 +93,17 @@ netperf_cli_udp6 = ctl.get_module("Netperf",
> "-L %s -6" %
h2.get_ip("vlan10", 1)
> })
>
>+# configure mtu
>+h1.run("ip link set dev %s mtu %s" % (h1.get_devname("nic"),
mtu))
>+h1.run("ip link set dev %s mtu %s" % (h1.get_devname("tap"),
mtu))
>+h1.run("ip link set dev %s mtu %s" % (h1.get_devname("vlan10"),
mtu))
>+h1.run("ip link set dev %s mtu %s" % (h1.get_devname("br"),
mtu))
>+
>+g1.run("ip link set dev %s mtu %s" % (g1.get_devname("guestnic"),
mtu))
>+
>+h2.run("ip link set dev %s mtu %s" % (h2.get_devname("nic"),
mtu))
>+h2.run("ip link set dev %s mtu %s" % (h2.get_devname("vlan10"),
mtu))
>+
> ctl.wait(15)
>
> for offload in offloads:
>diff --git a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.xml
b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.xml
>index 3649ca9..82a7642 100644
>--- a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.xml
>+++ b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.xml
>@@ -2,6 +2,7 @@
> <define>
> <alias name="ipv" value="both" />
> <alias name="netperf_duration" value="60" />
>+ <alias name="mtu" value="1500" />
> </define>
> <network>
> <host id="host1">
>--
>2.1.0
>
>_______________________________________________
>LNST-developers mailing list
>LNST-developers(a)lists.fedorahosted.org
>https://lists.fedorahosted.org/mailman/listinfo/lnst-developers