<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body lang="en-IL" link="blue" vlink="purple" style="word-wrap:break-word"> <div class="WordSection1"> Understood. Thanks.<o:p></o:p> <o:p> </o:p> <div> Regards,<o:p></o:p> Maayan Kashani<o:p></o:p> </div> <o:p> </o:p> <div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt"> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0cm 0cm 0cm"> From: Stephen Hemminger <stephen@networkplumber.org> Sent: Wednesday, 8 July 2026 10:11 To: Maayan Kashani <mkashani@nvidia.com> Cc: Bruce Richardson <bruce.richardson@intel.com>; dev <dev@dpdk.org>; Raslan Darawsheh <rasland@nvidia.com>; stable <stable@dpdk.org>; David Marchand <david.marchand@redhat.com> Subject: Re: [PATCH] build: fix libpcap detection with unusable header<o:p></o:p> </div> </div> <o:p> </o:p> <table class="MsoNormalTable" border="1" cellpadding="0" style="background:#FFEB9C"> <tbody> <tr> <td style="padding:.75pt .75pt .75pt .75pt"> External email: Use caution opening links or attachments <o:p></o:p> </td> </tr> </tbody> </table> <o:p> </o:p> <div> <div> Not sure. Working around busted packages in DPDK is a bad idea.<o:p></o:p> </div> <o:p> </o:p> <div> <div> On Wed, Jul 8, 2026, 08:38 Maayan Kashani <<a href="mailto:mkashani@nvidia.com">mkashani@nvidia.com</a>> wrote:<o:p></o:p> </div> <blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm"> Hi, Bruce, we encountered DPDK build issue on Tencent 4.6. It is not expected; the package/image appears corrupted. RPM verification confirms the headers differ from the packaged contents. The DPDK change is still valid because optional dependency detection should test for usable declarations, not just file existence and library presence.” Regards, Maayan Kashani > -----Original Message----- > From: Bruce Richardson <<a href="mailto:bruce.richardson@intel.com" target="_blank">bruce.richardson@intel.com</a>> > Sent: Tuesday, 7 July 2026 17:16 > To: Maayan Kashani <<a href="mailto:mkashani@nvidia.com" target="_blank">mkashani@nvidia.com</a>> > Cc: <a href="mailto:dev@dpdk.org" target="_blank">dev@dpdk.org</a>; Raslan Darawsheh <<a href="mailto:rasland@nvidia.com" target="_blank">rasland@nvidia.com</a>>; > <a href="mailto:stable@dpdk.org" target="_blank">stable@dpdk.org</a>; David Marchand <<a href="mailto:david.marchand@redhat.com" target="_blank">david.marchand@redhat.com</a>>; Stephen > Hemminger <<a href="mailto:stephen@networkplumber.org" target="_blank">stephen@networkplumber.org</a>> > Subject: Re: [PATCH] build: fix libpcap detection with unusable header > > External email: Use caution opening links or attachments > > > On Tue, Jul 07, 2026 at 04:30:28PM +0300, Maayan Kashani wrote: > > The libpcap detection only checked that the library was present, that > > <pcap.h> could be included, and that a trivial program links. None of > > these fail when a distribution ships a libpcap-devel package whose > > <pcap.h> is empty (present but declaring nothing) while libpcap.so is > > installed: has_header() passes on an empty file and the link test does > > not reference any pcap symbol. RTE_HAS_LIBPCAP was then set and the > > pcap-dependent code (bpf_convert.c, port source/sink, dumpcap) failed > > to build with errors such as: > > > > Why would a distribution ship a devel package with an empty header file for > pcap.h? Is that expected behaviour in some scenario that I'm unaware of, > because it seems strange to me? > > /Bruce > > > bpf_convert.c: error: invalid use of undefined type > > 'const struct bpf_insn' > > > > Require an actual pcap declaration (pcap_create) to be visible in the > > header before enabling libpcap support, so a broken or empty header > > disables the optional pcap features instead of breaking the build. > > > > Fixes: d6024c0a6757 ("build: cleanup libpcap dependent components") > > Cc: <a href="mailto:stable@dpdk.org" target="_blank">stable@dpdk.org</a> > > > > Signed-off-by: Maayan Kashani <<a href="mailto:mkashani@nvidia.com" target="_blank">mkashani@nvidia.com</a>> > > --- > > config/meson.build | 5 ++++- > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > > diff --git a/config/meson.build b/config/meson.build index > > d7f5e55c18f..4bfb8535781 100644 > > --- a/config/meson.build > > +++ b/config/meson.build > > @@ -290,7 +290,10 @@ if not pcap_dep.found() > > # pcap got a pkg-config file only in 1.9.0 > > pcap_dep = cc.find_library(pcap_lib, required: false) endif -if > > (pcap_dep.found() and cc.has_header('pcap.h', dependencies: pcap_dep) > > +# has_header() passes even when pcap.h is empty; require a real > declaration. > > +if (pcap_dep.found() > > + and cc.has_header_symbol('pcap.h', 'pcap_create', dependencies: > pcap_dep, > > + args: '-D_GNU_SOURCE') > > and cc.links(min_c_code, dependencies: pcap_dep)) > > dpdk_conf.set('RTE_HAS_LIBPCAP', 1) > > dpdk_extra_ldflags += '-l@0@'.format(pcap_lib) > > -- > > 2.21.0 > ><o:p></o:p> </blockquote> </div> </div> </div> </div> </body> </html>