From: Hongbo Li <hongbox.li@intel.com>
To: dts@dpdk.org
Cc: Hongbo Li <hongbox.li@intel.com>
Subject: [dts][PATCH V2 2/2] tests/vf_pf_reset: add new case and plan for pf_reset_trigger_vf_reset
Date: Fri, 6 Jan 2023 18:27:58 +0800 [thread overview]
Message-ID: <20230106102758.11043-2-hongbox.li@intel.com> (raw)
In-Reply-To: <20230106102758.11043-1-hongbox.li@intel.com>
1.add case test_pf_reset_trigger_vf_reset according to test plan.
2.optimize the logic of finding the tester port in the script
Signed-off-by: Hongbo Li <hongbox.li@intel.com>
---
tests/TestSuite_vf_pf_reset.py | 111 +++++++++++++++++++++++++++++++--
1 file changed, 106 insertions(+), 5 deletions(-)
diff --git a/tests/TestSuite_vf_pf_reset.py b/tests/TestSuite_vf_pf_reset.py
index 6a9b5969..31cdbb39 100644
--- a/tests/TestSuite_vf_pf_reset.py
+++ b/tests/TestSuite_vf_pf_reset.py
@@ -61,12 +61,12 @@ class TestVfPfReset(TestCase):
self.host_intf_1 = self.dut.ports_info[self.used_dut_port_1]["intf"]
tester_port_0 = self.tester.get_local_port(self.used_dut_port_0)
- self.tester_intf_0 = self.tester.get_interface(self.used_dut_port_0)
- self.tester_mac_0 = self.tester.get_mac(self.used_dut_port_0)
+ self.tester_intf_0 = self.tester.get_interface(tester_port_0)
+ self.tester_mac_0 = self.tester.get_mac(tester_port_0)
tester_port_1 = self.tester.get_local_port(self.used_dut_port_1)
- self.tester_intf_1 = self.tester.get_interface(self.used_dut_port_1)
- self.tester_mac_1 = self.tester.get_mac(self.used_dut_port_1)
+ self.tester_intf_1 = self.tester.get_interface(tester_port_1)
+ self.tester_mac_1 = self.tester.get_mac(tester_port_1)
self.vf_mac1 = "00:11:22:33:44:11"
self.vf_mac2 = "00:11:22:33:44:12"
@@ -251,7 +251,8 @@ class TestVfPfReset(TestCase):
pmd_output.execute_cmd(f"port reset {port}")
pmd_output.execute_cmd(f"port start {port}")
pmd_output.execute_cmd("start")
- pmd_output.execute_cmd(f"show port info {port}")
+ port_info = pmd_output.execute_cmd(f"show port info {port}")
+ return port_info
def setup_vm_env(self):
"""
@@ -1305,6 +1306,106 @@ class TestVfPfReset(TestCase):
self.vm0_testpmd.quit()
self.vm1_testpmd.quit()
+ def test_pf_reset_trigger_vf_reset(self):
+ """
+ check pf reset and trigger vf reset
+ """
+ # Set mac
+ self.ip_link_set(
+ host_intf=self.host_intf_0,
+ cmd="vf",
+ port=0,
+ types="mac",
+ value=self.vf_mac1,
+ )
+ self.ip_link_set(
+ host_intf=self.host_intf_0,
+ cmd="vf",
+ port=1,
+ types="mac",
+ value=self.vf_mac2,
+ )
+
+ # Set the VLAN id of VF0 and VF1
+ self.ip_link_set(
+ host_intf=self.host_intf_0, cmd="vf", port=0, types="vlan", value="1"
+ )
+ self.ip_link_set(
+ host_intf=self.host_intf_0, cmd="vf", port=1, types="vlan", value="1"
+ )
+
+ # Launch pmd
+ param = "--portmask=0x3" + " --port-topology=paired"
+ self.pmd_output.start_testpmd(
+ cores="1S/4C/1T",
+ ports=[self.sriov_vfs_port[0].pci, self.sriov_vfs_port[1].pci],
+ param=param,
+ )
+
+ # Input pmd command
+ self.pmd_output.execute_cmd("set fwd mac")
+ self.pmd_output.execute_cmd("start")
+ self.pmd_output.execute_cmd("set allmulti all on")
+ self.pmd_output.execute_cmd("set promisc all off")
+ self.pmd_output.execute_cmd("set verbose 1")
+ out = self.pmd_output.execute_cmd("show port info all")
+ # Diable Promiscuous mode and enable Allmulticast mode
+ self.verify(
+ "Promiscuous mode: enabled" not in out, "disable promiscuous mode failed."
+ )
+ self.verify(
+ "Allmulticast mode: disabled" not in out,
+ "enabled allmulticast mode failed.",
+ )
+
+ # The packets can be received
+ # By one VF and can be forward to another VF correctly
+ self.verify_send_packets(
+ self.tester_intf_0,
+ self.vf_mac1,
+ expect_value=1000,
+ count=1000,
+ vlan=1,
+ tx_port=self.tester_intf_0,
+ )
+ # Pf reset
+ self.used_dut_port = self.dut_ports[0]
+ self.domain_id_0 = self.dut.ports_info[self.used_dut_port]["port"].domain_id
+ self.bus_id_0 = self.dut.ports_info[self.used_dut_port]["port"].bus_id
+ self.devfun_id_0 = self.dut.ports_info[self.used_dut_port]["port"].devfun_id
+ reset_cmd = (
+ "echo 1 > /sys/bus/pci/devices/"
+ + self.domain_id_0
+ + "\\:"
+ + self.bus_id_0
+ + "\\:"
+ + self.devfun_id_0
+ + "/reset"
+ )
+ self.dut_new_session.send_expect(reset_cmd, "#")
+ # Check testpmd output "reset event"
+ out = self.pmd_output.get_output()
+ self.verify("reset event" in out, "testpmd did not report reset event")
+ # Send packets and verify again
+ port_info = self.reset_vf_ports(self.pmd_output)
+ # Check Diable Promiscuous mode and enable Allmulticast mode
+ self.verify(
+ "Promiscuous mode: enabled" not in port_info,
+ "disable promiscuous mode failed.",
+ )
+ self.verify(
+ "Allmulticast mode: disabled" not in port_info,
+ "enabled allmulticast mode failed.",
+ )
+ self.verify_send_packets(
+ self.tester_intf_0,
+ self.vf_mac1,
+ expect_value=1000,
+ count=1000,
+ vlan=1,
+ tx_port=self.tester_intf_0,
+ )
+
def tear_down(self):
"""
Run after each test case.
--
2.17.1
next prev parent reply other threads:[~2023-01-06 2:07 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-06 10:27 [dts][PATCH V2 1/2] test_plans/vf_pf_reset: " Hongbo Li
2023-01-06 10:27 ` Hongbo Li [this message]
2023-01-10 7:26 ` [dts][PATCH V2 2/2] tests/vf_pf_reset: " Jiale, SongX
2023-01-10 7:36 ` lijuan.tu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230106102758.11043-2-hongbox.li@intel.com \
--to=hongbox.li@intel.com \
--cc=dts@dpdk.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).