Re: [dpdk-dev] [PATCH V2] doc: add missing loopback option in testpmd guide

2019-01-17 Thread Iremonger, Bernard
> -Original Message-
> From: Rami Rosen [mailto:ramir...@gmail.com]
> Sent: Thursday, January 17, 2019 11:39 AM
> To: dev@dpdk.org
> Cc: sta...@dpdk.org; Mcnamara, John ;
> Iremonger, Bernard ; Rami Rosen
> 
> Subject: [PATCH V2] doc: add missing loopback option in testpmd guide
> 
> This patch adds a missing topology "loop" mode for port-topology option in
> testpmd guide. The testpmd "loop" option
> (PORT_TOPOLOGY_LOOP)  was added to testpmd parameter port-topology back
> in 2014, but it seems that it was never added to the testpmd guide. This patch
> adds it in the testpmd guide.
> 
> v2: Change "loopback" to "loop" (following Bernard review)
> 
> Fixes: 3e2006d6186c ("app/testpmd: add loopback topology")
> Cc: sta...@dpdk.org
> Signed-off-by: Rami Rosen 

Acked-by: Bernard Iremonger 



[dpdk-dev] [PATCH V2] doc: add missing loopback option in testpmd guide

2019-01-17 Thread Rami Rosen
This patch adds a missing topology "loop" mode for port-topology
option in testpmd guide. The testpmd "loop" option
(PORT_TOPOLOGY_LOOP)  was added to testpmd parameter
port-topology back in 2014, but it seems that it was never added
to the testpmd guide. This patch adds it in the testpmd guide.

v2: Change "loopback" to "loop" (following Bernard review)

Fixes: 3e2006d6186c ("app/testpmd: add loopback topology")
Cc: sta...@dpdk.org
Signed-off-by: Rami Rosen 
---
 doc/guides/testpmd_app_ug/run_app.rst | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/doc/guides/testpmd_app_ug/run_app.rst 
b/doc/guides/testpmd_app_ug/run_app.rst
index f717bd3..ee51b1d 100644
--- a/doc/guides/testpmd_app_ug/run_app.rst
+++ b/doc/guides/testpmd_app_ug/run_app.rst
@@ -196,7 +196,7 @@ The commandline options are:
 
 *   ``--port-topology=mode``
 
-Set port topology, where mode is ``paired`` (the default) or ``chained``.
+Set port topology, where mode is ``paired`` (the default), ``chained`` or 
``loop``.
 
 In ``paired`` mode, the forwarding is between pairs of ports, for example: 
(0,1), (2,3), (4,5).
 
@@ -204,6 +204,8 @@ The commandline options are:
 
 The ordering of the ports can be changed using the portlist testpmd 
runtime function.
 
+In ``loop`` mode, ingress traffic is simply transmitted back on the same 
interface.
+
 *   ``--forward-mode=mode``
 
 Set the forwarding mode where ``mode`` is one of the following::
-- 
1.8.3.1