Fri, Jul 24, 2015 at 04:08:35PM CEST, jiri@resnulli.us wrote:

Fri, Jul 24, 2015 at 12:06:21PM CEST, jtluka@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@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@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/lnst-developers