Re: [net-next PATCH v2 0/9] Multiple cleanup and feature for qca8k

From: Ansuel Smith <ansuelsmth@gmail.com>
To: Vladimir Oltean <olteanv@gmail.com>
Cc: Andrew Lunn <andrew@lunn.ch>,
	Vivien Didelot <vivien.didelot@gmail.com>,
	Florian Fainelli <f.fainelli@gmail.com>,
	"David S. Miller" <davem@davemloft.net>,
	Jakub Kicinski <kuba@kernel.org>,
	Russell King <linux@armlinux.org.uk>,
	netdev@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [net-next PATCH v2 0/9] Multiple cleanup and feature for qca8k
Date: Mon, 22 Nov 2021 02:43:46 +0100	[thread overview]
Message-ID: <619af5d3.1c69fb81.36176.ca79@mx.google.com> (raw)
In-Reply-To: <20211122012910.bd33slbrfk4h6xbw@skbuf>

On Mon, Nov 22, 2021 at 03:29:10AM +0200, Vladimir Oltean wrote:
> On Mon, Nov 22, 2021 at 02:03:04AM +0100, Ansuel Smith wrote:
> > This is a reduced version of the old massive series.
> > Refer to the changelog to know what is removed from this.
> > 
> > THIS IS BASED ON net-next WITH THE 2 FIXES FROM net ALREADY REVIEWED
> > net: dsa: qca8k: fix MTU calculation
> > net: dsa: qca8k: fix internal delay applied to the wrong PAD config
> 
> Since patchwork has auto build hooks now, it doesn't detect dependencies
> to other trees like "net" in this case, and your patches will fail to
> apply without the other ones you've mentioned, which in turn will make
> the builds fail. Patches without clean build reports aren't accepted, so
> you'll have to resend either way. Your options are:
> (a) wait until the bugfix patches get applied to "net", and Jakub and/or
>     David send the networking pull request for v5.16-rc3 to Linus, then
>     they'll merge the "net" tree into "net-next" quickly afterwards and
>     your patches apply cleanly. Last two "net" pull requests were
>     submitted on Nov 18th and 12th, if that is any indication as to when
>     the next one is going to be.
> (b) base your patches on "net-next" without the bug fixes, and let
>     Jakub/David handle the merge conflict when the'll merge "net" into
>     "net-next" next time. Please note that if you do this, there is a
>     small chance that mistakes can be made, and you can't easily
>     backport patches to a stable tree such as OpenWRT if that's what
>     you're into, since part of the delta will be in a merge commit, and
>     there isn't any simple way in which you can linearize that during
>     cherry-pick time, if you're picking from divergent branches.

Mhhh I honestly think b option can be accepted here (due to the fact
that fixes patch are very small) but the backport part can be
problematic.
Think it's better to just wait and get the reviewed by tag.

Is it problematic to add stuff to this series while the fixes are
merged? (for example the LAGs or mirror part / the code split)
Or having big series is still problematic even if half of the patch are
already reviewed?
Just asking if there is a way to continue the review process while we
wait for the merge process.

-- 
	Ansuel