[dpdk-dev] [PATCH] doc: update commands for pdump

[dpdk-dev] [PATCH] doc: update commands for pdump

[Tiwei Bie]

The --server-socket-path and --client-socket-path options
have already been removed. So also remove them from the doc
to avoid confusion.

Fixes: 09f4aa2b955f ("app/pdump: remove unused socket path options")
Cc: stable@dpdk.org

Signed-off-by: Tiwei Bie <tiwei.bie@intel.com>
---
 doc/guides/tools/pdump.rst | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/doc/guides/tools/pdump.rst b/doc/guides/tools/pdump.rst
index 5168c81ab..7c2b73e72 100644
--- a/doc/guides/tools/pdump.rst
+++ b/doc/guides/tools/pdump.rst
@@ -42,8 +42,6 @@ The tool has a number of command line options:
                                    [ring-size=<ring size>],
                                    [mbuf-size=<mbuf data size>],
                                    [total-num-mbufs=<number of mbufs>]'
-                          [--server-socket-path=<server socket dir>]
-                          [--client-socket-path=<client socket dir>]
 
 The ``--pdump`` command line option is mandatory and it takes various sub arguments which are described in
 below section.
@@ -56,14 +54,6 @@ below section.
 
       * Multiple instances of ``--pdump`` can be passed to capture packets on different port and queue combinations.
 
-The ``--server-socket-path`` command line option is optional. This represents the server socket directory.
-If no value is passed default values are used i.e. ``/var/run/.dpdk/`` for root users and ``~/.dpdk/``
-for non root users.
-
-The ``--client-socket-path`` command line option is optional. This represents the client socket directory.
-If no value is passed default values are used i.e. ``/var/run/.dpdk/`` for root users and ``~/.dpdk/``
-for non root users.
-
 
 The ``--pdump`` parameters
 ~~~~~~~~~~~~~~~~~~~~~~~~~~
-- 
2.14.5

Re: [dpdk-dev] [PATCH] doc: update commands for pdump

[Ferruh Yigit]

On 11/22/2018 2:51 AM, Tiwei Bie wrote:
> The --server-socket-path and --client-socket-path options
> have already been removed. So also remove them from the doc
> to avoid confusion.
> 
> Fixes: 09f4aa2b955f ("app/pdump: remove unused socket path options")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Tiwei Bie <tiwei.bie@intel.com>

Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>

Re: [dpdk-dev] [PATCH] doc: update commands for pdump

[Mcnamara, John]

> -----Original Message-----
> From: Bie, Tiwei
> Sent: Thursday, November 22, 2018 2:52 AM
> To: dev@dpdk.org
> Cc: Pattan, Reshma <reshma.pattan@intel.com>; Mcnamara, John
> <john.mcnamara@intel.com>; Kovacevic, Marko <marko.kovacevic@intel.com>;
> stable@dpdk.org
> Subject: [PATCH] doc: update commands for pdump
> 
> The --server-socket-path and --client-socket-path options have already
> been removed. So also remove them from the doc to avoid confusion.


Acked-by: John McNamara <john.mcnamara@intel.com>

Re: [dpdk-dev] [dpdk-stable] [PATCH] doc: update commands for pdump

[Thomas Monjalon]

22/11/2018 18:54, Mcnamara, John:
> From: Bie, Tiwei
> > The --server-socket-path and --client-socket-path options have already
> > been removed. So also remove them from the doc to avoid confusion.
> 
> Acked-by: John McNamara <john.mcnamara@intel.com>

Applied, thanks