From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerin Jacob Subject: Re: [PATCH v3 2/7] service cores: EAL init changes Date: Tue, 4 Jul 2017 17:05:32 +0530 Message-ID: <20170704113531.GA14921@jerin> References: <1498735421-100164-1-git-send-email-harry.van.haaren@intel.com> <1499031314-7172-1-git-send-email-harry.van.haaren@intel.com> <1499031314-7172-3-git-send-email-harry.van.haaren@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org, thomas@monjalon.net, keith.wiles@intel.com, bruce.richardson@intel.com To: Harry van Haaren Return-path: Received: from NAM01-BN3-obe.outbound.protection.outlook.com (mail-bn3nam01on0066.outbound.protection.outlook.com [104.47.33.66]) by dpdk.org (Postfix) with ESMTP id 2B5982C8 for ; Tue, 4 Jul 2017 13:35:51 +0200 (CEST) Content-Disposition: inline In-Reply-To: <1499031314-7172-3-git-send-email-harry.van.haaren@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" -----Original Message----- > Date: Sun, 2 Jul 2017 22:35:09 +0100 > From: Harry van Haaren > To: dev@dpdk.org > CC: jerin.jacob@caviumnetworks.com, thomas@monjalon.net, > keith.wiles@intel.com, bruce.richardson@intel.com, Harry van Haaren > > Subject: [PATCH v3 2/7] service cores: EAL init changes > X-Mailer: git-send-email 2.7.4 > > This commit shows the changes required in rte_eal_init() > to transparently launch the service threads. The threads > are launched into the service worker functions here because > after rte_eal_init() the application is not gauranteed to > call any other DPDK API. > > As the registration of services happens at initialization > time, the services that require CPU time are already available > when we reach the end of rte_eal_init(). > > Signed-off-by: Harry van Haaren > > --- > > v2 comments: > - Include BSD implementation (Jerin) > - Move details of core-tracking into rte_service_lcore_add(Jerin) > - Given there are changes other to suggested, not using Ack > --- > lib/librte_eal/bsdapp/eal/eal.c | 22 ++++++++++++++++++++++ > lib/librte_eal/linuxapp/eal/eal.c | 23 +++++++++++++++++++++++ > 2 files changed, 45 insertions(+) > > diff --git a/lib/librte_eal/bsdapp/eal/eal.c b/lib/librte_eal/bsdapp/eal/eal.c > index 05f0c1f..4f7dcb3 100644 > --- a/lib/librte_eal/bsdapp/eal/eal.c > +++ b/lib/librte_eal/bsdapp/eal/eal.c > @@ -653,6 +653,17 @@ rte_eal_init(int argc, char **argv) > rte_eal_mp_remote_launch(sync_func, NULL, SKIP_MASTER); > rte_eal_mp_wait_lcore(); > > + /* initialize services first so vdevs can register during bus_probe. > + * Ignore return value of already initialized, this means EAL parameter > + * -s was used to set a service-core mask. > + */ > + ret = rte_service_init(); > + if (ret) { > + rte_eal_init_alert("rte_service_init() failed\n"); > + rte_errno = ENOEXEC; > + return -1; > + } > + > /* Probe all the buses and devices/drivers on them */ > if (rte_bus_probe()) { > rte_eal_init_alert("Cannot probe devices\n"); > @@ -660,6 +671,17 @@ rte_eal_init(int argc, char **argv) > return -1; > } > > + /* initialize default services configuration */ > + uint32_t service_cores[RTE_MAX_LCORE]; > + int count = rte_service_lcore_list(service_cores, RTE_MAX_LCORE); > + for (i = 0; i < count; i++) > + rte_service_lcore_start(service_cores[i]); > + ret = rte_service_set_default_mapping(); > + if (ret) { > + rte_errno = ENOEXEC; > + return -1; > + } > + > rte_eal_mcfg_complete(); > > return fctret; > diff --git a/lib/librte_eal/linuxapp/eal/eal.c b/lib/librte_eal/linuxapp/eal/eal.c > index 7c78f2d..d63dd87 100644 > --- a/lib/librte_eal/linuxapp/eal/eal.c > +++ b/lib/librte_eal/linuxapp/eal/eal.c > @@ -78,6 +78,7 @@ > #include > #include > #include > +#include Not included this header file for bsdapp compilation. It should fail to compile. Right?