Skip to content

Commit 02514a0

Browse files
kuba-moodavem330
authored andcommitted
docs: netdev: document that patch series length limit
We had been asking people to avoid massive patch series but it does not appear in the FAQ. Signed-off-by: Jakub Kicinski <[email protected]> Reviewed-by: Andrew Lunn <[email protected]> Signed-off-by: David S. Miller <[email protected]>
1 parent 280e3a8 commit 02514a0

File tree

1 file changed

+14
-0
lines changed

1 file changed

+14
-0
lines changed

Documentation/process/maintainer-netdev.rst

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -136,6 +136,20 @@ it to the maintainer to figure out what is the most recent and current
136136
version that should be applied. If there is any doubt, the maintainer
137137
will reply and ask what should be done.
138138

139+
How do I divide my work into patches?
140+
-------------------------------------
141+
142+
Put yourself in the shoes of the reviewer. Each patch is read separately
143+
and therefore should constitute a comprehensible step towards your stated
144+
goal.
145+
146+
Avoid sending series longer than 15 patches. Larger series takes longer
147+
to review as reviewers will defer looking at it until they find a large
148+
chunk of time. A small series can be reviewed in a short time, so Maintainers
149+
just do it. As a result, a sequence of smaller series gets merged quicker and
150+
with better review coverage. Re-posting large series also increases the mailing
151+
list traffic.
152+
139153
I made changes to only a few patches in a patch series should I resend only those changed?
140154
------------------------------------------------------------------------------------------
141155
No, please resend the entire patch series and make sure you do number your

0 commit comments

Comments
 (0)