03/07/2025 18:29, Dylan Schneider:
> I did a clean build on system where libpcap was NOT installed.
> 
> My changes make rte_pcapng lib depend on libpcap (this is for libpcaps 
> DLT_USER Macros).
> So now, in turn, everything that depends on rte_pcapng also depends on 
> libpcap. Dumpcap, rte_graph and rte_pdump all depend on rte_pcapng, so I 
> think we may need to add the libpcap dependency to all of these? We would 
> then need to add the libpcap dependency to any libraries that depend on 
> these, which seems like a slippery slope.  Maybe that's not the best approach 
> though. Can you say whether it is okay to add libpcap dependency to all of 
> these libraries? Or maybe there's a better way to enforce the dependency for 
> DPDK build? Or is it altogether incorrect to have rte_pcapng depend on 
> libpcap in the first place?

The best is to have only rte_pcapng to depend on libpcap, as you did.

If the problem is only in ABI check, this is what we need to fix.
I'm not sure how to fix this.



> From: Thomas Monjalon <tho...@monjalon.net>
> Sent: Thursday, July 3, 2025 7:27 AM
> To: Dylan Schneider <schne...@qti.qualcomm.com>
> Cc: dev@dpdk.org <dev@dpdk.org>; Reshma Pattan <reshma.pat...@intel.com>; 
> Stephen Hemminger <step...@networkplumber.org>; Jerin Jacob 
> <jer...@marvell.com>; Kiran Kumar K <kirankum...@marvell.com>; Nithin 
> Dabilpuram <ndabilpu...@marvell.com>; Zhirun Yan <yanzhirun_...@163.com>
> Subject: Re: [PATCH v2] pcapng: allow any protocol link type for the 
> interface block
> 
> WARNING: This email originated from outside of Qualcomm. Please be wary of 
> any links or attachments, and do not enable macros.
> 
> Please can you compare which libraries are built before/after your patch?
> 
> 
> 02/07/2025 21:30, Dylan Schneider:
> > Hey Thomas,
> > I retried a build when libpcap was absent and was able to reproduce the 
> > same errors as you did.
> >
> > Error: cannot find librte_pcapng.so.25.1 in 
> > dpdk-build/build-arm64-generic-gcc/install
> > Error: cannot find librte_graph.so.25.1 in 
> > dpdk-build/build-arm64-generic-gcc/install
> > Error: cannot find librte_node.so.25.1 in 
> > dpdk-build/build-arm64-generic-gcc/install
> > Error: cannot find librte_pdump.so.25.1 in 
> > dpdk-build/build-arm64-generic-gcc/install
> >
> > These Errors are coming from ABI check: dpdk/devtools/check-abi.sh:49
> >
> > I see in this function that it is checking for the existence of these 
> > libraries, which will be absent when libpcap is absent. Is there a way to 
> > have the abi-check skip these libraries if they are not built?
> > Or better question: How does the abi-check even know about these libraries 
> > if they are never built?
> >
> > Thanks!
> > Dylan
> >
> >
> > ________________________________
> > From: Thomas Monjalon <tho...@monjalon.net>
> > Sent: Monday, June 30, 2025 8:28 AM
> > To: Dylan Schneider <schne...@qti.qualcomm.com>
> > Cc: dev@dpdk.org <dev@dpdk.org>; Reshma Pattan <reshma.pat...@intel.com>; 
> > Stephen Hemminger <step...@networkplumber.org>; Jerin Jacob 
> > <jer...@marvell.com>; Kiran Kumar K <kirankum...@marvell.com>; Nithin 
> > Dabilpuram <ndabilpu...@marvell.com>; Zhirun Yan <yanzhirun_...@163.com>
> > Subject: Re: [PATCH v2] pcapng: allow any protocol link type for the 
> > interface block
> >
> > WARNING: This email originated from outside of Qualcomm. Please be wary of 
> > any links or attachments, and do not enable macros.
> >
> > Sorry I did not investigate, but there is probably a different installation 
> > of the pcap library.
> >
> > I have this in a config file:
> >
> > pcaparm=$deps/libpcap/build-aarch64/install # requires big changes in DPDK
> > #export PKG_CONFIG_PATH=$PKG_CONFIG_PATH:$pcaparm/lib/pkgconfig
> >
> > I don't remember what are the big changes required, and why it is disabled.
> >
> > Checking Meson, libpcap is indeed not found on aarch64:
> >
> >         Run-time dependency libpcap found: NO (tried pkgconfig)
> >         Library pcap found: NO
> >
> > Can you check whether the error can be related to the absence of libpcap?
> >
> >
> >
> > 30/06/2025 15:50, Dylan Schneider:
> > > Hey Thomas, just wanted to confirm,  I see the ARM build passing in CI. 
> > > Is there a reason it would pass there but not on your dev machine?
> > >
> > >
> > > ________________________________
> > > From: Dylan Schneider <schne...@qti.qualcomm.com>
> > > Sent: Friday, June 27, 2025 11:46 AM
> > > To: Thomas Monjalon <tho...@monjalon.net>
> > > Cc: dev@dpdk.org <dev@dpdk.org>; Reshma Pattan <reshma.pat...@intel.com>; 
> > > Stephen Hemminger <step...@networkplumber.org>; Jerin Jacob 
> > > <jer...@marvell.com>; Kiran Kumar K <kirankum...@marvell.com>; Nithin 
> > > Dabilpuram <ndabilpu...@marvell.com>; Zhirun Yan <yanzhirun_...@163.com>
> > > Subject: Re: [PATCH v2] pcapng: allow any protocol link type for the 
> > > interface block
> > >
> > >
> > > WARNING: This email originated from outside of Qualcomm. Please be wary 
> > > of any links or attachments, and do not enable macros.
> > >
> > > Hey Thomas,
> > > Thanks for the reply.
> > > Can you tell me the build command you're using to produce those errors? I 
> > > have ran the builds on my test machine and cannot reproduce these. Thanks!
> > > Dylan
> > > ________________________________
> > > From: Thomas Monjalon <tho...@monjalon.net>
> > > Sent: Friday, June 27, 2025 11:39 AM
> > > To: Dylan Schneider <schne...@qti.qualcomm.com>
> > > Cc: dev@dpdk.org <dev@dpdk.org>; Reshma Pattan <reshma.pat...@intel.com>; 
> > > Stephen Hemminger <step...@networkplumber.org>; Jerin Jacob 
> > > <jer...@marvell.com>; Kiran Kumar K <kirankum...@marvell.com>; Nithin 
> > > Dabilpuram <ndabilpu...@marvell.com>; Zhirun Yan <yanzhirun_...@163.com>
> > > Subject: Re: [PATCH v2] pcapng: allow any protocol link type for the 
> > > interface block
> > >
> > > WARNING: This email originated from outside of Qualcomm. Please be wary 
> > > of any links or attachments, and do not enable macros.
> > >
> > > 09/06/2025 23:19, Schneide:
> > > > From: Dylan Schneider <schne...@qti.qualcomm.com>
> > > >
> > > > Allow the user to specify protocol link type when creating pcapng files.
> > > > This change is needed to specify the protocol type in the pcapng file,
> > > > DLT_EN10MB specifies ethernet packets only. This will allow dissectors
> > > > for other protocols to be used on files generated by pcapng.
> > > >
> > > > Includes a breaking change to rte_pcapng_add_interface to add link_type
> > > > parameter. Existing calls to the function have been updated to pass
> > > > DLT_EN10MB for the link type argument.
> > > >
> > > > Fixes: d1da6d0d04c7 ("pcapng: require per-interface information")
> > > > Signed-off-by: Dylan Schneider <schne...@qti.qualcomm.com>
> > > > Cc: step...@networkplumber.org
> > >
> > > I doesn't pass compilation test on my machine:
> > >
> > > Error: cannot find librte_pcapng.so.25.1 in 
> > > dpdk-build/build-arm64-generic-gcc/install
> > > Error: cannot find librte_graph.so.25.1 in 
> > > dpdk-build/build-arm64-generic-gcc/install
> > > Error: cannot find librte_node.so.25.1 in 
> > > dpdk-build/build-arm64-generic-gcc/install
> > > Error: cannot find librte_pdump.so.25.1 in 
> > > dpdk-build/build-arm64-generic-gcc/install





Reply via email to