From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by dpdk.space (Postfix) with ESMTP id 98EDDA046B for ; Mon, 24 Jun 2019 17:26:32 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 90C661BEEC; Mon, 24 Jun 2019 17:26:32 +0200 (CEST) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 919111BEEA for ; Mon, 24 Jun 2019 17:26:31 +0200 (CEST) Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 032E0882FB; Mon, 24 Jun 2019 15:26:26 +0000 (UTC) Received: from rh.redhat.com (ovpn-116-250.ams2.redhat.com [10.36.116.250]) by smtp.corp.redhat.com (Postfix) with ESMTP id 0DAAA19936; Mon, 24 Jun 2019 15:26:22 +0000 (UTC) From: Kevin Traynor To: Michal Krawczyk Cc: David Marchand , dpdk stable Date: Mon, 24 Jun 2019 16:24:43 +0100 Message-Id: <20190624152525.19349-19-ktraynor@redhat.com> In-Reply-To: <20190624152525.19349-1-ktraynor@redhat.com> References: <20190624152525.19349-1-ktraynor@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.28]); Mon, 24 Jun 2019 15:26:31 +0000 (UTC) Subject: [dpdk-stable] patch 'net/ena: fix assigning NUMA node to IO queue' has been queued to LTS release 18.11.3 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" Hi, FYI, your patch has been queued to LTS release 18.11.3 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 06/27/19. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Queued patches are on a temporary branch at: https://github.com/kevintraynor/dpdk-stable-queue This queued commit can be viewed at: https://github.com/kevintraynor/dpdk-stable-queue/commit/6043b435acec7e04a274308980736e4a37829f40 Thanks. Kevin Traynor --- >From 6043b435acec7e04a274308980736e4a37829f40 Mon Sep 17 00:00:00 2001 From: Michal Krawczyk Date: Tue, 4 Jun 2019 12:59:36 +0200 Subject: [PATCH] net/ena: fix assigning NUMA node to IO queue [ upstream commit 4217cb0b7d2c5385f06f531af7f14b860927aba7 ] Previous solution was using memzones in invalid way in hope to assign IO queue to the appropriate NUMA zone. The right way is to use socket_id from the rx/tx queue setup function and then pass it to the IO queue. Fixes: 3d3edc265fc8 ("net/ena: make coherent memory allocation NUMA-aware") Signed-off-by: Michal Krawczyk Reviewed-by: David Marchand --- drivers/net/ena/ena_ethdev.c | 25 +++++-------------------- drivers/net/ena/ena_ethdev.h | 1 + 2 files changed, 6 insertions(+), 20 deletions(-) diff --git a/drivers/net/ena/ena_ethdev.c b/drivers/net/ena/ena_ethdev.c index 82161f41c..9359d4d76 100644 --- a/drivers/net/ena/ena_ethdev.c +++ b/drivers/net/ena/ena_ethdev.c @@ -40,5 +40,4 @@ #include #include -#include #include @@ -282,20 +281,4 @@ static const struct eth_dev_ops ena_dev_ops = { }; -#define NUMA_NO_NODE SOCKET_ID_ANY - -static inline int ena_cpu_to_node(int cpu) -{ - struct rte_config *config = rte_eal_get_configuration(); - struct rte_fbarray *arr = &config->mem_config->memzones; - const struct rte_memzone *mz; - - if (unlikely(cpu >= RTE_MAX_MEMZONE)) - return NUMA_NO_NODE; - - mz = rte_fbarray_get(arr, cpu); - - return mz->socket_id; -} - static inline void ena_rx_mbuf_prepare(struct rte_mbuf *mbuf, struct ena_com_rx_ctx *ena_rx_ctx) @@ -1072,5 +1055,5 @@ static int ena_create_io_queue(struct ena_ring *ring) ctx.qid = ena_qid; ctx.msix_vector = -1; /* interrupts not used */ - ctx.numa_node = ena_cpu_to_node(ring->id); + ctx.numa_node = ring->numa_socket_id; rc = ena_com_create_io_queue(ena_dev, &ctx); @@ -1166,5 +1149,5 @@ static int ena_tx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx, uint16_t nb_desc, - __rte_unused unsigned int socket_id, + unsigned int socket_id, const struct rte_eth_txconf *tx_conf) { @@ -1200,4 +1183,5 @@ static int ena_tx_queue_setup(struct rte_eth_dev *dev, txq->next_to_use = 0; txq->ring_size = nb_desc; + txq->numa_socket_id = socket_id; txq->tx_buffer_info = rte_zmalloc("txq->tx_buffer_info", @@ -1237,5 +1221,5 @@ static int ena_rx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx, uint16_t nb_desc, - __rte_unused unsigned int socket_id, + unsigned int socket_id, __rte_unused const struct rte_eth_rxconf *rx_conf, struct rte_mempool *mp) @@ -1271,4 +1255,5 @@ static int ena_rx_queue_setup(struct rte_eth_dev *dev, rxq->next_to_use = 0; rxq->ring_size = nb_desc; + rxq->numa_socket_id = socket_id; rxq->mb_pool = mp; diff --git a/drivers/net/ena/ena_ethdev.h b/drivers/net/ena/ena_ethdev.h index 322e90ace..a4aba7fdf 100644 --- a/drivers/net/ena/ena_ethdev.h +++ b/drivers/net/ena/ena_ethdev.h @@ -106,4 +106,5 @@ struct ena_ring { uint64_t offloads; u16 sgl_size; + unsigned int numa_socket_id; } __rte_cache_aligned; -- 2.20.1 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2019-06-24 16:18:56.068499662 +0100 +++ 0019-net-ena-fix-assigning-NUMA-node-to-IO-queue.patch 2019-06-24 16:18:54.997432026 +0100 @@ -1 +1 @@ -From 4217cb0b7d2c5385f06f531af7f14b860927aba7 Mon Sep 17 00:00:00 2001 +From 6043b435acec7e04a274308980736e4a37829f40 Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit 4217cb0b7d2c5385f06f531af7f14b860927aba7 ] + @@ -13 +14,0 @@ -Cc: stable@dpdk.org @@ -19,2 +20,2 @@ - drivers/net/ena/ena_ethdev.h | 2 ++ - 2 files changed, 7 insertions(+), 20 deletions(-) + drivers/net/ena/ena_ethdev.h | 1 + + 2 files changed, 6 insertions(+), 20 deletions(-) @@ -23 +24 @@ -index a42ec8c79..f58334080 100644 +index 82161f41c..9359d4d76 100644 @@ -26 +27 @@ -@@ -41,5 +41,4 @@ +@@ -40,5 +40,4 @@ @@ -32 +33 @@ -@@ -273,20 +272,4 @@ static const struct eth_dev_ops ena_dev_ops = { +@@ -282,20 +281,4 @@ static const struct eth_dev_ops ena_dev_ops = { @@ -53 +54 @@ -@@ -1117,5 +1100,5 @@ static int ena_create_io_queue(struct ena_ring *ring) +@@ -1072,5 +1055,5 @@ static int ena_create_io_queue(struct ena_ring *ring) @@ -60 +61 @@ -@@ -1214,5 +1197,5 @@ static int ena_tx_queue_setup(struct rte_eth_dev *dev, +@@ -1166,5 +1149,5 @@ static int ena_tx_queue_setup(struct rte_eth_dev *dev, @@ -67 +68 @@ -@@ -1251,4 +1234,5 @@ static int ena_tx_queue_setup(struct rte_eth_dev *dev, +@@ -1200,4 +1183,5 @@ static int ena_tx_queue_setup(struct rte_eth_dev *dev, @@ -73 +74 @@ -@@ -1298,5 +1282,5 @@ static int ena_rx_queue_setup(struct rte_eth_dev *dev, +@@ -1237,5 +1221,5 @@ static int ena_rx_queue_setup(struct rte_eth_dev *dev, @@ -80 +81 @@ -@@ -1335,4 +1319,5 @@ static int ena_rx_queue_setup(struct rte_eth_dev *dev, +@@ -1271,4 +1255,5 @@ static int ena_rx_queue_setup(struct rte_eth_dev *dev, @@ -87 +88 @@ -index dcc8690a5..9067e9007 100644 +index 322e90ace..a4aba7fdf 100644 @@ -90,4 +91,3 @@ -@@ -144,4 +144,6 @@ struct ena_ring { - struct ena_stats_tx tx_stats; - }; -+ +@@ -106,4 +106,5 @@ struct ena_ring { + uint64_t offloads; + u16 sgl_size;