Skip to content

Improved smart pointer usages to KniDevice and DpdkDevice. #1901

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

Open
wants to merge 8 commits into
base: dev
Choose a base branch
from

Conversation

Dimi1010
Copy link
Collaborator

This PR rewrites some functions that were using manual lifetime tracking with std::unique_ptr.

@Dimi1010 Dimi1010 marked this pull request as ready for review July 26, 2025 17:13
@Dimi1010 Dimi1010 requested a review from seladb as a code owner July 26, 2025 17:13
@seladb
Copy link
Owner

seladb commented Jul 27, 2025

@Dimi1010 did you have a chance to run the DPDK test locally?
Unfortunately, the CI doesn't run them because setting up DPDK in CI is not easy, so we need to run them locally...

@Dimi1010
Copy link
Collaborator Author

@Dimi1010 did you have a chance to run the DPDK test locally? Unfortunately, the CI doesn't run them because setting up DPDK in CI is not easy, so we need to run them locally...

Unfortunately, no. :/ I mostly have it setup to validate it can build, but unsure how, or even if it can be setup to run on WSL, and I don't have a bare-metal Unix OS currently.

@seladb
Copy link
Owner

seladb commented Jul 27, 2025

@Dimi1010 did you have a chance to run the DPDK test locally? Unfortunately, the CI doesn't run them because setting up DPDK in CI is not easy, so we need to run them locally...

Unfortunately, no. :/ I mostly have it setup to validate it can build, but unsure how, or even if it can be setup to run on WSL, and I don't have a bare-metal Unix OS currently.

I don't think DPDK can run in WSL 😕
I also don't have a bare-metal Linux device, but I have Ubuntu running inside a VirtualBox VM, and DPDK can be easily set up there. Do you think you can set one up? If not, I can test it on my VM...

@Dimi1010
Copy link
Collaborator Author

@Dimi1010 did you have a chance to run the DPDK test locally? Unfortunately, the CI doesn't run them because setting up DPDK in CI is not easy, so we need to run them locally...

Unfortunately, no. :/ I mostly have it setup to validate it can build, but unsure how, or even if it can be setup to run on WSL, and I don't have a bare-metal Unix OS currently.

I don't think DPDK can run in WSL 😕 I also don't have a bare-metal Linux device, but I have Ubuntu running inside a VirtualBox VM, and DPDK can be easily set up there. Do you think you can set one up? If not, I can test it on my VM...

Sure, I can try to set one up, if you know it can run there. :)

@seladb
Copy link
Owner

seladb commented Jul 27, 2025

Sure, I can try to set one up, if you know it can run there. :)

You can follow the instructions here: https://pcapplusplus.github.io/docs/dpdk
It's a good opportunity to see if these instructions are accurate, up to date, and clear 🙂

@seladb
Copy link
Owner

seladb commented Jul 27, 2025

BTW, maybe at some point we need to figure out how to run DPDK tests in CI, but I gave some thought to it and it won't be easy...

@Dimi1010
Copy link
Collaborator Author

Sure, I can try to set one up, if you know it can run there. :)

You can follow the instructions here: https://pcapplusplus.github.io/docs/dpdk It's a good opportunity to see if these instructions are accurate, up to date, and clear 🙂

the docs still say we test against ubuntu 20.04 and 22.04 when the CI runs on 22.04 and 24.04

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

Successfully merging this pull request may close these issues.

2 participants