Skip to content

Commit

Permalink
Merge branch 'dsa-fast-ageing'
Browse files Browse the repository at this point in the history
Vladimir Oltean says:

====================
DSA fast ageing fixes/improvements

These are 2 small improvements brought to the DSA fast ageing changes
merged earlier today.

Patch 1 restores the behavior for DSA drivers that don't implement the
.port_bridge_flags function (I don't think there is any breakage due
to the new behavior, but just to be sure). This came as a result of
Andrew's review.

Patch 2 reduces the number of fast ages of a port from 2 to 1 when it
leaves a bridge.
====================

Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
David S. Miller committed Aug 9, 2021
2 parents cfe908c + bee7c57 commit 0033523
Showing 1 changed file with 20 additions and 2 deletions.
22 changes: 20 additions & 2 deletions net/dsa/port.c
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,21 @@ static void dsa_port_fast_age(const struct dsa_port *dp)
dsa_port_notify_bridge_fdb_flush(dp);
}

static bool dsa_port_can_configure_learning(struct dsa_port *dp)
{
struct switchdev_brport_flags flags = {
.mask = BR_LEARNING,
};
struct dsa_switch *ds = dp->ds;
int err;

if (!ds->ops->port_bridge_flags || !ds->ops->port_pre_bridge_flags)
return false;

err = ds->ops->port_pre_bridge_flags(ds, dp->index, flags, NULL);
return !err;
}

int dsa_port_set_state(struct dsa_port *dp, u8 state, bool do_fast_age)
{
struct dsa_switch *ds = dp->ds;
Expand All @@ -70,7 +85,8 @@ int dsa_port_set_state(struct dsa_port *dp, u8 state, bool do_fast_age)

ds->ops->port_stp_state_set(ds, port, state);

if (do_fast_age && dp->learning) {
if (!dsa_port_can_configure_learning(dp) ||
(do_fast_age && dp->learning)) {
/* Fast age FDB entries or flush appropriate forwarding database
* for the given port, if we are moving it from Learning or
* Forwarding state, to Disabled or Blocking or Listening state.
Expand Down Expand Up @@ -683,7 +699,9 @@ int dsa_port_bridge_flags(struct dsa_port *dp,
if (learning == dp->learning)
return 0;

if (dp->learning && !learning)
if ((dp->learning && !learning) &&
(dp->stp_state == BR_STATE_LEARNING ||
dp->stp_state == BR_STATE_FORWARDING))
dsa_port_fast_age(dp);

dp->learning = learning;
Expand Down

0 comments on commit 0033523

Please sign in to comment.