Thread (31 messages) 31 messages, 3 authors, 2022-03-30

Re: [PATCH net v2 07/14] docs: netdev: rephrase the 'Under review' question

From: Florian Fainelli <f.fainelli@gmail.com>
Date: 2022-03-30 03:33:11
Also in: bpf, linux-doc


On 3/28/2022 10:08 PM, Jakub Kicinski wrote:
The semantics of "Under review" have shifted. Reword the question
about it a bit and focus it on the response time.

Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>

Nit below:
quoted hunk ↗ jump to hunk
---
  Documentation/networking/netdev-FAQ.rst | 8 +++++---
  1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/Documentation/networking/netdev-FAQ.rst b/Documentation/networking/netdev-FAQ.rst
index e10a8140d642..00ac300ebe6a 100644
--- a/Documentation/networking/netdev-FAQ.rst
+++ b/Documentation/networking/netdev-FAQ.rst
@@ -116,10 +116,12 @@ patch. Patches are indexed by the ``Message-ID`` header of the emails
  which carried them so if you have trouble finding your patch append
  the value of ``Message-ID`` to the URL above.
  
-The above only says "Under Review".  How can I find out more?
--------------------------------------------------------------
+How long before my patch is accepted?
+-------------------------------------
  Generally speaking, the patches get triaged quickly (in less than
-48h).  So be patient.  Asking the maintainer for status updates on your
+48h). But be patient, if your patch is active in patchwork (i.e. it's
+listed on the project's patch list) the chances it was missed are close to zero.
+Asking the maintainer for status updates on your
  patch is a good way to ensure your patch is ignored or pushed to the
  bottom of the priority list.
I would reword that at some point, this feels a little bit retaliating.
-- 
Florian
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help