test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH] test_plans checksum_offload: fix name of a parameter
@ 2019-03-03 16:07 Rami Rosen
  2019-03-04  6:47 ` Tu, Lijuan
  0 siblings, 1 reply; 2+ messages in thread
From: Rami Rosen @ 2019-03-03 16:07 UTC (permalink / raw)
  To: dts; +Cc: Rami Rosen

Testpmd has no longer a paramater called enable-rx-checksum, 
it should be: enable-rx-cksum. This patch fixes it in the 
checksum offload test plan.

See:
https://git.dpdk.org/dpdk/tree/app/test-pmd/parameters.c#n134

Signed-off-by: Rami Rosen <ramirose@gmail.com>
---
 test_plans/checksum_offload_test_plan.rst | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/test_plans/checksum_offload_test_plan.rst b/test_plans/checksum_offload_test_plan.rst
index 710b98a..82fd12f 100644
--- a/test_plans/checksum_offload_test_plan.rst
+++ b/test_plans/checksum_offload_test_plan.rst
@@ -56,7 +56,7 @@ On the TX side:
 RX side, the L3/L4 checksum offload by hardware can be enabled with the
 following command of the ``testpmd`` application::
 
-   enable-rx-checksum
+   enable-rx-cksum
 
 TX side, the insertion of a L3/L4 checksum by hardware can be enabled with the
 following command of the ``testpmd`` application and running in a dedicated
@@ -93,7 +93,7 @@ launch the ``testpmd`` with the following arguments::
 
   ./build/app/testpmd -cffffff -n 1 -- -i --burst=1 --txpt=32 \
   --txht=8 --txwt=0 --txfreet=0 --rxfreet=64 --mbcache=250 --portmask=0x5
-  enable-rx-checksum
+  enable-rx-cksum
 
 Set the verbose level to 1 to display information for each received packet::
 
-- 
1.8.3.1

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dts] [PATCH] test_plans checksum_offload: fix name of a parameter
  2019-03-03 16:07 [dts] [PATCH] test_plans checksum_offload: fix name of a parameter Rami Rosen
@ 2019-03-04  6:47 ` Tu, Lijuan
  0 siblings, 0 replies; 2+ messages in thread
From: Tu, Lijuan @ 2019-03-04  6:47 UTC (permalink / raw)
  To: Rami Rosen, dts

Applied, thanks

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Rami Rosen
> Sent: Monday, March 4, 2019 12:08 AM
> To: dts@dpdk.org
> Cc: Rami Rosen <ramirose@gmail.com>
> Subject: [dts] [PATCH] test_plans checksum_offload: fix name of a parameter
> 
> Testpmd has no longer a paramater called enable-rx-checksum, it should be:
> enable-rx-cksum. This patch fixes it in the checksum offload test plan.
> 
> See:
> https://git.dpdk.org/dpdk/tree/app/test-pmd/parameters.c#n134
> 
> Signed-off-by: Rami Rosen <ramirose@gmail.com>
> ---
>  test_plans/checksum_offload_test_plan.rst | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/test_plans/checksum_offload_test_plan.rst
> b/test_plans/checksum_offload_test_plan.rst
> index 710b98a..82fd12f 100644
> --- a/test_plans/checksum_offload_test_plan.rst
> +++ b/test_plans/checksum_offload_test_plan.rst
> @@ -56,7 +56,7 @@ On the TX side:
>  RX side, the L3/L4 checksum offload by hardware can be enabled with the
> following command of the ``testpmd`` application::
> 
> -   enable-rx-checksum
> +   enable-rx-cksum
> 
>  TX side, the insertion of a L3/L4 checksum by hardware can be enabled with the
> following command of the ``testpmd`` application and running in a dedicated
> @@ -93,7 +93,7 @@ launch the ``testpmd`` with the following arguments::
> 
>    ./build/app/testpmd -cffffff -n 1 -- -i --burst=1 --txpt=32 \
>    --txht=8 --txwt=0 --txfreet=0 --rxfreet=64 --mbcache=250 --portmask=0x5
> -  enable-rx-checksum
> +  enable-rx-cksum
> 
>  Set the verbose level to 1 to display information for each received packet::
> 
> --
> 1.8.3.1

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-03-04  6:47 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-03 16:07 [dts] [PATCH] test_plans checksum_offload: fix name of a parameter Rami Rosen
2019-03-04  6:47 ` Tu, Lijuan

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).