* [dpdk-dev] [PATCH v1] app/pdump: add check for PCAP PMD
@ 2018-03-05 7:57 Vipin Varghese
2018-03-05 14:32 ` Ferruh Yigit
2018-03-06 12:12 ` Pattan, Reshma
0 siblings, 2 replies; 6+ messages in thread
From: Vipin Varghese @ 2018-03-05 7:57 UTC (permalink / raw)
To: dev, reshma.pattan; +Cc: john.mcnamara, Vipin Varghese
dpdk-pdump makes use of LIBRTE_PMD_PCAP for interfacing the ring to
the device-queue pair. Updating Makefile to check for the same.
Signed-off-by: Vipin Varghese <vipin.varghese@intel.com>
---
app/pdump/Makefile | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/app/pdump/Makefile b/app/pdump/Makefile
index bd3c208..038a34f 100644
--- a/app/pdump/Makefile
+++ b/app/pdump/Makefile
@@ -3,6 +3,10 @@
include $(RTE_SDK)/mk/rte.vars.mk
+ifeq ($(CONFIG_RTE_LIBRTE_PMD_PCAP),n)
+$(error "Please enable CONFIG_RTE_LIBRTE_PMD_PCAP")
+endif
+
ifeq ($(CONFIG_RTE_LIBRTE_PDUMP),y)
APP = dpdk-pdump
--
1.9.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH v1] app/pdump: add check for PCAP PMD
2018-03-05 7:57 [dpdk-dev] [PATCH v1] app/pdump: add check for PCAP PMD Vipin Varghese
@ 2018-03-05 14:32 ` Ferruh Yigit
2018-03-06 8:45 ` Varghese, Vipin
2018-03-06 12:12 ` Pattan, Reshma
1 sibling, 1 reply; 6+ messages in thread
From: Ferruh Yigit @ 2018-03-05 14:32 UTC (permalink / raw)
To: Vipin Varghese, dev, reshma.pattan; +Cc: john.mcnamara
On 3/5/2018 7:57 AM, Vipin Varghese wrote:
> dpdk-pdump makes use of LIBRTE_PMD_PCAP for interfacing the ring to
> the device-queue pair. Updating Makefile to check for the same.
>
> Signed-off-by: Vipin Varghese <vipin.varghese@intel.com>
> ---
> app/pdump/Makefile | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/app/pdump/Makefile b/app/pdump/Makefile
> index bd3c208..038a34f 100644
> --- a/app/pdump/Makefile
> +++ b/app/pdump/Makefile
> @@ -3,6 +3,10 @@
>
> include $(RTE_SDK)/mk/rte.vars.mk
>
> +ifeq ($(CONFIG_RTE_LIBRTE_PMD_PCAP),n)
> +$(error "Please enable CONFIG_RTE_LIBRTE_PMD_PCAP")
> +endif
pdump is enabled default, so won't this break the default build?
What about moving this to lib/librte_pdump, convert $(error ..) to $(warning ..)
and disable CONFIG_RTE_LIBRTE_PDUMP there?
> +
> ifeq ($(CONFIG_RTE_LIBRTE_PDUMP),y)
>
> APP = dpdk-pdump
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH v1] app/pdump: add check for PCAP PMD
2018-03-05 14:32 ` Ferruh Yigit
@ 2018-03-06 8:45 ` Varghese, Vipin
2018-03-06 11:57 ` Ferruh Yigit
0 siblings, 1 reply; 6+ messages in thread
From: Varghese, Vipin @ 2018-03-06 8:45 UTC (permalink / raw)
To: Yigit, Ferruh, dev, Pattan, Reshma; +Cc: Mcnamara, John
Hi Ferruh,
> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Monday, March 5, 2018 2:33 PM
> To: Varghese, Vipin <vipin.varghese@intel.com>; dev@dpdk.org; Pattan,
> Reshma <reshma.pattan@intel.com>
> Cc: Mcnamara, John <john.mcnamara@intel.com>
> Subject: Re: [dpdk-dev] [PATCH v1] app/pdump: add check for PCAP PMD
>
> On 3/5/2018 7:57 AM, Vipin Varghese wrote:
> > dpdk-pdump makes use of LIBRTE_PMD_PCAP for interfacing the ring to
> > the device-queue pair. Updating Makefile to check for the same.
> >
> > Signed-off-by: Vipin Varghese <vipin.varghese@intel.com>
> > ---
> > app/pdump/Makefile | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/app/pdump/Makefile b/app/pdump/Makefile index
> > bd3c208..038a34f 100644
> > --- a/app/pdump/Makefile
> > +++ b/app/pdump/Makefile
> > @@ -3,6 +3,10 @@
> >
> > include $(RTE_SDK)/mk/rte.vars.mk
> >
> > +ifeq ($(CONFIG_RTE_LIBRTE_PMD_PCAP),n) $(error "Please enable
> > +CONFIG_RTE_LIBRTE_PMD_PCAP") endif
>
> pdump is enabled default, so won't this break the default build?
Yes, you are right it will fail. Which then forces the user to enable PCAP.
>
> What about moving this to lib/librte_pdump, convert $(error ..) to $(warning ..)
> and disable CONFIG_RTE_LIBRTE_PDUMP there?
If we set to warning and there are no PCAP headers in build system. The application gets built, but will fail internally becz the pcap API will fails during execution.
>
> > +
> > ifeq ($(CONFIG_RTE_LIBRTE_PDUMP),y)
> >
> > APP = dpdk-pdump
> >
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH v1] app/pdump: add check for PCAP PMD
2018-03-06 8:45 ` Varghese, Vipin
@ 2018-03-06 11:57 ` Ferruh Yigit
0 siblings, 0 replies; 6+ messages in thread
From: Ferruh Yigit @ 2018-03-06 11:57 UTC (permalink / raw)
To: Varghese, Vipin, dev, Pattan, Reshma; +Cc: Mcnamara, John
On 3/6/2018 8:45 AM, Varghese, Vipin wrote:
> Hi Ferruh,
>
>> -----Original Message-----
>> From: Yigit, Ferruh
>> Sent: Monday, March 5, 2018 2:33 PM
>> To: Varghese, Vipin <vipin.varghese@intel.com>; dev@dpdk.org; Pattan,
>> Reshma <reshma.pattan@intel.com>
>> Cc: Mcnamara, John <john.mcnamara@intel.com>
>> Subject: Re: [dpdk-dev] [PATCH v1] app/pdump: add check for PCAP PMD
>>
>> On 3/5/2018 7:57 AM, Vipin Varghese wrote:
>>> dpdk-pdump makes use of LIBRTE_PMD_PCAP for interfacing the ring to
>>> the device-queue pair. Updating Makefile to check for the same.
>>>
>>> Signed-off-by: Vipin Varghese <vipin.varghese@intel.com>
>>> ---
>>> app/pdump/Makefile | 4 ++++
>>> 1 file changed, 4 insertions(+)
>>>
>>> diff --git a/app/pdump/Makefile b/app/pdump/Makefile index
>>> bd3c208..038a34f 100644
>>> --- a/app/pdump/Makefile
>>> +++ b/app/pdump/Makefile
>>> @@ -3,6 +3,10 @@
>>>
>>> include $(RTE_SDK)/mk/rte.vars.mk
>>>
>>> +ifeq ($(CONFIG_RTE_LIBRTE_PMD_PCAP),n) $(error "Please enable
>>> +CONFIG_RTE_LIBRTE_PMD_PCAP") endif
>>
>> pdump is enabled default, so won't this break the default build?
>
> Yes, you are right it will fail. Which then forces the user to enable PCAP.
We shouldn't break the default build because of missing dependencies.
>
>>
>> What about moving this to lib/librte_pdump, convert $(error ..) to $(warning ..)
>> and disable CONFIG_RTE_LIBRTE_PDUMP there?
>
> If we set to warning and there are no PCAP headers in build system. The application gets built, but will fail internally becz the pcap API will fails during execution.
if CONFIG_RTE_LIBRTE_PDUMP disabled application won't be compiled
>
>>
>>> +
>>> ifeq ($(CONFIG_RTE_LIBRTE_PDUMP),y)
>>>
>>> APP = dpdk-pdump
>>>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH v1] app/pdump: add check for PCAP PMD
2018-03-05 7:57 [dpdk-dev] [PATCH v1] app/pdump: add check for PCAP PMD Vipin Varghese
2018-03-05 14:32 ` Ferruh Yigit
@ 2018-03-06 12:12 ` Pattan, Reshma
2018-03-12 17:46 ` Varghese, Vipin
1 sibling, 1 reply; 6+ messages in thread
From: Pattan, Reshma @ 2018-03-06 12:12 UTC (permalink / raw)
To: Varghese, Vipin, dev; +Cc: Mcnamara, John
>
> +ifeq ($(CONFIG_RTE_LIBRTE_PMD_PCAP),n)
> +$(error "Please enable CONFIG_RTE_LIBRTE_PMD_PCAP") endif
> +
How about combining If(($(CONFIG_RTE_LIBRTE_PMD_PCAP),y) check with below existing if check?
with this, dpdk-pdump will be compiled only when both the flags are enabled.
> ifeq ($(CONFIG_RTE_LIBRTE_PDUMP),y)
>
> APP = dpdk-pdump
> --
> 1.9.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH v1] app/pdump: add check for PCAP PMD
2018-03-06 12:12 ` Pattan, Reshma
@ 2018-03-12 17:46 ` Varghese, Vipin
0 siblings, 0 replies; 6+ messages in thread
From: Varghese, Vipin @ 2018-03-12 17:46 UTC (permalink / raw)
To: Pattan, Reshma, dev; +Cc: Mcnamara, John
Hi Reshma,
I tried your suggestion, but the build fails even using warning or not using warning. The reason is DPDK build expects pdump to be build.
I am looking at other alternatives, how to get this done. In last effort, we may need to disable PDUMP to be build default from '.config' file.
Thanks
Vipin Varghese
> -----Original Message-----
> From: Pattan, Reshma
> Sent: Tuesday, March 6, 2018 5:42 PM
> To: Varghese, Vipin <vipin.varghese@intel.com>; dev@dpdk.org
> Cc: Mcnamara, John <john.mcnamara@intel.com>
> Subject: RE: [PATCH v1] app/pdump: add check for PCAP PMD
>
> >
> > +ifeq ($(CONFIG_RTE_LIBRTE_PMD_PCAP),n) $(error "Please enable
> > +CONFIG_RTE_LIBRTE_PMD_PCAP") endif
> > +
>
> How about combining If(($(CONFIG_RTE_LIBRTE_PMD_PCAP),y) check with
> below existing if check?
> with this, dpdk-pdump will be compiled only when both the flags are enabled.
>
> > ifeq ($(CONFIG_RTE_LIBRTE_PDUMP),y)
> >
> > APP = dpdk-pdump
> > --
> > 1.9.1
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-03-12 17:57 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-05 7:57 [dpdk-dev] [PATCH v1] app/pdump: add check for PCAP PMD Vipin Varghese
2018-03-05 14:32 ` Ferruh Yigit
2018-03-06 8:45 ` Varghese, Vipin
2018-03-06 11:57 ` Ferruh Yigit
2018-03-06 12:12 ` Pattan, Reshma
2018-03-12 17:46 ` Varghese, Vipin
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).