* [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).