Skip to content

Commit ee8ab74

Browse files
kuba-moodavem330
authored andcommitted
docs: netdev: document patchwork patch states
The patchwork states are largely self-explanatory but small ambiguities may still come up. Document how we interpret the states in networking. Signed-off-by: Jakub Kicinski <kuba@kernel.org> Signed-off-by: David S. Miller <davem@davemloft.net>
1 parent 719c5e3 commit ee8ab74

File tree

1 file changed

+31
-1
lines changed

1 file changed

+31
-1
lines changed

Documentation/process/maintainer-netdev.rst

Lines changed: 31 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,37 @@ queue for netdev:
120120
https://patchwork.kernel.org/project/netdevbpf/list/
121121

122122
The "State" field will tell you exactly where things are at with your
123-
patch. Patches are indexed by the ``Message-ID`` header of the emails
123+
patch:
124+
125+
================== =============================================================
126+
Patch state Description
127+
================== =============================================================
128+
New, Under review pending review, patch is in the maintainer’s queue for
129+
review; the two states are used interchangeably (depending on
130+
the exact co-maintainer handling patchwork at the time)
131+
Accepted patch was applied to the appropriate networking tree, this is
132+
usually set automatically by the pw-bot
133+
Needs ACK waiting for an ack from an area expert or testing
134+
Changes requested patch has not passed the review, new revision is expected
135+
with appropriate code and commit message changes
136+
Rejected patch has been rejected and new revision is not expected
137+
Not applicable patch is expected to be applied outside of the networking
138+
subsystem
139+
Awaiting upstream patch should be reviewed and handled by appropriate
140+
sub-maintainer, who will send it on to the networking trees;
141+
patches set to ``Awaiting upstream`` in netdev's patchwork
142+
will usually remain in this state, whether the sub-maintainer
143+
requested changes, accepted or rejected the patch
144+
Deferred patch needs to be reposted later, usually due to dependency
145+
or because it was posted for a closed tree
146+
Superseded new version of the patch was posted, usually set by the
147+
pw-bot
148+
RFC not to be applied, usually not in maintainer’s review queue,
149+
pw-bot can automatically set patches to this state based
150+
on subject tags
151+
================== =============================================================
152+
153+
Patches are indexed by the ``Message-ID`` header of the emails
124154
which carried them so if you have trouble finding your patch append
125155
the value of ``Message-ID`` to the URL above.
126156

0 commit comments

Comments
 (0)