DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc: remove file listings
@ 2019-01-17  9:05 David Marchand
  2019-01-17  9:16 ` Thomas Monjalon
                   ` (2 more replies)
  0 siblings, 3 replies; 13+ messages in thread
From: David Marchand @ 2019-01-17  9:05 UTC (permalink / raw)
  To: dev; +Cc: thomas, john.mcnamara, ferruh.yigit

No need to keep those file listings, they are very likely to become
outdated.

Signed-off-by: David Marchand <david.marchand@redhat.com>
---
 doc/guides/prog_guide/dev_kit_build_system.rst | 43 --------------
 doc/guides/prog_guide/source_org.rst           | 79 +-------------------------
 2 files changed, 2 insertions(+), 120 deletions(-)

diff --git a/doc/guides/prog_guide/dev_kit_build_system.rst b/doc/guides/prog_guide/dev_kit_build_system.rst
index da83a31..855b5da 100644
--- a/doc/guides/prog_guide/dev_kit_build_system.rst
+++ b/doc/guides/prog_guide/dev_kit_build_system.rst
@@ -52,49 +52,6 @@ which is equivalent to:
 
     make O=my_sdk_build_dir
 
-The content of the my_sdk_build_dir is then:
-
-::
-
-    -- .config                         # used configuration
-
-    -- Makefile                        # wrapper that calls head Makefile
-                                       # with $PWD as build directory
-
-
-        -- build                              #All temporary files used during build
-        +--app                                # process, including . o, .d, and .cmd files.
-            |  +-- test                       # For libraries, we have the .a file.
-            |  +-- test.o                     # For applications, we have the elf file.
-            |  `-- ...
-            +-- lib
-                +-- librte_eal
-                |   `-- ...
-                +-- librte_mempool
-                |  +--  mempool-file1.o
-                |  +--  .mempool-file1.o.cmd
-                |  +--  .mempool-file1.o.d
-                |  +--   mempool-file2.o
-                |  +--  .mempool-file2.o.cmd
-                |  +--  .mempool-file2.o.d
-                |  `--  mempool.a
-                `-- ...
-
-    -- include                # All include files installed by libraries
-        +-- librte_mempool.h  # and applications are located in this
-        +-- rte_eal.h         # directory. The installed files can depend
-        +-- rte_spinlock.h    # on configuration if needed (environment,
-        +-- rte_atomic.h      # architecture, ..)
-        `-- \*.h ...
-
-    -- lib                    # all compiled libraries are copied in this
-        +-- librte_eal.a      # directory
-        +-- librte_mempool.a
-        `-- \*.a ...
-
-    -- app                    # All compiled applications are installed
-    + --test                  # here. It includes the binary in elf format
-
 Refer to
 :ref:`Development Kit Root Makefile Help <Development_Kit_Root_Makefile_Help>`
 for details about make commands that can be used from the root of DPDK.
diff --git a/doc/guides/prog_guide/source_org.rst b/doc/guides/prog_guide/source_org.rst
index cee4ce6..1fb7abc 100644
--- a/doc/guides/prog_guide/source_org.rst
+++ b/doc/guides/prog_guide/source_org.rst
@@ -34,27 +34,6 @@ Libraries are located in subdirectories of ``$(RTE_SDK)/lib``.
 By convention a library refers to any code that provides an API to an application.
 Typically, it generates an archive file (``.a``), but a kernel module would also go in the same directory.
 
-The lib directory contains::
-
-    lib
-    +-- librte_cmdline      # Command line interface helper
-    +-- librte_distributor  # Packet distributor
-    +-- librte_eal          # Environment abstraction layer
-    +-- librte_ethdev       # Generic interface to poll mode driver
-    +-- librte_hash         # Hash library
-    +-- librte_ip_frag      # IP fragmentation library
-    +-- librte_kni          # Kernel NIC interface
-    +-- librte_kvargs       # Argument parsing library
-    +-- librte_lpm          # Longest prefix match library
-    +-- librte_mbuf         # Packet buffer manipulation
-    +-- librte_mempool      # Memory pool manager (fixed sized objects)
-    +-- librte_meter        # QoS metering library
-    +-- librte_net          # Various IP-related headers
-    +-- librte_power        # Power management library
-    +-- librte_ring         # Software rings (act as lockless FIFOs)
-    +-- librte_sched        # QoS scheduler and dropper library
-    +-- librte_timer        # Timer library
-
 Drivers
 -------
 
@@ -63,25 +42,6 @@ devices: either hardware devices or pseudo/virtual devices. They are contained
 in the *drivers* subdirectory, classified by type, and each compiles to a
 library with the format ``librte_pmd_X.a`` where ``X`` is the driver name.
 
-The drivers directory has a *net* subdirectory which contains::
-
-    drivers/net
-    +-- af_packet          # Poll mode driver based on Linux af_packet
-    +-- bonding            # Bonding poll mode driver
-    +-- cxgbe              # Chelsio Terminator 10GbE/40GbE poll mode driver
-    +-- e1000              # 1GbE poll mode drivers (igb and em)
-    +-- enic               # Cisco VIC Ethernet NIC Poll-mode Driver
-    +-- fm10k              # Host interface PMD driver for FM10000 Series
-    +-- i40e               # 40GbE poll mode driver
-    +-- ixgbe              # 10GbE poll mode driver
-    +-- mlx4               # Mellanox ConnectX-3 poll mode driver
-    +-- null               # NULL poll mode driver for testing
-    +-- pcap               # PCAP poll mode driver
-    +-- ring               # Ring poll mode driver
-    +-- szedata2           # SZEDATA2 poll mode driver
-    +-- virtio             # Virtio poll mode driver
-    +-- vmxnet3            # VMXNET3 poll mode driver
-
 .. note::
 
    Several of the ``driver/net`` directories contain a ``base``
@@ -98,41 +58,6 @@ Applications are source files that contain a ``main()`` function.
 They are located in the ``$(RTE_SDK)/app`` and ``$(RTE_SDK)/examples`` directories.
 
 The app directory contains sample applications that are used to test DPDK (such as autotests)
-or the Poll Mode Drivers (test-pmd)::
-
-    app
-    +-- chkincs            # Test program to check include dependencies
-    +-- cmdline_test       # Test the commandline library
-    +-- test               # Autotests to validate DPDK features
-    +-- test-acl           # Test the ACL library
-    +-- test-pipeline      # Test the IP Pipeline framework
-    +-- test-pmd           # Test and benchmark poll mode drivers
-
-The examples directory contains sample applications that show how libraries can be used::
-
-    examples
-    +-- cmdline            # Example of using the cmdline library
-    +-- exception_path     # Sending packets to and from Linux TAP device
-    +-- helloworld         # Basic Hello World example
-    +-- ip_reassembly      # Example showing IP reassembly
-    +-- ip_fragmentation   # Example showing IPv4 fragmentation
-    +-- ipv4_multicast     # Example showing IPv4 multicast
-    +-- kni                # Kernel NIC Interface (KNI) example
-    +-- l2fwd              # L2 forwarding with and without SR-IOV
-    +-- l3fwd              # L3 forwarding example
-    +-- l3fwd-power        # L3 forwarding example with power management
-    +-- l3fwd-vf           # L3 forwarding example with SR-IOV
-    +-- link_status_interrupt # Link status change interrupt example
-    +-- load_balancer      # Load balancing across multiple cores/sockets
-    +-- multi_process      # Example apps using multiple DPDK processes
-    +-- qos_meter          # QoS metering example
-    +-- qos_sched          # QoS scheduler and dropper example
-    +-- timer              # Example of using librte_timer library
-    +-- vmdq_dcb           # Example of VMDQ and DCB receiving
-    +-- vmdq               # Example of VMDQ receiving
-    +-- vhost              # Example of userspace vhost and switch
-
-.. note::
+or the Poll Mode Drivers (test-pmd).
 
-    The actual examples directory may contain additional sample applications to those shown above.
-    Check the latest DPDK source files for details.
+The examples directory contains sample applications that show how libraries can be used.
-- 
1.8.3.1

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

* Re: [dpdk-dev] [PATCH] doc: remove file listings
  2019-01-17  9:05 [dpdk-dev] [PATCH] doc: remove file listings David Marchand
@ 2019-01-17  9:16 ` Thomas Monjalon
  2019-01-17 15:42   ` Kovacevic, Marko
  2019-01-17 14:36 ` Ferruh Yigit
  2019-01-18 15:31 ` [dpdk-dev] [PATCH v2] " David Marchand
  2 siblings, 1 reply; 13+ messages in thread
From: Thomas Monjalon @ 2019-01-17  9:16 UTC (permalink / raw)
  To: David Marchand; +Cc: dev, john.mcnamara, ferruh.yigit, marko.kovacevic

+Cc Marko

17/01/2019 10:05, David Marchand:
> No need to keep those file listings, they are very likely to become
> outdated.
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
>  doc/guides/prog_guide/dev_kit_build_system.rst | 43 --------------
>  doc/guides/prog_guide/source_org.rst           | 79 +-------------------------
>  2 files changed, 2 insertions(+), 120 deletions(-)
> 
> diff --git a/doc/guides/prog_guide/dev_kit_build_system.rst b/doc/guides/prog_guide/dev_kit_build_system.rst
> index da83a31..855b5da 100644
> --- a/doc/guides/prog_guide/dev_kit_build_system.rst
> +++ b/doc/guides/prog_guide/dev_kit_build_system.rst
> @@ -52,49 +52,6 @@ which is equivalent to:
>  
>      make O=my_sdk_build_dir
>  
> -The content of the my_sdk_build_dir is then:
> -
> -::
> -
> -    -- .config                         # used configuration
> -
> -    -- Makefile                        # wrapper that calls head Makefile
> -                                       # with $PWD as build directory
> -
> -
> -        -- build                              #All temporary files used during build
> -        +--app                                # process, including . o, .d, and .cmd files.
> -            |  +-- test                       # For libraries, we have the .a file.
> -            |  +-- test.o                     # For applications, we have the elf file.
> -            |  `-- ...
> -            +-- lib
> -                +-- librte_eal
> -                |   `-- ...
> -                +-- librte_mempool
> -                |  +--  mempool-file1.o
> -                |  +--  .mempool-file1.o.cmd
> -                |  +--  .mempool-file1.o.d
> -                |  +--   mempool-file2.o
> -                |  +--  .mempool-file2.o.cmd
> -                |  +--  .mempool-file2.o.d
> -                |  `--  mempool.a
> -                `-- ...
> -
> -    -- include                # All include files installed by libraries
> -        +-- librte_mempool.h  # and applications are located in this
> -        +-- rte_eal.h         # directory. The installed files can depend
> -        +-- rte_spinlock.h    # on configuration if needed (environment,
> -        +-- rte_atomic.h      # architecture, ..)
> -        `-- \*.h ...
> -
> -    -- lib                    # all compiled libraries are copied in this
> -        +-- librte_eal.a      # directory
> -        +-- librte_mempool.a
> -        `-- \*.a ...
> -
> -    -- app                    # All compiled applications are installed
> -    + --test                  # here. It includes the binary in elf format
> -
>  Refer to
>  :ref:`Development Kit Root Makefile Help <Development_Kit_Root_Makefile_Help>`
>  for details about make commands that can be used from the root of DPDK.
> diff --git a/doc/guides/prog_guide/source_org.rst b/doc/guides/prog_guide/source_org.rst
> index cee4ce6..1fb7abc 100644
> --- a/doc/guides/prog_guide/source_org.rst
> +++ b/doc/guides/prog_guide/source_org.rst
> @@ -34,27 +34,6 @@ Libraries are located in subdirectories of ``$(RTE_SDK)/lib``.
>  By convention a library refers to any code that provides an API to an application.
>  Typically, it generates an archive file (``.a``), but a kernel module would also go in the same directory.
>  
> -The lib directory contains::
> -
> -    lib
> -    +-- librte_cmdline      # Command line interface helper
> -    +-- librte_distributor  # Packet distributor
> -    +-- librte_eal          # Environment abstraction layer
> -    +-- librte_ethdev       # Generic interface to poll mode driver
> -    +-- librte_hash         # Hash library
> -    +-- librte_ip_frag      # IP fragmentation library
> -    +-- librte_kni          # Kernel NIC interface
> -    +-- librte_kvargs       # Argument parsing library
> -    +-- librte_lpm          # Longest prefix match library
> -    +-- librte_mbuf         # Packet buffer manipulation
> -    +-- librte_mempool      # Memory pool manager (fixed sized objects)
> -    +-- librte_meter        # QoS metering library
> -    +-- librte_net          # Various IP-related headers
> -    +-- librte_power        # Power management library
> -    +-- librte_ring         # Software rings (act as lockless FIFOs)
> -    +-- librte_sched        # QoS scheduler and dropper library
> -    +-- librte_timer        # Timer library
> -
>  Drivers
>  -------
>  
> @@ -63,25 +42,6 @@ devices: either hardware devices or pseudo/virtual devices. They are contained
>  in the *drivers* subdirectory, classified by type, and each compiles to a
>  library with the format ``librte_pmd_X.a`` where ``X`` is the driver name.
>  
> -The drivers directory has a *net* subdirectory which contains::
> -
> -    drivers/net
> -    +-- af_packet          # Poll mode driver based on Linux af_packet
> -    +-- bonding            # Bonding poll mode driver
> -    +-- cxgbe              # Chelsio Terminator 10GbE/40GbE poll mode driver
> -    +-- e1000              # 1GbE poll mode drivers (igb and em)
> -    +-- enic               # Cisco VIC Ethernet NIC Poll-mode Driver
> -    +-- fm10k              # Host interface PMD driver for FM10000 Series
> -    +-- i40e               # 40GbE poll mode driver
> -    +-- ixgbe              # 10GbE poll mode driver
> -    +-- mlx4               # Mellanox ConnectX-3 poll mode driver
> -    +-- null               # NULL poll mode driver for testing
> -    +-- pcap               # PCAP poll mode driver
> -    +-- ring               # Ring poll mode driver
> -    +-- szedata2           # SZEDATA2 poll mode driver
> -    +-- virtio             # Virtio poll mode driver
> -    +-- vmxnet3            # VMXNET3 poll mode driver
> -
>  .. note::
>  
>     Several of the ``driver/net`` directories contain a ``base``
> @@ -98,41 +58,6 @@ Applications are source files that contain a ``main()`` function.
>  They are located in the ``$(RTE_SDK)/app`` and ``$(RTE_SDK)/examples`` directories.
>  
>  The app directory contains sample applications that are used to test DPDK (such as autotests)
> -or the Poll Mode Drivers (test-pmd)::
> -
> -    app
> -    +-- chkincs            # Test program to check include dependencies
> -    +-- cmdline_test       # Test the commandline library
> -    +-- test               # Autotests to validate DPDK features
> -    +-- test-acl           # Test the ACL library
> -    +-- test-pipeline      # Test the IP Pipeline framework
> -    +-- test-pmd           # Test and benchmark poll mode drivers
> -
> -The examples directory contains sample applications that show how libraries can be used::
> -
> -    examples
> -    +-- cmdline            # Example of using the cmdline library
> -    +-- exception_path     # Sending packets to and from Linux TAP device
> -    +-- helloworld         # Basic Hello World example
> -    +-- ip_reassembly      # Example showing IP reassembly
> -    +-- ip_fragmentation   # Example showing IPv4 fragmentation
> -    +-- ipv4_multicast     # Example showing IPv4 multicast
> -    +-- kni                # Kernel NIC Interface (KNI) example
> -    +-- l2fwd              # L2 forwarding with and without SR-IOV
> -    +-- l3fwd              # L3 forwarding example
> -    +-- l3fwd-power        # L3 forwarding example with power management
> -    +-- l3fwd-vf           # L3 forwarding example with SR-IOV
> -    +-- link_status_interrupt # Link status change interrupt example
> -    +-- load_balancer      # Load balancing across multiple cores/sockets
> -    +-- multi_process      # Example apps using multiple DPDK processes
> -    +-- qos_meter          # QoS metering example
> -    +-- qos_sched          # QoS scheduler and dropper example
> -    +-- timer              # Example of using librte_timer library
> -    +-- vmdq_dcb           # Example of VMDQ and DCB receiving
> -    +-- vmdq               # Example of VMDQ receiving
> -    +-- vhost              # Example of userspace vhost and switch
> -
> -.. note::
> +or the Poll Mode Drivers (test-pmd).
>  
> -    The actual examples directory may contain additional sample applications to those shown above.
> -    Check the latest DPDK source files for details.
> +The examples directory contains sample applications that show how libraries can be used.
> 

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

* Re: [dpdk-dev] [PATCH] doc: remove file listings
  2019-01-17  9:05 [dpdk-dev] [PATCH] doc: remove file listings David Marchand
  2019-01-17  9:16 ` Thomas Monjalon
@ 2019-01-17 14:36 ` Ferruh Yigit
  2019-01-18 15:31 ` [dpdk-dev] [PATCH v2] " David Marchand
  2 siblings, 0 replies; 13+ messages in thread
From: Ferruh Yigit @ 2019-01-17 14:36 UTC (permalink / raw)
  To: David Marchand, dev; +Cc: thomas, john.mcnamara

On 1/17/2019 9:05 AM, David Marchand wrote:
> No need to keep those file listings, they are very likely to become
> outdated.
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>

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

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

* Re: [dpdk-dev] [PATCH] doc: remove file listings
  2019-01-17  9:16 ` Thomas Monjalon
@ 2019-01-17 15:42   ` Kovacevic, Marko
  2019-01-17 16:08     ` Thomas Monjalon
  0 siblings, 1 reply; 13+ messages in thread
From: Kovacevic, Marko @ 2019-01-17 15:42 UTC (permalink / raw)
  To: Thomas Monjalon, David Marchand; +Cc: dev, Mcnamara, John, Yigit, Ferruh

> > Signed-off-by: David Marchand <david.marchand@redhat.com>
> > ---
> >  doc/guides/prog_guide/dev_kit_build_system.rst | 43 --------------
> >  doc/guides/prog_guide/source_org.rst           | 79 +-------------------------
> >  2 files changed, 2 insertions(+), 120 deletions(-)
> >
> > diff --git a/doc/guides/prog_guide/dev_kit_build_system.rst
> > b/doc/guides/prog_guide/dev_kit_build_system.rst
> > index da83a31..855b5da 100644
> > --- a/doc/guides/prog_guide/dev_kit_build_system.rst
> > +++ b/doc/guides/prog_guide/dev_kit_build_system.rst
> > @@ -52,49 +52,6 @@ which is equivalent to:
> >

I agree with the change and deletion of the sections there,
But I feel like there still needs to be a clickable reference to each section in the docs.
So if they want to see the drivers or applications they can still be brought there.

Marko K

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

* Re: [dpdk-dev] [PATCH] doc: remove file listings
  2019-01-17 15:42   ` Kovacevic, Marko
@ 2019-01-17 16:08     ` Thomas Monjalon
  2019-01-17 16:53       ` Kovacevic, Marko
  0 siblings, 1 reply; 13+ messages in thread
From: Thomas Monjalon @ 2019-01-17 16:08 UTC (permalink / raw)
  To: Kovacevic, Marko; +Cc: David Marchand, dev, Mcnamara, John, Yigit, Ferruh

17/01/2019 16:42, Kovacevic, Marko:
> > > Signed-off-by: David Marchand <david.marchand@redhat.com>
> > > ---
> > >  doc/guides/prog_guide/dev_kit_build_system.rst | 43 --------------
> > >  doc/guides/prog_guide/source_org.rst           | 79 +-------------------------
> > >  2 files changed, 2 insertions(+), 120 deletions(-)
> > >
> > > diff --git a/doc/guides/prog_guide/dev_kit_build_system.rst
> > > b/doc/guides/prog_guide/dev_kit_build_system.rst
> > > index da83a31..855b5da 100644
> > > --- a/doc/guides/prog_guide/dev_kit_build_system.rst
> > > +++ b/doc/guides/prog_guide/dev_kit_build_system.rst
> > > @@ -52,49 +52,6 @@ which is equivalent to:
> > >
> 
> I agree with the change and deletion of the sections there,
> But I feel like there still needs to be a clickable reference to each section in the docs.
> So if they want to see the drivers or applications they can still be brought there.

I'm not sure to understand your proposal.
Please could you explain?

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

* Re: [dpdk-dev] [PATCH] doc: remove file listings
  2019-01-17 16:08     ` Thomas Monjalon
@ 2019-01-17 16:53       ` Kovacevic, Marko
  2019-01-17 17:01         ` Thomas Monjalon
  0 siblings, 1 reply; 13+ messages in thread
From: Kovacevic, Marko @ 2019-01-17 16:53 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: David Marchand, dev, Mcnamara, John, Yigit, Ferruh

Sorry for not making it clearer,
So if someone was reading it and saw this line

"The examples directory contains sample applications that show how libraries can be used"
 
It would be nice to have a link to the sample applications in the docs like so, which has all the 
Info that was deleted. 

"The examples directory contains :doc:`Sample applications<../sample_app_ug/index>` that show how libraries can be used."


-    examples
-    +-- cmdline            # Example of using the cmdline library
-    +-- exception_path     # Sending packets to and from Linux TAP device
-    +-- helloworld         # Basic Hello World example
-    +-- ip_reassembly      # Example showing IP reassembly
-    +-- ip_fragmentation   # Example showing IPv4 fragmentation
-    +-- ipv4_multicast     # Example showing IPv4 multicast
-    +-- kni                # Kernel NIC Interface (KNI) example
-    +-- l2fwd              # L2 forwarding with and without SR-IOV
-    +-- l3fwd              # L3 forwarding example
-    +-- l3fwd-power        # L3 forwarding example with power management
-    +-- l3fwd-vf           # L3 forwarding example with SR-IOV
-    +-- link_status_interrupt # Link status change interrupt example
-    +-- load_balancer      # Load balancing across multiple cores/sockets
-    +-- multi_process      # Example apps using multiple DPDK processes
-    +-- qos_meter          # QoS metering example
-    +-- qos_sched          # QoS scheduler and dropper example
-    +-- timer              # Example of using librte_timer library
-    +-- vmdq_dcb           # Example of VMDQ and DCB receiving
-    +-- vmdq               # Example of VMDQ receiving
-    +-- vhost              # Example of userspace vhost and switch

Thanks, 
Marko K

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

* Re: [dpdk-dev] [PATCH] doc: remove file listings
  2019-01-17 16:53       ` Kovacevic, Marko
@ 2019-01-17 17:01         ` Thomas Monjalon
  2019-01-18  9:29           ` Kovacevic, Marko
  0 siblings, 1 reply; 13+ messages in thread
From: Thomas Monjalon @ 2019-01-17 17:01 UTC (permalink / raw)
  To: Kovacevic, Marko; +Cc: David Marchand, dev, Mcnamara, John, Yigit, Ferruh

17/01/2019 17:53, Kovacevic, Marko:
> Sorry for not making it clearer,
> So if someone was reading it and saw this line
> 
> "The examples directory contains sample applications that show how libraries can be used"
>  
> It would be nice to have a link to the sample applications in the docs like so, which has all the 
> Info that was deleted. 
> 
> "The examples directory contains :doc:`Sample applications<../sample_app_ug/index>` that show how libraries can be used."
> 
> 
> -    examples
> -    +-- cmdline            # Example of using the cmdline library
> -    +-- exception_path     # Sending packets to and from Linux TAP device
> -    +-- helloworld         # Basic Hello World example
> -    +-- ip_reassembly      # Example showing IP reassembly
> -    +-- ip_fragmentation   # Example showing IPv4 fragmentation
> -    +-- ipv4_multicast     # Example showing IPv4 multicast
> -    +-- kni                # Kernel NIC Interface (KNI) example
> -    +-- l2fwd              # L2 forwarding with and without SR-IOV
> -    +-- l3fwd              # L3 forwarding example
> -    +-- l3fwd-power        # L3 forwarding example with power management
> -    +-- l3fwd-vf           # L3 forwarding example with SR-IOV
> -    +-- link_status_interrupt # Link status change interrupt example
> -    +-- load_balancer      # Load balancing across multiple cores/sockets
> -    +-- multi_process      # Example apps using multiple DPDK processes
> -    +-- qos_meter          # QoS metering example
> -    +-- qos_sched          # QoS scheduler and dropper example
> -    +-- timer              # Example of using librte_timer library
> -    +-- vmdq_dcb           # Example of VMDQ and DCB receiving
> -    +-- vmdq               # Example of VMDQ receiving
> -    +-- vhost              # Example of userspace vhost and switch

We have a full guide about examples:
	http://doc.dpdk.org/guides/sample_app_ug/index.html

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

* Re: [dpdk-dev] [PATCH] doc: remove file listings
  2019-01-17 17:01         ` Thomas Monjalon
@ 2019-01-18  9:29           ` Kovacevic, Marko
  2019-01-18  9:33             ` Thomas Monjalon
  0 siblings, 1 reply; 13+ messages in thread
From: Kovacevic, Marko @ 2019-01-18  9:29 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: David Marchand, dev, Mcnamara, John, Yigit, Ferruh

> > Sorry for not making it clearer,
> > So if someone was reading it and saw this line
> >
> > "The examples directory contains sample applications that show how
> libraries can be used"
> >
> > It would be nice to have a link to the sample applications in the docs
> > like so, which has all the Info that was deleted.
> >
> > "The examples directory contains :doc: ../sample_app_ug/index> that show how libraries can be
> used."
> >
> >
> > -    examples
> > -    +-- cmdline            # Example of using the cmdline library
> > -    +-- exception_path     # Sending packets to and from Linux TAP device
> > -    +-- helloworld         # Basic Hello World example
> > -    +-- ip_reassembly      # Example showing IP reassembly
> > -    +-- ip_fragmentation   # Example showing IPv4 fragmentation
> > -    +-- ipv4_multicast     # Example showing IPv4 multicast
> > -    +-- kni                # Kernel NIC Interface (KNI) example
> > -    +-- l2fwd              # L2 forwarding with and without SR-IOV
> > -    +-- l3fwd              # L3 forwarding example
> > -    +-- l3fwd-power        # L3 forwarding example with power management
> > -    +-- l3fwd-vf           # L3 forwarding example with SR-IOV
> > -    +-- link_status_interrupt # Link status change interrupt example
> > -    +-- load_balancer      # Load balancing across multiple cores/sockets
> > -    +-- multi_process      # Example apps using multiple DPDK processes
> > -    +-- qos_meter          # QoS metering example
> > -    +-- qos_sched          # QoS scheduler and dropper example
> > -    +-- timer              # Example of using librte_timer library
> > -    +-- vmdq_dcb           # Example of VMDQ and DCB receiving
> > -    +-- vmdq               # Example of VMDQ receiving
> > -    +-- vhost              # Example of userspace vhost and switch
> 
> We have a full guide about examples:
> 	http://doc.dpdk.org/guides/sample_app_ug/index.html

Yes I know but I'm saying since we are removing the list from here would it not 
Be a good idea to add a link to the sample guides in the line it says

The examples directory contains sample applications that show how libraries can be used

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

* Re: [dpdk-dev] [PATCH] doc: remove file listings
  2019-01-18  9:29           ` Kovacevic, Marko
@ 2019-01-18  9:33             ` Thomas Monjalon
  2019-01-18 10:15               ` David Marchand
  0 siblings, 1 reply; 13+ messages in thread
From: Thomas Monjalon @ 2019-01-18  9:33 UTC (permalink / raw)
  To: Kovacevic, Marko; +Cc: David Marchand, dev, Mcnamara, John, Yigit, Ferruh

18/01/2019 10:29, Kovacevic, Marko:
> > > Sorry for not making it clearer,
> > > So if someone was reading it and saw this line
> > >
> > > "The examples directory contains sample applications that show how
> > libraries can be used"
> > >
> > > It would be nice to have a link to the sample applications in the docs
> > > like so, which has all the Info that was deleted.
> > >
> > > "The examples directory contains :doc: ../sample_app_ug/index> that show how libraries can be
> > used."
> > >
> > >
> > > -    examples
> > > -    +-- cmdline            # Example of using the cmdline library
> > > -    +-- exception_path     # Sending packets to and from Linux TAP device
> > > -    +-- helloworld         # Basic Hello World example
> > > -    +-- ip_reassembly      # Example showing IP reassembly
> > > -    +-- ip_fragmentation   # Example showing IPv4 fragmentation
> > > -    +-- ipv4_multicast     # Example showing IPv4 multicast
> > > -    +-- kni                # Kernel NIC Interface (KNI) example
> > > -    +-- l2fwd              # L2 forwarding with and without SR-IOV
> > > -    +-- l3fwd              # L3 forwarding example
> > > -    +-- l3fwd-power        # L3 forwarding example with power management
> > > -    +-- l3fwd-vf           # L3 forwarding example with SR-IOV
> > > -    +-- link_status_interrupt # Link status change interrupt example
> > > -    +-- load_balancer      # Load balancing across multiple cores/sockets
> > > -    +-- multi_process      # Example apps using multiple DPDK processes
> > > -    +-- qos_meter          # QoS metering example
> > > -    +-- qos_sched          # QoS scheduler and dropper example
> > > -    +-- timer              # Example of using librte_timer library
> > > -    +-- vmdq_dcb           # Example of VMDQ and DCB receiving
> > > -    +-- vmdq               # Example of VMDQ receiving
> > > -    +-- vhost              # Example of userspace vhost and switch
> > 
> > We have a full guide about examples:
> > 	http://doc.dpdk.org/guides/sample_app_ug/index.html
> 
> Yes I know but I'm saying since we are removing the list from here would it not 
> Be a good idea to add a link to the sample guides in the line it says
> 
> The examples directory contains sample applications that show how libraries can be used

Yes, good idea.

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

* Re: [dpdk-dev] [PATCH] doc: remove file listings
  2019-01-18  9:33             ` Thomas Monjalon
@ 2019-01-18 10:15               ` David Marchand
  0 siblings, 0 replies; 13+ messages in thread
From: David Marchand @ 2019-01-18 10:15 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: Kovacevic, Marko, dev, Mcnamara, John, Yigit, Ferruh

On Fri, Jan 18, 2019 at 10:33 AM Thomas Monjalon <thomas@monjalon.net>
wrote:

> 18/01/2019 10:29, Kovacevic, Marko:
> > > > Sorry for not making it clearer,
> > > > So if someone was reading it and saw this line
> > > >
> > > > "The examples directory contains sample applications that show how
> > > libraries can be used"
> > > >
> > > > It would be nice to have a link to the sample applications in the
> docs
> > > > like so, which has all the Info that was deleted.
> > > >
> > > > "The examples directory contains :doc: ../sample_app_ug/index> that
> show how libraries can be
> > > used."
> > > >
>
> Yes, good idea.
>

Ok, I will prepare a v2 with this.

-- 
David Marchand

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

* [dpdk-dev] [PATCH v2] doc: remove file listings
  2019-01-17  9:05 [dpdk-dev] [PATCH] doc: remove file listings David Marchand
  2019-01-17  9:16 ` Thomas Monjalon
  2019-01-17 14:36 ` Ferruh Yigit
@ 2019-01-18 15:31 ` David Marchand
  2019-01-18 15:35   ` Kovacevic, Marko
  2 siblings, 1 reply; 13+ messages in thread
From: David Marchand @ 2019-01-18 15:31 UTC (permalink / raw)
  To: dev; +Cc: thomas, john.mcnamara, ferruh.yigit, marko.kovacevic

No need to keep those file listings, they are very likely to become
outdated.

Signed-off-by: David Marchand <david.marchand@redhat.com>
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
Changelog since v1:
- added a link to the sample applications index

---
 doc/guides/prog_guide/dev_kit_build_system.rst | 43 --------------
 doc/guides/prog_guide/source_org.rst           | 79 +-------------------------
 2 files changed, 2 insertions(+), 120 deletions(-)

diff --git a/doc/guides/prog_guide/dev_kit_build_system.rst b/doc/guides/prog_guide/dev_kit_build_system.rst
index da83a31..855b5da 100644
--- a/doc/guides/prog_guide/dev_kit_build_system.rst
+++ b/doc/guides/prog_guide/dev_kit_build_system.rst
@@ -52,49 +52,6 @@ which is equivalent to:
 
     make O=my_sdk_build_dir
 
-The content of the my_sdk_build_dir is then:
-
-::
-
-    -- .config                         # used configuration
-
-    -- Makefile                        # wrapper that calls head Makefile
-                                       # with $PWD as build directory
-
-
-        -- build                              #All temporary files used during build
-        +--app                                # process, including . o, .d, and .cmd files.
-            |  +-- test                       # For libraries, we have the .a file.
-            |  +-- test.o                     # For applications, we have the elf file.
-            |  `-- ...
-            +-- lib
-                +-- librte_eal
-                |   `-- ...
-                +-- librte_mempool
-                |  +--  mempool-file1.o
-                |  +--  .mempool-file1.o.cmd
-                |  +--  .mempool-file1.o.d
-                |  +--   mempool-file2.o
-                |  +--  .mempool-file2.o.cmd
-                |  +--  .mempool-file2.o.d
-                |  `--  mempool.a
-                `-- ...
-
-    -- include                # All include files installed by libraries
-        +-- librte_mempool.h  # and applications are located in this
-        +-- rte_eal.h         # directory. The installed files can depend
-        +-- rte_spinlock.h    # on configuration if needed (environment,
-        +-- rte_atomic.h      # architecture, ..)
-        `-- \*.h ...
-
-    -- lib                    # all compiled libraries are copied in this
-        +-- librte_eal.a      # directory
-        +-- librte_mempool.a
-        `-- \*.a ...
-
-    -- app                    # All compiled applications are installed
-    + --test                  # here. It includes the binary in elf format
-
 Refer to
 :ref:`Development Kit Root Makefile Help <Development_Kit_Root_Makefile_Help>`
 for details about make commands that can be used from the root of DPDK.
diff --git a/doc/guides/prog_guide/source_org.rst b/doc/guides/prog_guide/source_org.rst
index cee4ce6..31c153a 100644
--- a/doc/guides/prog_guide/source_org.rst
+++ b/doc/guides/prog_guide/source_org.rst
@@ -34,27 +34,6 @@ Libraries are located in subdirectories of ``$(RTE_SDK)/lib``.
 By convention a library refers to any code that provides an API to an application.
 Typically, it generates an archive file (``.a``), but a kernel module would also go in the same directory.
 
-The lib directory contains::
-
-    lib
-    +-- librte_cmdline      # Command line interface helper
-    +-- librte_distributor  # Packet distributor
-    +-- librte_eal          # Environment abstraction layer
-    +-- librte_ethdev       # Generic interface to poll mode driver
-    +-- librte_hash         # Hash library
-    +-- librte_ip_frag      # IP fragmentation library
-    +-- librte_kni          # Kernel NIC interface
-    +-- librte_kvargs       # Argument parsing library
-    +-- librte_lpm          # Longest prefix match library
-    +-- librte_mbuf         # Packet buffer manipulation
-    +-- librte_mempool      # Memory pool manager (fixed sized objects)
-    +-- librte_meter        # QoS metering library
-    +-- librte_net          # Various IP-related headers
-    +-- librte_power        # Power management library
-    +-- librte_ring         # Software rings (act as lockless FIFOs)
-    +-- librte_sched        # QoS scheduler and dropper library
-    +-- librte_timer        # Timer library
-
 Drivers
 -------
 
@@ -63,25 +42,6 @@ devices: either hardware devices or pseudo/virtual devices. They are contained
 in the *drivers* subdirectory, classified by type, and each compiles to a
 library with the format ``librte_pmd_X.a`` where ``X`` is the driver name.
 
-The drivers directory has a *net* subdirectory which contains::
-
-    drivers/net
-    +-- af_packet          # Poll mode driver based on Linux af_packet
-    +-- bonding            # Bonding poll mode driver
-    +-- cxgbe              # Chelsio Terminator 10GbE/40GbE poll mode driver
-    +-- e1000              # 1GbE poll mode drivers (igb and em)
-    +-- enic               # Cisco VIC Ethernet NIC Poll-mode Driver
-    +-- fm10k              # Host interface PMD driver for FM10000 Series
-    +-- i40e               # 40GbE poll mode driver
-    +-- ixgbe              # 10GbE poll mode driver
-    +-- mlx4               # Mellanox ConnectX-3 poll mode driver
-    +-- null               # NULL poll mode driver for testing
-    +-- pcap               # PCAP poll mode driver
-    +-- ring               # Ring poll mode driver
-    +-- szedata2           # SZEDATA2 poll mode driver
-    +-- virtio             # Virtio poll mode driver
-    +-- vmxnet3            # VMXNET3 poll mode driver
-
 .. note::
 
    Several of the ``driver/net`` directories contain a ``base``
@@ -98,41 +58,6 @@ Applications are source files that contain a ``main()`` function.
 They are located in the ``$(RTE_SDK)/app`` and ``$(RTE_SDK)/examples`` directories.
 
 The app directory contains sample applications that are used to test DPDK (such as autotests)
-or the Poll Mode Drivers (test-pmd)::
-
-    app
-    +-- chkincs            # Test program to check include dependencies
-    +-- cmdline_test       # Test the commandline library
-    +-- test               # Autotests to validate DPDK features
-    +-- test-acl           # Test the ACL library
-    +-- test-pipeline      # Test the IP Pipeline framework
-    +-- test-pmd           # Test and benchmark poll mode drivers
-
-The examples directory contains sample applications that show how libraries can be used::
-
-    examples
-    +-- cmdline            # Example of using the cmdline library
-    +-- exception_path     # Sending packets to and from Linux TAP device
-    +-- helloworld         # Basic Hello World example
-    +-- ip_reassembly      # Example showing IP reassembly
-    +-- ip_fragmentation   # Example showing IPv4 fragmentation
-    +-- ipv4_multicast     # Example showing IPv4 multicast
-    +-- kni                # Kernel NIC Interface (KNI) example
-    +-- l2fwd              # L2 forwarding with and without SR-IOV
-    +-- l3fwd              # L3 forwarding example
-    +-- l3fwd-power        # L3 forwarding example with power management
-    +-- l3fwd-vf           # L3 forwarding example with SR-IOV
-    +-- link_status_interrupt # Link status change interrupt example
-    +-- load_balancer      # Load balancing across multiple cores/sockets
-    +-- multi_process      # Example apps using multiple DPDK processes
-    +-- qos_meter          # QoS metering example
-    +-- qos_sched          # QoS scheduler and dropper example
-    +-- timer              # Example of using librte_timer library
-    +-- vmdq_dcb           # Example of VMDQ and DCB receiving
-    +-- vmdq               # Example of VMDQ receiving
-    +-- vhost              # Example of userspace vhost and switch
-
-.. note::
+or the Poll Mode Drivers (test-pmd).
 
-    The actual examples directory may contain additional sample applications to those shown above.
-    Check the latest DPDK source files for details.
+The examples directory contains :doc:`Sample applications<../sample_app_ug/index>` that show how libraries can be used.
-- 
1.8.3.1

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

* Re: [dpdk-dev] [PATCH v2] doc: remove file listings
  2019-01-18 15:31 ` [dpdk-dev] [PATCH v2] " David Marchand
@ 2019-01-18 15:35   ` Kovacevic, Marko
  2019-01-20 12:10     ` Thomas Monjalon
  0 siblings, 1 reply; 13+ messages in thread
From: Kovacevic, Marko @ 2019-01-18 15:35 UTC (permalink / raw)
  To: David Marchand, dev; +Cc: thomas, Mcnamara, John, Yigit, Ferruh

Acked-by: Marko Kovacevic <marko.kovacevic@intel.com>

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

* Re: [dpdk-dev] [PATCH v2] doc: remove file listings
  2019-01-18 15:35   ` Kovacevic, Marko
@ 2019-01-20 12:10     ` Thomas Monjalon
  0 siblings, 0 replies; 13+ messages in thread
From: Thomas Monjalon @ 2019-01-20 12:10 UTC (permalink / raw)
  To: David Marchand; +Cc: dev, Kovacevic, Marko, Mcnamara, John, Yigit, Ferruh

18/01/2019 16:35, Kovacevic, Marko:
> Acked-by: Marko Kovacevic <marko.kovacevic@intel.com>

Applied, thanks

Marko, please keep previous SoB and acks so we have a bit of history
in replies. Usually we also the keep the commit description in acks.

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

end of thread, other threads:[~2019-01-20 12:10 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-17  9:05 [dpdk-dev] [PATCH] doc: remove file listings David Marchand
2019-01-17  9:16 ` Thomas Monjalon
2019-01-17 15:42   ` Kovacevic, Marko
2019-01-17 16:08     ` Thomas Monjalon
2019-01-17 16:53       ` Kovacevic, Marko
2019-01-17 17:01         ` Thomas Monjalon
2019-01-18  9:29           ` Kovacevic, Marko
2019-01-18  9:33             ` Thomas Monjalon
2019-01-18 10:15               ` David Marchand
2019-01-17 14:36 ` Ferruh Yigit
2019-01-18 15:31 ` [dpdk-dev] [PATCH v2] " David Marchand
2019-01-18 15:35   ` Kovacevic, Marko
2019-01-20 12:10     ` Thomas Monjalon

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