On Fri, 2024-11-08 at 14:42 +0100, Andrew Lunn wrote: > This sounds a bit hacky. Could you add a new optional attribute to the > netlink message indicating the roam destination, so there is no need > to play games with the actual port? Yes that's another option. > I'm not too deep into how these all works, but i also wounder about > backwards compatibility. Old code which does not look for > FDB_NOTIFY_ROAMING_BIT is going to think it really has moved, with > your code. By using a new attribute, and not changing the port, old > code just sees a notification it is on the port it always was on, > which is less likely to cause issues? Thanks that's a good point. I'll have a look at making it an attribute instead. > And do we want to differentiate between it wants to roam, but the > sticky bit has stopped that, and it really has roamed? That is partly what this patch is trying to do, since actually roaming hosts will already trigger a notification with the new port. I will try your suggestion as it seems better than what I have here. I also think moving away from relying on the sticky bit might be good. This behaviour relies on the port being locked in hardware so it might be more appropriate for this to be part of locked behaviour in the kernel also?
