This patch modifies all tests that set rx offload. We saw that setting this offload off for certain NICs such as ixgbe does reset of the device. Therefore the link goes down and a delay has to be added after the offload setting so that data sent over the device pass through.

Signed-off-by: Jan Tluka jtluka@redhat.com --- recipes/regression_tests/phase1/3_vlans.py | 4 ++++ recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.py | 5 +++++ recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.py | 5 +++++ recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_guest.py | 5 +++++ recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_host.py | 5 +++++ 5 files changed, 24 insertions(+)

diff --git a/recipes/regression_tests/phase1/3_vlans.py b/recipes/regression_tests/phase1/3_vlans.py index ac20b60..fa2e02d 100644 --- a/recipes/regression_tests/phase1/3_vlans.py +++ b/recipes/regression_tests/phase1/3_vlans.py @@ -149,6 +149,10 @@ for setting in offload_settings: dev_features)) m2.run("ethtool -K %s %s" % (m2_phy1.get_devname(), dev_features)) + if ("rx", "off") in setting: + # when rx offload is turned off some of the cards might get reset + # and link goes down, so wait a few seconds until NIC is ready + ctl.wait(15)

# Ping test for vlan1 in vlans: 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 d3cc786..d15929a 100644 --- a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.py +++ b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.py @@ -166,6 +166,11 @@ for setting in offload_settings: h1.run("ethtool -K %s %s" % (h1_nic.get_devname(), dev_features)) h2.run("ethtool -K %s %s" % (h2_nic.get_devname(), dev_features))

+ if ("rx", "off") in setting: + # when rx offload is turned off some of the cards might get reset + # and link goes down, so wait a few seconds until NIC is ready + ctl.wait(15) + if ipv in [ 'ipv4', 'both' ]: g1.run(ping_mod)

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 ca169af..ed12b67 100644 --- a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.py +++ b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.py @@ -166,6 +166,11 @@ for setting in offload_settings: h1.run("ethtool -K %s %s" % (h1_nic.get_devname(), dev_features)) h2.run("ethtool -K %s %s" % (h2_nic.get_devname(), dev_features))

+ if ("rx", "off") in setting: + # when rx offload is turned off some of the cards might get reset + # and link goes down, so wait a few seconds until NIC is ready + ctl.wait(15) + if ipv in [ 'ipv4', 'both' ]: g1.run(ping_mod)

diff --git a/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_guest.py b/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_guest.py index 76aa386..8ac0da4 100644 --- a/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_guest.py +++ b/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_guest.py @@ -153,6 +153,11 @@ for setting in offload_settings: g1.run("ethtool -K %s %s" % (g1_guestnic.get_devname(), dev_features)) h2.run("ethtool -K %s %s" % (h2_nic.get_devname(), dev_features))

+ if ("rx", "off") in setting: + # when rx offload is turned off some of the cards might get reset + # and link goes down, so wait a few seconds until NIC is ready + ctl.wait(15) + if ipv in [ 'ipv4', 'both' ]: g1.run(ping_mod)

diff --git a/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_host.py b/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_host.py index dcde1b2..5dae9da 100644 --- a/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_host.py +++ b/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_host.py @@ -152,6 +152,11 @@ for setting in offload_settings: g1.run("ethtool -K %s %s" % (g1_guestnic.get_devname(), dev_features)) h2.run("ethtool -K %s %s" % (h2_nic.get_devname(), dev_features))

+ if ("rx", "off") in setting: + # when rx offload is turned off some of the cards might get reset + # and link goes down, so wait a few seconds until NIC is ready + ctl.wait(15) + if ipv in [ 'ipv4', 'both' ]: g1.run(ping_mod)