From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by dpdk.org (Postfix) with ESMTP id CF7C61B19 for ; Mon, 23 Apr 2018 00:29:57 +0200 (CEST) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id 3B22821B38; Sun, 22 Apr 2018 18:29:57 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute1.internal (MEProxy); Sun, 22 Apr 2018 18:29:57 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-sender :x-me-sender:x-sasl-enc; s=mesmtp; bh=VFs5uUt8ZqRUQiX+5YSKXZ7GG1 kVfuZQUVRR9+5uwNI=; b=mIH22ZbdbztaNpskro8BzgNHs6LhkEBKmJ1xWn0hzN rcxmMv0SgY7dt9bwg6z+RmMIU2Rn15LLXVVvpsdrESRvKSi4U+gQ+rLfjNUPmdXC h8blrDxyG2WhgXKLrecSowAliLBGqRK8UT6hrfUou/s/3klaYCnXmlIgZB1MxJLF E= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-sender:x-me-sender:x-sasl-enc; s=fm2; bh=VFs5uU t8ZqRUQiX+5YSKXZ7GG1kVfuZQUVRR9+5uwNI=; b=FThjuVbbuESuzH9OeIL8zK uuHheWbK3Mce3gUVxzlsphRllK1tQTXFnaJDkCBfLosx4PwDtOj+RQetyxq+e0Fe bMd35tjJltfWBOzSUSlHZ/tF2jwXkFE8x+tRcj8dzwCi4/bk0fL6x24VNkrZIOGx NyAXotO7qHI0bEjmTnVcGZqa65Waank/YCzNEpN7K2RJngYMvGsEk6GZzT0J7RKI 5vE8/IrVQf7f68eQ2sdwUQwaBjgNmb2mtR6JLcBZdSyX6AUaezirQIEIe9egBNBG 9yQDj412lI6Th4aWe5GzDqsu3gjUpQbhNq9PIKwHigHSKtHccW24FmQtZWg1bIMA == X-ME-Sender: Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id 6BB5EE4122; Sun, 22 Apr 2018 18:29:56 -0400 (EDT) From: Thomas Monjalon To: dev@dpdk.org Cc: Neil Horman , =?ISO-8859-1?Q?Ga=EBtan?= Rivet , "Richardson, Bruce" , "Wiles, Keith" Date: Mon, 23 Apr 2018 00:29:55 +0200 Message-ID: <3840634.tRnGvbobzp@xps> In-Reply-To: <20180328111730.GA9514@hmswarspite.think-freely.org> References: <20180328081007.balvfgy5cwnxdd7k@bidouze.vm.6wind.com> <20180328111730.GA9514@hmswarspite.think-freely.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="iso-8859-1" Subject: Re: [dpdk-dev] [PATCH v3 10/20] eal/dev: implement device iteration initialization X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 22 Apr 2018 22:29:58 -0000 28/03/2018 13:17, Neil Horman: > On Wed, Mar 28, 2018 at 10:10:07AM +0200, Ga=EBtan Rivet wrote: > > On Tue, Mar 27, 2018 at 07:53:46PM -0400, Neil Horman wrote: > > > On Tue, Mar 27, 2018 at 08:48:01PM +0000, Richardson, Bruce wrote: > > > > From: Ga=EBtan Rivet [mailto:gaetan.rivet@6wind.com] > > > > > On Tue, Mar 27, 2018 at 09:28:07PM +0100, Bruce Richardson wrote: > > > > > > On Tue, Mar 27, 2018 at 10:20:40PM +0200, Ga=EBtan Rivet wrote: > > > > > > > Neat. The issue is that rte_log.h is not symlink-ed until lib= rte_eal > > > > > > > is processed. rte_log cannot be included. > > > > > > > > > > > > > Sure it can - just pass -I/path/to/eal as a cflag. > > > > > > > > > > > > /Bruce > > > > >=20 > > > > > When you put it this way... :) > > > > >=20 > > > > > Sure, this is a solution, of which early symlink was a genericiza= tion. > > > > > I'm just not a fan of having co-dependent libraries. > > > > >=20 > > > > > But this will probably come to that. > > > > >=20 > > > >=20 > > > > The other alternative is what was done with rte_compat.h - create a= new lib > > > > just with a single header file in it. Works ok too, and may seem le= ss hacky > > > > to some folks. > > > >=20 > > > > /Bruce > > > >=20 > > > This seems like a good alternative to me. I'm not entirely sure why = the logging > > > functions are integrated to EAL anyway. =20 > > >=20 > > > Neil > > > =20 > >=20 > > As I said earlier: > >=20 > > > > > > > > > Ideally rte_log could be made independent (starting skimm= ing EAL > > > > > > > > > from all the fat), but this is much less trivial. > >=20 > > rte_log could certainly stand on its own. I quickly attempted to make it > > a library, but this is too much work at this point. I think the EAL > > should be broken down, it is too monolithic. Problem is that many > > other libraries / applications, now relies on parts of the EAL that > > would need to be moved. > >=20 > > So any effort in this direction will be difficult to undertake (and > > badly received by the community, with good reasons), especially when > > the workaround is an additional -I cflag. > >=20 > I'm fine with just adding another include path to the CFLAGS for this pur= pose in > the short term, just saying I'm not sure why the log library got integrat= ed into > EAL in the first place. >=20 > Neil The log functions must be in EAL because they are different per-environment. The basic functions are the reason of EAL. I think the problem is somewhere else: we are trying to implemement the com= plex device logic in EAL. We should move all the bus and device logics outside of EAL, in my opinion. We should also move all the options parsing outside of EAL, and make it opt= ional. Anyway, these are considerations for the long run.