Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pcap-dpdk.c fails to compile using DPDK 22.11.7 #1461

Open
infrastation opened this issue Feb 12, 2025 · 2 comments
Open

pcap-dpdk.c fails to compile using DPDK 22.11.7 #1461

infrastation opened this issue Feb 12, 2025 · 2 comments

Comments

@infrastation
Copy link
Member

Steps to reproduce:

  1. Get a Debian 12 host.
  2. apt-get install pkg-config dpdk-dev
  3. Build libpcap normally.
pcap-dpdk.c:200:18: error: 'struct rte_eth_rxmode' has no member named 'split_hdr_size'
pcap-dpdk.c:203:28: error: 'ETH_MQ_TX_NONE' undeclared here (not in a function); did you mean 'RTE_ETH_MQ_TX_NONE'?
pcap-dpdk.c:328:22: error: variable 'gather_len' set but not used [-Werror=unused-but-set-variable]
pcap-dpdk.c:500:38: error: 'ETH_LINK_UP' undeclared (first use in this function); did you mean 'RTE_ETH_LINK_UP'?
pcap-dpdk.c:836:48: error: 'DEV_TX_OFFLOAD_MBUF_FAST_FREE' undeclared (first use in this function); did you mean 'RTE_ETH_TX_OFFLOAD_MBUF_FAST_FREE'?
pcap-dpdk.c:973:62: error: 'ETH_LINK_FULL_DUPLEX' undeclared (first use in this function); did you mean 'RTE_ETH_LINK_FULL_DUPLEX'?
pcap-dpdk.c:501:1: error: control reaches end of non-void function [-Werror=return-type]

@guyharris
Copy link
Member

Is it worth the effort to provide a libpcap module that depends on a set of APIs that appears to continuously change in incompatible ways? Or should we just get rid of pcap-dpdk.c? At some point I remember either an email to tcpdump-workers or another list, or a comment on an issue or pull request, saying that the current pcap-dpdk.c isn't doing DPDK capture right (and indicating what at least one problem was), suggesting what a better way would be and, I think, talking about submitting a replacement.

@infrastation
Copy link
Member Author

There is quite a bit of traffic on DPDK mailing lists, perhaps there is someone there would would be willing to make it good?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants