Re: [PATCH net-next 0/9] XDP for NXP ENETC

From: Vladimir Oltean <olteanv@gmail.com>
To: patchwork-bot+netdevbpf@kernel.org
Cc: kuba@kernel.org, ast@kernel.org, daniel@iogearbox.net,
	hawk@kernel.org, john.fastabend@gmail.com, andrii@kernel.org,
	kafai@fb.com, songliubraving@fb.com, yhs@fb.com,
	kpsingh@kernel.org, davem@davemloft.net, netdev@vger.kernel.org,
	bpf@vger.kernel.org, alexander.duyck@gmail.com,
	ioana.ciornei@nxp.com, alexandru.marginean@nxp.com,
	claudiu.manoil@nxp.com, ilias.apalodimas@linaro.org,
	vladimir.oltean@nxp.com
Subject: Re: [PATCH net-next 0/9] XDP for NXP ENETC
Date: Thu, 1 Apr 2021 01:55:43 +0300	[thread overview]
Message-ID: <20210331225543.oelvapw3pli45k5q@skbuf> (raw)
In-Reply-To: <161722921847.2890.11454275035323776176.git-patchwork-notify@kernel.org>

On Wed, Mar 31, 2021 at 10:20:18PM +0000, patchwork-bot+netdevbpf@kernel.org wrote:
> Hello:
>
> This series was applied to netdev/net-next.git (refs/heads/master):
>
> On Wed, 31 Mar 2021 23:08:48 +0300 you wrote:
> > From: Vladimir Oltean <vladimir.oltean@nxp.com>
> >
> > This series adds support to the enetc driver for the basic XDP primitives.
> > The ENETC is a network controller found inside the NXP LS1028A SoC,
> > which is a dual-core Cortex A72 device for industrial networking,
> > with the CPUs clocked at up to 1.3 GHz. On this platform, there are 4
> > ENETC ports and a 6-port embedded DSA switch, in a topology that looks
> > like this:
> >
> > [...]
>
> Here is the summary with links:
>   - [net-next,1/9] net: enetc: consume the error RX buffer descriptors in a dedicated function
>     https://git.kernel.org/netdev/net-next/c/2fa423f5f0c6
>   - [net-next,2/9] net: enetc: move skb creation into enetc_build_skb
>     https://git.kernel.org/netdev/net-next/c/a800abd3ecb9
>   - [net-next,3/9] net: enetc: add a dedicated is_eof bit in the TX software BD
>     https://git.kernel.org/netdev/net-next/c/d504498d2eb3
>   - [net-next,4/9] net: enetc: clean the TX software BD on the TX confirmation path
>     https://git.kernel.org/netdev/net-next/c/1ee8d6f3bebb
>   - [net-next,5/9] net: enetc: move up enetc_reuse_page and enetc_page_reusable
>     https://git.kernel.org/netdev/net-next/c/65d0cbb414ce
>   - [net-next,6/9] net: enetc: add support for XDP_DROP and XDP_PASS
>     https://git.kernel.org/netdev/net-next/c/d1b15102dd16
>   - [net-next,7/9] net: enetc: add support for XDP_TX
>     https://git.kernel.org/netdev/net-next/c/7ed2bc80074e
>   - [net-next,8/9] net: enetc: increase RX ring default size
>     https://git.kernel.org/netdev/net-next/c/d6a2829e82cf
>   - [net-next,9/9] net: enetc: add support for XDP_REDIRECT
>     https://git.kernel.org/netdev/net-next/c/9d2b68cc108d
>
> You are awesome, thank you!
> --
> Deet-doot-dot, I am a bot.
> https://korg.docs.kernel.org/patchwork/pwbot.html

Let's play a drinking game, the winner is who doesn't get drunk every
time Dave merges a 9-patch series with no review in less than two hours
after it was posted :D

Now in all seriousness, I'm very much open to receiving feedback still.
In a way, I appreciate seeing the patches merged as is, since further
fixups made based on review will add more nuance and color to the git
log, and it will be easier to understand why things were done or
modified in a certain way, etc, as opposed to the alternative which is
to keep amending the same blank 'add support for XDP_TX / XDP_REDIRECT /
whatever', with never enough attention given to the finer points.