DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] drivers/net: only compile PFE and ENETC when DPAA was selected
@ 2019-10-29 13:01 Thierry Herbelot
  2019-10-30  5:46 ` Gagandeep Singh
  0 siblings, 1 reply; 4+ messages in thread
From: Thierry Herbelot @ 2019-10-29 13:01 UTC (permalink / raw)
  To: dev; +Cc: Thomas Monjalon, Gagandeep Singh

PFE and ENETC depend on DPAA features.

Fixes: e56463ec47f0 ('net/enetc: enable dpaax library')
Fixes: 67fc3ff97c39 ('net/pfe: introduce basic functions')
Cc: Gagandeep Singh <g.singh@nxp.com>

Signed-off-by: Thierry Herbelot <thierry.herbelot@6wind.com>
---
 drivers/net/Makefile | 4 ++++
 mk/rte.app.mk        | 4 ++++
 2 files changed, 8 insertions(+)

diff --git a/drivers/net/Makefile b/drivers/net/Makefile
index cee30367a1c3..966133427276 100644
--- a/drivers/net/Makefile
+++ b/drivers/net/Makefile
@@ -25,7 +25,9 @@ DIRS-$(CONFIG_RTE_LIBRTE_DPAA2_PMD) += dpaa2
 endif
 DIRS-$(CONFIG_RTE_LIBRTE_E1000_PMD) += e1000
 DIRS-$(CONFIG_RTE_LIBRTE_ENA_PMD) += ena
+ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
 DIRS-$(CONFIG_RTE_LIBRTE_ENETC_PMD) += enetc
+endif
 DIRS-$(CONFIG_RTE_LIBRTE_ENIC_PMD) += enic
 DIRS-$(CONFIG_RTE_LIBRTE_PMD_FAILSAFE) += failsafe
 DIRS-$(CONFIG_RTE_LIBRTE_FM10K_PMD) += fm10k
@@ -50,7 +52,9 @@ DIRS-$(CONFIG_RTE_LIBRTE_PMD_NULL) += null
 DIRS-$(CONFIG_RTE_LIBRTE_OCTEONTX_PMD) += octeontx
 DIRS-$(CONFIG_RTE_LIBRTE_OCTEONTX2_PMD) += octeontx2
 DIRS-$(CONFIG_RTE_LIBRTE_PMD_PCAP) += pcap
+ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
 DIRS-$(CONFIG_RTE_LIBRTE_PFE_PMD) += pfe
+endif
 DIRS-$(CONFIG_RTE_LIBRTE_QEDE_PMD) += qede
 DIRS-$(CONFIG_RTE_LIBRTE_PMD_RING) += ring
 DIRS-$(CONFIG_RTE_LIBRTE_SFC_EFX_PMD) += sfc
diff --git a/mk/rte.app.mk b/mk/rte.app.mk
index 1f5c74841fb3..e440dffd7575 100644
--- a/mk/rte.app.mk
+++ b/mk/rte.app.mk
@@ -170,7 +170,9 @@ _LDLIBS-$(CONFIG_RTE_LIBRTE_DPAA2_PMD)      += -lrte_pmd_dpaa2
 endif
 _LDLIBS-$(CONFIG_RTE_LIBRTE_E1000_PMD)      += -lrte_pmd_e1000
 _LDLIBS-$(CONFIG_RTE_LIBRTE_ENA_PMD)        += -lrte_pmd_ena
+ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
 _LDLIBS-$(CONFIG_RTE_LIBRTE_ENETC_PMD)      += -lrte_pmd_enetc
+endif
 _LDLIBS-$(CONFIG_RTE_LIBRTE_ENIC_PMD)       += -lrte_pmd_enic
 _LDLIBS-$(CONFIG_RTE_LIBRTE_FM10K_PMD)      += -lrte_pmd_fm10k
 _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_FAILSAFE)   += -lrte_pmd_failsafe
@@ -204,7 +206,9 @@ _LDLIBS-$(CONFIG_RTE_LIBRTE_NFP_PMD)        += -lrte_pmd_nfp
 _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_NULL)       += -lrte_pmd_null
 _LDLIBS-$(CONFIG_RTE_LIBRTE_OCTEONTX2_PMD)  += -lrte_pmd_octeontx2 -lm
 _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_PCAP)       += -lrte_pmd_pcap -lpcap
+ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
 _LDLIBS-$(CONFIG_RTE_LIBRTE_PFE_PMD)        += -lrte_pmd_pfe
+endif
 _LDLIBS-$(CONFIG_RTE_LIBRTE_QEDE_PMD)       += -lrte_pmd_qede
 _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_RING)       += -lrte_pmd_ring
 ifeq ($(CONFIG_RTE_LIBRTE_SCHED),y)
-- 
2.20.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] drivers/net: only compile PFE and ENETC when DPAA was selected
  2019-10-29 13:01 [dpdk-dev] [PATCH] drivers/net: only compile PFE and ENETC when DPAA was selected Thierry Herbelot
@ 2019-10-30  5:46 ` Gagandeep Singh
  2019-10-30 10:11   ` Thierry Herbelot
  0 siblings, 1 reply; 4+ messages in thread
From: Gagandeep Singh @ 2019-10-30  5:46 UTC (permalink / raw)
  To: Thierry Herbelot, dev; +Cc: Thomas Monjalon

Hi Thierry,

> -----Original Message-----
> From: Thierry Herbelot <thierry.herbelot@6wind.com>
> Sent: Tuesday, October 29, 2019 6:32 PM
> To: dev@dpdk.org
> Cc: Thomas Monjalon <thomas@monjalon.net>; Gagandeep Singh
> <G.Singh@nxp.com>
> Subject: [PATCH] drivers/net: only compile PFE and ENETC when DPAA was
> selected
> 
> PFE and ENETC depend on DPAA features.
> 
> Fixes: e56463ec47f0 ('net/enetc: enable dpaax library')
> Fixes: 67fc3ff97c39 ('net/pfe: introduce basic functions')
> Cc: Gagandeep Singh <g.singh@nxp.com>
> 
> Signed-off-by: Thierry Herbelot <thierry.herbelot@6wind.com>
> ---
>  drivers/net/Makefile | 4 ++++
>  mk/rte.app.mk        | 4 ++++
>  2 files changed, 8 insertions(+)
> 
> diff --git a/drivers/net/Makefile b/drivers/net/Makefile
> index cee30367a1c3..966133427276 100644
> --- a/drivers/net/Makefile
> +++ b/drivers/net/Makefile
> @@ -25,7 +25,9 @@ DIRS-$(CONFIG_RTE_LIBRTE_DPAA2_PMD) += dpaa2
>  endif
>  DIRS-$(CONFIG_RTE_LIBRTE_E1000_PMD) += e1000
>  DIRS-$(CONFIG_RTE_LIBRTE_ENA_PMD) += ena
> +ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
>  DIRS-$(CONFIG_RTE_LIBRTE_ENETC_PMD) += enetc
> +endif
>  DIRS-$(CONFIG_RTE_LIBRTE_ENIC_PMD) += enic
>  DIRS-$(CONFIG_RTE_LIBRTE_PMD_FAILSAFE) += failsafe
>  DIRS-$(CONFIG_RTE_LIBRTE_FM10K_PMD) += fm10k
> @@ -50,7 +52,9 @@ DIRS-$(CONFIG_RTE_LIBRTE_PMD_NULL) += null
>  DIRS-$(CONFIG_RTE_LIBRTE_OCTEONTX_PMD) += octeontx
>  DIRS-$(CONFIG_RTE_LIBRTE_OCTEONTX2_PMD) += octeontx2
>  DIRS-$(CONFIG_RTE_LIBRTE_PMD_PCAP) += pcap
> +ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
>  DIRS-$(CONFIG_RTE_LIBRTE_PFE_PMD) += pfe
> +endif
>  DIRS-$(CONFIG_RTE_LIBRTE_QEDE_PMD) += qede
>  DIRS-$(CONFIG_RTE_LIBRTE_PMD_RING) += ring
>  DIRS-$(CONFIG_RTE_LIBRTE_SFC_EFX_PMD) += sfc
> diff --git a/mk/rte.app.mk b/mk/rte.app.mk
> index 1f5c74841fb3..e440dffd7575 100644
> --- a/mk/rte.app.mk
> +++ b/mk/rte.app.mk
> @@ -170,7 +170,9 @@ _LDLIBS-$(CONFIG_RTE_LIBRTE_DPAA2_PMD)      += -
> lrte_pmd_dpaa2
>  endif
>  _LDLIBS-$(CONFIG_RTE_LIBRTE_E1000_PMD)      += -lrte_pmd_e1000
>  _LDLIBS-$(CONFIG_RTE_LIBRTE_ENA_PMD)        += -lrte_pmd_ena
> +ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
>  _LDLIBS-$(CONFIG_RTE_LIBRTE_ENETC_PMD)      += -lrte_pmd_enetc
> +endif
>  _LDLIBS-$(CONFIG_RTE_LIBRTE_ENIC_PMD)       += -lrte_pmd_enic
>  _LDLIBS-$(CONFIG_RTE_LIBRTE_FM10K_PMD)      += -lrte_pmd_fm10k
>  _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_FAILSAFE)   += -lrte_pmd_failsafe
> @@ -204,7 +206,9 @@ _LDLIBS-$(CONFIG_RTE_LIBRTE_NFP_PMD)        += -
> lrte_pmd_nfp
>  _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_NULL)       += -lrte_pmd_null
>  _LDLIBS-$(CONFIG_RTE_LIBRTE_OCTEONTX2_PMD)  += -lrte_pmd_octeontx2 -
> lm
>  _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_PCAP)       += -lrte_pmd_pcap -lpcap
> +ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
>  _LDLIBS-$(CONFIG_RTE_LIBRTE_PFE_PMD)        += -lrte_pmd_pfe
> +endif
>  _LDLIBS-$(CONFIG_RTE_LIBRTE_QEDE_PMD)       += -lrte_pmd_qede
>  _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_RING)       += -lrte_pmd_ring
>  ifeq ($(CONFIG_RTE_LIBRTE_SCHED),y)
> --
> 2.20.1

I didn't see any compilation issue on '19.11-rc1' tag and also these both PMDs are independent of DPAA_BUS, so we cannot accept these changes, can you please share your exact steps of compilation?

Thanks,
Gagan

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] drivers/net: only compile PFE and ENETC when DPAA was selected
  2019-10-30  5:46 ` Gagandeep Singh
@ 2019-10-30 10:11   ` Thierry Herbelot
  2019-10-30 11:59     ` Gagandeep Singh
  0 siblings, 1 reply; 4+ messages in thread
From: Thierry Herbelot @ 2019-10-30 10:11 UTC (permalink / raw)
  To: Gagandeep Singh, dev; +Cc: Thomas Monjalon

On 10/30/19 6:46 AM, Gagandeep Singh wrote:
> Hi Thierry,
> 
>> -----Original Message-----
>> From: Thierry Herbelot <thierry.herbelot@6wind.com>
>> Sent: Tuesday, October 29, 2019 6:32 PM
>> To: dev@dpdk.org
>> Cc: Thomas Monjalon <thomas@monjalon.net>; Gagandeep Singh
>> <G.Singh@nxp.com>
>> Subject: [PATCH] drivers/net: only compile PFE and ENETC when DPAA was
>> selected
>>
>> PFE and ENETC depend on DPAA features.
>>
>> Fixes: e56463ec47f0 ('net/enetc: enable dpaax library')
>> Fixes: 67fc3ff97c39 ('net/pfe: introduce basic functions')
>> Cc: Gagandeep Singh <g.singh@nxp.com>
>>
>> Signed-off-by: Thierry Herbelot <thierry.herbelot@6wind.com>
>> ---
>>   drivers/net/Makefile | 4 ++++
>>   mk/rte.app.mk        | 4 ++++
>>   2 files changed, 8 insertions(+)
>>
>> diff --git a/drivers/net/Makefile b/drivers/net/Makefile
>> index cee30367a1c3..966133427276 100644
>> --- a/drivers/net/Makefile
>> +++ b/drivers/net/Makefile
>> @@ -25,7 +25,9 @@ DIRS-$(CONFIG_RTE_LIBRTE_DPAA2_PMD) += dpaa2
>>   endif
>>   DIRS-$(CONFIG_RTE_LIBRTE_E1000_PMD) += e1000
>>   DIRS-$(CONFIG_RTE_LIBRTE_ENA_PMD) += ena
>> +ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
>>   DIRS-$(CONFIG_RTE_LIBRTE_ENETC_PMD) += enetc
>> +endif
>>   DIRS-$(CONFIG_RTE_LIBRTE_ENIC_PMD) += enic
>>   DIRS-$(CONFIG_RTE_LIBRTE_PMD_FAILSAFE) += failsafe
>>   DIRS-$(CONFIG_RTE_LIBRTE_FM10K_PMD) += fm10k
>> @@ -50,7 +52,9 @@ DIRS-$(CONFIG_RTE_LIBRTE_PMD_NULL) += null
>>   DIRS-$(CONFIG_RTE_LIBRTE_OCTEONTX_PMD) += octeontx
>>   DIRS-$(CONFIG_RTE_LIBRTE_OCTEONTX2_PMD) += octeontx2
>>   DIRS-$(CONFIG_RTE_LIBRTE_PMD_PCAP) += pcap
>> +ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
>>   DIRS-$(CONFIG_RTE_LIBRTE_PFE_PMD) += pfe
>> +endif
>>   DIRS-$(CONFIG_RTE_LIBRTE_QEDE_PMD) += qede
>>   DIRS-$(CONFIG_RTE_LIBRTE_PMD_RING) += ring
>>   DIRS-$(CONFIG_RTE_LIBRTE_SFC_EFX_PMD) += sfc
>> diff --git a/mk/rte.app.mk b/mk/rte.app.mk
>> index 1f5c74841fb3..e440dffd7575 100644
>> --- a/mk/rte.app.mk
>> +++ b/mk/rte.app.mk
>> @@ -170,7 +170,9 @@ _LDLIBS-$(CONFIG_RTE_LIBRTE_DPAA2_PMD)      += -
>> lrte_pmd_dpaa2
>>   endif
>>   _LDLIBS-$(CONFIG_RTE_LIBRTE_E1000_PMD)      += -lrte_pmd_e1000
>>   _LDLIBS-$(CONFIG_RTE_LIBRTE_ENA_PMD)        += -lrte_pmd_ena
>> +ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
>>   _LDLIBS-$(CONFIG_RTE_LIBRTE_ENETC_PMD)      += -lrte_pmd_enetc
>> +endif
>>   _LDLIBS-$(CONFIG_RTE_LIBRTE_ENIC_PMD)       += -lrte_pmd_enic
>>   _LDLIBS-$(CONFIG_RTE_LIBRTE_FM10K_PMD)      += -lrte_pmd_fm10k
>>   _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_FAILSAFE)   += -lrte_pmd_failsafe
>> @@ -204,7 +206,9 @@ _LDLIBS-$(CONFIG_RTE_LIBRTE_NFP_PMD)        += -
>> lrte_pmd_nfp
>>   _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_NULL)       += -lrte_pmd_null
>>   _LDLIBS-$(CONFIG_RTE_LIBRTE_OCTEONTX2_PMD)  += -lrte_pmd_octeontx2 -
>> lm
>>   _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_PCAP)       += -lrte_pmd_pcap -lpcap
>> +ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
>>   _LDLIBS-$(CONFIG_RTE_LIBRTE_PFE_PMD)        += -lrte_pmd_pfe
>> +endif
>>   _LDLIBS-$(CONFIG_RTE_LIBRTE_QEDE_PMD)       += -lrte_pmd_qede
>>   _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_RING)       += -lrte_pmd_ring
>>   ifeq ($(CONFIG_RTE_LIBRTE_SCHED),y)
>> --
>> 2.20.1
> 
> I didn't see any compilation issue on '19.11-rc1' tag and also these both PMDs are independent of DPAA_BUS, so we cannot accept these changes, can you please share your exact steps of compilation?

Hello, Gagan

Indeed, the compilation procedure was not described.

These are the reproduction steps (on an Intel machine):

git clone git://dpdk.org/dpdk
cd dpdk
make config T=x86_64-native-linux-gcc
cd build
vi .config
   => disable RTE_LIBRTE_DPAA_BUS and RTE_LIBRTE_FSLMC_BUS
make
...
== Build app/test
   LD test
/build/dpdk/build/lib/librte_pmd_caam_jr.a(caam_jr.o): In function 
`cryptodev_caam_jr_probe':
caam_jr.c:(.text+0x123a): undefined reference to `of_find_compatible_node'
caam_jr.c:(.text+0x125b): undefined reference to `of_find_compatible_node'
caam_jr.c:(.text+0x1276): undefined reference to `of_get_property'
/build/dpdk/build/lib/librte_pmd_caam_jr.a(caam_jr.o): In function 
`caam_jr_dequeue_burst':
caam_jr.c:(.text+0x1dfb): undefined reference to `dpaax_iova_table_p'
/build/dpdk/build/lib/librte_pmd_enetc.a(enetc_ethdev.o): In function 
`enetc_dev_uninit':
enetc_ethdev.c:(.text+0x679): undefined reference to 
`dpaax_iova_table_depopulate'
/build/dpdk/build/lib/librte_pmd_enetc.a(enetc_ethdev.o): In function 
`enetc_dev_init':
enetc_ethdev.c:(.text+0xda1): undefined reference to 
`dpaax_iova_table_populate'
/build/dpdk/build/lib/librte_pmd_enetc.a(enetc_ethdev.o): In function 
`enetc_pci_remove':
enetc_ethdev.c:(.text+0xfe9): undefined reference to 
`dpaax_iova_table_depopulate'
collect2: error: ld returned 1 exit status

	Regards

	Thierry

> 
> Thanks,
> Gagan
> 


-- 
Thierry Herbelot
6WIND
Senior Software Engineer

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] drivers/net: only compile PFE and ENETC when DPAA was selected
  2019-10-30 10:11   ` Thierry Herbelot
@ 2019-10-30 11:59     ` Gagandeep Singh
  0 siblings, 0 replies; 4+ messages in thread
From: Gagandeep Singh @ 2019-10-30 11:59 UTC (permalink / raw)
  To: Thierry Herbelot, dev; +Cc: Thomas Monjalon

Hi Thierry,

I got this issue now, will send the fix soon.

> -----Original Message-----
> From: Thierry Herbelot <thierry.herbelot@6wind.com>
> Sent: Wednesday, October 30, 2019 3:41 PM
> To: Gagandeep Singh <G.Singh@nxp.com>; dev@dpdk.org
> Cc: Thomas Monjalon <thomas@monjalon.net>
> Subject: Re: [PATCH] drivers/net: only compile PFE and ENETC when DPAA was
> selected
> 
> On 10/30/19 6:46 AM, Gagandeep Singh wrote:
> > Hi Thierry,
> >
> >> -----Original Message-----
> >> From: Thierry Herbelot <thierry.herbelot@6wind.com>
> >> Sent: Tuesday, October 29, 2019 6:32 PM
> >> To: dev@dpdk.org
> >> Cc: Thomas Monjalon <thomas@monjalon.net>; Gagandeep Singh
> >> <G.Singh@nxp.com>
> >> Subject: [PATCH] drivers/net: only compile PFE and ENETC when DPAA was
> >> selected
> >>
> >> PFE and ENETC depend on DPAA features.
> >>
> >> Fixes: e56463ec47f0 ('net/enetc: enable dpaax library')
> >> Fixes: 67fc3ff97c39 ('net/pfe: introduce basic functions')
> >> Cc: Gagandeep Singh <g.singh@nxp.com>
> >>
> >> Signed-off-by: Thierry Herbelot <thierry.herbelot@6wind.com>
> >> ---
> >>   drivers/net/Makefile | 4 ++++
> >>   mk/rte.app.mk        | 4 ++++
> >>   2 files changed, 8 insertions(+)
> >>
> >> diff --git a/drivers/net/Makefile b/drivers/net/Makefile
> >> index cee30367a1c3..966133427276 100644
> >> --- a/drivers/net/Makefile
> >> +++ b/drivers/net/Makefile
> >> @@ -25,7 +25,9 @@ DIRS-$(CONFIG_RTE_LIBRTE_DPAA2_PMD) += dpaa2
> >>   endif
> >>   DIRS-$(CONFIG_RTE_LIBRTE_E1000_PMD) += e1000
> >>   DIRS-$(CONFIG_RTE_LIBRTE_ENA_PMD) += ena
> >> +ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
> >>   DIRS-$(CONFIG_RTE_LIBRTE_ENETC_PMD) += enetc
> >> +endif
> >>   DIRS-$(CONFIG_RTE_LIBRTE_ENIC_PMD) += enic
> >>   DIRS-$(CONFIG_RTE_LIBRTE_PMD_FAILSAFE) += failsafe
> >>   DIRS-$(CONFIG_RTE_LIBRTE_FM10K_PMD) += fm10k
> >> @@ -50,7 +52,9 @@ DIRS-$(CONFIG_RTE_LIBRTE_PMD_NULL) += null
> >>   DIRS-$(CONFIG_RTE_LIBRTE_OCTEONTX_PMD) += octeontx
> >>   DIRS-$(CONFIG_RTE_LIBRTE_OCTEONTX2_PMD) += octeontx2
> >>   DIRS-$(CONFIG_RTE_LIBRTE_PMD_PCAP) += pcap
> >> +ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
> >>   DIRS-$(CONFIG_RTE_LIBRTE_PFE_PMD) += pfe
> >> +endif
> >>   DIRS-$(CONFIG_RTE_LIBRTE_QEDE_PMD) += qede
> >>   DIRS-$(CONFIG_RTE_LIBRTE_PMD_RING) += ring
> >>   DIRS-$(CONFIG_RTE_LIBRTE_SFC_EFX_PMD) += sfc
> >> diff --git a/mk/rte.app.mk b/mk/rte.app.mk
> >> index 1f5c74841fb3..e440dffd7575 100644
> >> --- a/mk/rte.app.mk
> >> +++ b/mk/rte.app.mk
> >> @@ -170,7 +170,9 @@ _LDLIBS-$(CONFIG_RTE_LIBRTE_DPAA2_PMD)      +=
> -
> >> lrte_pmd_dpaa2
> >>   endif
> >>   _LDLIBS-$(CONFIG_RTE_LIBRTE_E1000_PMD)      += -lrte_pmd_e1000
> >>   _LDLIBS-$(CONFIG_RTE_LIBRTE_ENA_PMD)        += -lrte_pmd_ena
> >> +ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
> >>   _LDLIBS-$(CONFIG_RTE_LIBRTE_ENETC_PMD)      += -lrte_pmd_enetc
> >> +endif
> >>   _LDLIBS-$(CONFIG_RTE_LIBRTE_ENIC_PMD)       += -lrte_pmd_enic
> >>   _LDLIBS-$(CONFIG_RTE_LIBRTE_FM10K_PMD)      += -lrte_pmd_fm10k
> >>   _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_FAILSAFE)   += -lrte_pmd_failsafe
> >> @@ -204,7 +206,9 @@ _LDLIBS-$(CONFIG_RTE_LIBRTE_NFP_PMD)        += -
> >> lrte_pmd_nfp
> >>   _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_NULL)       += -lrte_pmd_null
> >>   _LDLIBS-$(CONFIG_RTE_LIBRTE_OCTEONTX2_PMD)  += -
> lrte_pmd_octeontx2 -
> >> lm
> >>   _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_PCAP)       += -lrte_pmd_pcap -lpcap
> >> +ifeq ($(CONFIG_RTE_LIBRTE_DPAA_BUS),y)
> >>   _LDLIBS-$(CONFIG_RTE_LIBRTE_PFE_PMD)        += -lrte_pmd_pfe
> >> +endif
> >>   _LDLIBS-$(CONFIG_RTE_LIBRTE_QEDE_PMD)       += -lrte_pmd_qede
> >>   _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_RING)       += -lrte_pmd_ring
> >>   ifeq ($(CONFIG_RTE_LIBRTE_SCHED),y)
> >> --
> >> 2.20.1
> >
> > I didn't see any compilation issue on '19.11-rc1' tag and also these both PMDs
> are independent of DPAA_BUS, so we cannot accept these changes, can you
> please share your exact steps of compilation?
> 
> Hello, Gagan
> 
> Indeed, the compilation procedure was not described.
> 
> These are the reproduction steps (on an Intel machine):
> 
> git clone git://dpdk.org/dpdk
> cd dpdk
> make config T=x86_64-native-linux-gcc
> cd build
> vi .config
>    => disable RTE_LIBRTE_DPAA_BUS and RTE_LIBRTE_FSLMC_BUS
> make
> ...
> == Build app/test
>    LD test
> /build/dpdk/build/lib/librte_pmd_caam_jr.a(caam_jr.o): In function
> `cryptodev_caam_jr_probe':
> caam_jr.c:(.text+0x123a): undefined reference to `of_find_compatible_node'
> caam_jr.c:(.text+0x125b): undefined reference to `of_find_compatible_node'
> caam_jr.c:(.text+0x1276): undefined reference to `of_get_property'
> /build/dpdk/build/lib/librte_pmd_caam_jr.a(caam_jr.o): In function
> `caam_jr_dequeue_burst':
> caam_jr.c:(.text+0x1dfb): undefined reference to `dpaax_iova_table_p'
> /build/dpdk/build/lib/librte_pmd_enetc.a(enetc_ethdev.o): In function
> `enetc_dev_uninit':
> enetc_ethdev.c:(.text+0x679): undefined reference to
> `dpaax_iova_table_depopulate'
> /build/dpdk/build/lib/librte_pmd_enetc.a(enetc_ethdev.o): In function
> `enetc_dev_init':
> enetc_ethdev.c:(.text+0xda1): undefined reference to
> `dpaax_iova_table_populate'
> /build/dpdk/build/lib/librte_pmd_enetc.a(enetc_ethdev.o): In function
> `enetc_pci_remove':
> enetc_ethdev.c:(.text+0xfe9): undefined reference to
> `dpaax_iova_table_depopulate'
> collect2: error: ld returned 1 exit status
> 
> 	Regards
> 
> 	Thierry
> 
> >
> > Thanks,
> > Gagan
> >
> 
> 
> --
> Thierry Herbelot
> 6WIND
> Senior Software Engineer

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-10-30 11:59 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-29 13:01 [dpdk-dev] [PATCH] drivers/net: only compile PFE and ENETC when DPAA was selected Thierry Herbelot
2019-10-30  5:46 ` Gagandeep Singh
2019-10-30 10:11   ` Thierry Herbelot
2019-10-30 11:59     ` Gagandeep Singh

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).