On 16/02/17 22:38, Roopa Prabhu wrote:
> From: Roopa Prabhu <ro...@cumulusnetworks.com>
>
> added_by_external_learn fdb entries are added and expired by
> external entities like switchdev driver or external controllers.
> ageing is already disabled for such entries. Hence, don't
> indicate expiry for such fdb entries.
>
> CC: Nikolay Aleksandrov <niko...@cumulusnetworks.com>
> CC: Jiri Pirko <j...@resnulli.us>
> CC: Ido Schimmel <ido...@mellanox.com>
> Signed-off-by: Roopa Prabhu <ro...@cumulusnetworks.com>
> ---
> net/bridge/br_fdb.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/net/bridge/br_fdb.c b/net/bridge/br_fdb.c
> index 4ac1157..4f598dc 100644
> --- a/net/bridge/br_fdb.c
> +++ b/net/bridge/br_fdb.c
> @@ -65,7 +65,7 @@ static inline unsigned long hold_time(const struct
> net_bridge *br)
> static inline int has_expired(const struct net_bridge *br,
> const struct net_bridge_fdb_entry *fdb)
> {
> - return !fdb->is_static &&
> + return !fdb->is_static && !fdb->added_by_external_learn &&
> time_before_eq(fdb->updated + hold_time(br), jiffies);
> }
>
>
Good catch,
Reviewed-by: Nikolay Aleksandrov <niko...@cumulusnetworks.com>
