From: David Miller <davem@davemloft.net>
To: sfeldma@gmail.com
Cc: netdev@vger.kernel.org, jiri@resnulli.us,
siva.mannem.lnx@gmail.com, pjonnala@broadcom.com,
stephen@networkplumber.org, roopa@cumulusnetworks.com,
andrew@lunn.ch, f.fainelli@gmail.com,
vivien.didelot@savoirfairelinux.com
Subject: Re: [PATCH net-next v3 0/7] bridge: don't age out externally added FDB entries
Date: Wed, 23 Sep 2015 14:36:46 -0700 (PDT) [thread overview]
Message-ID: <20150923.143646.1111978912183035009.davem@davemloft.net> (raw)
In-Reply-To: <1443022760-21301-1-git-send-email-sfeldma@gmail.com>
From: sfeldma@gmail.com
Date: Wed, 23 Sep 2015 08:39:13 -0700
> From: Scott Feldman <sfeldma@gmail.com>
>
> v3: Per davem review: add del_timer_sync on rocker port remove.
>
> v2: Per Jiri review comment: add BR_DEFAULT_AGEING_TIME to defines
>
> Siva originally proposed skipping externally added FDB entries in the bridge's
> FDB garbage collection func, and moving the ageing of externally added entries
> to the port driver/device. This broke rocker, since rocker didn't have a
> hardware (or software) mechanism for ageing out its learned FDB entries.
>
> This patchset reintroduces Siva's bridge driver patch to skip externally added
> entries and adds support in rocker so rocker can age out its own entries.
> Rocker does this using a software timer similar to the bridge's FDB garbage
> collection timer. Other switchdev devices/drivers can use this software timer
> method or program the device to nofity aged-out entries to the driver.
>
> Updated switchdev.txt documentation to reflect current state-of-the-art. This
> removes one more XXX todo comment in switchdev.txt.
Series applied, thanks Scott.
prev parent reply other threads:[~2015-09-23 21:36 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-23 15:39 [PATCH net-next v3 0/7] bridge: don't age out externally added FDB entries sfeldma
2015-09-23 15:39 ` [PATCH net-next v3 1/7] rocker: track when FDB entry is touched sfeldma
2015-09-23 15:39 ` [PATCH net-next v3 2/7] rocker: store rocker_port in fdb key rather than pport sfeldma
2015-09-23 15:39 ` [PATCH net-next v3 3/7] bridge: define some min/max/default ageing time constants sfeldma
2015-09-23 15:49 ` Jiri Pirko
2015-09-23 15:39 ` [PATCH net-next v3 4/7] rocker: adding port ageing_time for ageing out FDB entries sfeldma
2015-09-23 15:49 ` Jiri Pirko
2015-09-23 15:39 ` [PATCH net-next v3 5/7] rocker: add FDB cleanup timer sfeldma
2015-09-23 15:39 ` [PATCH net-next v3 6/7] bridge: don't age externally added FDB entries sfeldma
2015-09-23 15:39 ` [PATCH net-next v3 7/7] switchdev: update documentation on FDB ageing_time sfeldma
2015-09-23 21:36 ` David Miller [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150923.143646.1111978912183035009.davem@davemloft.net \
--to=davem@davemloft.net \
--cc=andrew@lunn.ch \
--cc=f.fainelli@gmail.com \
--cc=jiri@resnulli.us \
--cc=netdev@vger.kernel.org \
--cc=pjonnala@broadcom.com \
--cc=roopa@cumulusnetworks.com \
--cc=sfeldma@gmail.com \
--cc=siva.mannem.lnx@gmail.com \
--cc=stephen@networkplumber.org \
--cc=vivien.didelot@savoirfairelinux.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).