From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-qk0-f170.google.com (mail-qk0-f170.google.com [209.85.220.170]) by dpdk.org (Postfix) with ESMTP id 92F282C4D for ; Tue, 11 Oct 2016 09:27:42 +0200 (CEST) Received: by mail-qk0-f170.google.com with SMTP id n189so17192263qke.0 for ; Tue, 11 Oct 2016 00:27:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:user-agent:in-reply-to :references:mime-version:content-transfer-encoding; bh=kP+u94d0rwhVz866Fb5ioVDRc9EA8+EurBPN954WNlo=; b=yU0GDxMAOCLtjv+xnhXQ8GwW2rUYvSTaA5iN6yz71+H2u63nQly6BX7Ie8CXyZ2ZfK EhE3ivAxtO3JSNSefAEA70DiYM4KYjkH17odvVs4/c3fKvMfm+NsozJQWfaDE0mi8aSg VTP7GleUkIwCNK+Z93Jiatbg3bTNZN9VeCbNXX9ioIDJwfci69ctadBLFPxvt6tInKKy dgB4byVWkWIPCwIuSK1QwmpUOSGCpeJFQJXxTpDwOjRmRh+OnEk6Eebp1AinX49aFub9 5q1Oh7aYcKkAdrx1bgSYslv/0nqFTnQNhDqKp+wyaXDfSzSzpl8fAn1RNYqdt3BZtGe2 2/MQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:user-agent :in-reply-to:references:mime-version:content-transfer-encoding; bh=kP+u94d0rwhVz866Fb5ioVDRc9EA8+EurBPN954WNlo=; b=SWp1wmFS31skRiFY0VxRQU8xDewFopJL7aJ/L8tMo3mc/+VBpvfuaIvzdYLV2d2tFs WIVm8A9GTGdwcePpK1G5cqj+rjxFyxmtJt+N7Q/++axAZJbgU3eGUwhO6sCe/E5VCnNx EDs0pP6IsNlUrToLeC3gdmMJEY61kNSpO21Ed88EA/KeDfyBJPCp+l4XVks/s193NRys Qsk9S51Iw9+yJusAzvuikr6u2ulScAKJBIjuVfkM1VJyREjvaNIi1NouZAbNen3ZdzXU hN8X/lWsZ23dW2cGdVYNybEOJxCShmRvfHbqFQbcOohp8THQ7RI5Z/HGUbBImbSykXPa 1obg== X-Gm-Message-State: AA6/9RkqUqkpyR54iIGNWxELEtkFN9TOzm12Ukwl4OLDPWLO8Iar20Dr+9PVLQ4AmXHMVF/a X-Received: by 10.195.6.97 with SMTP id ct1mr2803355wjd.165.1476170861944; Tue, 11 Oct 2016 00:27:41 -0700 (PDT) Received: from xps13.localnet (184.203.134.77.rev.sfr.net. [77.134.203.184]) by smtp.gmail.com with ESMTPSA id pe5sm4005674wjb.15.2016.10.11.00.27.41 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 11 Oct 2016 00:27:41 -0700 (PDT) From: Thomas Monjalon To: Shreyansh Jain Cc: Neil Horman , "david.marchand@6wind.com" , dev@dpdk.org Date: Tue, 11 Oct 2016 09:27:40 +0200 Message-ID: <47619133.Ocuh29YQlD@xps13> User-Agent: KMail/4.14.10 (Linux/4.5.4-1-ARCH; KDE/4.14.11; x86_64; ; ) In-Reply-To: References: <1475847187-28967-1-git-send-email-shreyansh.jain@nxp.com> <20161010125624.GA19423@hmsreliant.think-freely.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH 1/3] eal/drivers: prefix driver REGISTER macros with EAL 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: Tue, 11 Oct 2016 07:27:42 -0000 2016-10-11 12:06, Shreyansh Jain: > On Monday 10 October 2016 06:26 PM, Neil Horman wrote: > > On Sat, Oct 08, 2016 at 01:00:59PM +0000, Shreyansh Jain wrote: > >> Hi Thomas, > >> > >>> -----Original Message----- > >>> From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > >>> Sent: Friday, October 07, 2016 7:22 PM > >>> To: Shreyansh Jain > >>> Cc: david.marchand@6wind.com; dev@dpdk.org > >>> Subject: Re: [PATCH 1/3] eal/drivers: prefix driver REGISTER macros with EAL > >>> > >>> 2016-10-07 19:11, Shreyansh Jain: > >>>> --- a/mk/internal/rte.compile-pre.mk > >>>> +++ b/mk/internal/rte.compile-pre.mk > >>>> @@ -87,7 +87,7 @@ endif > >>>> PMDINFO_GEN = $(RTE_SDK_BIN)/app/dpdk-pmdinfogen $@ $@.pmd.c > >>>> PMDINFO_CC = $(CC) $(CPPFLAGS) $(CFLAGS) -c -o $@.pmd.o $@.pmd.c > >>>> PMDINFO_LD = $(CROSS)ld $(LDFLAGS) -r -o $@.o $@.pmd.o $@ > >>>> -PMDINFO_TO_O = if grep -q 'DRIVER_REGISTER_.*(.*)' $<; then \ > >>>> +PMDINFO_TO_O = if grep 'EAL_REGISTER_.*(.*)' $<; then \ > >>>> echo "$(if $V,$(PMDINFO_GEN), PMDINFO $@.pmd.c)" && \ > >>>> $(PMDINFO_GEN) && \ > >>>> echo "$(if $V,$(PMDINFO_CC), CC $@.pmd.o)" && \ > >>>> > >>>> --->8--- > >>>> CC eal_pci_vfio.o > >>>> PMDINFO eal_pci_vfio.o.pmd.c > >>>> /bin/sh: 1: > >>>> /home/shreyansh/build/DPDK/02_dpdk/x86_64-native-linuxapp-gcc/app/dpdk- > >>> pmdinfogen: > >>>> not found > >>>> /home/shreyansh/build/DPDK/02_dpdk/mk/internal/rte.compile-pre.mk:138: > >>>> recipe for target 'eal_pci_vfio.o' failed > >>>> --->8--- > >>>> > >>>> I don't think PMDINFO should be running on eal_pci_vfio file. Isn't it? > >>> > >>> Every files are scanned for the pattern. > >> > >> Sorry, I should have been clearer in my statement. > >> I meant, I didn't think eal_pci_vfio.o had anything of interest for the PMD tool and hence the mk files would have skipped over it in absence of a match. > >> I understand that PMDINFO would run on all files. > >> > > Thats incorrect, the Makefile does a REGEX search for appropriate registration > > macros that imply the need for pmdinfo to run. If its running on an > > inappropriate file its because your new macros inadvertently match the current > > regex, hence my suggestion that the regex should be tuned to be more specific > > Agree. Thats is what I wanted to clarify as stated below: > "...EAL_REGISTER_* (macro name has changed since) is matching > EAL_REGISTER_TAILQ..". > > As for 'more specific' match - I did suggest [2] a longer more specific > version but Thomas had a different view point [1]. You can have a look > at [2] and let me know your suggestion or if that is wrong. > > [1] http://dpdk.org/ml/archives/dev/2016-October/048425.html > [2] http://dpdk.org/ml/archives/dev/2016-October/048407.html I do not have a different point of view :) We need to be more specific than EAL_REGISTER_*. And RTE_PMD_REGISTER_* is specific enough. That's all :)