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

dpdk: new recipe #24817

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

dpdk: new recipe #24817

wants to merge 12 commits into from

Conversation

valgur
Copy link
Contributor

@valgur valgur commented Aug 4, 2024

Summary

Changes to recipe: dpdk/24.07

Motivation

DPDK: a set of libraries and drivers for fast packet processing

https://www.dpdk.org/
https://github.com/DPDK/dpdk

Packaging status

Details

Continues from #24801 and fills many gaps in that recipe.


@valgur valgur changed the title New/dpdk dpdk: new recipe Aug 4, 2024
@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@valgur valgur marked this pull request as draft August 4, 2024 18:10
@valgur
Copy link
Contributor Author

valgur commented Aug 5, 2024

@wadehunk Can you give this recipe version a try? It builds ok for me locally. I think the issue in CI is related to an old Linux kernel version being used there.

@wadehunk
Copy link
Contributor

wadehunk commented Aug 5, 2024

@valgur Can confirm this builds locally with gcc11-ubuntu22.04 and gcc13-almalinux9.

Looks like the ubuntu16 CI image has kernel version 4.18, but installed kernel headers are for v4.4. The compilation error looks like it will need kernel headers >= 4.6.

@wadehunk wadehunk mentioned this pull request Aug 5, 2024
3 tasks
@conan-center-bot

This comment has been minimized.

@valgur valgur marked this pull request as ready for review August 5, 2024 20:40
@conan-center-bot

This comment has been minimized.

@valgur valgur mentioned this pull request Aug 6, 2024
3 tasks
@valgur
Copy link
Contributor Author

valgur commented Aug 6, 2024

@AbrilRBS @uilianries @perseoGI The PR should be ready for review but the linter marks it as failing due to a dubious error. Maybe you can review the PR regardless of the CI failure. I tagged you just for visibility, not because it's urgent.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@perseoGI
Copy link
Contributor

perseoGI commented Aug 7, 2024

Hi @valgur soon we will introduce a new linter, right now you can ignore that error...
We will try to review this PR soon!
Thank you as always :)

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@AbrilRBS AbrilRBS self-assigned this Oct 31, 2024
@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ✔️

Warning

Conan Center will stop receiving updates for Conan 1.x packages soon - please see announcement.

All green in build 10 (445791435ee6d24f439aecab94cd020051f60e76):

  • dpdk/24.07:
    No packages built ⚠️ (All logs)

Conan v2 pipeline ✔️

Note: Conan v2 builds are now mandatory. Please read our discussion about it.

All green in build 10 (445791435ee6d24f439aecab94cd020051f60e76):

  • dpdk/24.07:
    Built 1 packages out of 10 (All logs)

if self.options.with_libarchive:
self.requires("libarchive/3.7.6")
if self.options.with_libbpf:
self.requires("libbpf/1.4.6")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @valgur!
There is not such version in CCI, could you please check it?
Thank you!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a PR for it: #25931. Will revert it for now, though.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh I see, I'll take a look at that one thanks

@perseoGI perseoGI self-assigned this Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[question] Availability of dpdk recipe in release version [request] dpdk/21.02
5 participants