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.

Sign up for GitHub

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

Jump to bottom

AF_XDP support #507

Open
vodik opened this issue Jan 29, 2023 · 8 comments · May be fixed by #1096
Open

AF_XDP support #507

vodik opened this issue Jan 29, 2023 · 8 comments · May be fixed by #1096
Assignees
@vodik

Comments

@vodik
Copy link

vodik commented Jan 29, 2023

Hey, so I see no support for AF_XDP. I'm interested in using that, so I'm currently thinking of working on it.

I'm starting with support for xsk_umem and will work up from there. Opening an issue mostly to see if there's any advice, interest, or if I'm duplicating ongoing work.
@alessandrod
Copy link
Collaborator

Hi! Just assigned the issue to you 😊

Sounds great! There's definitely interest and it's been close to the top of my todo list for a while. Very happy to hand it off to you and also happy to assist with testing, reviews etc along the way.

@vodik
Copy link
Author

vodik commented Jan 29, 2023 •
edited
Loading

I started stumbling around with support here: https://github.com/vodik/aya/tree/xdp-tools

I haven't gotten far yet. So far I've added bindgen support to pull definitions of xdp-tools and started playing with them. I'm still figuring out how this library and xdp-tools work.

In the meanwhile, any thoughts on the bindgen part of the equation? xdp/xsk stuff has been split out of libpbf and moved into xdp-tools. Should we make it optional? Put it in another bindings file? In another crate?

@alessandrod
Copy link
Collaborator

I started stumbling around with support here: https://github.com/vodik/aya/tree/xdp-tools

In the meanwhile, any thoughts on the bindgen part of the equation? xdp/xsk stuff has been split out of libpbf and moved into xdp-tools. Should we make it optional? Put it in another bindings file? In another crate?

I skimmed through your branch and now you've put it with the other bindings right? That seems perfectly ok to me. I don't think we need to make it optional.

@vodik
Copy link
Author

vodik commented Feb 5, 2023

Status update, I think I have umem part setup correctly - however terrible the code is.

@brevilo brevilo mentioned this issue Feb 8, 2023
Open
@Tuetuopay Tuetuopay mentioned this issue Feb 20, 2023
Merged
3 tasks
@maryamtahhan
Copy link

Hey Folks
have you seen https://github.com/CloudNativeDataPlane/cndp/tree/main/lang/rs, would it be of interest from an integration POV?

@kjetilkjeka
Copy link

What is the status of this feature? Do you still have ambitions of completing it @vodik? Or is it currently not being worked on? Does anyone have any estimates of the work remaining?

@emanuele-em
Copy link

News on this feature? Is it still under development or the has it been abandoned?

@vegkams
Copy link

vegkams commented Oct 30, 2024

As mentioned on Discord, I'll have a look at this issue

vegkams pushed a commit to vegkams/aya that referenced this issue Nov 26, 2024
aya: adds support for AF_XDP
b3aad24 
Contains user-space functionality for doing
AF_XDP socket interactions. Much is copy-paste
from xdpilone, but with higher-level abstractions
put on top.

It should be usable at a high level without requiring
unsafe code on the library user's side. The more
low-level control is still available, for power users
who need that level of control (and accept the safety
implications).

Fixes: aya-rs#507
@vegkams vegkams linked a pull request Nov 26, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vodik vodik

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

aya: adds support for AF_XDP vegkams/aya
6 participants
@alessandrod @vodik @maryamtahhan @kjetilkjeka @vegkams @emanuele-em