-
Notifications
You must be signed in to change notification settings - Fork 340
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
T6944: adds option to enable switchdev mode on ethernet interface #4235
base: current
Are you sure you want to change the base?
Conversation
👍 |
8cc339c
to
f8cb50d
Compare
Tested on a system with Mellanox ConnectX-5 ✅ |
I'd probably limit this feature to a NIC that was tested good, like the MLX CX5 and only allow it here. |
Works on CX4 and CX5, so don't see why it shouldn't work on CX6. AFAIK it also works on Intel cards like E810. Could be done, but other offload methods also don't work on all cards and end in a warning if not supported. |
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.
I'd be uneasy about hardcoded HCLs in scripts. We should certainly have lists of tested NICs in the docs. But if we don't allow people to try that option on new NICs, people will be unable to test them and report their findings.
We may add a list of knowingly problematic NICs, if any such NICs show up, and if trying to add the option makes the NIC entirely stop working until a hard reset or similar.
From my testing so far, more consumer NICs like I210-AT or virto just don't support it and fail to enable it (currently resulting in a message like enabling unsupported offload options). Afaik it must be supported by the NIC driver as well. |
475e27f
to
6159426
Compare
CI integration ❌ failed! Details
|
Change Summary
Adds ethernet interface config option to enable switchdev mode.
Types of changes
Related Task(s)
Related PR(s)
Component(s) name
ethernet
Proposed changes
Adds ethernet interface config option to enable switchdev mode.
How to test
On a capable NIC set
Get PCIe address of that NIC to verify state
Smoketest result
Does not work on virtio_net adapters, requires manual testing on capable hardware
Checklist: