From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 9BF4DC75E for ; Thu, 16 Jun 2016 16:06:53 +0200 (CEST) Received: from int-mx13.intmail.prod.int.phx2.redhat.com (int-mx13.intmail.prod.int.phx2.redhat.com [10.5.11.26]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 0A5AD693F5; Thu, 16 Jun 2016 14:06:52 +0000 (UTC) Received: from sopuli.koti.laiskiainen.org (vpn1-4-166.ams2.redhat.com [10.36.4.166]) by int-mx13.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id u5GE6oXQ004722; Thu, 16 Jun 2016 10:06:50 -0400 To: Neil Horman 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> Cc: dev@dpdk.org, Bruce Richardson , Thomas Monjalon , Stephen Hemminger From: Panu Matilainen Message-ID: <1624507a-dcfe-ec4d-3d61-a532130a3cc0@redhat.com> Date: Thu, 16 Jun 2016 17:06:49 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 MIME-Version: 1.0 In-Reply-To: <20160616133343.GA20718@neilslaptop.think-freely.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.68 on 10.5.11.26 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.25]); Thu, 16 Jun 2016 14:06:53 +0000 (UTC) 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:06:54 -0000 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. - Panu - > > Neil > >> - Panu - >> >>