DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [RFC PATCH 0/4 v2] Extending DPDK with multiple device support
@ 2015-04-13 19:44 Keith Wiles
  2015-04-13 19:44 ` [dpdk-dev] [RFC PATCH 1/4 v2] Adding the common device files for " Keith Wiles
                   ` (5 more replies)
  0 siblings, 6 replies; 28+ messages in thread
From: Keith Wiles @ 2015-04-13 19:44 UTC (permalink / raw)
  To: dev

Hi All,

Here is a set of RFC patches to update DPDK to support a generic set of
devices. The patches that follow create two files eal_common_device.h
and rte_common_device.c and then a few items in rte_ethdev.[ch] were moved
to cleanup those files by moving the device generic values and functions
into a common set of device generic files.

In the rte_common_device.h file are a couple of macros to abstract a few
common items from rte_ethdev.h which are not ethernet specific. These
generic device specific structure members are place into macros to be
placed at the top of each new device type crypto, hardware offload, dpi,
compression and possible others to be defined later. In version 2 I used
nested structures a bit cleaner from the macro design, but modified a lot
more files.

Most of the changes are to update the code to locate the new location of
the members in the nested structures. To not try and rewrite code I
used macros to help hide the changes, but these constructs are now used
in a lot of files withint DPDK.

I did not pull the Rx/Tx Routines into a common function, but it could be
done if everyone agrees. It does not mean every device will use a common
Rx/Tx routine. Without pulling Rx/Tx routines into a common file allows
the device writer to have similar or different set of routines.

These patches do not contain any new devices as these are being work on
today. I could have included the DPI (dpidev) routines we did in the PoC,
but the DPI is not open source.

More cleanup of ethdev could be done to remove NIC specific features not
supported in all devices and someone needs to do that cleanup IMO.

The code is untested and I wanted to get something our for others to poke
at today and more could be pulled out of ethdev as well. I/We will be
looking at testing the code as we get more completed.

I have not finished up the crypto APIs yet, but I am planning to work on
those additions today. The crypto code we are using is the Quick Assist
code found on 01.org, but we need to update the code to be move DPDK
friendly.

The QAT code does have a modified like API similar to Linux Kernel crypto
API and I want to review that code first.

Regards,
++Keith


Keith Wiles (4):
  Adding the common device files for multiple device support
  Add the ethdev changes for multiple device support
  Add the test file changes for common device support
  Update PMD files for new common device support

 app/test-pmd/config.c                             |   6 +-
 app/test-pmd/testpmd.h                            |   4 +-
 app/test/test_kni.c                               |  12 +-
 app/test/test_link_bonding.c                      |  24 +-
 app/test/virtual_pmd.c                            | 106 +--
 examples/link_status_interrupt/main.c             |   6 +-
 lib/librte_eal/bsdapp/eal/Makefile                |   1 +
 lib/librte_eal/common/Makefile                    |   1 +
 lib/librte_eal/common/eal_common_device.c         | 185 +++++
 lib/librte_eal/common/include/rte_common_device.h | 674 +++++++++++++++
 lib/librte_eal/common/include/rte_log.h           |   1 +
 lib/librte_eal/linuxapp/eal/Makefile              |   1 +
 lib/librte_ether/rte_ethdev.c                     | 944 +++++++++-------------
 lib/librte_ether/rte_ethdev.h                     | 340 ++------
 lib/librte_kni/rte_kni.c                          |   4 +-
 lib/librte_pmd_af_packet/rte_eth_af_packet.c      |  38 +-
 lib/librte_pmd_bond/rte_eth_bond_8023ad.c         |  18 +-
 lib/librte_pmd_bond/rte_eth_bond_alb.c            |  10 +-
 lib/librte_pmd_bond/rte_eth_bond_api.c            | 142 ++--
 lib/librte_pmd_bond/rte_eth_bond_args.c           |   2 +-
 lib/librte_pmd_bond/rte_eth_bond_pmd.c            | 164 ++--
 lib/librte_pmd_bond/rte_eth_bond_private.h        |   2 +-
 lib/librte_pmd_e1000/em_ethdev.c                  | 156 ++--
 lib/librte_pmd_e1000/em_rxtx.c                    |  94 +--
 lib/librte_pmd_e1000/igb_ethdev.c                 | 302 +++----
 lib/librte_pmd_e1000/igb_pf.c                     |  86 +-
 lib/librte_pmd_e1000/igb_rxtx.c                   | 168 ++--
 lib/librte_pmd_enic/enic.h                        |   4 +-
 lib/librte_pmd_enic/enic_ethdev.c                 | 140 ++--
 lib/librte_pmd_enic/enic_main.c                   |  30 +-
 lib/librte_pmd_fm10k/fm10k_ethdev.c               | 154 ++--
 lib/librte_pmd_fm10k/fm10k_rxtx.c                 |   4 +-
 lib/librte_pmd_i40e/i40e_ethdev.c                 | 172 ++--
 lib/librte_pmd_i40e/i40e_ethdev_vf.c              | 194 ++---
 lib/librte_pmd_i40e/i40e_fdir.c                   |  28 +-
 lib/librte_pmd_i40e/i40e_pf.c                     |   8 +-
 lib/librte_pmd_i40e/i40e_rxtx.c                   |  88 +-
 lib/librte_pmd_ixgbe/ixgbe_ethdev.c               | 392 ++++-----
 lib/librte_pmd_ixgbe/ixgbe_fdir.c                 |  50 +-
 lib/librte_pmd_ixgbe/ixgbe_pf.c                   | 114 ++-
 lib/librte_pmd_ixgbe/ixgbe_rxtx.c                 | 276 +++----
 lib/librte_pmd_ixgbe/ixgbe_rxtx_vec.c             |   6 +-
 lib/librte_pmd_mlx4/mlx4.c                        |   2 +-
 lib/librte_pmd_null/rte_eth_null.c                |  36 +-
 lib/librte_pmd_pcap/rte_eth_pcap.c                |   2 +-
 lib/librte_pmd_ring/rte_eth_ring.c                |  36 +-
 lib/librte_pmd_virtio/virtio_ethdev.c             | 120 +--
 lib/librte_pmd_virtio/virtio_rxtx.c               |  20 +-
 lib/librte_pmd_vmxnet3/vmxnet3_ethdev.c           |  64 +-
 lib/librte_pmd_vmxnet3/vmxnet3_rxtx.c             |  40 +-
 lib/librte_pmd_xenvirt/rte_eth_xenvirt.c          |   2 +-
 51 files changed, 2990 insertions(+), 2483 deletions(-)
 create mode 100644 lib/librte_eal/common/eal_common_device.c
 create mode 100644 lib/librte_eal/common/include/rte_common_device.h

-- 
2.3.0

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

end of thread, other threads:[~2015-05-04 14:44 UTC | newest]

Thread overview: 28+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-13 19:44 [dpdk-dev] [RFC PATCH 0/4 v2] Extending DPDK with multiple device support Keith Wiles
2015-04-13 19:44 ` [dpdk-dev] [RFC PATCH 1/4 v2] Adding the common device files for " Keith Wiles
2015-05-04 13:13   ` Marc Sune
2015-05-04 14:44     ` Wiles, Keith
2015-04-13 19:44 ` [dpdk-dev] [RFC PATCH 2/4 v2] Add the ethdev changes " Keith Wiles
2015-04-13 19:44 ` [dpdk-dev] [RFC PATCH 3/4 v2] Add the test file changes for common " Keith Wiles
2015-04-13 19:44 ` [dpdk-dev] [RFC PATCH 4/4 v2] Update PMD files for new " Keith Wiles
2015-04-17 15:16 ` [dpdk-dev] [RFC PATCH 0/4] pktdev Bruce Richardson
2015-04-17 15:16   ` [dpdk-dev] [RFC PATCH 1/4] Add example pktdev implementation Bruce Richardson
2015-04-20 11:26     ` Ananyev, Konstantin
2015-04-20 15:02       ` Bruce Richardson
2015-04-21  8:40         ` Ananyev, Konstantin
2015-04-21  9:23           ` Bruce Richardson
2015-04-17 15:16   ` [dpdk-dev] [RFC PATCH 2/4] Make ethdev explicitly a subclass of pktdev Bruce Richardson
2015-04-17 15:16   ` [dpdk-dev] [RFC PATCH 3/4] add support for a ring to be a pktdev Bruce Richardson
2015-04-17 17:31     ` Neil Horman
2015-04-18  0:00     ` Ouyang, Changchun
2015-04-20 10:32     ` Ananyev, Konstantin
2015-04-17 15:16   ` [dpdk-dev] [RFC PATCH 4/4] example app showing pktdevs used in a chain Bruce Richardson
2015-04-17 17:28   ` [dpdk-dev] [RFC PATCH 0/4] pktdev Neil Horman
2015-04-17 18:49   ` Marc Sune
2015-04-17 19:50     ` Wiles, Keith
2015-04-20  6:51       ` Marc Sune
2015-04-20 10:43         ` Bruce Richardson
2015-04-20 17:03           ` Marc Sune
2015-04-20 13:19         ` Wiles, Keith
2015-04-20 13:30           ` Wiles, Keith
2015-05-04 13:13 ` [dpdk-dev] [RFC PATCH 0/4 v2] Extending DPDK with multiple device support Marc Sune

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