From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 2FDD7A0613 for ; Tue, 27 Aug 2019 11:31:58 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 23E3F1BFE1; Tue, 27 Aug 2019 11:31:58 +0200 (CEST) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 42E661C0C6 for ; Tue, 27 Aug 2019 11:31:56 +0200 (CEST) Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id BB8DC30821C2; Tue, 27 Aug 2019 09:31:55 +0000 (UTC) Received: from rh.redhat.com (ovpn-117-48.ams2.redhat.com [10.36.117.48]) by smtp.corp.redhat.com (Postfix) with ESMTP id E30245C1B2; Tue, 27 Aug 2019 09:31:54 +0000 (UTC) From: Kevin Traynor To: Marcin Zapolski Cc: dpdk stable Date: Tue, 27 Aug 2019 10:30:27 +0100 Message-Id: <20190827093032.20423-50-ktraynor@redhat.com> In-Reply-To: <20190827093032.20423-1-ktraynor@redhat.com> References: <20190827093032.20423-1-ktraynor@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Scanned-By: MIMEDefang 2.79 on 10.5.11.16 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.47]); Tue, 27 Aug 2019 09:31:55 +0000 (UTC) Subject: [dpdk-stable] patch 'examples: fix use of ethdev internal device array' 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 09/03/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/6a734ed87214cd1018864f7ce006a1f2b13f956f Thanks. Kevin Traynor --- >From 6a734ed87214cd1018864f7ce006a1f2b13f956f Mon Sep 17 00:00:00 2001 From: Marcin Zapolski Date: Wed, 17 Jul 2019 09:58:04 +0200 Subject: [PATCH] examples: fix use of ethdev internal device array [ upstream commit 5c5c1f99a923986b5b2d2d34e874ca14ffabbcb0 ] Modify l3fwd and related example apps to use locally defined port_conf instead of global rte_eth_devices which is not a part of public API. Apps should not be using internal DPDK data structures directly. Fixes: 1ef9600b2d20 ("examples/l3fwd: convert to ethdev offloads API") Fixes: ba8c103d2455 ("examples/l3fwd-acl: convert to new ethdev offloads API") Fixes: 40df1d7a695d ("examples/l3fwd-power: convert to new ethdev offloads API") Fixes: 43fc038262b8 ("examples/l3fwd-vf: convert to new ethdev offloads API") Fixes: 373149c631fe ("examples/performance-thread: convert to new offloads API") Signed-off-by: Marcin Zapolski --- examples/l3fwd-acl/main.c | 6 +----- examples/l3fwd-power/main.c | 6 +----- examples/l3fwd-vf/main.c | 7 +------ examples/l3fwd/main.c | 6 +----- examples/performance-thread/l3fwd-thread/main.c | 6 +----- 5 files changed, 5 insertions(+), 26 deletions(-) diff --git a/examples/l3fwd-acl/main.c b/examples/l3fwd-acl/main.c index a322ce4f2..8ed0c07ec 100644 --- a/examples/l3fwd-acl/main.c +++ b/examples/l3fwd-acl/main.c @@ -2019,12 +2019,8 @@ main(int argc, char **argv) /* init RX queues */ for (queue = 0; queue < qconf->n_rx_queue; ++queue) { - struct rte_eth_dev *dev; - struct rte_eth_conf *conf; struct rte_eth_rxconf rxq_conf; portid = qconf->rx_queue_list[queue].port_id; queueid = qconf->rx_queue_list[queue].queue_id; - dev = &rte_eth_devices[portid]; - conf = &dev->data->dev_conf; if (numa_on) @@ -2039,5 +2035,5 @@ main(int argc, char **argv) rte_eth_dev_info_get(portid, &dev_info); rxq_conf = dev_info.default_rxconf; - rxq_conf.offloads = conf->rxmode.offloads; + rxq_conf.offloads = port_conf.rxmode.offloads; ret = rte_eth_rx_queue_setup(portid, queueid, nb_rxd, socketid, &rxq_conf, diff --git a/examples/l3fwd-power/main.c b/examples/l3fwd-power/main.c index 9c7b31564..3421de8ec 100644 --- a/examples/l3fwd-power/main.c +++ b/examples/l3fwd-power/main.c @@ -2119,11 +2119,7 @@ main(int argc, char **argv) for(queue = 0; queue < qconf->n_rx_queue; ++queue) { struct rte_eth_rxconf rxq_conf; - struct rte_eth_dev *dev; - struct rte_eth_conf *conf; portid = qconf->rx_queue_list[queue].port_id; queueid = qconf->rx_queue_list[queue].queue_id; - dev = &rte_eth_devices[portid]; - conf = &dev->data->dev_conf; if (numa_on) @@ -2138,5 +2134,5 @@ main(int argc, char **argv) rte_eth_dev_info_get(portid, &dev_info); rxq_conf = dev_info.default_rxconf; - rxq_conf.offloads = conf->rxmode.offloads; + rxq_conf.offloads = port_conf.rxmode.offloads; ret = rte_eth_rx_queue_setup(portid, queueid, nb_rxd, socketid, &rxq_conf, diff --git a/examples/l3fwd-vf/main.c b/examples/l3fwd-vf/main.c index 036f86854..ce052544f 100644 --- a/examples/l3fwd-vf/main.c +++ b/examples/l3fwd-vf/main.c @@ -1023,11 +1023,6 @@ main(int argc, char **argv) /* init RX queues */ for(queue = 0; queue < qconf->n_rx_queue; ++queue) { - struct rte_eth_dev *dev; - struct rte_eth_conf *conf; - portid = qconf->rx_queue_list[queue].port_id; queueid = qconf->rx_queue_list[queue].queue_id; - dev = &rte_eth_devices[portid]; - conf = &dev->data->dev_conf; if (numa_on) @@ -1041,5 +1036,5 @@ main(int argc, char **argv) rte_eth_dev_info_get(portid, &dev_info); rxq_conf = dev_info.default_rxconf; - rxq_conf.offloads = conf->rxmode.offloads; + rxq_conf.offloads = port_conf.rxmode.offloads; ret = rte_eth_rx_queue_setup(portid, queueid, nb_rxd, socketid, &rxq_conf, diff --git a/examples/l3fwd/main.c b/examples/l3fwd/main.c index e4b99efe0..71a67f422 100644 --- a/examples/l3fwd/main.c +++ b/examples/l3fwd/main.c @@ -946,12 +946,8 @@ main(int argc, char **argv) /* init RX queues */ for(queue = 0; queue < qconf->n_rx_queue; ++queue) { - struct rte_eth_dev *dev; - struct rte_eth_conf *conf; struct rte_eth_rxconf rxq_conf; portid = qconf->rx_queue_list[queue].port_id; queueid = qconf->rx_queue_list[queue].queue_id; - dev = &rte_eth_devices[portid]; - conf = &dev->data->dev_conf; if (numa_on) @@ -966,5 +962,5 @@ main(int argc, char **argv) rte_eth_dev_info_get(portid, &dev_info); rxq_conf = dev_info.default_rxconf; - rxq_conf.offloads = conf->rxmode.offloads; + rxq_conf.offloads = port_conf.rxmode.offloads; ret = rte_eth_rx_queue_setup(portid, queueid, nb_rxd, socketid, diff --git a/examples/performance-thread/l3fwd-thread/main.c b/examples/performance-thread/l3fwd-thread/main.c index 4f8747bc3..8ddcfa2f4 100644 --- a/examples/performance-thread/l3fwd-thread/main.c +++ b/examples/performance-thread/l3fwd-thread/main.c @@ -3635,12 +3635,8 @@ main(int argc, char **argv) /* init RX queues */ for (queue = 0; queue < rx_thread[i].n_rx_queue; ++queue) { - struct rte_eth_dev *dev; - struct rte_eth_conf *conf; struct rte_eth_rxconf rxq_conf; portid = rx_thread[i].rx_queue_list[queue].port_id; queueid = rx_thread[i].rx_queue_list[queue].queue_id; - dev = &rte_eth_devices[portid]; - conf = &dev->data->dev_conf; if (numa_on) @@ -3654,5 +3650,5 @@ main(int argc, char **argv) rte_eth_dev_info_get(portid, &dev_info); rxq_conf = dev_info.default_rxconf; - rxq_conf.offloads = conf->rxmode.offloads; + rxq_conf.offloads = port_conf.rxmode.offloads; ret = rte_eth_rx_queue_setup(portid, queueid, nb_rxd, socketid, -- 2.20.1 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2019-08-27 09:40:13.634769662 +0100 +++ 0050-examples-fix-use-of-ethdev-internal-device-array.patch 2019-08-27 09:40:10.959143441 +0100 @@ -1 +1 @@ -From 5c5c1f99a923986b5b2d2d34e874ca14ffabbcb0 Mon Sep 17 00:00:00 2001 +From 6a734ed87214cd1018864f7ce006a1f2b13f956f Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit 5c5c1f99a923986b5b2d2d34e874ca14ffabbcb0 ] + @@ -15 +16,0 @@ -Cc: stable@dpdk.org @@ -27 +28 @@ -index 09b7c3850..0c44df767 100644 +index a322ce4f2..8ed0c07ec 100644 @@ -30 +31 @@ -@@ -2023,12 +2023,8 @@ main(int argc, char **argv) +@@ -2019,12 +2019,8 @@ main(int argc, char **argv) @@ -43 +44 @@ -@@ -2043,5 +2039,5 @@ main(int argc, char **argv) +@@ -2039,5 +2035,5 @@ main(int argc, char **argv) @@ -51 +52 @@ -index cbe8f8e3f..fd8d9528f 100644 +index 9c7b31564..3421de8ec 100644 @@ -54 +55 @@ -@@ -2386,11 +2386,7 @@ main(int argc, char **argv) +@@ -2119,11 +2119,7 @@ main(int argc, char **argv) @@ -66 +67 @@ -@@ -2405,5 +2401,5 @@ main(int argc, char **argv) +@@ -2138,5 +2134,5 @@ main(int argc, char **argv) @@ -74 +75 @@ -index 1432f8969..572e74cf5 100644 +index 036f86854..ce052544f 100644 @@ -97 +98 @@ -index 570693b32..3800bad19 100644 +index e4b99efe0..71a67f422 100644 @@ -100 +101 @@ -@@ -972,12 +972,8 @@ main(int argc, char **argv) +@@ -946,12 +946,8 @@ main(int argc, char **argv) @@ -113 +114 @@ -@@ -992,5 +988,5 @@ main(int argc, char **argv) +@@ -966,5 +962,5 @@ main(int argc, char **argv) @@ -118,2 +119,2 @@ - if (!per_port_pool) - ret = rte_eth_rx_queue_setup(portid, queueid, + ret = rte_eth_rx_queue_setup(portid, queueid, nb_rxd, + socketid, @@ -121 +122 @@ -index dd468958a..0121c71cb 100644 +index 4f8747bc3..8ddcfa2f4 100644 @@ -124 +125 @@ -@@ -3646,12 +3646,8 @@ main(int argc, char **argv) +@@ -3635,12 +3635,8 @@ main(int argc, char **argv) @@ -137 +138 @@ -@@ -3665,5 +3661,5 @@ main(int argc, char **argv) +@@ -3654,5 +3650,5 @@ main(int argc, char **argv)