From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.tuxdriver.com (charlotte.tuxdriver.com [70.61.120.58]) by dpdk.org (Postfix) with ESMTP id 2279AC636 for ; Thu, 16 Jun 2016 16:41:25 +0200 (CEST) Received: from [107.15.76.160] (helo=localhost) by smtp.tuxdriver.com with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.63) (envelope-from ) id 1bDYTv-0005iv-Hn; Thu, 16 Jun 2016 10:41:21 -0400 Date: Thu, 16 Jun 2016 10:41:18 -0400 From: Neil Horman To: Panu Matilainen Cc: dev@dpdk.org, Bruce Richardson , Thomas Monjalon , Stephen Hemminger Message-ID: <20160616144118.GB20718@neilslaptop.think-freely.org> References: <1463431287-4551-1-git-send-email-nhorman@tuxdriver.com> <1465494421-6210-1-git-send-email-nhorman@tuxdriver.com> <1465494421-6210-2-git-send-email-nhorman@tuxdriver.com> <2e4a75e2-54fb-e449-a146-5b7bdf47f9f2@redhat.com> <20160616133343.GA20718@neilslaptop.think-freely.org> <1624507a-dcfe-ec4d-3d61-a532130a3cc0@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1624507a-dcfe-ec4d-3d61-a532130a3cc0@redhat.com> User-Agent: Mutt/1.6.1 (2016-04-27) X-Spam-Score: -1.0 (-) X-Spam-Status: No Subject: Re: [dpdk-dev] [PATCHv7 1/6] pmdinfogen: Add buildtools and pmdinfogen utility X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 16 Jun 2016 14:41:25 -0000 On Thu, Jun 16, 2016 at 05:06:49PM +0300, Panu Matilainen wrote: > On 06/16/2016 04:33 PM, Neil Horman wrote: > > On Thu, Jun 16, 2016 at 03:29:57PM +0300, Panu Matilainen wrote: > > > On 06/09/2016 08:46 PM, Neil Horman wrote: > > > > pmdinfogen is a tool used to parse object files and build json strings for > > > > use in later determining hardware support in a dso or application binary. > > > > pmdinfo looks for the non-exported symbol names this_pmd_name and > > > > this_pmd_tbl (where n is a integer counter). It records the name of > > > > each of these tuples, using the later to find the symbolic name of the > > > > pci_table for physical devices that the object supports. With this > > > > information, it outputs a C file with a single line of the form: > > > > > > > > static char *_driver_info[] __attribute__((used)) = " \ > > > > PMD_DRIVER_INFO="; > > > > > > > > Where is the arbitrary name of the pmd, and is the > > > > json encoded string that hold relevant pmd information, including the pmd > > > > name, type and optional array of pci device/vendor ids that the driver > > > > supports. > > > > > > > > This c file is suitable for compiling to object code, then relocatably > > > > linking into the parent file from which the C was generated. This creates > > > > an entry in the string table of the object that can inform a later tool > > > > about hardware support. > > > > > > > > Signed-off-by: Neil Horman > > > > CC: Bruce Richardson > > > > CC: Thomas Monjalon > > > > CC: Stephen Hemminger > > > > CC: Panu Matilainen > > > > --- > > > > > > Unlike earlier versions, pmdinfogen ends up installed in bindir during "make > > > install". Is that intentional, or just a side-effect from using > > > rte.hostapp.mk? If its intentional it probably should be prefixed with dpdk_ > > > like the other tools. > > > > > Im not sure what the answer is here. As you can see, Thomas and I argued at > > length over which makefile to use, and I gave up, so I suppose you can call it > > intentional. Being in bindir makes a reasonable amount of sense I suppose, as > > 3rd party developers can use it during their independent driver development. > > Right, it'd be useful for 3rd party driver developer, so lets consider it > intentional :) > > > I'm not sure I agree with prefixing it though. Given that the hostapp.mk file > > installs everything there, and nothing that previously used that make file had a > > dpdk_ prefix that I can tell, I'm not sure why this would. pmdinfogen seems > > like a pretty unique name, and I know of no other project that uses the term pmd > > to describe anything. > > I agree about "pmd" being fairly unique as is, but if pmdinfo is dpdk_ > prefixed then this should be too, or neither should be prefixed. I dont > personally care which way, but it should be consistent. > I caved on the pmdinfo issue, I guess I'll cave here too.