From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from nbfkord-smmo01.seg.att.com (nbfkord-smmo01.seg.att.com [209.65.160.76]) by dpdk.org (Postfix) with ESMTP id 024322BA7 for ; Thu, 24 Nov 2016 16:45:56 +0100 (CET) Received: from unknown [193.34.186.16] (EHLO nbfkord-smmo01.seg.att.com) by nbfkord-smmo01.seg.att.com(mxl_mta-7.2.4-7) with ESMTP id 53b07385.2ba83a863940.1069662.00-2487.2286875.nbfkord-smmo01.seg.att.com (envelope-from ); Thu, 24 Nov 2016 15:45:57 +0000 (UTC) X-MXL-Hash: 58370b3541b55a3e-42a10c362712bd28746ef5f065f4f2a72c78e3e1 Received: from unknown [193.34.186.16] by nbfkord-smmo01.seg.att.com(mxl_mta-7.2.4-7) with SMTP id ffa07385.0.1069479.00-2371.2286708.nbfkord-smmo01.seg.att.com (envelope-from ); Thu, 24 Nov 2016 15:45:07 +0000 (UTC) X-MXL-Hash: 58370b030351e7cb-2eafbc21b96cd12362f8de74b070fe1a00c3c026 Received: from [192.168.38.17] (84.52.89.52) by ukex01.SolarFlarecom.com (10.17.10.4) with Microsoft SMTP Server (TLS) id 15.0.1044.25; Thu, 24 Nov 2016 15:44:23 +0000 To: Ferruh Yigit , References: <1479740470-6723-1-git-send-email-arybchenko@solarflare.com> <1479740470-6723-31-git-send-email-arybchenko@solarflare.com> <18cbe35d-ee10-961b-f6a9-abfb0232c974@intel.com> CC: Artem Andreev From: Andrew Rybchenko Message-ID: <0cc9d168-a15f-abb7-303f-8c22132227ed@solarflare.com> Date: Thu, 24 Nov 2016 18:44:19 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.0 MIME-Version: 1.0 In-Reply-To: <18cbe35d-ee10-961b-f6a9-abfb0232c974@intel.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [84.52.89.52] X-ClientProxiedBy: ocex03.SolarFlarecom.com (10.20.40.36) To ukex01.SolarFlarecom.com (10.17.10.4) X-TM-AS-Product-Ver: SMEX-11.0.0.1191-8.000.1202-22718.003 X-TM-AS-Result: No--16.278800-0.000000-31 X-TM-AS-User-Approved-Sender: Yes X-TM-AS-User-Blocked-Sender: No X-AnalysisOut: [v=2.1 cv=P5Y8830u c=1 sm=1 tr=0 a=8P+NB+fYZDP74ap4g4d9Kw==] X-AnalysisOut: [:17 a=RB3BGLmKESwA:10 a=N659UExz7-8A:10 a=L24OOQBejmoA:10 ] X-AnalysisOut: [a=pK7X0mNQAAAA:8 a=zRKbQ67AAAAA:8 a=G5H1aCjo4Ju0FcVmDYQA:9] X-AnalysisOut: [ a=pILNOxqGKmIA:10 a=5HA-qpC1VU4iIGLgRoNS:22 a=PA03WX8tBze] X-AnalysisOut: [izutn5_OT:22] X-Spam: [F=0.3729405382; CM=0.500; S=0.372(2015072901)] X-MAIL-FROM: X-SOURCE-IP: [193.34.186.16] Subject: Re: [dpdk-dev] [PATCH 30/56] net/sfc: include libefx in build 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, 24 Nov 2016 15:45:57 -0000 See one question below. On 11/23/2016 06:26 PM, Ferruh Yigit wrote: > On 11/21/2016 3:00 PM, Andrew Rybchenko wrote: >> From: Artem Andreev >> >> Implement efsys.h for the PMD. >> >> Reviewed-by: Andy Moreton >> Signed-off-by: Artem Andreev >> Signed-off-by: Andrew Rybchenko >> --- >> drivers/net/sfc/efx/Makefile | 54 +++ >> drivers/net/sfc/efx/efsys.h | 767 +++++++++++++++++++++++++++++++++++++++++++ >> 2 files changed, 821 insertions(+) >> create mode 100644 drivers/net/sfc/efx/efsys.h >> >> diff --git a/drivers/net/sfc/efx/Makefile b/drivers/net/sfc/efx/Makefile >> index 71f07ca..de95ea8 100644 >> --- a/drivers/net/sfc/efx/Makefile >> +++ b/drivers/net/sfc/efx/Makefile >> @@ -33,6 +33,8 @@ include $(RTE_SDK)/mk/rte.vars.mk >> # >> LIB = librte_pmd_sfc_efx.a >> >> +CFLAGS += -I$(SRCDIR)/base/ >> +CFLAGS += -I$(SRCDIR) >> CFLAGS += -O3 >> >> # Enable basic warnings but disable some which are accepted >> @@ -60,6 +62,17 @@ CFLAGS += -Wstrict-prototypes >> CFLAGS += -Wundef >> CFLAGS += -Wwrite-strings >> >> +# Extra CFLAGS for base driver files >> +CFLAGS_BASE_DRIVER += -Wno-unused-variable >> +CFLAGS_BASE_DRIVER += -Wno-unused-but-set-variable > clang complain about this one: > warning: unknown warning option '-Wno-unused-but-set-variable'; did you > mean '-Wno-unused-const-variable'? [-Wunknown-warning-option] Will fix in v2 >> + >> +# >> +# List of base driver object files for which >> +# special CFLAGS above should be applied >> +# >> +BASE_DRIVER_OBJS=$(patsubst %.c,%.o,$(notdir $(wildcard $(SRCDIR)/base/*.c))) >> +$(foreach obj, $(BASE_DRIVER_OBJS), $(eval CFLAGS+=$(CFLAGS_BASE_DRIVER))) > This cause multiple "-Wno-unused-variable -Wno-unused-but-set-variable" > params in final command, I guess the intention is: > > $(foreach obj, $(BASE_DRIVER_OBJS), $(eval > CFLAGS_$(obj)+=$(CFLAGS_BASE_DRIVER))) > > Fixing this may generate a few compiler warnings. Many thanks, will fix in v2. > <...> > >> diff --git a/drivers/net/sfc/efx/efsys.h b/drivers/net/sfc/efx/efsys.h >> new file mode 100644 >> index 0000000..2eef996 >> --- /dev/null >> +++ b/drivers/net/sfc/efx/efsys.h >> @@ -0,0 +1,767 @@ > <...> > > I guess below is hardcoded compile time configuration for libefx, do you > think does it make sense to document this default configuration? Yes, it is libefx configuration and more options will be enabled when corresponding functionality is supported in the PMD. I'm sorry, but I don't understand what would you like to see in the documentation. Could you clarify, please? >> + >> +#define EFSYS_OPT_NAMES 0 >> + >> +#define EFSYS_OPT_SIENA 0 >> +#define EFSYS_OPT_HUNTINGTON 1 >> +#define EFSYS_OPT_MEDFORD 1 >> +#ifdef RTE_LIBRTE_SFC_EFX_DEBUG >> +#define EFSYS_OPT_CHECK_REG 1 >> +#else >> +#define EFSYS_OPT_CHECK_REG 0 >> +#endif >> + >> +#define EFSYS_OPT_MCDI 1 >> +#define EFSYS_OPT_MCDI_LOGGING 0 >> +#define EFSYS_OPT_MCDI_PROXY_AUTH 0 >> + >> +#define EFSYS_OPT_MAC_STATS 0 >> + >> +#define EFSYS_OPT_LOOPBACK 0 >> + >> +#define EFSYS_OPT_MON_MCDI 0 >> +#define EFSYS_OPT_MON_STATS 0 >> + >> +#define EFSYS_OPT_PHY_STATS 0 >> +#define EFSYS_OPT_BIST 0 >> +#define EFSYS_OPT_PHY_LED_CONTROL 0 >> +#define EFSYS_OPT_PHY_FLAGS 0 >> + >> +#define EFSYS_OPT_VPD 0 >> +#define EFSYS_OPT_NVRAM 0 >> +#define EFSYS_OPT_BOOTCFG 0 >> + >> +#define EFSYS_OPT_DIAG 0 >> +#define EFSYS_OPT_RX_SCALE 0 >> +#define EFSYS_OPT_QSTATS 0 >> +#define EFSYS_OPT_FILTER 1 >> +#define EFSYS_OPT_RX_SCATTER 0 >> + >> +#define EFSYS_OPT_EV_PREFETCH 0 >> + >> +#define EFSYS_OPT_DECODE_INTR_FATAL 0 >> + >> +#define EFSYS_OPT_LICENSING 0 >> + >> +#define EFSYS_OPT_ALLOW_UNCONFIGURED_NIC 0 >> + >> +#define EFSYS_OPT_RX_PACKED_STREAM 0 > <...>