-
Notifications
You must be signed in to change notification settings - Fork 526
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
Plumb QEO to XDP #3632
Plumb QEO to XDP #3632
Conversation
// | ||
// The following logic just tries all interfaces and if it's able to offload | ||
// to any of them, it considers it a success. Long term though, this should | ||
// only offload to the interface that the socket is bound to. | ||
// | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we allow enabling some of interfaces, we need to know which interfaces are offloading? to disable the offloading ones later
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm, but not sure the operation is _ADD or _REMOVE...
Code could be really complex then
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right now, the goal isn't to make this perfect, but to just unblock experimentation. So I think this is good enough right now. What do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, offloading functionality point of view, it should be OK for now
Description
Wire up the initial plumbing to QEO in XDP.
Testing
Must wait for real HW or official tests.
Documentation
N/A