[dpdk-dev] [PATCH] doc: fix a parameter name in testpmd guide

Iremonger, Bernard bernard.iremonger at intel.com
Thu Jan 3 14:49:09 CET 2019


Hi Rami,

> -----Original Message-----
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Rami Rosen
> Sent: Tuesday, December 25, 2018 1:37 PM
> To: dev at dpdk.org
> Cc: stable at dpdk.org; Rami Rosen <ramirose at gmail.com>
> Subject: [dpdk-dev] [PATCH] doc: fix a parameter name in testpmd guide
> 
> There is no parameter called "eth-peer-configfile" in testpmd. It should be
> "eth-peers-configfile". See the usage() method in app/test-
> pmd/parameters.c.
> 
> Fixes: a67857e97ba8 ("doc: clarify usage of testpmd MAC forward mode")
> Cc: stable at dpdk.org
> 
> Signed-off-by: Rami Rosen <ramirose at gmail.com>
> ---
>  doc/guides/testpmd_app_ug/testpmd_funcs.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> index cbf23e9..1ceb000 100644
> --- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> +++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> @@ -304,7 +304,7 @@ The available information categories are:
>  * ``mac``: Changes the source and the destination Ethernet addresses of
> packets before forwarding them.
>    Default application behaviour is to set source Ethernet address to that of
> the transmitting interface, and destination
>    address to a dummy value (set during init). The user may specify a target
> destination Ethernet address via the 'eth-peer' or
> -  'eth-peer-configfile' command-line options. It is not currently possible to
> specify a specific source Ethernet address.
> +  'eth-peers-configfile' command-line options. It is not currently possible to
> specify a specific source Ethernet address.
> 
>  * ``macswap``: MAC swap forwarding mode.
>    Swaps the source and the destination Ethernet addresses of packets
> before forwarding them.
> --
> 1.8.3.1

./dpdk/devtools/check-git-log.sh -1
Line too long:
        "eth-peers-configfile". See the usage() method in app/test-pmd/parameters.c.

./dpdk/devtools/checkpatches.sh /home/bairemon/patches_19_02_rc1_patchwork_testpmd/doc-fix-a-parameter-name-in-testpmd-guide.patch

### doc: fix a parameter name in testpmd guide

WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#18: 
There is no parameter called "eth-peer-configfile" in testpmd. It should be 

total: 0 errors, 1 warnings, 8 lines checked

The above warnings should be fixed.

Regards,

Bernard.





More information about the dev mailing list