Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1445793

Re: [PATCH v2 net-next v2 12/12] net: dsa: mv88e6xxx: add support for DSA ageing time

From Vivien Didelot <vivien.didelot@savoirfairelinux.com>
Newsgroups linux.kernel
Subject Re: [PATCH v2 net-next v2 12/12] net: dsa: mv88e6xxx: add support for DSA ageing time
Date 2016-07-18 22:00 +0200
Message-ID <rWmRQ-26O-25@gated-at.bofh.it> (permalink)
References <rWlM5-1sH-3@gated-at.bofh.it> <rWlM5-1sH-1@gated-at.bofh.it> <rWmf9-1SQ-47@gated-at.bofh.it> <rWmoO-1Wa-27@gated-at.bofh.it> <rWmyu-1ZC-45@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Andrew Lunn <andrew@lunn.ch> writes:

>> Nope, the bridge ageing time is not per-port, even though switchdev ops
>> are per-port by design. This is a switch-wide attribute.
>
> So you are saying the core is doing all the reference counting, etc,
> when swapping between fast and slow ageing?

I don't see how checking for the fastest ageing time would fix support
for multiple bridges... I think that would make the code much more
complex for a small value. Multiple logical bridges on top of a single
physical switch is still a tricky topic.

Maybe we can keep it simple for the moment with this switch-wide
set_ageing_time operation, and later add a patch for the DSA layer to
cache and elect the ageing time per-port or per-bridge.

Thanks,

        Vivien

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v2 net-next v2 12/12] net: dsa: mv88e6xxx: add support for DSA ageing time Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2016-07-18 20:50 +0200
  Re: [PATCH v2 net-next v2 12/12] net: dsa: mv88e6xxx: add support  for DSA ageing time Andrew Lunn <andrew@lunn.ch> - 2016-07-18 21:20 +0200
    Re: [PATCH v2 net-next v2 12/12] net: dsa: mv88e6xxx: add support for DSA ageing time Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2016-07-18 21:30 +0200
      Re: [PATCH v2 net-next v2 12/12] net: dsa: mv88e6xxx: add support  for DSA ageing time Andrew Lunn <andrew@lunn.ch> - 2016-07-18 21:40 +0200
        Re: [PATCH v2 net-next v2 12/12] net: dsa: mv88e6xxx: add support for DSA ageing time Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2016-07-18 22:00 +0200
          Re: [PATCH v2 net-next v2 12/12] net: dsa: mv88e6xxx: add support  for DSA ageing time Andrew Lunn <andrew@lunn.ch> - 2016-07-18 22:20 +0200
            Re: [PATCH v2 net-next v2 12/12] net: dsa: mv88e6xxx: add support for DSA ageing time Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2016-07-18 22:50 +0200
              Re: [PATCH v2 net-next v2 12/12] net: dsa: mv88e6xxx: add support  for DSA ageing time Andrew Lunn <andrew@lunn.ch> - 2016-07-18 23:30 +0200

csiph-web