From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shreyansh Jain Subject: Re: error in testpmd when CONFIG_RTE_BUILD_SHARED_LIB=y Date: Wed, 12 Apr 2017 10:33:10 +0000 Message-ID: References: <357262EF-AA7B-4064-963E-FF130E4607E6@atomicrules.com> <1675520.qVeAGBHxLM@xps13> <20170412100521.GB20092@bricha3-MOBL3.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Cc: Thomas Monjalon , john miller , "dev@dpdk.org" , "olivier.matz@6wind.com" To: Bruce Richardson Return-path: Received: from EUR01-VE1-obe.outbound.protection.outlook.com (mail-ve1eur01on0049.outbound.protection.outlook.com [104.47.1.49]) by dpdk.org (Postfix) with ESMTP id 2D1C62E8A for ; Wed, 12 Apr 2017 12:33:27 +0200 (CEST) In-Reply-To: Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" My bad - I was too quick in replying - some clarification beneath. > -----Original Message----- > From: Shreyansh Jain > Sent: Wednesday, April 12, 2017 3:55 PM > To: 'Bruce Richardson' > Cc: Thomas Monjalon ; john miller > ; dev@dpdk.org; olivier.matz@6wind.com > Subject: RE: [dpdk-dev] error in testpmd when CONFIG_RTE_BUILD_SHARED_LIB= =3Dy >=20 > > -----Original Message----- > > From: Bruce Richardson [mailto:bruce.richardson@intel.com] > > Sent: Wednesday, April 12, 2017 3:35 PM > > To: Shreyansh Jain > > Cc: Thomas Monjalon ; john miller > > ; dev@dpdk.org; olivier.matz@6wind.com > > Subject: Re: [dpdk-dev] error in testpmd when CONFIG_RTE_BUILD_SHARED_L= IB=3Dy > > > > On Wed, Apr 12, 2017 at 04:52:47AM +0000, Shreyansh Jain wrote: > > > > -----Original Message----- > > > > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > > > > Sent: Wednesday, April 12, 2017 12:58 AM > > > > To: john miller > > > > Cc: dev@dpdk.org; olivier.matz@6wind.com; Shreyansh Jain > > > > > > > > Subject: Re: [dpdk-dev] error in testpmd when > > CONFIG_RTE_BUILD_SHARED_LIB=3Dy > > > > > > > > 2017-04-11 14:02, john miller: > > > > > > > > > > We are seeing an issue when running from the head of the master > branch > > in > > > > dpdk-next-net and building with CONFIG_RTE_BUILD_SHARED_LIB=3Dy. = When > we > > run > > > > testpmd using -d to point to our PMD we get this error > > > > > > > > > > EAL: Error - exiting with code: 1 > > > > > Cause: Creation of mbuf pool for socket 0 failed: Invalid argum= ent > > > > > > > > > > This error occurs as a result of the rte mempool ops table having= 0 > > > > entries. This table is populated from a call to > > rte_mempool_register_ops(). > > > > This function gets called in rte_mempool_ring.c via the static > > initialization > > > > MACRO MEMPOOL_REGISTER_OPS and exists in librte_mempool_ring.so. > However > > > > this library is not loaded when the rte_eal_init() gets called so t= he > > static > > > > initializers are not yet loaded. > > > > > > > > > > I am requesting advice on the proper way to repair this. > > > > > > "-d" the ring library (rte_mempool_ring) - just like any other shared > lib. > > > > > > > I think this is a bug that should be fixed. The user should not need to > > have to specify a mempool driver just to get testpmd working, so I thin= k > > the ring handler as default should be compiled in automatically so as t= o > > allow regular mempools to just work as before. >=20 > For Ring Mempool as default enabled, +1 =20 Actually, Ring mempool is enabled by default. But, obviously for shared lib= rary case, this still means "-d". >=20 > > > > > This change was done recently to move ring handler into its separate > > drivers/mempool/ring directory. That also means it no longer is compile= d > into > > the librte_mempool. > > > > > > > > > > > We should just add a better error message if no mempool driver is > > available. > > > > > > Yes, that is something to be improved. > > > > This should be fixed by always having a mempool driver installed. >=20 > Agree. =20 Probably, as ring mempool is a driver and compiled in shared mode, enabled = by default will not fix this. > Otherwise also, the framework seems to be replying with a valid code 'EIN= VAL' > for an invalid mempool specified. >=20 > > > > /Bruce