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 9271CA046B for ; Fri, 23 Aug 2019 11:44:26 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 89F9A1BF9B; Fri, 23 Aug 2019 11:44:26 +0200 (CEST) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 8F51C1BF9B for ; Fri, 23 Aug 2019 11:44:24 +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 F3B5F88FFF8; Fri, 23 Aug 2019 09:44:22 +0000 (UTC) Received: from rh.redhat.com (ovpn-116-48.ams2.redhat.com [10.36.116.48]) by smtp.corp.redhat.com (Postfix) with ESMTP id A1C4E5F9D3; Fri, 23 Aug 2019 09:44:17 +0000 (UTC) From: Kevin Traynor To: David Marchand Cc: Adrien Mazarguil , Stephen Hemminger , Neil Horman , dpdk stable Date: Fri, 23 Aug 2019 10:43:02 +0100 Message-Id: <20190823094336.12078-12-ktraynor@redhat.com> In-Reply-To: <20190823094336.12078-1-ktraynor@redhat.com> References: <20190823094336.12078-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.6.2 (mx1.redhat.com [10.5.110.68]); Fri, 23 Aug 2019 09:44:24 +0000 (UTC) Subject: [dpdk-stable] patch 'eal: hide internal hotplug function' 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 08/28/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/d2d1dbb6f51aadad579a3f0fac8d1589ca99f67a Thanks. Kevin Traynor --- >From d2d1dbb6f51aadad579a3f0fac8d1589ca99f67a Mon Sep 17 00:00:00 2001 From: David Marchand Date: Sat, 29 Jun 2019 13:58:44 +0200 Subject: [PATCH] eal: hide internal hotplug function [ upstream commit f319d9937977479528870c6859bd3a227c212f75 ] This API was experimental and not properly marked in the map file. But looking more closely, this is just an internal wrapper for EAL init. Hide it in the hotplug code. Fixes: 244d5130719c ("eal: enable hotplug on multi-process") Signed-off-by: David Marchand Acked-by: Adrien Mazarguil Acked-by: Stephen Hemminger Acked-by: Neil Horman --- lib/librte_eal/common/hotplug_mp.c | 2 +- lib/librte_eal/common/hotplug_mp.h | 9 +++++++++ lib/librte_eal/common/include/rte_eal.h | 9 --------- lib/librte_eal/linuxapp/eal/eal.c | 3 ++- 4 files changed, 12 insertions(+), 11 deletions(-) diff --git a/lib/librte_eal/common/hotplug_mp.c b/lib/librte_eal/common/hotplug_mp.c index 7c3f38dba..287f3df0b 100644 --- a/lib/librte_eal/common/hotplug_mp.c +++ b/lib/librte_eal/common/hotplug_mp.c @@ -419,5 +419,5 @@ int eal_dev_hotplug_request_to_secondary(struct eal_dev_mp_req *req) } -int rte_mp_dev_hotplug_init(void) +int eal_mp_dev_hotplug_init(void) { int ret; diff --git a/lib/librte_eal/common/hotplug_mp.h b/lib/librte_eal/common/hotplug_mp.h index 597fde3d7..8fcf9b52e 100644 --- a/lib/librte_eal/common/hotplug_mp.h +++ b/lib/librte_eal/common/hotplug_mp.h @@ -29,4 +29,13 @@ struct eal_dev_mp_req { }; +/** + * Register all mp action callbacks for hotplug. + * + * @return + * 0 on success, negative on error. + */ +int +eal_mp_dev_hotplug_init(void); + /** * This is a synchronous wrapper for secondary process send diff --git a/lib/librte_eal/common/include/rte_eal.h b/lib/librte_eal/common/include/rte_eal.h index 9951228e0..595efdca5 100644 --- a/lib/librte_eal/common/include/rte_eal.h +++ b/lib/librte_eal/common/include/rte_eal.h @@ -400,13 +400,4 @@ int __rte_experimental rte_mp_reply(struct rte_mp_msg *msg, const char *peer); -/** - * Register all mp action callbacks for hotplug. - * - * @return - * 0 on success, negative on error. - */ -int __rte_experimental -rte_mp_dev_hotplug_init(void); - /** * Usage function typedef used by the application usage function. diff --git a/lib/librte_eal/linuxapp/eal/eal.c b/lib/librte_eal/linuxapp/eal/eal.c index fa97b1154..f453337f7 100644 --- a/lib/librte_eal/linuxapp/eal/eal.c +++ b/lib/librte_eal/linuxapp/eal/eal.c @@ -60,4 +60,5 @@ #include "eal_options.h" #include "eal_vfio.h" +#include "hotplug_mp.h" #define MEMSIZE_IF_NO_HUGE_PAGE (64ULL * 1024ULL * 1024ULL) @@ -1019,5 +1020,5 @@ rte_eal_init(int argc, char **argv) /* register multi-process action callbacks for hotplug */ - if (rte_mp_dev_hotplug_init() < 0) { + if (eal_mp_dev_hotplug_init() < 0) { rte_eal_init_alert("failed to register mp callback for hotplug"); return -1; -- 2.20.1 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2019-08-22 19:38:21.257554824 +0100 +++ 0012-eal-hide-internal-hotplug-function.patch 2019-08-22 19:38:20.407027595 +0100 @@ -1 +1 @@ -From f319d9937977479528870c6859bd3a227c212f75 Mon Sep 17 00:00:00 2001 +From d2d1dbb6f51aadad579a3f0fac8d1589ca99f67a Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit f319d9937977479528870c6859bd3a227c212f75 ] + @@ -11 +12,0 @@ -Cc: stable@dpdk.org @@ -21 +22 @@ - lib/librte_eal/linux/eal/eal.c | 3 ++- + lib/librte_eal/linuxapp/eal/eal.c | 3 ++- @@ -25 +26 @@ -index 4f316ba9a..ee791903b 100644 +index 7c3f38dba..287f3df0b 100644 @@ -28 +29 @@ -@@ -439,5 +439,5 @@ int eal_dev_hotplug_request_to_secondary(struct eal_dev_mp_req *req) +@@ -419,5 +419,5 @@ int eal_dev_hotplug_request_to_secondary(struct eal_dev_mp_req *req) @@ -54 +55 @@ -index cf701e177..27808a336 100644 +index 9951228e0..595efdca5 100644 @@ -57 +58 @@ -@@ -409,13 +409,4 @@ int __rte_experimental +@@ -400,13 +400,4 @@ int __rte_experimental @@ -71,4 +72,4 @@ -diff --git a/lib/librte_eal/linux/eal/eal.c b/lib/librte_eal/linux/eal/eal.c -index aa0137ecd..8a0b387ce 100644 ---- a/lib/librte_eal/linux/eal/eal.c -+++ b/lib/librte_eal/linux/eal/eal.c +diff --git a/lib/librte_eal/linuxapp/eal/eal.c b/lib/librte_eal/linuxapp/eal/eal.c +index fa97b1154..f453337f7 100644 +--- a/lib/librte_eal/linuxapp/eal/eal.c ++++ b/lib/librte_eal/linuxapp/eal/eal.c @@ -81 +82 @@ -@@ -1063,5 +1064,5 @@ rte_eal_init(int argc, char **argv) +@@ -1019,5 +1020,5 @@ rte_eal_init(int argc, char **argv)