Re: [PATCH net-next 0/8] Bridge FDB refactoring

From: Vladimir Oltean <olteanv@gmail.com>
To: Nikolay Aleksandrov <nikolay@nvidia.com>
Cc: patchwork-bot+netdevbpf@kernel.org,
	Vladimir Oltean <vladimir.oltean@nxp.com>,
	netdev <netdev@vger.kernel.org>, Ido Schimmel <idosch@nvidia.com>,
	Jakub Kicinski <kuba@kernel.org>,
	"David S. Miller" <davem@davemloft.net>,
	Roopa Prabhu <roopa@nvidia.com>, Andrew Lunn <andrew@lunn.ch>,
	Florian Fainelli <f.fainelli@gmail.com>,
	Vivien Didelot <vivien.didelot@gmail.com>,
	Jiri Pirko <jiri@nvidia.com>
Subject: Re: [PATCH net-next 0/8] Bridge FDB refactoring
Date: Wed, 27 Oct 2021 17:46:05 +0300	[thread overview]
Message-ID: <CA+h21hoYXvATh5_MCDvCUYit11T9Za9DvRnLeNs3yeRRRQ+H=A@mail.gmail.com> (raw)
In-Reply-To: <c9da923f-42b8-8771-8867-9ea35f10da91@nvidia.com>

On Wed, 27 Oct 2021 at 17:44, Nikolay Aleksandrov <nikolay@nvidia.com> wrote:
>
> On 27/10/2021 17:40, patchwork-bot+netdevbpf@kernel.org wrote:
> > Hello:
> >
> > This series was applied to netdev/net-next.git (master)
> > by David S. Miller <davem@davemloft.net>:
> >
> > On Tue, 26 Oct 2021 17:27:35 +0300 you wrote:
> >> This series refactors the br_fdb.c, br_switchdev.c and switchdev.c files
> >> to offer the same level of functionality with a bit less code, and to
> >> clarify the purpose of some functions.
> >>
> >> No functional change intended.
> >>
> >> Vladimir Oltean (8):
> >>   net: bridge: remove fdb_notify forward declaration
> >>   net: bridge: remove fdb_insert forward declaration
> >>   net: bridge: rename fdb_insert to fdb_add_local
> >>   net: bridge: rename br_fdb_insert to br_fdb_add_local
> >>   net: bridge: reduce indentation level in fdb_create
> >>   net: bridge: move br_fdb_replay inside br_switchdev.c
> >>   net: bridge: create a common function for populating switchdev FDB
> >>     entries
> >>   net: switchdev: merge switchdev_handle_fdb_{add,del}_to_device
> >>
> >> [...]
> >
> > Here is the summary with links:
> >   - [net-next,1/8] net: bridge: remove fdb_notify forward declaration
> >     https://git.kernel.org/netdev/net-next/c/4682048af0c8
> >   - [net-next,2/8] net: bridge: remove fdb_insert forward declaration
> >     https://git.kernel.org/netdev/net-next/c/5f94a5e276ae
> >   - [net-next,3/8] net: bridge: rename fdb_insert to fdb_add_local
> >     https://git.kernel.org/netdev/net-next/c/4731b6d6b257
> >   - [net-next,4/8] net: bridge: rename br_fdb_insert to br_fdb_add_local
> >     https://git.kernel.org/netdev/net-next/c/f6814fdcfe1b
> >   - [net-next,5/8] net: bridge: reduce indentation level in fdb_create
> >     https://git.kernel.org/netdev/net-next/c/9574fb558044
> >   - [net-next,6/8] net: bridge: move br_fdb_replay inside br_switchdev.c
> >     https://git.kernel.org/netdev/net-next/c/5cda5272a460
> >   - [net-next,7/8] net: bridge: create a common function for populating switchdev FDB entries
> >     https://git.kernel.org/netdev/net-next/c/fab9eca88410
> >   - [net-next,8/8] net: switchdev: merge switchdev_handle_fdb_{add,del}_to_device
> >     https://git.kernel.org/netdev/net-next/c/716a30a97a52
> >
> > You are awesome, thank you!
> >
>
> There was a discussion about patch 06 which we agreed have to turn into its own series
> with more changes. Vladimir, since the set got applied please send a follow-up to
> finish those changes.

Wait a minute, even I got the impression that the next series I'll be
sending would be completely separate from this one...

>
> Thanks,
>  Nik
>