From 74b46340e2d4f6b4b00476ed3790580004f21341 Mon Sep 17 00:00:00 2001 From: Ciara Loftus Date: Wed, 23 Sep 2020 07:34:39 +0000 Subject: [PATCH] net/af_xdp: support shared UMEM Kernel v5.10 will introduce the ability to efficiently share a UMEM between AF_XDP sockets bound to different queue ids on the same or different devices. This patch integrates that functionality into the AF_XDP PMD. A PMD will attempt to share a UMEM with others if the shared_umem=1 vdev arg is set. UMEMs can only be shared across PMDs with the same mempool, up to a limited number of PMDs goverened by the size of the given mempool. Sharing UMEMs is not supported for non-zero-copy (aligned) mode. The benefit of sharing UMEM across PMDs is a saving in memory due to not having to register the UMEM multiple times. Throughput was measured to remain within 2% of the default mode (not sharing UMEM). A version of libbpf >= v0.2.0 is required and the appropriate pkg-config file for libbpf must be installed such that meson can determine the version. Signed-off-by: Ciara Loftus --- doc/guides/nics/af_xdp.rst | 6 +- drivers/net/af_xdp/compat.h | 41 ++++ drivers/net/af_xdp/meson.build | 4 + drivers/net/af_xdp/rte_eth_af_xdp.c | 303 ++++++++++++++++++++++------ 4 files changed, 289 insertions(+), 65 deletions(-) create mode 100644 drivers/net/af_xdp/compat.h diff --git a/doc/guides/nics/af_xdp.rst b/doc/guides/nics/af_xdp.rst index 07bdd29e29..78a088468f 100644 --- a/doc/guides/nics/af_xdp.rst +++ b/doc/guides/nics/af_xdp.rst @@ -1,5 +1,5 @@ .. SPDX-License-Identifier: BSD-3-Clause - Copyright(c) 2019 Intel Corporation. + Copyright(c) 2019-2020 Intel Corporation. AF_XDP Poll Mode Driver ========================== @@ -32,6 +32,8 @@ The following options can be provided to set up an af_xdp port in DPDK. * ``iface`` - name of the Kernel interface to attach to (required); * ``start_queue`` - starting netdev queue id (optional, default 0); * ``queue_count`` - total netdev queue number (optional, default 1); +* ``shared_umem`` - PMD will attempt to share UMEM with others (optional, + default 0); Prerequisites ------------- @@ -45,6 +47,8 @@ This is a Linux-specific PMD, thus the following prerequisites apply: * A Kernel bound interface to attach to; * For need_wakeup feature, it requires kernel version later than v5.3-rc1; * For PMD zero copy, it requires kernel version later than v5.4-rc1; +* For shared_umem, it requires kernel version v5.10 or later and libbpf version + v0.2.0 or later. Set up an af_xdp interface ----------------------------- diff --git a/drivers/net/af_xdp/compat.h b/drivers/net/af_xdp/compat.h new file mode 100644 index 0000000000..7aa40d522e --- /dev/null +++ b/drivers/net/af_xdp/compat.h @@ -0,0 +1,41 @@ +/* SPDX-License-Identifier: BSD-3-Clause + * Copyright(c) 2020 Intel Corporation. + */ + +#include +#include + +#if KERNEL_VERSION(5, 10, 0) <= LINUX_VERSION_CODE && \ + defined(RTE_LIBRTE_AF_XDP_PMD_SHARED_UMEM) +#define ETH_AF_XDP_SHARED_UMEM 1 +#endif + +#ifdef ETH_AF_XDP_SHARED_UMEM +static __rte_always_inline int +create_shared_socket(struct xsk_socket **xsk_ptr, + const char *ifname, + __u32 queue_id, struct xsk_umem *umem, + struct xsk_ring_cons *rx, + struct xsk_ring_prod *tx, + struct xsk_ring_prod *fill, + struct xsk_ring_cons *comp, + const struct xsk_socket_config *config) +{ + return xsk_socket__create_shared(xsk_ptr, ifname, queue_id, umem, rx, + tx, fill, comp, config); +} +#else +static __rte_always_inline int +create_shared_socket(struct xsk_socket **xsk_ptr __rte_unused, + const char *ifname __rte_unused, + __u32 queue_id __rte_unused, + struct xsk_umem *umem __rte_unused, + struct xsk_ring_cons *rx __rte_unused, + struct xsk_ring_prod *tx __rte_unused, + struct xsk_ring_prod *fill __rte_unused, + struct xsk_ring_cons *comp __rte_unused, + const struct xsk_socket_config *config __rte_unused) +{ + return -1; +} +#endif diff --git a/drivers/net/af_xdp/meson.build b/drivers/net/af_xdp/meson.build index 307aa0e388..fead8dd99f 100644 --- a/drivers/net/af_xdp/meson.build +++ b/drivers/net/af_xdp/meson.build @@ -10,6 +10,10 @@ endif if bpf_dep.found() and cc.has_header('bpf/xsk.h') and cc.has_header('linux/if_xdp.h') ext_deps += bpf_dep + bpf_ver_dep = dependency('libbpf', version : '>=0.2.0', required: false) + if bpf_ver_dep.found() + dpdk_conf.set('RTE_LIBRTE_AF_XDP_PMD_SHARED_UMEM', 1) + endif else build = false reason = 'missing dependency, "libbpf"' diff --git a/drivers/net/af_xdp/rte_eth_af_xdp.c b/drivers/net/af_xdp/rte_eth_af_xdp.c index 00de671841..01f462b465 100644 --- a/drivers/net/af_xdp/rte_eth_af_xdp.c +++ b/drivers/net/af_xdp/rte_eth_af_xdp.c @@ -1,5 +1,5 @@ /* SPDX-License-Identifier: BSD-3-Clause - * Copyright(c) 2019 Intel Corporation. + * Copyright(c) 2019-2020 Intel Corporation. */ #include #include @@ -37,6 +37,10 @@ #include #include #include +#include + +#include "compat.h" + #ifndef SOL_XDP #define SOL_XDP 283 @@ -67,13 +71,13 @@ RTE_LOG_REGISTER(af_xdp_logtype, pmd.net.af_xdp, NOTICE); struct xsk_umem_info { - struct xsk_ring_prod fq; - struct xsk_ring_cons cq; struct xsk_umem *umem; struct rte_ring *buf_ring; const struct rte_memzone *mz; struct rte_mempool *mb_pool; void *buffer; + uint8_t refcnt; + uint32_t max_xsks; }; struct rx_stats { @@ -90,6 +94,9 @@ struct pkt_rx_queue { struct rx_stats stats; + struct xsk_ring_prod fq; + struct xsk_ring_cons cq; + struct pkt_tx_queue *pair; struct pollfd fds[1]; int xsk_queue_idx; @@ -118,6 +125,7 @@ struct pmd_internals { int queue_cnt; int max_queue_cnt; int combined_queue_cnt; + bool shared_umem; struct rte_ether_addr eth_addr; @@ -128,11 +136,13 @@ struct pmd_internals { #define ETH_AF_XDP_IFACE_ARG "iface" #define ETH_AF_XDP_START_QUEUE_ARG "start_queue" #define ETH_AF_XDP_QUEUE_COUNT_ARG "queue_count" +#define ETH_AF_XDP_SHARED_UMEM_ARG "shared_umem" static const char * const valid_arguments[] = { ETH_AF_XDP_IFACE_ARG, ETH_AF_XDP_START_QUEUE_ARG, ETH_AF_XDP_QUEUE_COUNT_ARG, + ETH_AF_XDP_SHARED_UMEM_ARG, NULL }; @@ -143,12 +153,23 @@ static const struct rte_eth_link pmd_link = { .link_autoneg = ETH_LINK_AUTONEG }; +/* List which tracks PMDs to facilitate sharing UMEMs across them. */ +struct internal_list { + TAILQ_ENTRY(internal_list) next; + struct rte_eth_dev *eth_dev; +}; + +TAILQ_HEAD(internal_list_head, internal_list); +static struct internal_list_head internal_list = + TAILQ_HEAD_INITIALIZER(internal_list); + +static pthread_mutex_t internal_list_lock = PTHREAD_MUTEX_INITIALIZER; + #if defined(XDP_UMEM_UNALIGNED_CHUNK_FLAG) static inline int reserve_fill_queue_zc(struct xsk_umem_info *umem, uint16_t reserve_size, - struct rte_mbuf **bufs) + struct rte_mbuf **bufs, struct xsk_ring_prod *fq) { - struct xsk_ring_prod *fq = &umem->fq; uint32_t idx; uint16_t i; @@ -176,9 +197,9 @@ reserve_fill_queue_zc(struct xsk_umem_info *umem, uint16_t reserve_size, #else static inline int reserve_fill_queue_cp(struct xsk_umem_info *umem, uint16_t reserve_size, - struct rte_mbuf **bufs __rte_unused) + struct rte_mbuf **bufs __rte_unused, + struct xsk_ring_prod *fq) { - struct xsk_ring_prod *fq = &umem->fq; void *addrs[reserve_size]; uint32_t idx; uint16_t i; @@ -211,12 +232,12 @@ reserve_fill_queue_cp(struct xsk_umem_info *umem, uint16_t reserve_size, static inline int reserve_fill_queue(struct xsk_umem_info *umem, uint16_t reserve_size, - struct rte_mbuf **bufs) + struct rte_mbuf **bufs, struct xsk_ring_prod *fq) { #if defined(XDP_UMEM_UNALIGNED_CHUNK_FLAG) - return reserve_fill_queue_zc(umem, reserve_size, bufs); + return reserve_fill_queue_zc(umem, reserve_size, bufs, fq); #else - return reserve_fill_queue_cp(umem, reserve_size, bufs); + return reserve_fill_queue_cp(umem, reserve_size, bufs, fq); #endif } @@ -226,6 +247,7 @@ af_xdp_rx_zc(void *queue, struct rte_mbuf **bufs, uint16_t nb_pkts) { struct pkt_rx_queue *rxq = queue; struct xsk_ring_cons *rx = &rxq->rx; + struct xsk_ring_prod *fq = &rxq->fq; struct xsk_umem_info *umem = rxq->umem; uint32_t idx_rx = 0; unsigned long rx_bytes = 0; @@ -243,7 +265,7 @@ af_xdp_rx_zc(void *queue, struct rte_mbuf **bufs, uint16_t nb_pkts) if (rcvd == 0) { #if defined(XDP_USE_NEED_WAKEUP) - if (xsk_ring_prod__needs_wakeup(&umem->fq)) + if (xsk_ring_prod__needs_wakeup(fq)) (void)poll(rxq->fds, 1, 1000); #endif @@ -277,7 +299,7 @@ af_xdp_rx_zc(void *queue, struct rte_mbuf **bufs, uint16_t nb_pkts) xsk_ring_cons__release(rx, rcvd); - (void)reserve_fill_queue(umem, rcvd, fq_bufs); + (void)reserve_fill_queue(umem, rcvd, fq_bufs, fq); /* statistics */ rxq->stats.rx_pkts += rcvd; @@ -297,7 +319,7 @@ af_xdp_rx_cp(void *queue, struct rte_mbuf **bufs, uint16_t nb_pkts) struct pkt_rx_queue *rxq = queue; struct xsk_ring_cons *rx = &rxq->rx; struct xsk_umem_info *umem = rxq->umem; - struct xsk_ring_prod *fq = &umem->fq; + struct xsk_ring_prod *fq = &rxq->fq; uint32_t idx_rx = 0; unsigned long rx_bytes = 0; int rcvd, i; @@ -305,8 +327,8 @@ af_xdp_rx_cp(void *queue, struct rte_mbuf **bufs, uint16_t nb_pkts) struct rte_mbuf *mbufs[ETH_AF_XDP_RX_BATCH_SIZE]; if (xsk_prod_nb_free(fq, free_thresh) >= free_thresh) - (void)reserve_fill_queue(umem, ETH_AF_XDP_RX_BATCH_SIZE, NULL); - + (void)reserve_fill_queue(umem, ETH_AF_XDP_RX_BATCH_SIZE, + NULL, fq); if (unlikely(rte_pktmbuf_alloc_bulk(rxq->mb_pool, mbufs, nb_pkts) != 0)) return 0; @@ -368,9 +390,8 @@ eth_af_xdp_rx(void *queue, struct rte_mbuf **bufs, uint16_t nb_pkts) } static void -pull_umem_cq(struct xsk_umem_info *umem, int size) +pull_umem_cq(struct xsk_umem_info *umem, int size, struct xsk_ring_cons *cq) { - struct xsk_ring_cons *cq = &umem->cq; size_t i, n; uint32_t idx_cq = 0; @@ -393,11 +414,11 @@ pull_umem_cq(struct xsk_umem_info *umem, int size) } static void -kick_tx(struct pkt_tx_queue *txq) +kick_tx(struct pkt_tx_queue *txq, struct xsk_ring_cons *cq) { struct xsk_umem_info *umem = txq->umem; - pull_umem_cq(umem, XSK_RING_CONS__DEFAULT_NUM_DESCS); + pull_umem_cq(umem, XSK_RING_CONS__DEFAULT_NUM_DESCS, cq); #if defined(XDP_USE_NEED_WAKEUP) if (xsk_ring_prod__needs_wakeup(&txq->tx)) @@ -411,7 +432,8 @@ kick_tx(struct pkt_tx_queue *txq) /* pull from completion queue to leave more space */ if (errno == EAGAIN) pull_umem_cq(umem, - XSK_RING_CONS__DEFAULT_NUM_DESCS); + XSK_RING_CONS__DEFAULT_NUM_DESCS, + cq); } } @@ -428,17 +450,18 @@ af_xdp_tx_zc(void *queue, struct rte_mbuf **bufs, uint16_t nb_pkts) uint16_t count = 0; struct xdp_desc *desc; uint64_t addr, offset; - uint32_t free_thresh = umem->cq.size >> 1; + struct xsk_ring_cons *cq = &txq->pair->cq; + uint32_t free_thresh = cq->size >> 1; - if (xsk_cons_nb_avail(&umem->cq, free_thresh) >= free_thresh) - pull_umem_cq(umem, XSK_RING_CONS__DEFAULT_NUM_DESCS); + if (xsk_cons_nb_avail(cq, free_thresh) >= free_thresh) + pull_umem_cq(umem, XSK_RING_CONS__DEFAULT_NUM_DESCS, cq); for (i = 0; i < nb_pkts; i++) { mbuf = bufs[i]; if (mbuf->pool == umem->mb_pool) { if (!xsk_ring_prod__reserve(&txq->tx, 1, &idx_tx)) { - kick_tx(txq); + kick_tx(txq, cq); if (!xsk_ring_prod__reserve(&txq->tx, 1, &idx_tx)) goto out; @@ -463,7 +486,7 @@ af_xdp_tx_zc(void *queue, struct rte_mbuf **bufs, uint16_t nb_pkts) if (!xsk_ring_prod__reserve(&txq->tx, 1, &idx_tx)) { rte_pktmbuf_free(local_mbuf); - kick_tx(txq); + kick_tx(txq, cq); goto out; } @@ -487,7 +510,7 @@ af_xdp_tx_zc(void *queue, struct rte_mbuf **bufs, uint16_t nb_pkts) tx_bytes += mbuf->pkt_len; } - kick_tx(txq); + kick_tx(txq, cq); out: xsk_ring_prod__submit(&txq->tx, count); @@ -509,10 +532,11 @@ af_xdp_tx_cp(void *queue, struct rte_mbuf **bufs, uint16_t nb_pkts) unsigned long tx_bytes = 0; int i; uint32_t idx_tx; + struct xsk_ring_cons *cq = &txq->pair->cq; nb_pkts = RTE_MIN(nb_pkts, ETH_AF_XDP_TX_BATCH_SIZE); - pull_umem_cq(umem, nb_pkts); + pull_umem_cq(umem, nb_pkts, cq); nb_pkts = rte_ring_dequeue_bulk(umem->buf_ring, addrs, nb_pkts, NULL); @@ -520,7 +544,7 @@ af_xdp_tx_cp(void *queue, struct rte_mbuf **bufs, uint16_t nb_pkts) return 0; if (xsk_ring_prod__reserve(&txq->tx, nb_pkts, &idx_tx) != nb_pkts) { - kick_tx(txq); + kick_tx(txq, cq); rte_ring_enqueue_bulk(umem->buf_ring, addrs, nb_pkts, NULL); return 0; } @@ -543,7 +567,7 @@ af_xdp_tx_cp(void *queue, struct rte_mbuf **bufs, uint16_t nb_pkts) xsk_ring_prod__submit(&txq->tx, nb_pkts); - kick_tx(txq); + kick_tx(txq, cq); txq->stats.tx_pkts += nb_pkts; txq->stats.tx_bytes += tx_bytes; @@ -577,13 +601,101 @@ eth_dev_stop(struct rte_eth_dev *dev) dev->data->dev_link.link_status = ETH_LINK_DOWN; } +/* Find ethdev in list */ +static inline struct internal_list * +find_internal_resource(struct pmd_internals *port_int) +{ + int found = 0; + struct internal_list *list = NULL; + + if (port_int == NULL) + return NULL; + + pthread_mutex_lock(&internal_list_lock); + + TAILQ_FOREACH(list, &internal_list, next) { + struct pmd_internals *list_int = + list->eth_dev->data->dev_private; + if (list_int == port_int) { + found = 1; + break; + } + } + + pthread_mutex_unlock(&internal_list_lock); + + if (!found) + return NULL; + + return list; +} + +/* Get a pointer to an existing UMEM which overlays the rxq's mb_pool */ +static inline struct xsk_umem_info * +get_shared_umem(struct pkt_rx_queue *rxq) { + struct internal_list *list; + struct pmd_internals *internals; + int i = 0; + struct rte_mempool *mb_pool = rxq->mb_pool; + + if (mb_pool == NULL) + return NULL; + + pthread_mutex_lock(&internal_list_lock); + + TAILQ_FOREACH(list, &internal_list, next) { + internals = list->eth_dev->data->dev_private; + for (i = 0; i < internals->queue_cnt; i++) { + struct pkt_rx_queue *list_rxq = + &internals->rx_queues[i]; + if (rxq == list_rxq) + continue; + if (mb_pool == internals->rx_queues[i].mb_pool) { + if (__atomic_load_n( + &internals->rx_queues[i].umem->refcnt, + __ATOMIC_ACQUIRE)) { + pthread_mutex_unlock( + &internal_list_lock); + return internals->rx_queues[i].umem; + } + } + } + } + + pthread_mutex_unlock(&internal_list_lock); + + return NULL; +} + static int eth_dev_configure(struct rte_eth_dev *dev) { + struct pmd_internals *internal = dev->data->dev_private; + /* rx/tx must be paired */ if (dev->data->nb_rx_queues != dev->data->nb_tx_queues) return -EINVAL; + if (internal->shared_umem) { + struct internal_list *list = NULL; + const char *name = dev->device->name; + + /* Ensure PMD is not already inserted into the list */ + list = find_internal_resource(internal); + if (list) + return 0; + + list = rte_zmalloc_socket(name, sizeof(*list), 0, + dev->device->numa_node); + if (list == NULL) + return -1; + + list->eth_dev = dev; + pthread_mutex_lock(&internal_list_lock); + TAILQ_INSERT_TAIL(&internal_list, list, next); + pthread_mutex_unlock(&internal_list_lock); + } + return 0; } @@ -717,8 +829,12 @@ eth_dev_close(struct rte_eth_dev *dev) if (rxq->umem == NULL) break; xsk_socket__delete(rxq->xsk); - (void)xsk_umem__delete(rxq->umem->umem); - xdp_umem_destroy(rxq->umem); + + if (__atomic_sub_fetch(&rxq->umem->refcnt, 1, __ATOMIC_ACQUIRE) + == 0) { + (void)xsk_umem__delete(rxq->umem->umem); + xdp_umem_destroy(rxq->umem); + } /* free pkt_tx_queue */ rte_free(rxq->pair); @@ -732,6 +848,19 @@ eth_dev_close(struct rte_eth_dev *dev) dev->data->mac_addrs = NULL; remove_xdp_program(internals); + + if (internals->shared_umem) { + struct internal_list *list; + + /* Remove ethdev from list used to track and share UMEMs */ + list = find_internal_resource(internals); + if (list) { + pthread_mutex_lock(&internal_list_lock); + TAILQ_REMOVE(&internal_list, list, next); + pthread_mutex_unlock(&internal_list_lock); + rte_free(list); + } + } } static void @@ -761,10 +890,10 @@ static inline uint64_t get_base_addr(struct rte_mempool *mp, uint64_t *align) } static struct -xsk_umem_info *xdp_umem_configure(struct pmd_internals *internals __rte_unused, +xsk_umem_info *xdp_umem_configure(struct pmd_internals *internals, struct pkt_rx_queue *rxq) { - struct xsk_umem_info *umem; + struct xsk_umem_info *umem = NULL; int ret; struct xsk_umem_config usr_config = { .fill_size = ETH_AF_XDP_DFLT_NUM_DESCS * 2, @@ -774,32 +903,55 @@ xsk_umem_info *xdp_umem_configure(struct pmd_internals *internals __rte_unused, struct rte_mempool *mb_pool = rxq->mb_pool; uint64_t umem_size, align = 0; - usr_config.frame_size = rte_mempool_calc_obj_size(mb_pool->elt_size, - mb_pool->flags, - NULL); - usr_config.frame_headroom = mb_pool->header_size + - sizeof(struct rte_mbuf) + - rte_pktmbuf_priv_size(mb_pool) + - RTE_PKTMBUF_HEADROOM; + if (internals->shared_umem) { + umem = get_shared_umem(rxq); + if (umem != NULL && + __atomic_load_n(&umem->refcnt, __ATOMIC_ACQUIRE) < + umem->max_xsks) { + AF_XDP_LOG(INFO, "%s,qid%i sharing UMEM\n", + internals->if_name, rxq->xsk_queue_idx); + __atomic_fetch_add(&umem->refcnt, 1, __ATOMIC_ACQUIRE); + } + } - umem = rte_zmalloc_socket("umem", sizeof(*umem), 0, rte_socket_id()); if (umem == NULL) { - AF_XDP_LOG(ERR, "Failed to allocate umem info"); - return NULL; - } + usr_config.frame_size = + rte_mempool_calc_obj_size(mb_pool->elt_size, + mb_pool->flags, NULL); + usr_config.frame_headroom = mb_pool->header_size + + sizeof(struct rte_mbuf) + + rte_pktmbuf_priv_size(mb_pool) + + RTE_PKTMBUF_HEADROOM; + + umem = rte_zmalloc_socket("umem", sizeof(*umem), 0, + rte_socket_id()); + if (umem == NULL) { + AF_XDP_LOG(ERR, "Failed to allocate umem info"); + return NULL; + } + + umem->mb_pool = mb_pool; + base_addr = (void *)get_base_addr(mb_pool, &align); + umem_size = mb_pool->populated_size * usr_config.frame_size + + align; - umem->mb_pool = mb_pool; - base_addr = (void *)get_base_addr(mb_pool, &align); - umem_size = mb_pool->populated_size * usr_config.frame_size + align; + ret = xsk_umem__create(&umem->umem, base_addr, umem_size, + &rxq->fq, &rxq->cq, &usr_config); + if (ret) { + AF_XDP_LOG(ERR, "Failed to create umem"); + goto err; + } + umem->buffer = base_addr; - ret = xsk_umem__create(&umem->umem, base_addr, umem_size, - &umem->fq, &umem->cq, &usr_config); + if (internals->shared_umem) { + umem->max_xsks = mb_pool->populated_size / + ETH_AF_XDP_NUM_BUFFERS; + AF_XDP_LOG(INFO, "Max xsks for UMEM %s: %u\n", + mb_pool->name, umem->max_xsks); + } - if (ret) { - AF_XDP_LOG(ERR, "Failed to create umem"); - goto err; + __atomic_store_n(&umem->refcnt, 1, __ATOMIC_RELEASE); } - umem->buffer = base_addr; #else static struct @@ -852,7 +1004,7 @@ xsk_umem_info *xdp_umem_configure(struct pmd_internals *internals, ret = xsk_umem__create(&umem->umem, mz->addr, ETH_AF_XDP_NUM_BUFFERS * ETH_AF_XDP_FRAME_SIZE, - &umem->fq, &umem->cq, + &rxq->fq, &rxq->cq, &usr_config); if (ret) { @@ -894,9 +1046,15 @@ xsk_configure(struct pmd_internals *internals, struct pkt_rx_queue *rxq, cfg.bind_flags |= XDP_USE_NEED_WAKEUP; #endif - ret = xsk_socket__create(&rxq->xsk, internals->if_name, - rxq->xsk_queue_idx, rxq->umem->umem, &rxq->rx, - &txq->tx, &cfg); + if (internals->shared_umem) + ret = create_shared_socket(&rxq->xsk, internals->if_name, + rxq->xsk_queue_idx, rxq->umem->umem, &rxq->rx, + &txq->tx, &rxq->fq, &rxq->cq, &cfg); + else + ret = xsk_socket__create(&rxq->xsk, internals->if_name, + rxq->xsk_queue_idx, rxq->umem->umem, &rxq->rx, + &txq->tx, &cfg); + if (ret) { AF_XDP_LOG(ERR, "Failed to create xsk socket.\n"); goto err; @@ -908,7 +1066,7 @@ xsk_configure(struct pmd_internals *internals, struct pkt_rx_queue *rxq, goto err; } #endif - ret = reserve_fill_queue(rxq->umem, reserve_size, fq_bufs); + ret = reserve_fill_queue(rxq->umem, reserve_size, fq_bufs, &rxq->fq); if (ret) { xsk_socket__delete(rxq->xsk); AF_XDP_LOG(ERR, "Failed to reserve fill queue.\n"); @@ -918,7 +1076,8 @@ xsk_configure(struct pmd_internals *internals, struct pkt_rx_queue *rxq, return 0; err: - xdp_umem_destroy(rxq->umem); + if (__atomic_sub_fetch(&rxq->umem->refcnt, 1, __ATOMIC_ACQUIRE) == 0) + xdp_umem_destroy(rxq->umem); return ret; } @@ -1148,7 +1307,7 @@ xdp_get_channels_info(const char *if_name, int *max_queues, static int parse_parameters(struct rte_kvargs *kvlist, char *if_name, int *start_queue, - int *queue_cnt) + int *queue_cnt, int *shared_umem) { int ret; @@ -1169,6 +1328,11 @@ parse_parameters(struct rte_kvargs *kvlist, char *if_name, int *start_queue, goto free_kvlist; } + ret = rte_kvargs_process(kvlist, ETH_AF_XDP_SHARED_UMEM_ARG, + &parse_integer_arg, shared_umem); + if (ret < 0) + goto free_kvlist; + free_kvlist: rte_kvargs_free(kvlist); return ret; @@ -1206,7 +1370,7 @@ error: static struct rte_eth_dev * init_internals(struct rte_vdev_device *dev, const char *if_name, - int start_queue_idx, int queue_cnt) + int start_queue_idx, int queue_cnt, int shared_umem) { const char *name = rte_vdev_device_name(dev); const unsigned int numa_node = dev->device.numa_node; @@ -1223,6 +1387,15 @@ init_internals(struct rte_vdev_device *dev, const char *if_name, internals->queue_cnt = queue_cnt; strlcpy(internals->if_name, if_name, IFNAMSIZ); +#ifndef ETH_AF_XDP_SHARED_UMEM + if (shared_umem) { + AF_XDP_LOG(ERR, "Shared UMEM feature not available. " + "Check kernel and libbpf version\n"); + goto err_free_internals; + } +#endif + internals->shared_umem = shared_umem; + if (xdp_get_channels_info(if_name, &internals->max_queue_cnt, &internals->combined_queue_cnt)) { AF_XDP_LOG(ERR, "Failed to get channel info of interface: %s\n", @@ -1298,6 +1471,7 @@ rte_pmd_af_xdp_probe(struct rte_vdev_device *dev) char if_name[IFNAMSIZ] = {'\0'}; int xsk_start_queue_idx = ETH_AF_XDP_DFLT_START_QUEUE_IDX; int xsk_queue_cnt = ETH_AF_XDP_DFLT_QUEUE_COUNT; + int shared_umem = 0; struct rte_eth_dev *eth_dev = NULL; const char *name; @@ -1327,7 +1501,7 @@ rte_pmd_af_xdp_probe(struct rte_vdev_device *dev) dev->device.numa_node = rte_socket_id(); if (parse_parameters(kvlist, if_name, &xsk_start_queue_idx, - &xsk_queue_cnt) < 0) { + &xsk_queue_cnt, &shared_umem) < 0) { AF_XDP_LOG(ERR, "Invalid kvargs value\n"); return -EINVAL; } @@ -1338,7 +1512,7 @@ rte_pmd_af_xdp_probe(struct rte_vdev_device *dev) } eth_dev = init_internals(dev, if_name, xsk_start_queue_idx, - xsk_queue_cnt); + xsk_queue_cnt, shared_umem); if (eth_dev == NULL) { AF_XDP_LOG(ERR, "Failed to init internals\n"); return -1; @@ -1381,4 +1555,5 @@ RTE_PMD_REGISTER_VDEV(net_af_xdp, pmd_af_xdp_drv); RTE_PMD_REGISTER_PARAM_STRING(net_af_xdp, "iface= " "start_queue= " - "queue_count= "); + "queue_count= " + "shared_umem= "); -- 2.20.1