* [dpdk-dev] [PATCH] doc: fix interactive commands in testpmd guide
2019-04-16 6:33 [dpdk-dev] [PATCH] doc: fix interactive commands in testpmd guide Agalya Babu RadhaKrishnan
@ 2019-04-16 6:33 ` Agalya Babu RadhaKrishnan
2019-04-16 7:32 ` Rami Rosen
` (2 subsequent siblings)
3 siblings, 0 replies; 14+ messages in thread
From: Agalya Babu RadhaKrishnan @ 2019-04-16 6:33 UTC (permalink / raw)
To: dev
Cc: reshma.pattan, olivier.matz, bernard.iremonger,
Agalya Babu RadhaKrishnan
From: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
Added some missing documentation for interactive mode commands
Fixes: 01b2092a5e ("testpmd: add dump commands for debug")
Fixes: caf05a1b86 ("app/testpmd: new command to dump log types")
Signed-off-by: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
---
doc/guides/testpmd_app_ug/testpmd_funcs.rst | 101 ++++++++++++++++++++
1 file changed, 101 insertions(+)
diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
index 5d4dc6f0c..a8954c279 100644
--- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
+++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
@@ -467,6 +467,57 @@ Show Tx metadata value set for a specific port::
testpmd> show port (port_id) tx_metadata
+dump physmem
+~~~~~~~~~~~~
+
+Dumps all physical memory segment layouts::
+
+ testpmd> dump_physmem
+
+dump memzone
+~~~~~~~~~~~~
+
+Dumps the layout of all memory zones::
+
+ testpmd> dump_memzone
+
+
+dump struct size
+~~~~~~~~~~~~~~~~
+
+Dumps the size of all memory structures::
+
+ testpmd> dump_struct_sizes
+
+dump ring
+~~~~~~~~~
+
+Dumps the status of all or specific element in DPDK rings::
+
+ testpmd> dump_ring (string)
+
+dump mempool
+~~~~~~~~~~~~
+
+Dumps the statistics of all or specific memory pool::
+
+ testpmd> dump_mempool (string)
+ testpmd> dump_mempool (name)
+
+dump devargs
+~~~~~~~~~~~~
+
+Dumps the user device list::
+
+ testpmd> dump_devargs
+
+dump log types
+~~~~~~~~~~~~~~
+
+Dumps the log level for all the dpdk modules::
+
+ testpmd> dump_log_types
+
Configuration Functions
-----------------------
@@ -1041,6 +1092,20 @@ Display the status of TCP Segmentation Offload::
testpmd> tso show (port_id)
+tunnel tso set
+~~~~~~~~~~~~~~
+
+Set tso segment size of tunneled packets for a port in csum engine::
+
+ testpmd> tunnel_tso set (tso_segsz) (port_id)
+
+tunnel tso show
+~~~~~~~~~~~~~~~
+
+Display the status of tunneled TCP Segmentation Offload for a port::
+
+ testpmd> tunnel_tso show (port_id)
+
set port - gro
~~~~~~~~~~~~~~
@@ -1162,6 +1227,22 @@ Remove a MAC address from a port::
testpmd> mac_addr remove (port_id) (XX:XX:XX:XX:XX:XX)
+mcast_addr add
+~~~~~~~~~~~~~~
+
+To add the multicast MAC address to/from the set of multicast addresses
+filtered by port::
+
+ testpmd> mcast_addr add (port_id) (mcast_addr)
+
+mcast_addr remove
+~~~~~~~~~~~~~~~~~
+
+To remove the multicast MAC address to/from the set of multicast addresses
+filtered by port::
+
+ testpmd> mcast_addr remove (port_id) (mcast_addr)
+
mac_addr add (for VF)
~~~~~~~~~~~~~~~~~~~~~
@@ -2183,6 +2264,26 @@ testpmd will add this value to any Tx packet sent from this port::
testpmd> port config (port_id) tx_metadata (value)
+port config mtu
+~~~~~~~~~~~~~~~
+
+To configure MTU(Maximum Transmission Unit) on devices using testpmd::
+
+ testpmd> port config mtu (port_id) (value)
+
+port config rss hash key
+~~~~~~~~~~~~~~~~~~~~~~~~
+
+To configure the RSS hash key used to compute the RSS
+hash of input [IP] packets received on port::
+
+ testpmd> port config <port_id> rss-hash-key (ipv4|ipv4-frag|\
+ ipv4-tcp|ipv4-udp|ipv4-sctp|ipv4-other|\
+ ipv6|ipv6-frag|ipv6-tcp|ipv6-udp|ipv6-sctp|\
+ ipv6-other|l2-payload|ipv6-ex|ipv6-tcp-ex|\
+ ipv6-udp-ex <string of hex digits \
+ (variable length, NIC dependent)>)
+
Link Bonding Functions
----------------------
--
2.17.2
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [dpdk-dev] [PATCH] doc: fix interactive commands in testpmd guide
2019-04-16 6:33 [dpdk-dev] [PATCH] doc: fix interactive commands in testpmd guide Agalya Babu RadhaKrishnan
2019-04-16 6:33 ` Agalya Babu RadhaKrishnan
@ 2019-04-16 7:32 ` Rami Rosen
2019-04-16 7:32 ` Rami Rosen
2019-04-16 15:05 ` Ferruh Yigit
2019-04-23 10:44 ` [dpdk-dev] [PATCH v2] " Agalya Babu RadhaKrishnan
3 siblings, 1 reply; 14+ messages in thread
From: Rami Rosen @ 2019-04-16 7:32 UTC (permalink / raw)
To: Agalya Babu RadhaKrishnan
Cc: dev, Reshma Pattan, Olivier Matz, Iremonger, Bernard
On Tue, Apr 16, 2019 at 9:35 AM Agalya Babu RadhaKrishnan <
agalyax.babu.radhakrishnan@intel.com> wrote:
> From: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
>
> Added some missing documentation for interactive mode commands
>
> Fixes: 01b2092a5e ("testpmd: add dump commands for debug")
> Fixes: caf05a1b86 ("app/testpmd: new command to dump log types")
>
> Signed-off-by: Agalya Babu RadhaKrishnan <
> agalyax.babu.radhakrishnan@intel.com>
> ---
>
> Thanks for this work!
Reviewed-by: Rami Rosen <ramirose@gmail.com>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [dpdk-dev] [PATCH] doc: fix interactive commands in testpmd guide
2019-04-16 7:32 ` Rami Rosen
@ 2019-04-16 7:32 ` Rami Rosen
0 siblings, 0 replies; 14+ messages in thread
From: Rami Rosen @ 2019-04-16 7:32 UTC (permalink / raw)
To: Agalya Babu RadhaKrishnan
Cc: dev, Reshma Pattan, Olivier Matz, Iremonger, Bernard
On Tue, Apr 16, 2019 at 9:35 AM Agalya Babu RadhaKrishnan <
agalyax.babu.radhakrishnan@intel.com> wrote:
> From: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
>
> Added some missing documentation for interactive mode commands
>
> Fixes: 01b2092a5e ("testpmd: add dump commands for debug")
> Fixes: caf05a1b86 ("app/testpmd: new command to dump log types")
>
> Signed-off-by: Agalya Babu RadhaKrishnan <
> agalyax.babu.radhakrishnan@intel.com>
> ---
>
> Thanks for this work!
Reviewed-by: Rami Rosen <ramirose@gmail.com>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [dpdk-dev] [PATCH] doc: fix interactive commands in testpmd guide
2019-04-16 6:33 [dpdk-dev] [PATCH] doc: fix interactive commands in testpmd guide Agalya Babu RadhaKrishnan
2019-04-16 6:33 ` Agalya Babu RadhaKrishnan
2019-04-16 7:32 ` Rami Rosen
@ 2019-04-16 15:05 ` Ferruh Yigit
2019-04-16 15:05 ` Ferruh Yigit
` (2 more replies)
2019-04-23 10:44 ` [dpdk-dev] [PATCH v2] " Agalya Babu RadhaKrishnan
3 siblings, 3 replies; 14+ messages in thread
From: Ferruh Yigit @ 2019-04-16 15:05 UTC (permalink / raw)
To: Agalya Babu RadhaKrishnan, dev
Cc: reshma.pattan, olivier.matz, bernard.iremonger
On 4/16/2019 7:33 AM, Agalya Babu RadhaKrishnan wrote:
> From: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
>
> Added some missing documentation for interactive mode commands
>
> Fixes: 01b2092a5e ("testpmd: add dump commands for debug")
> Fixes: caf05a1b86 ("app/testpmd: new command to dump log types")
>
> Signed-off-by: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
Hi Agalya,
Good to see the documentation is updated to cover the gaps, this patch documents
various commands, are above fixes lines cover all commits?
> ---
> doc/guides/testpmd_app_ug/testpmd_funcs.rst | 101 ++++++++++++++++++++
> 1 file changed, 101 insertions(+)
>
> diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> index 5d4dc6f0c..a8954c279 100644
> --- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> +++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> @@ -467,6 +467,57 @@ Show Tx metadata value set for a specific port::
>
> testpmd> show port (port_id) tx_metadata
>
> +dump physmem
> +~~~~~~~~~~~~
> +
> +Dumps all physical memory segment layouts::
> +
> + testpmd> dump_physmem
> +
> +dump memzone
> +~~~~~~~~~~~~
> +
> +Dumps the layout of all memory zones::
> +
> + testpmd> dump_memzone
> +
> +
> +dump struct size
> +~~~~~~~~~~~~~~~~
> +
> +Dumps the size of all memory structures::
> +
> + testpmd> dump_struct_sizes
> +
> +dump ring
> +~~~~~~~~~
> +
> +Dumps the status of all or specific element in DPDK rings::
> +
> + testpmd> dump_ring (string)
> +
> +dump mempool
> +~~~~~~~~~~~~
> +
> +Dumps the statistics of all or specific memory pool::
> +
> + testpmd> dump_mempool (string)
> + testpmd> dump_mempool (name)
> +
> +dump devargs
> +~~~~~~~~~~~~
> +
> +Dumps the user device list::
> +
> + testpmd> dump_devargs
> +
> +dump log types
> +~~~~~~~~~~~~~~
> +
> +Dumps the log level for all the dpdk modules::
> +
> + testpmd> dump_log_types
> +
> Configuration Functions
> -----------------------
>
> @@ -1041,6 +1092,20 @@ Display the status of TCP Segmentation Offload::
>
> testpmd> tso show (port_id)
>
> +tunnel tso set
> +~~~~~~~~~~~~~~
> +
> +Set tso segment size of tunneled packets for a port in csum engine::
> +
> + testpmd> tunnel_tso set (tso_segsz) (port_id)
> +
> +tunnel tso show
> +~~~~~~~~~~~~~~~
> +
> +Display the status of tunneled TCP Segmentation Offload for a port::
> +
> + testpmd> tunnel_tso show (port_id)
> +
> set port - gro
> ~~~~~~~~~~~~~~
>
> @@ -1162,6 +1227,22 @@ Remove a MAC address from a port::
>
> testpmd> mac_addr remove (port_id) (XX:XX:XX:XX:XX:XX)
>
> +mcast_addr add
> +~~~~~~~~~~~~~~
> +
> +To add the multicast MAC address to/from the set of multicast addresses
> +filtered by port::
> +
> + testpmd> mcast_addr add (port_id) (mcast_addr)
> +
> +mcast_addr remove
> +~~~~~~~~~~~~~~~~~
> +
> +To remove the multicast MAC address to/from the set of multicast addresses
> +filtered by port::
> +
> + testpmd> mcast_addr remove (port_id) (mcast_addr)
> +
> mac_addr add (for VF)
> ~~~~~~~~~~~~~~~~~~~~~
>
> @@ -2183,6 +2264,26 @@ testpmd will add this value to any Tx packet sent from this port::
>
> testpmd> port config (port_id) tx_metadata (value)
>
> +port config mtu
> +~~~~~~~~~~~~~~~
> +
> +To configure MTU(Maximum Transmission Unit) on devices using testpmd::
> +
> + testpmd> port config mtu (port_id) (value)
> +
> +port config rss hash key
> +~~~~~~~~~~~~~~~~~~~~~~~~
> +
> +To configure the RSS hash key used to compute the RSS
> +hash of input [IP] packets received on port::
> +
> + testpmd> port config <port_id> rss-hash-key (ipv4|ipv4-frag|\
> + ipv4-tcp|ipv4-udp|ipv4-sctp|ipv4-other|\
> + ipv6|ipv6-frag|ipv6-tcp|ipv6-udp|ipv6-sctp|\
> + ipv6-other|l2-payload|ipv6-ex|ipv6-tcp-ex|\
> + ipv6-udp-ex <string of hex digits \
> + (variable length, NIC dependent)>)
> +
> Link Bonding Functions
> ----------------------
>
>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [dpdk-dev] [PATCH] doc: fix interactive commands in testpmd guide
2019-04-16 15:05 ` Ferruh Yigit
@ 2019-04-16 15:05 ` Ferruh Yigit
2019-04-17 9:25 ` Iremonger, Bernard
2019-04-18 19:51 ` Ferruh Yigit
2 siblings, 0 replies; 14+ messages in thread
From: Ferruh Yigit @ 2019-04-16 15:05 UTC (permalink / raw)
To: Agalya Babu RadhaKrishnan, dev
Cc: reshma.pattan, olivier.matz, bernard.iremonger
On 4/16/2019 7:33 AM, Agalya Babu RadhaKrishnan wrote:
> From: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
>
> Added some missing documentation for interactive mode commands
>
> Fixes: 01b2092a5e ("testpmd: add dump commands for debug")
> Fixes: caf05a1b86 ("app/testpmd: new command to dump log types")
>
> Signed-off-by: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
Hi Agalya,
Good to see the documentation is updated to cover the gaps, this patch documents
various commands, are above fixes lines cover all commits?
> ---
> doc/guides/testpmd_app_ug/testpmd_funcs.rst | 101 ++++++++++++++++++++
> 1 file changed, 101 insertions(+)
>
> diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> index 5d4dc6f0c..a8954c279 100644
> --- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> +++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> @@ -467,6 +467,57 @@ Show Tx metadata value set for a specific port::
>
> testpmd> show port (port_id) tx_metadata
>
> +dump physmem
> +~~~~~~~~~~~~
> +
> +Dumps all physical memory segment layouts::
> +
> + testpmd> dump_physmem
> +
> +dump memzone
> +~~~~~~~~~~~~
> +
> +Dumps the layout of all memory zones::
> +
> + testpmd> dump_memzone
> +
> +
> +dump struct size
> +~~~~~~~~~~~~~~~~
> +
> +Dumps the size of all memory structures::
> +
> + testpmd> dump_struct_sizes
> +
> +dump ring
> +~~~~~~~~~
> +
> +Dumps the status of all or specific element in DPDK rings::
> +
> + testpmd> dump_ring (string)
> +
> +dump mempool
> +~~~~~~~~~~~~
> +
> +Dumps the statistics of all or specific memory pool::
> +
> + testpmd> dump_mempool (string)
> + testpmd> dump_mempool (name)
> +
> +dump devargs
> +~~~~~~~~~~~~
> +
> +Dumps the user device list::
> +
> + testpmd> dump_devargs
> +
> +dump log types
> +~~~~~~~~~~~~~~
> +
> +Dumps the log level for all the dpdk modules::
> +
> + testpmd> dump_log_types
> +
> Configuration Functions
> -----------------------
>
> @@ -1041,6 +1092,20 @@ Display the status of TCP Segmentation Offload::
>
> testpmd> tso show (port_id)
>
> +tunnel tso set
> +~~~~~~~~~~~~~~
> +
> +Set tso segment size of tunneled packets for a port in csum engine::
> +
> + testpmd> tunnel_tso set (tso_segsz) (port_id)
> +
> +tunnel tso show
> +~~~~~~~~~~~~~~~
> +
> +Display the status of tunneled TCP Segmentation Offload for a port::
> +
> + testpmd> tunnel_tso show (port_id)
> +
> set port - gro
> ~~~~~~~~~~~~~~
>
> @@ -1162,6 +1227,22 @@ Remove a MAC address from a port::
>
> testpmd> mac_addr remove (port_id) (XX:XX:XX:XX:XX:XX)
>
> +mcast_addr add
> +~~~~~~~~~~~~~~
> +
> +To add the multicast MAC address to/from the set of multicast addresses
> +filtered by port::
> +
> + testpmd> mcast_addr add (port_id) (mcast_addr)
> +
> +mcast_addr remove
> +~~~~~~~~~~~~~~~~~
> +
> +To remove the multicast MAC address to/from the set of multicast addresses
> +filtered by port::
> +
> + testpmd> mcast_addr remove (port_id) (mcast_addr)
> +
> mac_addr add (for VF)
> ~~~~~~~~~~~~~~~~~~~~~
>
> @@ -2183,6 +2264,26 @@ testpmd will add this value to any Tx packet sent from this port::
>
> testpmd> port config (port_id) tx_metadata (value)
>
> +port config mtu
> +~~~~~~~~~~~~~~~
> +
> +To configure MTU(Maximum Transmission Unit) on devices using testpmd::
> +
> + testpmd> port config mtu (port_id) (value)
> +
> +port config rss hash key
> +~~~~~~~~~~~~~~~~~~~~~~~~
> +
> +To configure the RSS hash key used to compute the RSS
> +hash of input [IP] packets received on port::
> +
> + testpmd> port config <port_id> rss-hash-key (ipv4|ipv4-frag|\
> + ipv4-tcp|ipv4-udp|ipv4-sctp|ipv4-other|\
> + ipv6|ipv6-frag|ipv6-tcp|ipv6-udp|ipv6-sctp|\
> + ipv6-other|l2-payload|ipv6-ex|ipv6-tcp-ex|\
> + ipv6-udp-ex <string of hex digits \
> + (variable length, NIC dependent)>)
> +
> Link Bonding Functions
> ----------------------
>
>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [dpdk-dev] [PATCH] doc: fix interactive commands in testpmd guide
2019-04-16 15:05 ` Ferruh Yigit
2019-04-16 15:05 ` Ferruh Yigit
@ 2019-04-17 9:25 ` Iremonger, Bernard
2019-04-17 9:25 ` Iremonger, Bernard
2019-04-18 19:51 ` Ferruh Yigit
2 siblings, 1 reply; 14+ messages in thread
From: Iremonger, Bernard @ 2019-04-17 9:25 UTC (permalink / raw)
To: Yigit, Ferruh, Babu Radhakrishnan, AgalyaX, dev
Cc: Pattan, Reshma, olivier.matz
> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Tuesday, April 16, 2019 4:06 PM
> To: Babu Radhakrishnan, AgalyaX <agalyax.babu.radhakrishnan@intel.com>;
> dev@dpdk.org
> Cc: Pattan, Reshma <reshma.pattan@intel.com>; olivier.matz@6wind.com;
> Iremonger, Bernard <bernard.iremonger@intel.com>
> Subject: Re: [dpdk-dev] [PATCH] doc: fix interactive commands in testpmd
> guide
>
> On 4/16/2019 7:33 AM, Agalya Babu RadhaKrishnan wrote:
> > From: Agalya Babu RadhaKrishnan
> <agalyax.babu.radhakrishnan@intel.com>
> >
> > Added some missing documentation for interactive mode commands
> >
> > Fixes: 01b2092a5e ("testpmd: add dump commands for debug")
> > Fixes: caf05a1b86 ("app/testpmd: new command to dump log types")
> >
> > Signed-off-by: Agalya Babu RadhaKrishnan
> > <agalyax.babu.radhakrishnan@intel.com>
>
<snip>
Hi Babu,
./dpdk/devtools/check-git-log.sh -1
Line too long:
Signed-off-by: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
Is it candidate for Cc: stable@dpdk.org backport?
doc: fix interactive commands in testpmd guide
I am not sure what can be done about the length of the signoff line.
The Cc: stable@dpdk.org line should be added after the Fixes line.
Otherwise
Acked-by: Bernard Iremonger <bernard.iremonger@intel.com>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [dpdk-dev] [PATCH] doc: fix interactive commands in testpmd guide
2019-04-17 9:25 ` Iremonger, Bernard
@ 2019-04-17 9:25 ` Iremonger, Bernard
0 siblings, 0 replies; 14+ messages in thread
From: Iremonger, Bernard @ 2019-04-17 9:25 UTC (permalink / raw)
To: Yigit, Ferruh, Babu Radhakrishnan, AgalyaX, dev
Cc: Pattan, Reshma, olivier.matz
> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Tuesday, April 16, 2019 4:06 PM
> To: Babu Radhakrishnan, AgalyaX <agalyax.babu.radhakrishnan@intel.com>;
> dev@dpdk.org
> Cc: Pattan, Reshma <reshma.pattan@intel.com>; olivier.matz@6wind.com;
> Iremonger, Bernard <bernard.iremonger@intel.com>
> Subject: Re: [dpdk-dev] [PATCH] doc: fix interactive commands in testpmd
> guide
>
> On 4/16/2019 7:33 AM, Agalya Babu RadhaKrishnan wrote:
> > From: Agalya Babu RadhaKrishnan
> <agalyax.babu.radhakrishnan@intel.com>
> >
> > Added some missing documentation for interactive mode commands
> >
> > Fixes: 01b2092a5e ("testpmd: add dump commands for debug")
> > Fixes: caf05a1b86 ("app/testpmd: new command to dump log types")
> >
> > Signed-off-by: Agalya Babu RadhaKrishnan
> > <agalyax.babu.radhakrishnan@intel.com>
>
<snip>
Hi Babu,
./dpdk/devtools/check-git-log.sh -1
Line too long:
Signed-off-by: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
Is it candidate for Cc: stable@dpdk.org backport?
doc: fix interactive commands in testpmd guide
I am not sure what can be done about the length of the signoff line.
The Cc: stable@dpdk.org line should be added after the Fixes line.
Otherwise
Acked-by: Bernard Iremonger <bernard.iremonger@intel.com>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [dpdk-dev] [PATCH] doc: fix interactive commands in testpmd guide
2019-04-16 15:05 ` Ferruh Yigit
2019-04-16 15:05 ` Ferruh Yigit
2019-04-17 9:25 ` Iremonger, Bernard
@ 2019-04-18 19:51 ` Ferruh Yigit
2019-04-18 19:51 ` Ferruh Yigit
2 siblings, 1 reply; 14+ messages in thread
From: Ferruh Yigit @ 2019-04-18 19:51 UTC (permalink / raw)
To: Agalya Babu RadhaKrishnan, dev
Cc: reshma.pattan, olivier.matz, bernard.iremonger
On 4/16/2019 4:05 PM, Ferruh Yigit wrote:
> On 4/16/2019 7:33 AM, Agalya Babu RadhaKrishnan wrote:
>> From: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
>>
>> Added some missing documentation for interactive mode commands
>>
>> Fixes: 01b2092a5e ("testpmd: add dump commands for debug")
>> Fixes: caf05a1b86 ("app/testpmd: new command to dump log types")
>>
>> Signed-off-by: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
>
> Hi Agalya,
>
> Good to see the documentation is updated to cover the gaps, this patch documents
> various commands, are above fixes lines cover all commits?
Missing fixes are as following, please include them in next version:
Fixes: 0f62d63593ed ("app/testpmd: support tunneled TSO in checksum engine")
Fixes: 8fff667578a7 ("app/testpmd: new command to add/remove multicast MAC
addresses")
Fixes: ae03d0d18adf ("app/testpmd: command to configure MTU")
Fixes: f79959ea1504 ("app/testpmd: allow to configure RSS hash key")
Fixes: caf05a1b8608 ("app/testpmd: new command to dump log types")
>
>> ---
>> doc/guides/testpmd_app_ug/testpmd_funcs.rst | 101 ++++++++++++++++++++
>> 1 file changed, 101 insertions(+)
>>
>> diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
>> index 5d4dc6f0c..a8954c279 100644
>> --- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
>> +++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
>> @@ -467,6 +467,57 @@ Show Tx metadata value set for a specific port::
>>
>> testpmd> show port (port_id) tx_metadata
>>
>> +dump physmem
>> +~~~~~~~~~~~~
>> +
>> +Dumps all physical memory segment layouts::
>> +
>> + testpmd> dump_physmem
>> +
>> +dump memzone
>> +~~~~~~~~~~~~
>> +
>> +Dumps the layout of all memory zones::
>> +
>> + testpmd> dump_memzone
>> +
>> +
>> +dump struct size
>> +~~~~~~~~~~~~~~~~
>> +
>> +Dumps the size of all memory structures::
>> +
>> + testpmd> dump_struct_sizes
>> +
>> +dump ring
>> +~~~~~~~~~
>> +
>> +Dumps the status of all or specific element in DPDK rings::
>> +
>> + testpmd> dump_ring (string)
In documentation syntax '()' for variables, '[]' for optional paramters and rest
are keywords for commandline.
For this case 'string' is optional I think, because both "dump_ring" &
"dump_ring ring_name" are valid commands. So can you please fix the command
using "[]", and 'string' can be 'ring_name' perhaps.
Same is valid for below command.
>> +
>> +dump mempool
>> +~~~~~~~~~~~~
>> +
>> +Dumps the statistics of all or specific memory pool::
>> +
>> + testpmd> dump_mempool (string)
>> + testpmd> dump_mempool (name)
Why this is duplicated, there is no two different command with 'name' and 'string'
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [dpdk-dev] [PATCH] doc: fix interactive commands in testpmd guide
2019-04-18 19:51 ` Ferruh Yigit
@ 2019-04-18 19:51 ` Ferruh Yigit
0 siblings, 0 replies; 14+ messages in thread
From: Ferruh Yigit @ 2019-04-18 19:51 UTC (permalink / raw)
To: Agalya Babu RadhaKrishnan, dev
Cc: reshma.pattan, olivier.matz, bernard.iremonger
On 4/16/2019 4:05 PM, Ferruh Yigit wrote:
> On 4/16/2019 7:33 AM, Agalya Babu RadhaKrishnan wrote:
>> From: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
>>
>> Added some missing documentation for interactive mode commands
>>
>> Fixes: 01b2092a5e ("testpmd: add dump commands for debug")
>> Fixes: caf05a1b86 ("app/testpmd: new command to dump log types")
>>
>> Signed-off-by: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
>
> Hi Agalya,
>
> Good to see the documentation is updated to cover the gaps, this patch documents
> various commands, are above fixes lines cover all commits?
Missing fixes are as following, please include them in next version:
Fixes: 0f62d63593ed ("app/testpmd: support tunneled TSO in checksum engine")
Fixes: 8fff667578a7 ("app/testpmd: new command to add/remove multicast MAC
addresses")
Fixes: ae03d0d18adf ("app/testpmd: command to configure MTU")
Fixes: f79959ea1504 ("app/testpmd: allow to configure RSS hash key")
Fixes: caf05a1b8608 ("app/testpmd: new command to dump log types")
>
>> ---
>> doc/guides/testpmd_app_ug/testpmd_funcs.rst | 101 ++++++++++++++++++++
>> 1 file changed, 101 insertions(+)
>>
>> diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
>> index 5d4dc6f0c..a8954c279 100644
>> --- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
>> +++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
>> @@ -467,6 +467,57 @@ Show Tx metadata value set for a specific port::
>>
>> testpmd> show port (port_id) tx_metadata
>>
>> +dump physmem
>> +~~~~~~~~~~~~
>> +
>> +Dumps all physical memory segment layouts::
>> +
>> + testpmd> dump_physmem
>> +
>> +dump memzone
>> +~~~~~~~~~~~~
>> +
>> +Dumps the layout of all memory zones::
>> +
>> + testpmd> dump_memzone
>> +
>> +
>> +dump struct size
>> +~~~~~~~~~~~~~~~~
>> +
>> +Dumps the size of all memory structures::
>> +
>> + testpmd> dump_struct_sizes
>> +
>> +dump ring
>> +~~~~~~~~~
>> +
>> +Dumps the status of all or specific element in DPDK rings::
>> +
>> + testpmd> dump_ring (string)
In documentation syntax '()' for variables, '[]' for optional paramters and rest
are keywords for commandline.
For this case 'string' is optional I think, because both "dump_ring" &
"dump_ring ring_name" are valid commands. So can you please fix the command
using "[]", and 'string' can be 'ring_name' perhaps.
Same is valid for below command.
>> +
>> +dump mempool
>> +~~~~~~~~~~~~
>> +
>> +Dumps the statistics of all or specific memory pool::
>> +
>> + testpmd> dump_mempool (string)
>> + testpmd> dump_mempool (name)
Why this is duplicated, there is no two different command with 'name' and 'string'
^ permalink raw reply [flat|nested] 14+ messages in thread
* [dpdk-dev] [PATCH v2] doc: fix interactive commands in testpmd guide
2019-04-16 6:33 [dpdk-dev] [PATCH] doc: fix interactive commands in testpmd guide Agalya Babu RadhaKrishnan
` (2 preceding siblings ...)
2019-04-16 15:05 ` Ferruh Yigit
@ 2019-04-23 10:44 ` Agalya Babu RadhaKrishnan
2019-04-23 10:44 ` Agalya Babu RadhaKrishnan
2019-04-23 11:38 ` Ferruh Yigit
3 siblings, 2 replies; 14+ messages in thread
From: Agalya Babu RadhaKrishnan @ 2019-04-23 10:44 UTC (permalink / raw)
To: dev
Cc: reshma.pattan, olivier.matz, ramirose, bernard.iremonger,
ferruh.yigit, Agalya Babu RadhaKrishnan, stable
Added some missing documentation for interactive mode commands
Fixes: 01b2092a5e ("testpmd: add dump commands for debug")
Fixes: caf05a1b86 ("app/testpmd: new command to dump log types")
Fixes: 0f62d63593ed ("app/testpmd: support tunneled TSO in checksum engine")
Fixes: 8fff667578a7
("app/testpmd: new command to add/remove multicast MAC addresses")
Fixes: ae03d0d18adf ("app/testpmd: command to configure MTU")
Fixes: f79959ea1504 ("app/testpmd: allow to configure RSS hash key")
Fixes: caf05a1b8608 ("app/testpmd: new command to dump log types")
Cc: stable@dpdk.org
Signed-off-by: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
Reviewed-by: Rami Rosen <ramirose@gmail.com>
Acked-by: Bernard Iremonger <bernard.iremonger@intel.com>
---
v2: Added fixes lines and addressed comments.
---
doc/guides/testpmd_app_ug/testpmd_funcs.rst | 100 ++++++++++++++++++++
1 file changed, 100 insertions(+)
diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
index 5d4dc6f0c..cdcc51957 100644
--- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
+++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
@@ -467,6 +467,56 @@ Show Tx metadata value set for a specific port::
testpmd> show port (port_id) tx_metadata
+dump physmem
+~~~~~~~~~~~~
+
+Dumps all physical memory segment layouts::
+
+ testpmd> dump_physmem
+
+dump memzone
+~~~~~~~~~~~~
+
+Dumps the layout of all memory zones::
+
+ testpmd> dump_memzone
+
+
+dump struct size
+~~~~~~~~~~~~~~~~
+
+Dumps the size of all memory structures::
+
+ testpmd> dump_struct_sizes
+
+dump ring
+~~~~~~~~~
+
+Dumps the status of all or specific element in DPDK rings::
+
+ testpmd> dump_ring [ring_name]
+
+dump mempool
+~~~~~~~~~~~~
+
+Dumps the statistics of all or specific memory pool::
+
+ testpmd> dump_mempool [mempool_name]
+
+dump devargs
+~~~~~~~~~~~~
+
+Dumps the user device list::
+
+ testpmd> dump_devargs
+
+dump log types
+~~~~~~~~~~~~~~
+
+Dumps the log level for all the dpdk modules::
+
+ testpmd> dump_log_types
+
Configuration Functions
-----------------------
@@ -1041,6 +1091,20 @@ Display the status of TCP Segmentation Offload::
testpmd> tso show (port_id)
+tunnel tso set
+~~~~~~~~~~~~~~
+
+Set tso segment size of tunneled packets for a port in csum engine::
+
+ testpmd> tunnel_tso set (tso_segsz) (port_id)
+
+tunnel tso show
+~~~~~~~~~~~~~~~
+
+Display the status of tunneled TCP Segmentation Offload for a port::
+
+ testpmd> tunnel_tso show (port_id)
+
set port - gro
~~~~~~~~~~~~~~
@@ -1162,6 +1226,22 @@ Remove a MAC address from a port::
testpmd> mac_addr remove (port_id) (XX:XX:XX:XX:XX:XX)
+mcast_addr add
+~~~~~~~~~~~~~~
+
+To add the multicast MAC address to/from the set of multicast addresses
+filtered by port::
+
+ testpmd> mcast_addr add (port_id) (mcast_addr)
+
+mcast_addr remove
+~~~~~~~~~~~~~~~~~
+
+To remove the multicast MAC address to/from the set of multicast addresses
+filtered by port::
+
+ testpmd> mcast_addr remove (port_id) (mcast_addr)
+
mac_addr add (for VF)
~~~~~~~~~~~~~~~~~~~~~
@@ -2183,6 +2263,26 @@ testpmd will add this value to any Tx packet sent from this port::
testpmd> port config (port_id) tx_metadata (value)
+port config mtu
+~~~~~~~~~~~~~~~
+
+To configure MTU(Maximum Transmission Unit) on devices using testpmd::
+
+ testpmd> port config mtu (port_id) (value)
+
+port config rss hash key
+~~~~~~~~~~~~~~~~~~~~~~~~
+
+To configure the RSS hash key used to compute the RSS
+hash of input [IP] packets received on port::
+
+ testpmd> port config <port_id> rss-hash-key (ipv4|ipv4-frag|\
+ ipv4-tcp|ipv4-udp|ipv4-sctp|ipv4-other|\
+ ipv6|ipv6-frag|ipv6-tcp|ipv6-udp|ipv6-sctp|\
+ ipv6-other|l2-payload|ipv6-ex|ipv6-tcp-ex|\
+ ipv6-udp-ex <string of hex digits \
+ (variable length, NIC dependent)>)
+
Link Bonding Functions
----------------------
--
2.17.2
^ permalink raw reply [flat|nested] 14+ messages in thread* [dpdk-dev] [PATCH v2] doc: fix interactive commands in testpmd guide
2019-04-23 10:44 ` [dpdk-dev] [PATCH v2] " Agalya Babu RadhaKrishnan
@ 2019-04-23 10:44 ` Agalya Babu RadhaKrishnan
2019-04-23 11:38 ` Ferruh Yigit
1 sibling, 0 replies; 14+ messages in thread
From: Agalya Babu RadhaKrishnan @ 2019-04-23 10:44 UTC (permalink / raw)
To: dev
Cc: reshma.pattan, olivier.matz, ramirose, bernard.iremonger,
ferruh.yigit, Agalya Babu RadhaKrishnan, stable
Added some missing documentation for interactive mode commands
Fixes: 01b2092a5e ("testpmd: add dump commands for debug")
Fixes: caf05a1b86 ("app/testpmd: new command to dump log types")
Fixes: 0f62d63593ed ("app/testpmd: support tunneled TSO in checksum engine")
Fixes: 8fff667578a7
("app/testpmd: new command to add/remove multicast MAC addresses")
Fixes: ae03d0d18adf ("app/testpmd: command to configure MTU")
Fixes: f79959ea1504 ("app/testpmd: allow to configure RSS hash key")
Fixes: caf05a1b8608 ("app/testpmd: new command to dump log types")
Cc: stable@dpdk.org
Signed-off-by: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
Reviewed-by: Rami Rosen <ramirose@gmail.com>
Acked-by: Bernard Iremonger <bernard.iremonger@intel.com>
---
v2: Added fixes lines and addressed comments.
---
doc/guides/testpmd_app_ug/testpmd_funcs.rst | 100 ++++++++++++++++++++
1 file changed, 100 insertions(+)
diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
index 5d4dc6f0c..cdcc51957 100644
--- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
+++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
@@ -467,6 +467,56 @@ Show Tx metadata value set for a specific port::
testpmd> show port (port_id) tx_metadata
+dump physmem
+~~~~~~~~~~~~
+
+Dumps all physical memory segment layouts::
+
+ testpmd> dump_physmem
+
+dump memzone
+~~~~~~~~~~~~
+
+Dumps the layout of all memory zones::
+
+ testpmd> dump_memzone
+
+
+dump struct size
+~~~~~~~~~~~~~~~~
+
+Dumps the size of all memory structures::
+
+ testpmd> dump_struct_sizes
+
+dump ring
+~~~~~~~~~
+
+Dumps the status of all or specific element in DPDK rings::
+
+ testpmd> dump_ring [ring_name]
+
+dump mempool
+~~~~~~~~~~~~
+
+Dumps the statistics of all or specific memory pool::
+
+ testpmd> dump_mempool [mempool_name]
+
+dump devargs
+~~~~~~~~~~~~
+
+Dumps the user device list::
+
+ testpmd> dump_devargs
+
+dump log types
+~~~~~~~~~~~~~~
+
+Dumps the log level for all the dpdk modules::
+
+ testpmd> dump_log_types
+
Configuration Functions
-----------------------
@@ -1041,6 +1091,20 @@ Display the status of TCP Segmentation Offload::
testpmd> tso show (port_id)
+tunnel tso set
+~~~~~~~~~~~~~~
+
+Set tso segment size of tunneled packets for a port in csum engine::
+
+ testpmd> tunnel_tso set (tso_segsz) (port_id)
+
+tunnel tso show
+~~~~~~~~~~~~~~~
+
+Display the status of tunneled TCP Segmentation Offload for a port::
+
+ testpmd> tunnel_tso show (port_id)
+
set port - gro
~~~~~~~~~~~~~~
@@ -1162,6 +1226,22 @@ Remove a MAC address from a port::
testpmd> mac_addr remove (port_id) (XX:XX:XX:XX:XX:XX)
+mcast_addr add
+~~~~~~~~~~~~~~
+
+To add the multicast MAC address to/from the set of multicast addresses
+filtered by port::
+
+ testpmd> mcast_addr add (port_id) (mcast_addr)
+
+mcast_addr remove
+~~~~~~~~~~~~~~~~~
+
+To remove the multicast MAC address to/from the set of multicast addresses
+filtered by port::
+
+ testpmd> mcast_addr remove (port_id) (mcast_addr)
+
mac_addr add (for VF)
~~~~~~~~~~~~~~~~~~~~~
@@ -2183,6 +2263,26 @@ testpmd will add this value to any Tx packet sent from this port::
testpmd> port config (port_id) tx_metadata (value)
+port config mtu
+~~~~~~~~~~~~~~~
+
+To configure MTU(Maximum Transmission Unit) on devices using testpmd::
+
+ testpmd> port config mtu (port_id) (value)
+
+port config rss hash key
+~~~~~~~~~~~~~~~~~~~~~~~~
+
+To configure the RSS hash key used to compute the RSS
+hash of input [IP] packets received on port::
+
+ testpmd> port config <port_id> rss-hash-key (ipv4|ipv4-frag|\
+ ipv4-tcp|ipv4-udp|ipv4-sctp|ipv4-other|\
+ ipv6|ipv6-frag|ipv6-tcp|ipv6-udp|ipv6-sctp|\
+ ipv6-other|l2-payload|ipv6-ex|ipv6-tcp-ex|\
+ ipv6-udp-ex <string of hex digits \
+ (variable length, NIC dependent)>)
+
Link Bonding Functions
----------------------
--
2.17.2
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [dpdk-dev] [PATCH v2] doc: fix interactive commands in testpmd guide
2019-04-23 10:44 ` [dpdk-dev] [PATCH v2] " Agalya Babu RadhaKrishnan
2019-04-23 10:44 ` Agalya Babu RadhaKrishnan
@ 2019-04-23 11:38 ` Ferruh Yigit
2019-04-23 11:38 ` Ferruh Yigit
1 sibling, 1 reply; 14+ messages in thread
From: Ferruh Yigit @ 2019-04-23 11:38 UTC (permalink / raw)
To: Agalya Babu RadhaKrishnan, dev
Cc: reshma.pattan, olivier.matz, ramirose, bernard.iremonger, stable
On 4/23/2019 11:44 AM, Agalya Babu RadhaKrishnan wrote:
> Added some missing documentation for interactive mode commands
>
> Fixes: 01b2092a5e ("testpmd: add dump commands for debug")
> Fixes: caf05a1b86 ("app/testpmd: new command to dump log types")
> Fixes: 0f62d63593ed ("app/testpmd: support tunneled TSO in checksum engine")
> Fixes: 8fff667578a7
> ("app/testpmd: new command to add/remove multicast MAC addresses")
> Fixes: ae03d0d18adf ("app/testpmd: command to configure MTU")
> Fixes: f79959ea1504 ("app/testpmd: allow to configure RSS hash key")
> Fixes: caf05a1b8608 ("app/testpmd: new command to dump log types")
> Cc: stable@dpdk.org
>
> Signed-off-by: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
> Reviewed-by: Rami Rosen <ramirose@gmail.com>
> Acked-by: Bernard Iremonger <bernard.iremonger@intel.com>
Applied to dpdk-next-net/master, thanks.
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [dpdk-dev] [PATCH v2] doc: fix interactive commands in testpmd guide
2019-04-23 11:38 ` Ferruh Yigit
@ 2019-04-23 11:38 ` Ferruh Yigit
0 siblings, 0 replies; 14+ messages in thread
From: Ferruh Yigit @ 2019-04-23 11:38 UTC (permalink / raw)
To: Agalya Babu RadhaKrishnan, dev
Cc: reshma.pattan, olivier.matz, ramirose, bernard.iremonger, stable
On 4/23/2019 11:44 AM, Agalya Babu RadhaKrishnan wrote:
> Added some missing documentation for interactive mode commands
>
> Fixes: 01b2092a5e ("testpmd: add dump commands for debug")
> Fixes: caf05a1b86 ("app/testpmd: new command to dump log types")
> Fixes: 0f62d63593ed ("app/testpmd: support tunneled TSO in checksum engine")
> Fixes: 8fff667578a7
> ("app/testpmd: new command to add/remove multicast MAC addresses")
> Fixes: ae03d0d18adf ("app/testpmd: command to configure MTU")
> Fixes: f79959ea1504 ("app/testpmd: allow to configure RSS hash key")
> Fixes: caf05a1b8608 ("app/testpmd: new command to dump log types")
> Cc: stable@dpdk.org
>
> Signed-off-by: Agalya Babu RadhaKrishnan <agalyax.babu.radhakrishnan@intel.com>
> Reviewed-by: Rami Rosen <ramirose@gmail.com>
> Acked-by: Bernard Iremonger <bernard.iremonger@intel.com>
Applied to dpdk-next-net/master, thanks.
^ permalink raw reply [flat|nested] 14+ messages in thread