[V2] test*: Add ieee1588_enable on ICE
Checks
Commit Message
According to dpdk commit f5ada3bcf12d (net/ice: fix mbuf offload flag for Rx timestamp)
add ieee1588_enable on ICE.
Signed-off-by: Lingli Chen <linglix.chen@intel.com>
---
V2: update test plan accordingly
conf/test_case_checklist.json | 4 +---
test_plans/ieee1588_test_plan.rst | 4 ++++
tests/TestSuite_ieee1588.py | 12 +++++++++---
3 files changed, 14 insertions(+), 6 deletions(-)
Comments
On Tue, 3 Jan 2023 21:42:47 -0500, Lingli Chen <linglix.chen@intel.com> wrote:
> According to dpdk commit f5ada3bcf12d (net/ice: fix mbuf offload flag for Rx timestamp)
> add ieee1588_enable on ICE.
>
> Signed-off-by: Lingli Chen <linglix.chen@intel.com>
Applied, thanks
@@ -1293,9 +1293,7 @@
"NIC": [
"IGB_1G-I210_COPPER",
"cavium_a034",
- "cavium_0011",
- "ICE_25G-E810C_SFP",
- "ICE_100G-E810C_QSFP"
+ "cavium_0011"
],
"Target": [
"ALL"
@@ -68,6 +68,10 @@ The -n command is used to select the number of memory channels. It should match
--tx-offloads: for IEEE1588, the full-feature tx path needs to be enabled. Enabling any tx offload will force DPDK utilize full tx path.
Enabling multiple segment offload is more reasonable for user cases.
+If test on ICE, start testpmd with the following parameters::
+
+ -cffffff -n 3 -- -i --tx-offloads=0x00008000 --enable-rx-timestamp
+
Test Case: Enable IEEE1588 PTP packet reception and generation
==============================================================
@@ -37,9 +37,15 @@ class TestIeee1588(TestCase):
# For IEEE1588, the full-feature tx path needs to be enabled.
# Enabling any tx offload will force DPDK utilize full tx path.
# Enabling multiple segment offload is more reasonable for user cases.
- self.pmdout.start_testpmd(
- "Default", " --tx-offloads=%s" % DEV_TX_OFFLOAD_MULTI_SEGS
- )
+ if self.kdriver == "ice":
+ self.pmdout.start_testpmd(
+ "Default",
+ " --enable-rx-timestamp --tx-offloads=%s" % DEV_TX_OFFLOAD_MULTI_SEGS,
+ )
+ else:
+ self.pmdout.start_testpmd(
+ "Default", " --tx-offloads=%s" % DEV_TX_OFFLOAD_MULTI_SEGS
+ )
def set_up(self):
"""