Re: [PATCH net 1/3] docs: netdev: document that patch series length limit
From: Andrew Lunn <andrew@lunn.ch>
Date: 2022-07-01 07:48:53
Also in:
linux-doc
On Thu, Jun 30, 2022 at 10:46:05AM -0700, Jakub Kicinski wrote:
quoted hunk ↗ jump to hunk
We had been asking people to avoid massive patch series but it does not appear in the FAQ. Signed-off-by: Jakub Kicinski <kuba@kernel.org> --- Documentation/process/maintainer-netdev.rst | 8 ++++++++ 1 file changed, 8 insertions(+)diff --git a/Documentation/process/maintainer-netdev.rst b/Documentation/process/maintainer-netdev.rst index c456b5225d66..862b6508fc22 100644 --- a/Documentation/process/maintainer-netdev.rst +++ b/Documentation/process/maintainer-netdev.rst@@ -136,6 +136,14 @@ it to the maintainer to figure out what is the most recent and current version that should be applied. If there is any doubt, the maintainer will reply and ask what should be done. +How do I divide my work into patches? +------------------------------------- + +Put yourself in the shoes of the reviewer. Each patch is read separately +and therefore should constitute a comprehensible step towards your stated +goal. Avoid sending series longer than 15 patches, they clog review queue +and increase mailing list traffic when re-posted. +
Hi Jakub
I think a key concept is, big patch series takes longer to review, so
needs a bigger junk of time allocated to it, so often gets differed
until late. As a result, it will take longer to merge. A small series
can be reviewed in a short time, so Maintainers just do it, allowing
for quicker merging.
Andrew