[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CA+CtxLTjEV3eWLHO9f0cBM-T6b-+b6nkPdJ8iLDxsUjenTYxNQ@mail.gmail.com>
Date: Sat, 21 Feb 2015 06:53:04 +0530
From: Siva Mannem <siva.mannem.lnx@...il.com>
To: Scott Feldman <sfeldma@...il.com>
Cc: Netdev <netdev@...r.kernel.org>,
Jiří Pírko <jiri@...nulli.us>,
Roopa Prabhu <roopa@...ulusnetworks.com>, linux@...ck-us.net,
f.fainelli@...il.com, andrew@...n.ch, gospo@...ulusnetworks.com,
vbandaru@...adcom.com
Subject: Re: [PATCH net-next RFC 0/5] Add NTF_EXT_AGED to control FDB ageing
in SW or HW
On Fri, Feb 20, 2015 at 12:39 PM, <sfeldma@...il.com> wrote:
> From: Scott Feldman <sfeldma@...il.com>
>
> Add a new NTF_EXT_FLAG to mark an FDB as externally aged, for example by
> offload hardware. Switchdev driver/devices can set this flag when learning a
> new FDB entry and SW (the bridge driver) will skip this entry when running its
> ageing task. If flag is set, the driver/device is responsible for calling
> call_netdev_switch_notifiers(NETDEV_SWITCH_FDB_DEL, ...) when entry expires.
>
> This give the flexibility for driver/device to decide ageing policy based on
> its capabilities. For devices managing many FDB entries, it is desireable for
> the device to aged out its own entries. Devices not capable of aged entries
> can rely of SW to age out the entries.
>
> Scott Feldman (5):
> neighbour: add external aged flag
> switchdev: add ntf_flags to FDB notifier
> bridge: call external learn add if adding FDB entry with
> NTF_EXT_LEARNED set
> bridge: let HW control FDB ageing by setting NTF_EXT_AGED
> rocker: explicitly set SW ageing for rocker
>
> drivers/net/ethernet/rocker/rocker.c | 9 +++++----
> include/net/switchdev.h | 1 +
> include/uapi/linux/neighbour.h | 1 +
> net/bridge/br.c | 18 ++++++++++--------
> net/bridge/br_fdb.c | 16 +++++++++++++---
> net/bridge/br_private.h | 8 +++++---
> 6 files changed, 35 insertions(+), 18 deletions(-)
>
> --
> 1.7.10.4
>
Patches look good. With these, my earlier patch
(http://patchwork.ozlabs.org/patch/435597/) is no longer required.
--
Regards,
Siva Mannem.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists