On 8 November 2022 06:47:07 GMT-04:00, Petr Machata <petrm@xxxxxxxxxx> wrote: >From: Ido Schimmel <idosch@xxxxxxxxxx> > >Currently, FDB entries that are notified to the bridge via >'SWITCHDEV_FDB_ADD_TO_BRIDGE' are always marked as offloaded. With MAB >enabled, this will no longer be universally true. Device drivers will >report locked FDB entries to the bridge to let it know that the >corresponding hosts required authorization, but it does not mean that >these entries are necessarily programmed in the underlying hardware. > >Solve this by determining the offload indication based of the >'offloaded' bit in the FDB notification. > >Signed-off-by: Ido Schimmel <idosch@xxxxxxxxxx> >Reviewed-by: Petr Machata <petrm@xxxxxxxxxx> >Signed-off-by: Petr Machata <petrm@xxxxxxxxxx> >--- > net/bridge/br.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Acked-by: Nikolay Aleksandrov <razor@xxxxxxxxxxxxx> >diff --git a/net/bridge/br.c b/net/bridge/br.c >index 96e91d69a9a8..145999b8c355 100644 >--- a/net/bridge/br.c >+++ b/net/bridge/br.c >@@ -172,7 +172,7 @@ static int br_switchdev_event(struct notifier_block *unused, > break; > } > br_fdb_offloaded_set(br, p, fdb_info->addr, >- fdb_info->vid, true); >+ fdb_info->vid, fdb_info->offloaded); > break; > case SWITCHDEV_FDB_DEL_TO_BRIDGE: > fdb_info = ptr;
