-
Notifications
You must be signed in to change notification settings - Fork 325
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
linux: net: dsa: mt7530: add support for bridge port isolation #3286
base: main
Are you sure you want to change the base?
Conversation
Adds bridge port isolation to MT7621-based devices like the Ubiquiti EdgeRouter-X.
9f8a150
to
8f3b422
Compare
Updated to the version accepted for net-next and OpenWrt main |
for me, stuff that was accepted in net-next and openwrt main doesnt need a further review by gluon maintainers, it's only a question about whether we want to backport it. |
I did have to make some changes to the patches for 5.15, but yes, the main question is whether we want the backport. |
Is there a reason to track this backport downstream and not in the OpenWrt repo? Seems like something that's not gluon specific. Otherwise it seems justifiable to backport it. |
Arguably it's a breaking change, so I'd rather not backport it to OpenWrt stable branches. |
From my understanding bridge port isolation is only enabled if that setting has been enabled and AFAIK it's not by default in openwrt (only gluon enables it by default). |
The first patch affects all drivers, not just those that actually support the offloading; it also doesn't only affect port isolation, but also other bridge flags. It's difficult to say what the exact effects might be. |
let's test in master, we can revert later if we have issues with this @neocturne i suggest you try you build and if necessary refresh patch with current master branch and merge it youself |
Status: I want to include the qca8k patches as well (not sure if they have been backported to OpenWrt main already) |
Adds bridge port isolation to MT7621-based devices like the Ubiquiti EdgeRouter-X.
Part of #2679.