ring: move to drivers/net/
authorBruce Richardson <bruce.richardson@intel.com>
Fri, 15 May 2015 15:56:57 +0000 (16:56 +0100)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Fri, 22 May 2015 14:06:22 +0000 (16:06 +0200)
Move ring PMD to drivers directory

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>
14 files changed:
MAINTAINERS
doc/guides/prog_guide/source_org.rst
drivers/net/Makefile
drivers/net/ring/Makefile [new file with mode: 0644]
drivers/net/ring/rte_eth_ring.c [new file with mode: 0644]
drivers/net/ring/rte_eth_ring.h [new file with mode: 0644]
drivers/net/ring/rte_eth_ring_version.map [new file with mode: 0644]
lib/Makefile
lib/librte_eal/bsdapp/eal/Makefile
lib/librte_eal/linuxapp/eal/Makefile
lib/librte_pmd_ring/Makefile [deleted file]
lib/librte_pmd_ring/rte_eth_ring.c [deleted file]
lib/librte_pmd_ring/rte_eth_ring.h [deleted file]
lib/librte_pmd_ring/rte_eth_ring_version.map [deleted file]

index 64ccca5..c2557bd 100644 (file)
@@ -261,7 +261,7 @@ F: doc/guides/nics/pcap_ring.rst
 
 Ring PMD
 M: Bruce Richardson <bruce.richardson@intel.com>
-F: lib/librte_pmd_ring/
+F: drivers/net/ring/
 F: doc/guides/nics/pcap_ring.rst
 F: app/test/test_pmd_ring.c
 
index f463234..6e51bf7 100644 (file)
@@ -79,7 +79,6 @@ The lib directory contains::
     +-- librte_mempool      # memory pool manager (fixedsized objects)
     +-- librte_meter        # QoS metering library
     +-- librte_net          # various IP-related headers
-    +-- librte_pmd_ring     # ring poll mode driver
     +-- librte_pmd_virtio   # virtio poll mode driver
     +-- librte_pmd_vmxnet3  # VMXNET3 poll mode driver
     +-- librte_pmd_xenvirt  # Xen virtio poll mode driver
@@ -109,6 +108,7 @@ The drivers directory has a net subdirectory which contains::
     +-- mlx4               # Mellanox ConnectX-3 poll mode driver
     +-- null               # NULL poll mode driver for testing
     +-- pcap               # PCAP poll mode driver
+    +-- ring               # ring poll mode driver
 
 Applications
 ------------
index 29c9361..f641f0a 100644 (file)
@@ -41,6 +41,7 @@ DIRS-$(CONFIG_RTE_LIBRTE_IXGBE_PMD) += ixgbe
 DIRS-$(CONFIG_RTE_LIBRTE_MLX4_PMD) += mlx4
 DIRS-$(CONFIG_RTE_LIBRTE_PMD_NULL) += null
 DIRS-$(CONFIG_RTE_LIBRTE_PMD_PCAP) += pcap
+DIRS-$(CONFIG_RTE_LIBRTE_PMD_RING) += ring
 
 include $(RTE_SDK)/mk/rte.sharelib.mk
 include $(RTE_SDK)/mk/rte.subdir.mk
diff --git a/drivers/net/ring/Makefile b/drivers/net/ring/Makefile
new file mode 100644 (file)
index 0000000..e442d0b
--- /dev/null
@@ -0,0 +1,61 @@
+#   BSD LICENSE
+#
+#   Copyright(c) 2010-2014 Intel Corporation. All rights reserved.
+#   All rights reserved.
+#
+#   Redistribution and use in source and binary forms, with or without
+#   modification, are permitted provided that the following conditions
+#   are met:
+#
+#     * Redistributions of source code must retain the above copyright
+#       notice, this list of conditions and the following disclaimer.
+#     * Redistributions in binary form must reproduce the above copyright
+#       notice, this list of conditions and the following disclaimer in
+#       the documentation and/or other materials provided with the
+#       distribution.
+#     * Neither the name of Intel Corporation nor the names of its
+#       contributors may be used to endorse or promote products derived
+#       from this software without specific prior written permission.
+#
+#   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+#   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+#   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+#   A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+#   OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+#   SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+#   LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+#   DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+#   THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+#   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+#   OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+include $(RTE_SDK)/mk/rte.vars.mk
+
+#
+# library name
+#
+LIB = librte_pmd_ring.a
+
+CFLAGS += -O3
+CFLAGS += $(WERROR_FLAGS)
+
+EXPORT_MAP := rte_eth_ring_version.map
+
+LIBABIVER := 1
+
+#
+# all source are stored in SRCS-y
+#
+SRCS-$(CONFIG_RTE_LIBRTE_PMD_RING) += rte_eth_ring.c
+
+#
+# Export include files
+#
+SYMLINK-y-include += rte_eth_ring.h
+
+# this lib depends upon:
+DEPDIRS-$(CONFIG_RTE_LIBRTE_PMD_RING) += lib/librte_eal lib/librte_ring
+DEPDIRS-$(CONFIG_RTE_LIBRTE_PMD_RING) += lib/librte_mbuf lib/librte_ether
+DEPDIRS-$(CONFIG_RTE_LIBRTE_PMD_RING) += lib/librte_kvargs
+
+include $(RTE_SDK)/mk/rte.lib.mk
diff --git a/drivers/net/ring/rte_eth_ring.c b/drivers/net/ring/rte_eth_ring.c
new file mode 100644 (file)
index 0000000..6832f01
--- /dev/null
@@ -0,0 +1,579 @@
+/*-
+ *   BSD LICENSE
+ *
+ *   Copyright(c) 2010-2014 Intel Corporation. All rights reserved.
+ *   All rights reserved.
+ *
+ *   Redistribution and use in source and binary forms, with or without
+ *   modification, are permitted provided that the following conditions
+ *   are met:
+ *
+ *     * Redistributions of source code must retain the above copyright
+ *       notice, this list of conditions and the following disclaimer.
+ *     * Redistributions in binary form must reproduce the above copyright
+ *       notice, this list of conditions and the following disclaimer in
+ *       the documentation and/or other materials provided with the
+ *       distribution.
+ *     * Neither the name of Intel Corporation nor the names of its
+ *       contributors may be used to endorse or promote products derived
+ *       from this software without specific prior written permission.
+ *
+ *   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ *   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ *   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ *   A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ *   OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ *   SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ *   LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ *   DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ *   THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ *   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ *   OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include "rte_eth_ring.h"
+#include <rte_mbuf.h>
+#include <rte_ethdev.h>
+#include <rte_malloc.h>
+#include <rte_memcpy.h>
+#include <rte_string_fns.h>
+#include <rte_dev.h>
+#include <rte_kvargs.h>
+
+#define ETH_RING_NUMA_NODE_ACTION_ARG  "nodeaction"
+#define ETH_RING_ACTION_CREATE         "CREATE"
+#define ETH_RING_ACTION_ATTACH         "ATTACH"
+
+static const char *ring_ethdev_driver_name = "Ring PMD";
+
+static const char *valid_arguments[] = {
+       ETH_RING_NUMA_NODE_ACTION_ARG,
+       NULL
+};
+
+struct ring_queue {
+       struct rte_ring *rng;
+       rte_atomic64_t rx_pkts;
+       rte_atomic64_t tx_pkts;
+       rte_atomic64_t err_pkts;
+};
+
+struct pmd_internals {
+       unsigned nb_rx_queues;
+       unsigned nb_tx_queues;
+
+       struct ring_queue rx_ring_queues[RTE_PMD_RING_MAX_RX_RINGS];
+       struct ring_queue tx_ring_queues[RTE_PMD_RING_MAX_TX_RINGS];
+
+       struct ether_addr address;
+};
+
+
+static const char *drivername = "Rings PMD";
+static struct rte_eth_link pmd_link = {
+               .link_speed = 10000,
+               .link_duplex = ETH_LINK_FULL_DUPLEX,
+               .link_status = 0
+};
+
+static uint16_t
+eth_ring_rx(void *q, struct rte_mbuf **bufs, uint16_t nb_bufs)
+{
+       void **ptrs = (void *)&bufs[0];
+       struct ring_queue *r = q;
+       const uint16_t nb_rx = (uint16_t)rte_ring_dequeue_burst(r->rng,
+                       ptrs, nb_bufs);
+       if (r->rng->flags & RING_F_SC_DEQ)
+               r->rx_pkts.cnt += nb_rx;
+       else
+               rte_atomic64_add(&(r->rx_pkts), nb_rx);
+       return nb_rx;
+}
+
+static uint16_t
+eth_ring_tx(void *q, struct rte_mbuf **bufs, uint16_t nb_bufs)
+{
+       void **ptrs = (void *)&bufs[0];
+       struct ring_queue *r = q;
+       const uint16_t nb_tx = (uint16_t)rte_ring_enqueue_burst(r->rng,
+                       ptrs, nb_bufs);
+       if (r->rng->flags & RING_F_SP_ENQ) {
+               r->tx_pkts.cnt += nb_tx;
+               r->err_pkts.cnt += nb_bufs - nb_tx;
+       } else {
+               rte_atomic64_add(&(r->tx_pkts), nb_tx);
+               rte_atomic64_add(&(r->err_pkts), nb_bufs - nb_tx);
+       }
+       return nb_tx;
+}
+
+static int
+eth_dev_configure(struct rte_eth_dev *dev __rte_unused) { return 0; }
+
+static int
+eth_dev_start(struct rte_eth_dev *dev)
+{
+       dev->data->dev_link.link_status = 1;
+       return 0;
+}
+
+static void
+eth_dev_stop(struct rte_eth_dev *dev)
+{
+       dev->data->dev_link.link_status = 0;
+}
+
+static int
+eth_dev_set_link_down(struct rte_eth_dev *dev)
+{
+       dev->data->dev_link.link_status = 0;
+       return 0;
+}
+
+static int
+eth_dev_set_link_up(struct rte_eth_dev *dev)
+{
+       dev->data->dev_link.link_status = 1;
+       return 0;
+}
+
+static int
+eth_rx_queue_setup(struct rte_eth_dev *dev,uint16_t rx_queue_id,
+                                   uint16_t nb_rx_desc __rte_unused,
+                                   unsigned int socket_id __rte_unused,
+                                   const struct rte_eth_rxconf *rx_conf __rte_unused,
+                                   struct rte_mempool *mb_pool __rte_unused)
+{
+       struct pmd_internals *internals = dev->data->dev_private;
+       dev->data->rx_queues[rx_queue_id] = &internals->rx_ring_queues[rx_queue_id];
+       return 0;
+}
+
+static int
+eth_tx_queue_setup(struct rte_eth_dev *dev, uint16_t tx_queue_id,
+                                   uint16_t nb_tx_desc __rte_unused,
+                                   unsigned int socket_id __rte_unused,
+                                   const struct rte_eth_txconf *tx_conf __rte_unused)
+{
+       struct pmd_internals *internals = dev->data->dev_private;
+       dev->data->tx_queues[tx_queue_id] = &internals->tx_ring_queues[tx_queue_id];
+       return 0;
+}
+
+
+static void
+eth_dev_info(struct rte_eth_dev *dev,
+               struct rte_eth_dev_info *dev_info)
+{
+       struct pmd_internals *internals = dev->data->dev_private;
+       dev_info->driver_name = drivername;
+       dev_info->max_mac_addrs = 1;
+       dev_info->max_rx_pktlen = (uint32_t)-1;
+       dev_info->max_rx_queues = (uint16_t)internals->nb_rx_queues;
+       dev_info->max_tx_queues = (uint16_t)internals->nb_tx_queues;
+       dev_info->min_rx_bufsize = 0;
+       dev_info->pci_dev = NULL;
+}
+
+static void
+eth_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *igb_stats)
+{
+       unsigned i;
+       unsigned long rx_total = 0, tx_total = 0, tx_err_total = 0;
+       const struct pmd_internals *internal = dev->data->dev_private;
+
+       for (i = 0; i < RTE_ETHDEV_QUEUE_STAT_CNTRS &&
+                       i < internal->nb_rx_queues; i++) {
+               igb_stats->q_ipackets[i] = internal->rx_ring_queues[i].rx_pkts.cnt;
+               rx_total += igb_stats->q_ipackets[i];
+       }
+
+       for (i = 0; i < RTE_ETHDEV_QUEUE_STAT_CNTRS &&
+                       i < internal->nb_tx_queues; i++) {
+               igb_stats->q_opackets[i] = internal->tx_ring_queues[i].tx_pkts.cnt;
+               igb_stats->q_errors[i] = internal->tx_ring_queues[i].err_pkts.cnt;
+               tx_total += igb_stats->q_opackets[i];
+               tx_err_total += igb_stats->q_errors[i];
+       }
+
+       igb_stats->ipackets = rx_total;
+       igb_stats->opackets = tx_total;
+       igb_stats->oerrors = tx_err_total;
+}
+
+static void
+eth_stats_reset(struct rte_eth_dev *dev)
+{
+       unsigned i;
+       struct pmd_internals *internal = dev->data->dev_private;
+       for (i = 0; i < internal->nb_rx_queues; i++)
+               internal->rx_ring_queues[i].rx_pkts.cnt = 0;
+       for (i = 0; i < internal->nb_tx_queues; i++) {
+               internal->tx_ring_queues[i].tx_pkts.cnt = 0;
+               internal->tx_ring_queues[i].err_pkts.cnt = 0;
+       }
+}
+
+static void
+eth_mac_addr_remove(struct rte_eth_dev *dev __rte_unused,
+       uint32_t index __rte_unused)
+{
+}
+
+static void
+eth_mac_addr_add(struct rte_eth_dev *dev __rte_unused,
+       struct ether_addr *mac_addr __rte_unused,
+       uint32_t index __rte_unused,
+       uint32_t vmdq __rte_unused)
+{
+}
+
+static void
+eth_queue_release(void *q __rte_unused) { ; }
+static int
+eth_link_update(struct rte_eth_dev *dev __rte_unused,
+               int wait_to_complete __rte_unused) { return 0; }
+
+static const struct eth_dev_ops ops = {
+       .dev_start = eth_dev_start,
+       .dev_stop = eth_dev_stop,
+       .dev_set_link_up = eth_dev_set_link_up,
+       .dev_set_link_down = eth_dev_set_link_down,
+       .dev_configure = eth_dev_configure,
+       .dev_infos_get = eth_dev_info,
+       .rx_queue_setup = eth_rx_queue_setup,
+       .tx_queue_setup = eth_tx_queue_setup,
+       .rx_queue_release = eth_queue_release,
+       .tx_queue_release = eth_queue_release,
+       .link_update = eth_link_update,
+       .stats_get = eth_stats_get,
+       .stats_reset = eth_stats_reset,
+       .mac_addr_remove = eth_mac_addr_remove,
+       .mac_addr_add = eth_mac_addr_add,
+};
+
+int
+rte_eth_from_rings(const char *name, struct rte_ring *const rx_queues[],
+               const unsigned nb_rx_queues,
+               struct rte_ring *const tx_queues[],
+               const unsigned nb_tx_queues,
+               const unsigned numa_node)
+{
+       struct rte_eth_dev_data *data = NULL;
+       struct rte_pci_device *pci_dev = NULL;
+       struct pmd_internals *internals = NULL;
+       struct rte_eth_dev *eth_dev = NULL;
+       struct eth_driver *eth_drv = NULL;
+       struct rte_pci_id *id_table = NULL;
+
+       unsigned i;
+
+       /* do some parameter checking */
+       if (rx_queues == NULL && nb_rx_queues > 0)
+               goto error;
+       if (tx_queues == NULL && nb_tx_queues > 0)
+               goto error;
+
+       RTE_LOG(INFO, PMD, "Creating rings-backed ethdev on numa socket %u\n",
+                       numa_node);
+
+       /* now do all data allocation - for eth_dev structure, dummy pci driver
+        * and internal (private) data
+        */
+       data = rte_zmalloc_socket(name, sizeof(*data), 0, numa_node);
+       if (data == NULL)
+               goto error;
+
+       pci_dev = rte_zmalloc_socket(name, sizeof(*pci_dev), 0, numa_node);
+       if (pci_dev == NULL)
+               goto error;
+
+       id_table = rte_zmalloc_socket(name, sizeof(*id_table), 0, numa_node);
+       if (id_table == NULL)
+               goto error;
+
+       internals = rte_zmalloc_socket(name, sizeof(*internals), 0, numa_node);
+       if (internals == NULL)
+               goto error;
+
+       /* reserve an ethdev entry */
+       eth_dev = rte_eth_dev_allocate(name, RTE_ETH_DEV_VIRTUAL);
+       if (eth_dev == NULL)
+               goto error;
+
+       eth_drv = rte_zmalloc_socket(name, sizeof(*eth_drv), 0, numa_node);
+       if (eth_drv == NULL)
+               goto error;
+
+       /* now put it all together
+        * - store queue data in internals,
+        * - store numa_node info in pci_driver
+        * - point eth_dev_data to internals and pci_driver
+        * - and point eth_dev structure to new eth_dev_data structure
+        */
+       /* NOTE: we'll replace the data element, of originally allocated eth_dev
+        * so the rings are local per-process */
+
+       internals->nb_rx_queues = nb_rx_queues;
+       internals->nb_tx_queues = nb_tx_queues;
+       for (i = 0; i < nb_rx_queues; i++) {
+               internals->rx_ring_queues[i].rng = rx_queues[i];
+       }
+       for (i = 0; i < nb_tx_queues; i++) {
+               internals->tx_ring_queues[i].rng = tx_queues[i];
+       }
+
+       eth_drv->pci_drv.name = ring_ethdev_driver_name;
+       eth_drv->pci_drv.id_table = id_table;
+
+       pci_dev->numa_node = numa_node;
+       pci_dev->driver = &eth_drv->pci_drv;
+
+       data->dev_private = internals;
+       data->port_id = eth_dev->data->port_id;
+       data->nb_rx_queues = (uint16_t)nb_rx_queues;
+       data->nb_tx_queues = (uint16_t)nb_tx_queues;
+       data->dev_link = pmd_link;
+       data->mac_addrs = &internals->address;
+
+       eth_dev->data = data;
+       eth_dev->driver = eth_drv;
+       eth_dev->dev_ops = &ops;
+       eth_dev->pci_dev = pci_dev;
+       TAILQ_INIT(&(eth_dev->link_intr_cbs));
+
+       /* finally assign rx and tx ops */
+       eth_dev->rx_pkt_burst = eth_ring_rx;
+       eth_dev->tx_pkt_burst = eth_ring_tx;
+
+       return 0;
+
+error:
+       rte_free(data);
+       rte_free(pci_dev);
+       rte_free(internals);
+
+       return -1;
+}
+
+enum dev_action{
+       DEV_CREATE,
+       DEV_ATTACH
+};
+
+static int
+eth_dev_ring_create(const char *name, const unsigned numa_node,
+               enum dev_action action)
+{
+       /* rx and tx are so-called from point of view of first port.
+        * They are inverted from the point of view of second port
+        */
+       struct rte_ring *rxtx[RTE_PMD_RING_MAX_RX_RINGS];
+       unsigned i;
+       char rng_name[RTE_RING_NAMESIZE];
+       unsigned num_rings = RTE_MIN(RTE_PMD_RING_MAX_RX_RINGS,
+                       RTE_PMD_RING_MAX_TX_RINGS);
+
+       for (i = 0; i < num_rings; i++) {
+               snprintf(rng_name, sizeof(rng_name), "ETH_RXTX%u_%s", i, name);
+               rxtx[i] = (action == DEV_CREATE) ?
+                               rte_ring_create(rng_name, 1024, numa_node,
+                                               RING_F_SP_ENQ|RING_F_SC_DEQ) :
+                               rte_ring_lookup(rng_name);
+               if (rxtx[i] == NULL)
+                       return -1;
+       }
+
+       if (rte_eth_from_rings(name, rxtx, num_rings, rxtx, num_rings, numa_node))
+               return -1;
+
+       return 0;
+}
+
+
+static int
+eth_dev_ring_pair_create(const char *name, const unsigned numa_node,
+               enum dev_action action)
+{
+       /* rx and tx are so-called from point of view of first port.
+        * They are inverted from the point of view of second port
+        */
+       struct rte_ring *rx[RTE_PMD_RING_MAX_RX_RINGS];
+       struct rte_ring *tx[RTE_PMD_RING_MAX_TX_RINGS];
+       unsigned i;
+       char rx_rng_name[RTE_RING_NAMESIZE];
+       char tx_rng_name[RTE_RING_NAMESIZE];
+       unsigned num_rings = RTE_MIN(RTE_PMD_RING_MAX_RX_RINGS,
+                       RTE_PMD_RING_MAX_TX_RINGS);
+
+       for (i = 0; i < num_rings; i++) {
+               snprintf(rx_rng_name, sizeof(rx_rng_name), "ETH_RX%u_%s", i, name);
+               rx[i] = (action == DEV_CREATE) ?
+                               rte_ring_create(rx_rng_name, 1024, numa_node,
+                                               RING_F_SP_ENQ|RING_F_SC_DEQ) :
+                               rte_ring_lookup(rx_rng_name);
+               if (rx[i] == NULL)
+                       return -1;
+               snprintf(tx_rng_name, sizeof(tx_rng_name), "ETH_TX%u_%s", i, name);
+               tx[i] = (action == DEV_CREATE) ?
+                               rte_ring_create(tx_rng_name, 1024, numa_node,
+                                               RING_F_SP_ENQ|RING_F_SC_DEQ):
+                               rte_ring_lookup(tx_rng_name);
+               if (tx[i] == NULL)
+                       return -1;
+       }
+
+       if (rte_eth_from_rings(rx_rng_name, rx, num_rings, tx, num_rings,
+                       numa_node) || rte_eth_from_rings(tx_rng_name, tx, num_rings, rx,
+                                       num_rings, numa_node))
+               return -1;
+
+       return 0;
+}
+
+int
+rte_eth_ring_pair_create(const char *name, const unsigned numa_node)
+{
+       RTE_LOG(WARNING, PMD, "rte_eth_ring_pair_create is deprecated\n");
+       return eth_dev_ring_pair_create(name, numa_node, DEV_CREATE);
+}
+
+int
+rte_eth_ring_pair_attach(const char *name, const unsigned numa_node)
+{
+       RTE_LOG(WARNING, PMD, "rte_eth_ring_pair_attach is deprecated\n");
+       return eth_dev_ring_pair_create(name, numa_node, DEV_ATTACH);
+}
+
+struct node_action_pair {
+       char name[PATH_MAX];
+       unsigned node;
+       enum dev_action action;
+};
+
+struct node_action_list {
+       unsigned total;
+       unsigned count;
+       struct node_action_pair *list;
+};
+
+static int parse_kvlist (const char *key __rte_unused, const char *value, void *data)
+{
+       struct node_action_list *info = data;
+       int ret;
+       char *name;
+       char *action;
+       char *node;
+       char *end;
+
+       name = strdup(value);
+
+       ret = -EINVAL;
+
+       if (!name) {
+               RTE_LOG(WARNING, PMD, "command line paramter is empty for ring pmd!\n");
+               goto out;
+       }
+
+       node = strchr(name, ':');
+       if (!node) {
+               RTE_LOG(WARNING, PMD, "could not parse node value from %s", name);
+               goto out;
+       }
+
+       *node = '\0';
+       node++;
+
+       action = strchr(node, ':');
+       if (!action) {
+               RTE_LOG(WARNING, PMD, "could not action value from %s", node);
+               goto out;
+       }
+
+       *action = '\0';
+       action++;
+
+       /*
+        * Need to do some sanity checking here
+        */
+
+       if (strcmp(action, ETH_RING_ACTION_ATTACH) == 0)
+               info->list[info->count].action = DEV_ATTACH;
+       else if (strcmp(action, ETH_RING_ACTION_CREATE) == 0)
+               info->list[info->count].action = DEV_CREATE;
+       else
+               goto out;
+
+       errno = 0;
+       info->list[info->count].node = strtol(node, &end, 10);
+
+       if ((errno != 0) || (*end != '\0')) {
+               RTE_LOG(WARNING, PMD, "node value %s is unparseable as a number\n", node);
+               goto out;
+       }
+
+       snprintf(info->list[info->count].name, sizeof(info->list[info->count].name), "%s", name);
+
+       info->count++;
+
+       ret = 0;
+out:
+       free(name);
+       return ret;
+}
+
+static int
+rte_pmd_ring_devinit(const char *name, const char *params)
+{
+       struct rte_kvargs *kvlist = NULL;
+       int ret = 0;
+       struct node_action_list *info = NULL;
+
+       RTE_LOG(INFO, PMD, "Initializing pmd_ring for %s\n", name);
+
+       if (params == NULL || params[0] == '\0')
+               eth_dev_ring_create(name, rte_socket_id(), DEV_CREATE);
+       else {
+               kvlist = rte_kvargs_parse(params, valid_arguments);
+
+               if (!kvlist) {
+                       RTE_LOG(INFO, PMD, "Ignoring unsupported parameters when creating"
+                                       " rings-backed ethernet device\n");
+                       eth_dev_ring_create(name, rte_socket_id(), DEV_CREATE);
+                       return 0;
+               } else {
+                       ret = rte_kvargs_count(kvlist, ETH_RING_NUMA_NODE_ACTION_ARG);
+                       info = rte_zmalloc("struct node_action_list", sizeof(struct node_action_list) +
+                                          (sizeof(struct node_action_pair) * ret), 0);
+                       if (!info)
+                               goto out_free;
+
+                       info->total = ret;
+                       info->list = (struct node_action_pair*)(info + 1);
+
+                       ret = rte_kvargs_process(kvlist, ETH_RING_NUMA_NODE_ACTION_ARG,
+                                                parse_kvlist, info);
+
+                       if (ret < 0)
+                               goto out_free;
+
+                       for (info->count = 0; info->count < info->total; info->count++) {
+                               eth_dev_ring_create(name, info->list[info->count].node,
+                                                   info->list[info->count].action);
+                       }
+               }
+       }
+
+out_free:
+       rte_kvargs_free(kvlist);
+       rte_free(info);
+       return ret;
+}
+
+static struct rte_driver pmd_ring_drv = {
+       .name = "eth_ring",
+       .type = PMD_VDEV,
+       .init = rte_pmd_ring_devinit,
+};
+
+PMD_REGISTER_DRIVER(pmd_ring_drv);
diff --git a/drivers/net/ring/rte_eth_ring.h b/drivers/net/ring/rte_eth_ring.h
new file mode 100644 (file)
index 0000000..d36489a
--- /dev/null
@@ -0,0 +1,57 @@
+/*-
+ *   BSD LICENSE
+ *
+ *   Copyright(c) 2010-2014 Intel Corporation. All rights reserved.
+ *   All rights reserved.
+ *
+ *   Redistribution and use in source and binary forms, with or without
+ *   modification, are permitted provided that the following conditions
+ *   are met:
+ *
+ *     * Redistributions of source code must retain the above copyright
+ *       notice, this list of conditions and the following disclaimer.
+ *     * Redistributions in binary form must reproduce the above copyright
+ *       notice, this list of conditions and the following disclaimer in
+ *       the documentation and/or other materials provided with the
+ *       distribution.
+ *     * Neither the name of Intel Corporation nor the names of its
+ *       contributors may be used to endorse or promote products derived
+ *       from this software without specific prior written permission.
+ *
+ *   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ *   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ *   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ *   A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ *   OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ *   SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ *   LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ *   DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ *   THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ *   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ *   OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#ifndef _RTE_ETH_RING_H_
+#define _RTE_ETH_RING_H_
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+#include <rte_ring.h>
+
+int rte_eth_from_rings(const char *name,
+               struct rte_ring * const rx_queues[],
+               const unsigned nb_rx_queues,
+               struct rte_ring *const tx_queues[],
+               const unsigned nb_tx_queues,
+               const unsigned numa_node);
+
+int rte_eth_ring_pair_create(const char *name, const unsigned numa_node);
+int rte_eth_ring_pair_attach(const char *name, const unsigned numa_node);
+
+#ifdef __cplusplus
+}
+#endif
+
+#endif
diff --git a/drivers/net/ring/rte_eth_ring_version.map b/drivers/net/ring/rte_eth_ring_version.map
new file mode 100644 (file)
index 0000000..8ad107d
--- /dev/null
@@ -0,0 +1,9 @@
+DPDK_2.0 {
+       global:
+
+       rte_eth_from_rings;
+       rte_eth_ring_pair_attach;
+       rte_eth_ring_pair_create;
+
+       local: *;
+};
index e63f1fe..0bb87ba 100644 (file)
@@ -41,7 +41,6 @@ DIRS-$(CONFIG_RTE_LIBRTE_TIMER) += librte_timer
 DIRS-$(CONFIG_RTE_LIBRTE_CFGFILE) += librte_cfgfile
 DIRS-$(CONFIG_RTE_LIBRTE_CMDLINE) += librte_cmdline
 DIRS-$(CONFIG_RTE_LIBRTE_ETHER) += librte_ether
-DIRS-$(CONFIG_RTE_LIBRTE_PMD_RING) += librte_pmd_ring
 DIRS-$(CONFIG_RTE_LIBRTE_VIRTIO_PMD) += librte_pmd_virtio
 DIRS-$(CONFIG_RTE_LIBRTE_VMXNET3_PMD) += librte_pmd_vmxnet3
 DIRS-$(CONFIG_RTE_LIBRTE_PMD_XENVIRT) += librte_pmd_xenvirt
index c2fe933..3d1d9eb 100644 (file)
@@ -42,7 +42,7 @@ CFLAGS += -I$(RTE_SDK)/lib/librte_ring
 CFLAGS += -I$(RTE_SDK)/lib/librte_mempool
 CFLAGS += -I$(RTE_SDK)/lib/librte_malloc
 CFLAGS += -I$(RTE_SDK)/lib/librte_ether
-CFLAGS += -I$(RTE_SDK)/lib/librte_pmd_ring
+CFLAGS += -I$(RTE_SDK)/drivers/net/ring
 CFLAGS += -I$(RTE_SDK)/drivers/net/pcap
 CFLAGS += $(WERROR_FLAGS) -O3
 
index fdb3656..bc1e90d 100644 (file)
@@ -47,7 +47,7 @@ CFLAGS += -I$(RTE_SDK)/lib/librte_mempool
 CFLAGS += -I$(RTE_SDK)/lib/librte_malloc
 CFLAGS += -I$(RTE_SDK)/lib/librte_ether
 CFLAGS += -I$(RTE_SDK)/lib/librte_ivshmem
-CFLAGS += -I$(RTE_SDK)/lib/librte_pmd_ring
+CFLAGS += -I$(RTE_SDK)/drivers/net/ring
 CFLAGS += -I$(RTE_SDK)/drivers/net/pcap
 CFLAGS += -I$(RTE_SDK)/drivers/net/af_packet
 CFLAGS += -I$(RTE_SDK)/lib/librte_pmd_xenvirt
diff --git a/lib/librte_pmd_ring/Makefile b/lib/librte_pmd_ring/Makefile
deleted file mode 100644 (file)
index e442d0b..0000000
+++ /dev/null
@@ -1,61 +0,0 @@
-#   BSD LICENSE
-#
-#   Copyright(c) 2010-2014 Intel Corporation. All rights reserved.
-#   All rights reserved.
-#
-#   Redistribution and use in source and binary forms, with or without
-#   modification, are permitted provided that the following conditions
-#   are met:
-#
-#     * Redistributions of source code must retain the above copyright
-#       notice, this list of conditions and the following disclaimer.
-#     * Redistributions in binary form must reproduce the above copyright
-#       notice, this list of conditions and the following disclaimer in
-#       the documentation and/or other materials provided with the
-#       distribution.
-#     * Neither the name of Intel Corporation nor the names of its
-#       contributors may be used to endorse or promote products derived
-#       from this software without specific prior written permission.
-#
-#   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
-#   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
-#   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
-#   A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
-#   OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
-#   SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
-#   LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
-#   DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
-#   THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
-#   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
-#   OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-
-include $(RTE_SDK)/mk/rte.vars.mk
-
-#
-# library name
-#
-LIB = librte_pmd_ring.a
-
-CFLAGS += -O3
-CFLAGS += $(WERROR_FLAGS)
-
-EXPORT_MAP := rte_eth_ring_version.map
-
-LIBABIVER := 1
-
-#
-# all source are stored in SRCS-y
-#
-SRCS-$(CONFIG_RTE_LIBRTE_PMD_RING) += rte_eth_ring.c
-
-#
-# Export include files
-#
-SYMLINK-y-include += rte_eth_ring.h
-
-# this lib depends upon:
-DEPDIRS-$(CONFIG_RTE_LIBRTE_PMD_RING) += lib/librte_eal lib/librte_ring
-DEPDIRS-$(CONFIG_RTE_LIBRTE_PMD_RING) += lib/librte_mbuf lib/librte_ether
-DEPDIRS-$(CONFIG_RTE_LIBRTE_PMD_RING) += lib/librte_kvargs
-
-include $(RTE_SDK)/mk/rte.lib.mk
diff --git a/lib/librte_pmd_ring/rte_eth_ring.c b/lib/librte_pmd_ring/rte_eth_ring.c
deleted file mode 100644 (file)
index 6832f01..0000000
+++ /dev/null
@@ -1,579 +0,0 @@
-/*-
- *   BSD LICENSE
- *
- *   Copyright(c) 2010-2014 Intel Corporation. All rights reserved.
- *   All rights reserved.
- *
- *   Redistribution and use in source and binary forms, with or without
- *   modification, are permitted provided that the following conditions
- *   are met:
- *
- *     * Redistributions of source code must retain the above copyright
- *       notice, this list of conditions and the following disclaimer.
- *     * Redistributions in binary form must reproduce the above copyright
- *       notice, this list of conditions and the following disclaimer in
- *       the documentation and/or other materials provided with the
- *       distribution.
- *     * Neither the name of Intel Corporation nor the names of its
- *       contributors may be used to endorse or promote products derived
- *       from this software without specific prior written permission.
- *
- *   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- *   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- *   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- *   A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- *   OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- *   SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- *   LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- *   DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- *   THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- *   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- *   OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#include "rte_eth_ring.h"
-#include <rte_mbuf.h>
-#include <rte_ethdev.h>
-#include <rte_malloc.h>
-#include <rte_memcpy.h>
-#include <rte_string_fns.h>
-#include <rte_dev.h>
-#include <rte_kvargs.h>
-
-#define ETH_RING_NUMA_NODE_ACTION_ARG  "nodeaction"
-#define ETH_RING_ACTION_CREATE         "CREATE"
-#define ETH_RING_ACTION_ATTACH         "ATTACH"
-
-static const char *ring_ethdev_driver_name = "Ring PMD";
-
-static const char *valid_arguments[] = {
-       ETH_RING_NUMA_NODE_ACTION_ARG,
-       NULL
-};
-
-struct ring_queue {
-       struct rte_ring *rng;
-       rte_atomic64_t rx_pkts;
-       rte_atomic64_t tx_pkts;
-       rte_atomic64_t err_pkts;
-};
-
-struct pmd_internals {
-       unsigned nb_rx_queues;
-       unsigned nb_tx_queues;
-
-       struct ring_queue rx_ring_queues[RTE_PMD_RING_MAX_RX_RINGS];
-       struct ring_queue tx_ring_queues[RTE_PMD_RING_MAX_TX_RINGS];
-
-       struct ether_addr address;
-};
-
-
-static const char *drivername = "Rings PMD";
-static struct rte_eth_link pmd_link = {
-               .link_speed = 10000,
-               .link_duplex = ETH_LINK_FULL_DUPLEX,
-               .link_status = 0
-};
-
-static uint16_t
-eth_ring_rx(void *q, struct rte_mbuf **bufs, uint16_t nb_bufs)
-{
-       void **ptrs = (void *)&bufs[0];
-       struct ring_queue *r = q;
-       const uint16_t nb_rx = (uint16_t)rte_ring_dequeue_burst(r->rng,
-                       ptrs, nb_bufs);
-       if (r->rng->flags & RING_F_SC_DEQ)
-               r->rx_pkts.cnt += nb_rx;
-       else
-               rte_atomic64_add(&(r->rx_pkts), nb_rx);
-       return nb_rx;
-}
-
-static uint16_t
-eth_ring_tx(void *q, struct rte_mbuf **bufs, uint16_t nb_bufs)
-{
-       void **ptrs = (void *)&bufs[0];
-       struct ring_queue *r = q;
-       const uint16_t nb_tx = (uint16_t)rte_ring_enqueue_burst(r->rng,
-                       ptrs, nb_bufs);
-       if (r->rng->flags & RING_F_SP_ENQ) {
-               r->tx_pkts.cnt += nb_tx;
-               r->err_pkts.cnt += nb_bufs - nb_tx;
-       } else {
-               rte_atomic64_add(&(r->tx_pkts), nb_tx);
-               rte_atomic64_add(&(r->err_pkts), nb_bufs - nb_tx);
-       }
-       return nb_tx;
-}
-
-static int
-eth_dev_configure(struct rte_eth_dev *dev __rte_unused) { return 0; }
-
-static int
-eth_dev_start(struct rte_eth_dev *dev)
-{
-       dev->data->dev_link.link_status = 1;
-       return 0;
-}
-
-static void
-eth_dev_stop(struct rte_eth_dev *dev)
-{
-       dev->data->dev_link.link_status = 0;
-}
-
-static int
-eth_dev_set_link_down(struct rte_eth_dev *dev)
-{
-       dev->data->dev_link.link_status = 0;
-       return 0;
-}
-
-static int
-eth_dev_set_link_up(struct rte_eth_dev *dev)
-{
-       dev->data->dev_link.link_status = 1;
-       return 0;
-}
-
-static int
-eth_rx_queue_setup(struct rte_eth_dev *dev,uint16_t rx_queue_id,
-                                   uint16_t nb_rx_desc __rte_unused,
-                                   unsigned int socket_id __rte_unused,
-                                   const struct rte_eth_rxconf *rx_conf __rte_unused,
-                                   struct rte_mempool *mb_pool __rte_unused)
-{
-       struct pmd_internals *internals = dev->data->dev_private;
-       dev->data->rx_queues[rx_queue_id] = &internals->rx_ring_queues[rx_queue_id];
-       return 0;
-}
-
-static int
-eth_tx_queue_setup(struct rte_eth_dev *dev, uint16_t tx_queue_id,
-                                   uint16_t nb_tx_desc __rte_unused,
-                                   unsigned int socket_id __rte_unused,
-                                   const struct rte_eth_txconf *tx_conf __rte_unused)
-{
-       struct pmd_internals *internals = dev->data->dev_private;
-       dev->data->tx_queues[tx_queue_id] = &internals->tx_ring_queues[tx_queue_id];
-       return 0;
-}
-
-
-static void
-eth_dev_info(struct rte_eth_dev *dev,
-               struct rte_eth_dev_info *dev_info)
-{
-       struct pmd_internals *internals = dev->data->dev_private;
-       dev_info->driver_name = drivername;
-       dev_info->max_mac_addrs = 1;
-       dev_info->max_rx_pktlen = (uint32_t)-1;
-       dev_info->max_rx_queues = (uint16_t)internals->nb_rx_queues;
-       dev_info->max_tx_queues = (uint16_t)internals->nb_tx_queues;
-       dev_info->min_rx_bufsize = 0;
-       dev_info->pci_dev = NULL;
-}
-
-static void
-eth_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *igb_stats)
-{
-       unsigned i;
-       unsigned long rx_total = 0, tx_total = 0, tx_err_total = 0;
-       const struct pmd_internals *internal = dev->data->dev_private;
-
-       for (i = 0; i < RTE_ETHDEV_QUEUE_STAT_CNTRS &&
-                       i < internal->nb_rx_queues; i++) {
-               igb_stats->q_ipackets[i] = internal->rx_ring_queues[i].rx_pkts.cnt;
-               rx_total += igb_stats->q_ipackets[i];
-       }
-
-       for (i = 0; i < RTE_ETHDEV_QUEUE_STAT_CNTRS &&
-                       i < internal->nb_tx_queues; i++) {
-               igb_stats->q_opackets[i] = internal->tx_ring_queues[i].tx_pkts.cnt;
-               igb_stats->q_errors[i] = internal->tx_ring_queues[i].err_pkts.cnt;
-               tx_total += igb_stats->q_opackets[i];
-               tx_err_total += igb_stats->q_errors[i];
-       }
-
-       igb_stats->ipackets = rx_total;
-       igb_stats->opackets = tx_total;
-       igb_stats->oerrors = tx_err_total;
-}
-
-static void
-eth_stats_reset(struct rte_eth_dev *dev)
-{
-       unsigned i;
-       struct pmd_internals *internal = dev->data->dev_private;
-       for (i = 0; i < internal->nb_rx_queues; i++)
-               internal->rx_ring_queues[i].rx_pkts.cnt = 0;
-       for (i = 0; i < internal->nb_tx_queues; i++) {
-               internal->tx_ring_queues[i].tx_pkts.cnt = 0;
-               internal->tx_ring_queues[i].err_pkts.cnt = 0;
-       }
-}
-
-static void
-eth_mac_addr_remove(struct rte_eth_dev *dev __rte_unused,
-       uint32_t index __rte_unused)
-{
-}
-
-static void
-eth_mac_addr_add(struct rte_eth_dev *dev __rte_unused,
-       struct ether_addr *mac_addr __rte_unused,
-       uint32_t index __rte_unused,
-       uint32_t vmdq __rte_unused)
-{
-}
-
-static void
-eth_queue_release(void *q __rte_unused) { ; }
-static int
-eth_link_update(struct rte_eth_dev *dev __rte_unused,
-               int wait_to_complete __rte_unused) { return 0; }
-
-static const struct eth_dev_ops ops = {
-       .dev_start = eth_dev_start,
-       .dev_stop = eth_dev_stop,
-       .dev_set_link_up = eth_dev_set_link_up,
-       .dev_set_link_down = eth_dev_set_link_down,
-       .dev_configure = eth_dev_configure,
-       .dev_infos_get = eth_dev_info,
-       .rx_queue_setup = eth_rx_queue_setup,
-       .tx_queue_setup = eth_tx_queue_setup,
-       .rx_queue_release = eth_queue_release,
-       .tx_queue_release = eth_queue_release,
-       .link_update = eth_link_update,
-       .stats_get = eth_stats_get,
-       .stats_reset = eth_stats_reset,
-       .mac_addr_remove = eth_mac_addr_remove,
-       .mac_addr_add = eth_mac_addr_add,
-};
-
-int
-rte_eth_from_rings(const char *name, struct rte_ring *const rx_queues[],
-               const unsigned nb_rx_queues,
-               struct rte_ring *const tx_queues[],
-               const unsigned nb_tx_queues,
-               const unsigned numa_node)
-{
-       struct rte_eth_dev_data *data = NULL;
-       struct rte_pci_device *pci_dev = NULL;
-       struct pmd_internals *internals = NULL;
-       struct rte_eth_dev *eth_dev = NULL;
-       struct eth_driver *eth_drv = NULL;
-       struct rte_pci_id *id_table = NULL;
-
-       unsigned i;
-
-       /* do some parameter checking */
-       if (rx_queues == NULL && nb_rx_queues > 0)
-               goto error;
-       if (tx_queues == NULL && nb_tx_queues > 0)
-               goto error;
-
-       RTE_LOG(INFO, PMD, "Creating rings-backed ethdev on numa socket %u\n",
-                       numa_node);
-
-       /* now do all data allocation - for eth_dev structure, dummy pci driver
-        * and internal (private) data
-        */
-       data = rte_zmalloc_socket(name, sizeof(*data), 0, numa_node);
-       if (data == NULL)
-               goto error;
-
-       pci_dev = rte_zmalloc_socket(name, sizeof(*pci_dev), 0, numa_node);
-       if (pci_dev == NULL)
-               goto error;
-
-       id_table = rte_zmalloc_socket(name, sizeof(*id_table), 0, numa_node);
-       if (id_table == NULL)
-               goto error;
-
-       internals = rte_zmalloc_socket(name, sizeof(*internals), 0, numa_node);
-       if (internals == NULL)
-               goto error;
-
-       /* reserve an ethdev entry */
-       eth_dev = rte_eth_dev_allocate(name, RTE_ETH_DEV_VIRTUAL);
-       if (eth_dev == NULL)
-               goto error;
-
-       eth_drv = rte_zmalloc_socket(name, sizeof(*eth_drv), 0, numa_node);
-       if (eth_drv == NULL)
-               goto error;
-
-       /* now put it all together
-        * - store queue data in internals,
-        * - store numa_node info in pci_driver
-        * - point eth_dev_data to internals and pci_driver
-        * - and point eth_dev structure to new eth_dev_data structure
-        */
-       /* NOTE: we'll replace the data element, of originally allocated eth_dev
-        * so the rings are local per-process */
-
-       internals->nb_rx_queues = nb_rx_queues;
-       internals->nb_tx_queues = nb_tx_queues;
-       for (i = 0; i < nb_rx_queues; i++) {
-               internals->rx_ring_queues[i].rng = rx_queues[i];
-       }
-       for (i = 0; i < nb_tx_queues; i++) {
-               internals->tx_ring_queues[i].rng = tx_queues[i];
-       }
-
-       eth_drv->pci_drv.name = ring_ethdev_driver_name;
-       eth_drv->pci_drv.id_table = id_table;
-
-       pci_dev->numa_node = numa_node;
-       pci_dev->driver = &eth_drv->pci_drv;
-
-       data->dev_private = internals;
-       data->port_id = eth_dev->data->port_id;
-       data->nb_rx_queues = (uint16_t)nb_rx_queues;
-       data->nb_tx_queues = (uint16_t)nb_tx_queues;
-       data->dev_link = pmd_link;
-       data->mac_addrs = &internals->address;
-
-       eth_dev->data = data;
-       eth_dev->driver = eth_drv;
-       eth_dev->dev_ops = &ops;
-       eth_dev->pci_dev = pci_dev;
-       TAILQ_INIT(&(eth_dev->link_intr_cbs));
-
-       /* finally assign rx and tx ops */
-       eth_dev->rx_pkt_burst = eth_ring_rx;
-       eth_dev->tx_pkt_burst = eth_ring_tx;
-
-       return 0;
-
-error:
-       rte_free(data);
-       rte_free(pci_dev);
-       rte_free(internals);
-
-       return -1;
-}
-
-enum dev_action{
-       DEV_CREATE,
-       DEV_ATTACH
-};
-
-static int
-eth_dev_ring_create(const char *name, const unsigned numa_node,
-               enum dev_action action)
-{
-       /* rx and tx are so-called from point of view of first port.
-        * They are inverted from the point of view of second port
-        */
-       struct rte_ring *rxtx[RTE_PMD_RING_MAX_RX_RINGS];
-       unsigned i;
-       char rng_name[RTE_RING_NAMESIZE];
-       unsigned num_rings = RTE_MIN(RTE_PMD_RING_MAX_RX_RINGS,
-                       RTE_PMD_RING_MAX_TX_RINGS);
-
-       for (i = 0; i < num_rings; i++) {
-               snprintf(rng_name, sizeof(rng_name), "ETH_RXTX%u_%s", i, name);
-               rxtx[i] = (action == DEV_CREATE) ?
-                               rte_ring_create(rng_name, 1024, numa_node,
-                                               RING_F_SP_ENQ|RING_F_SC_DEQ) :
-                               rte_ring_lookup(rng_name);
-               if (rxtx[i] == NULL)
-                       return -1;
-       }
-
-       if (rte_eth_from_rings(name, rxtx, num_rings, rxtx, num_rings, numa_node))
-               return -1;
-
-       return 0;
-}
-
-
-static int
-eth_dev_ring_pair_create(const char *name, const unsigned numa_node,
-               enum dev_action action)
-{
-       /* rx and tx are so-called from point of view of first port.
-        * They are inverted from the point of view of second port
-        */
-       struct rte_ring *rx[RTE_PMD_RING_MAX_RX_RINGS];
-       struct rte_ring *tx[RTE_PMD_RING_MAX_TX_RINGS];
-       unsigned i;
-       char rx_rng_name[RTE_RING_NAMESIZE];
-       char tx_rng_name[RTE_RING_NAMESIZE];
-       unsigned num_rings = RTE_MIN(RTE_PMD_RING_MAX_RX_RINGS,
-                       RTE_PMD_RING_MAX_TX_RINGS);
-
-       for (i = 0; i < num_rings; i++) {
-               snprintf(rx_rng_name, sizeof(rx_rng_name), "ETH_RX%u_%s", i, name);
-               rx[i] = (action == DEV_CREATE) ?
-                               rte_ring_create(rx_rng_name, 1024, numa_node,
-                                               RING_F_SP_ENQ|RING_F_SC_DEQ) :
-                               rte_ring_lookup(rx_rng_name);
-               if (rx[i] == NULL)
-                       return -1;
-               snprintf(tx_rng_name, sizeof(tx_rng_name), "ETH_TX%u_%s", i, name);
-               tx[i] = (action == DEV_CREATE) ?
-                               rte_ring_create(tx_rng_name, 1024, numa_node,
-                                               RING_F_SP_ENQ|RING_F_SC_DEQ):
-                               rte_ring_lookup(tx_rng_name);
-               if (tx[i] == NULL)
-                       return -1;
-       }
-
-       if (rte_eth_from_rings(rx_rng_name, rx, num_rings, tx, num_rings,
-                       numa_node) || rte_eth_from_rings(tx_rng_name, tx, num_rings, rx,
-                                       num_rings, numa_node))
-               return -1;
-
-       return 0;
-}
-
-int
-rte_eth_ring_pair_create(const char *name, const unsigned numa_node)
-{
-       RTE_LOG(WARNING, PMD, "rte_eth_ring_pair_create is deprecated\n");
-       return eth_dev_ring_pair_create(name, numa_node, DEV_CREATE);
-}
-
-int
-rte_eth_ring_pair_attach(const char *name, const unsigned numa_node)
-{
-       RTE_LOG(WARNING, PMD, "rte_eth_ring_pair_attach is deprecated\n");
-       return eth_dev_ring_pair_create(name, numa_node, DEV_ATTACH);
-}
-
-struct node_action_pair {
-       char name[PATH_MAX];
-       unsigned node;
-       enum dev_action action;
-};
-
-struct node_action_list {
-       unsigned total;
-       unsigned count;
-       struct node_action_pair *list;
-};
-
-static int parse_kvlist (const char *key __rte_unused, const char *value, void *data)
-{
-       struct node_action_list *info = data;
-       int ret;
-       char *name;
-       char *action;
-       char *node;
-       char *end;
-
-       name = strdup(value);
-
-       ret = -EINVAL;
-
-       if (!name) {
-               RTE_LOG(WARNING, PMD, "command line paramter is empty for ring pmd!\n");
-               goto out;
-       }
-
-       node = strchr(name, ':');
-       if (!node) {
-               RTE_LOG(WARNING, PMD, "could not parse node value from %s", name);
-               goto out;
-       }
-
-       *node = '\0';
-       node++;
-
-       action = strchr(node, ':');
-       if (!action) {
-               RTE_LOG(WARNING, PMD, "could not action value from %s", node);
-               goto out;
-       }
-
-       *action = '\0';
-       action++;
-
-       /*
-        * Need to do some sanity checking here
-        */
-
-       if (strcmp(action, ETH_RING_ACTION_ATTACH) == 0)
-               info->list[info->count].action = DEV_ATTACH;
-       else if (strcmp(action, ETH_RING_ACTION_CREATE) == 0)
-               info->list[info->count].action = DEV_CREATE;
-       else
-               goto out;
-
-       errno = 0;
-       info->list[info->count].node = strtol(node, &end, 10);
-
-       if ((errno != 0) || (*end != '\0')) {
-               RTE_LOG(WARNING, PMD, "node value %s is unparseable as a number\n", node);
-               goto out;
-       }
-
-       snprintf(info->list[info->count].name, sizeof(info->list[info->count].name), "%s", name);
-
-       info->count++;
-
-       ret = 0;
-out:
-       free(name);
-       return ret;
-}
-
-static int
-rte_pmd_ring_devinit(const char *name, const char *params)
-{
-       struct rte_kvargs *kvlist = NULL;
-       int ret = 0;
-       struct node_action_list *info = NULL;
-
-       RTE_LOG(INFO, PMD, "Initializing pmd_ring for %s\n", name);
-
-       if (params == NULL || params[0] == '\0')
-               eth_dev_ring_create(name, rte_socket_id(), DEV_CREATE);
-       else {
-               kvlist = rte_kvargs_parse(params, valid_arguments);
-
-               if (!kvlist) {
-                       RTE_LOG(INFO, PMD, "Ignoring unsupported parameters when creating"
-                                       " rings-backed ethernet device\n");
-                       eth_dev_ring_create(name, rte_socket_id(), DEV_CREATE);
-                       return 0;
-               } else {
-                       ret = rte_kvargs_count(kvlist, ETH_RING_NUMA_NODE_ACTION_ARG);
-                       info = rte_zmalloc("struct node_action_list", sizeof(struct node_action_list) +
-                                          (sizeof(struct node_action_pair) * ret), 0);
-                       if (!info)
-                               goto out_free;
-
-                       info->total = ret;
-                       info->list = (struct node_action_pair*)(info + 1);
-
-                       ret = rte_kvargs_process(kvlist, ETH_RING_NUMA_NODE_ACTION_ARG,
-                                                parse_kvlist, info);
-
-                       if (ret < 0)
-                               goto out_free;
-
-                       for (info->count = 0; info->count < info->total; info->count++) {
-                               eth_dev_ring_create(name, info->list[info->count].node,
-                                                   info->list[info->count].action);
-                       }
-               }
-       }
-
-out_free:
-       rte_kvargs_free(kvlist);
-       rte_free(info);
-       return ret;
-}
-
-static struct rte_driver pmd_ring_drv = {
-       .name = "eth_ring",
-       .type = PMD_VDEV,
-       .init = rte_pmd_ring_devinit,
-};
-
-PMD_REGISTER_DRIVER(pmd_ring_drv);
diff --git a/lib/librte_pmd_ring/rte_eth_ring.h b/lib/librte_pmd_ring/rte_eth_ring.h
deleted file mode 100644 (file)
index d36489a..0000000
+++ /dev/null
@@ -1,57 +0,0 @@
-/*-
- *   BSD LICENSE
- *
- *   Copyright(c) 2010-2014 Intel Corporation. All rights reserved.
- *   All rights reserved.
- *
- *   Redistribution and use in source and binary forms, with or without
- *   modification, are permitted provided that the following conditions
- *   are met:
- *
- *     * Redistributions of source code must retain the above copyright
- *       notice, this list of conditions and the following disclaimer.
- *     * Redistributions in binary form must reproduce the above copyright
- *       notice, this list of conditions and the following disclaimer in
- *       the documentation and/or other materials provided with the
- *       distribution.
- *     * Neither the name of Intel Corporation nor the names of its
- *       contributors may be used to endorse or promote products derived
- *       from this software without specific prior written permission.
- *
- *   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- *   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- *   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- *   A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- *   OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- *   SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- *   LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- *   DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- *   THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- *   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- *   OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#ifndef _RTE_ETH_RING_H_
-#define _RTE_ETH_RING_H_
-
-#ifdef __cplusplus
-extern "C" {
-#endif
-
-#include <rte_ring.h>
-
-int rte_eth_from_rings(const char *name,
-               struct rte_ring * const rx_queues[],
-               const unsigned nb_rx_queues,
-               struct rte_ring *const tx_queues[],
-               const unsigned nb_tx_queues,
-               const unsigned numa_node);
-
-int rte_eth_ring_pair_create(const char *name, const unsigned numa_node);
-int rte_eth_ring_pair_attach(const char *name, const unsigned numa_node);
-
-#ifdef __cplusplus
-}
-#endif
-
-#endif
diff --git a/lib/librte_pmd_ring/rte_eth_ring_version.map b/lib/librte_pmd_ring/rte_eth_ring_version.map
deleted file mode 100644 (file)
index 8ad107d..0000000
+++ /dev/null
@@ -1,9 +0,0 @@
-DPDK_2.0 {
-       global:
-
-       rte_eth_from_rings;
-       rte_eth_ring_pair_attach;
-       rte_eth_ring_pair_create;
-
-       local: *;
-};