[dts] [PATCH V1 1/2] test_plans/rxtx_offload: add tx offload multi_segs setting

[Tu, Lijuan]

> -----Original Message-----
> From: Lin, Xueqin <xueqin.lin at intel.com>
> Sent: 2021年8月3日 18:25
> To: Tu, Lijuan <lijuan.tu at intel.com>; Jiang, YuX <yux.jiang at intel.com>;
> dts at dpdk.org
> Cc: Jiang, YuX <yux.jiang at intel.com>
> Subject: RE: [dts] [PATCH V1 1/2] test_plans/rxtx_offload: add tx offload
> multi_segs setting
> 
> > -----Original Message-----
> > From: Tu, Lijuan <lijuan.tu at intel.com>
> > Sent: Tuesday, August 3, 2021 2:47 PM
> > To: Lin, Xueqin <xueqin.lin at intel.com>; Jiang, YuX
> > <yux.jiang at intel.com>; dts at dpdk.org
> > Cc: Jiang, YuX <yux.jiang at intel.com>
> > Subject: RE: [dts] [PATCH V1 1/2] test_plans/rxtx_offload: add tx
> > offload multi_segs setting
> >
> >
> >
> > > -----Original Message-----
> > > From: dts <dts-bounces at dpdk.org> On Behalf Of Lin, Xueqin
> > > Sent: 2021年8月2日 17:29
> > > To: Jiang, YuX <yux.jiang at intel.com>; dts at dpdk.org
> > > Cc: Jiang, YuX <yux.jiang at intel.com>
> > > Subject: Re: [dts] [PATCH V1 1/2] test_plans/rxtx_offload: add tx
> > > offload multi_segs setting
> > >
> > > > -----Original Message-----
> > > > From: dts <dts-bounces at dpdk.org> On Behalf Of Yu Jiang
> > > > Sent: Thursday, July 29, 2021 2:16 PM
> > > > To: dts at dpdk.org
> > > > Cc: Jiang, YuX <yux.jiang at intel.com>
> > > > Subject: [dts] [PATCH V1 1/2] test_plans/rxtx_offload: add tx
> > > > offload multi_segs setting
> > > >
> > > > add tx offload multi_segs setting
> > > >
> > > > Signed-off-by: Yu Jiang <yux.jiang at intel.com>
> > > Acked-by: Xueqin Lin <Xueqin.lin at intel.com>
> > > > ---
> > > >  test_plans/rxtx_offload_test_plan.rst | 56
> > > > +++++++++++++++++++++++++++++++++++
> > > >  1 file changed, 56 insertions(+)
> > > >
> > > > diff --git a/test_plans/rxtx_offload_test_plan.rst
> > > > b/test_plans/rxtx_offload_test_plan.rst
> > > > index 246e1e1..962ee9f 100644
> > > > --- a/test_plans/rxtx_offload_test_plan.rst
> > > > +++ b/test_plans/rxtx_offload_test_plan.rst
> > > > @@ -677,3 +677,59 @@ Test case: FVL Tx offload per-queue setting
> > > >
> > > >     Note 1: there is no tx_offload per_queue parameter in ixgbe driver,
> > > >     so this case is just only for i40e.
> > > > +
> > > > +Test case: Tx offload multi_segs setting
> > > > +======================================================
> > > > +
> > > > +1. Start testpmd with "--tx-offloads=0x00008000" to enable
> > > > +tx_offload
> > > > multi_segs ::
> > > > +
> > > > +    ./testpmd -c 0xf -n 4  -- -i --tx-offloads==0x00008000
> > > > +    testpmd> show port 0 tx_offload configuration
> > > > +    Tx Offloading Configuration of port 0 :
> > > > +      Port : MULTI_SEGS
> > > > +      Queue[ 0] : MULTI_SEGS
> > > > +
> > > > +2. Set fwd to txonly, Set the length of each segment of the
> > > > +TX-ONLY packets,
> > > > Set the split policy for TX packets, then start to send pkgs::
> > > > +
> > > > +    testpmd> set fwd txonly
> > > > +    testpmd> set txpkts 64,64
> >
> > It requires packet size larger than mbuf size, else it won't use multi_segs.

sorry, I misunderstood `set txpkts`
> 
> Vector path doesn't support multi-segment packet transmit, only normal path
> supports multi-segment packet transmit.
> Use 'port config 0 tx_offload multi_segs on' to enable multi segment offload
> first.

Sure, it's not care the vector path.
> 
> >
> > > > +    testpmd> set txsplit rand

`set txpkts 64,64` means the packet has 2 segments, the first one is 64, and the second one is also 64.
`set txsplit rand` means random segment number, here it might be 1 or 2, it is random, so here comes the issue.
If it is 1, it is only 1 segment, so it doesn't validate the MULT_SEG offload feature. Furthemore it has 50% percentage don't validate it.

> > > > +    testpmd> start
> > > > +
> > > > +3. Check TX-packets will not hang and continue to increase::
> > > > +    Wait 15s or more, check TX-packets will continue to increase
> > > > +and can be more than 300K
> > > > +
> > > > +    testpmd> show port stats all
> > > > +    testpmd> stop
> > > > +    testpmd> quit
> > > > +
> > > > +4. Start testpmd again without "--tx-offloads", check multi-segs
> > > > +is disabled by
> > > > default::
> > > > +
> > > > +    ./testpmd -c 0xf -n 4  -- -i
> > > > +    testpmd> show port 0 tx_offload configuration
> > > > +    No MULTI_SEGS in Tx Offloading Configuration of ports
> > > > +
> > > > +5. Enable tx_offload multi_segs ::
> > > > +    testpmd> port stop all
> > > > +    testpmd> port config 0 tx_offload multi_segs on
> > > > +    testpmd> port config 1 tx_offload multi_segs on
> > > > +    testpmd> port start all
> > > > +    testpmd> show port 0 tx_offload configuration
> > > > +    Tx Offloading Configuration of port 0 :
> > > > +      Port : MULTI_SEGS
> > > > +      Queue[ 0] : MULTI_SEGS
> > > > +
> > > > +6. Set fwd to txonly, Set the length of each segment of the
> > > > +TX-ONLY packets,
> > > > Set the split policy for TX packets, then start to send pkgs::
> > > > +
> > > > +    testpmd> set fwd txonly
> > > > +    testpmd> set txpkts 64,64
> > > > +    testpmd> set txsplit rand
> > > > +    testpmd> start
> > > > +    testpmd> show port stats all
> > > > +
> > > > +7. Check TX-packets will not hang and continue to increase::
> > > > +    Wait 15s or more, check TX-packets will continue to increase
> > > > +and can be more than 300K
> > > > +
> > > > +    testpmd> show port stats all
> > > > +    testpmd> stop
> > > > +    testpmd> quit
> > > > --
> > > > 2.7.4
> >
>