diff options
| author | Vladimir Oltean <vladimir.oltean@nxp.com> | 2022-07-23 04:24:11 +0300 |
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org> | 2022-07-25 19:37:06 -0700 |
| commit | c7560d1203b7a1ea0b99a5c575547e95d564b2a8 (patch) | |
| tree | 9ae41522750add3ed40077f03e70f59393c95d94 | |
| parent | 5fcbb711024aac6d4db385623e6f2fdf019f7782 (diff) | |
| download | linux-gpio-intel-c7560d1203b7a1ea0b99a5c575547e95d564b2a8.tar.gz | |
net: dsa: fix reference counting for LAG FDBs
Due to an invalid conflict resolution on my side while working on 2
different series (LAG FDBs and FDB isolation), dsa_switch_do_lag_fdb_add()
does not store the database associated with a dsa_mac_addr structure.
So after adding an FDB entry associated with a LAG, dsa_mac_addr_find()
fails to find it while deleting it, because &a->db is zeroized memory
for all stored FDB entries of lag->fdbs, and dsa_switch_do_lag_fdb_del()
returns -ENOENT rather than deleting the entry.
Fixes: c26933639b54 ("net: dsa: request drivers to perform FDB isolation")
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Link: https://lore.kernel.org/r/20220723012411.1125066-1-vladimir.oltean@nxp.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
| -rw-r--r-- | net/dsa/switch.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/dsa/switch.c b/net/dsa/switch.c index 2b56218fc57c7..4dfd68cf61c5a 100644 --- a/net/dsa/switch.c +++ b/net/dsa/switch.c @@ -344,6 +344,7 @@ static int dsa_switch_do_lag_fdb_add(struct dsa_switch *ds, struct dsa_lag *lag, ether_addr_copy(a->addr, addr); a->vid = vid; + a->db = db; refcount_set(&a->refcount, 1); list_add_tail(&a->list, &lag->fdbs); |