From: Thorsten Leemhuis <linux@leemhuis.info>
To: Jonathan Corbet <corbet@lwn.net>,
workflows@vger.kernel.org, regressions@lists.linux.dev,
linux-doc@vger.kernel.org
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
Greg KH <gregkh@linuxfoundation.org>
Subject: [PATCH v1 6/6] docs: 6.Followthrough.rst: advice on handling regressions fixes
Date: Tue, 10 Dec 2024 11:15:15 +0100 [thread overview]
Message-ID: <e7344ff7a57b61380152defaa5ec13f06ac5d7d0.1733825632.git.linux@leemhuis.info> (raw)
In-Reply-To: <cover.1733825632.git.linux@leemhuis.info>
Add some advice on how to handle regressions as developer, reviewer, and
maintainer, as resolving regression without unnecessary delays requires
multiple people working hand in hand.
This removes equivalent paragraphs from a section in
Documentation/process/handling-regressions.rst, which will become mostly
obsolete through this and follow-up changes.
Signed-off-by: Thorsten Leemhuis <linux@leemhuis.info>
---
Documentation/process/6.Followthrough.rst | 24 ++++++++++++++++---
.../process/handling-regressions.rst | 16 -------------
2 files changed, 21 insertions(+), 19 deletions(-)
diff --git a/Documentation/process/6.Followthrough.rst b/Documentation/process/6.Followthrough.rst
index 587e80578f83a9..8ffa3cd142e8a0 100644
--- a/Documentation/process/6.Followthrough.rst
+++ b/Documentation/process/6.Followthrough.rst
@@ -198,6 +198,11 @@ maintainers and other developers will take note if you fail to handle regression
appropriately, especially if they then have to fix the problem themselves: this
could well make it harder for you to incorporate future changes.
+In general:
+
+ - Prioritize work on providing, reviewing, and mainlining regression fixes over
+ other upstream Linux kernel work.
+
On timing once the mainline change causing the regression became known:
- If the regression is severe or reported by many people within a short
@@ -229,9 +234,22 @@ On timing once the mainline change causing the regression became known:
On procedure:
- - Regression fixes are not required to spend time in linux-next, but depending
- on the fix and the alignment with pull requests it might be beneficial to
- have them in there for a day or two.
+ - Developers, when trying to reach the time periods mentioned above, remember
+ to account for the time it will take to test, review, commit, and mainline
+ fixes, ideally with them being in linux-next briefly. To achieve that, make
+ the appropriate urgency of a fix obvious in the submission and consider
+ opting for a revert to be on the safe side.
+
+ - Reviewers, you are kindly asked to prefer reviewing regression fixes over
+ other changes.
+
+ - Maintainers, you likewise are kindly asked to expedite the handling of
+ regression fixes. They for example are not required to spend time in
+ linux-next, but depending on the fix and the alignment with pull requests it
+ might be beneficial to have them in there for a day or two. When appropriate,
+ also consider sending an additional pull request. Furthermore try to avoid
+ holding onto regression fixes over weekends -- especially when some are
+ marked for backporting to stable series.
- If a regression seems tangly, precarious, or urgent, consider CCing Linus on
discussions and patch reviews; do the same if the responsible maintainers
diff --git a/Documentation/process/handling-regressions.rst b/Documentation/process/handling-regressions.rst
index 581f99675a9d52..7b8f0b122c1552 100644
--- a/Documentation/process/handling-regressions.rst
+++ b/Documentation/process/handling-regressions.rst
@@ -156,22 +156,6 @@ only these options:
How to realize that in practice depends on various factors. Use the rules of
thumb outlined in Documentation/process/6.Followthrough.rst as a guide.
-On patch flow:
-
- * Developers, when trying to reach the time periods mentioned above, remember
- to account for the time it takes to get fixes tested, reviewed, and merged by
- Linus, ideally with them being in linux-next at least briefly. Hence, if a
- fix is urgent, make it obvious to ensure others handle it appropriately.
-
- * Reviewers, you are kindly asked to assist developers in reaching the time
- periods mentioned above by reviewing regression fixes in a timely manner.
-
- * Subsystem maintainers, you likewise are encouraged to expedite the handling
- of regression fixes. Thus evaluate if skipping linux-next is an option for
- the particular fix. Also consider sending git pull requests more often than
- usual when needed. And try to avoid holding onto regression fixes over
- weekends -- especially when the fix is marked for backporting.
-
More aspects regarding regressions developers should be aware of
----------------------------------------------------------------
--
2.45.0
next prev parent reply other threads:[~2024-12-10 10:15 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-10 10:15 [PATCH v1 0/6] docs: add instruction about handling regressions to our core texts for contributors Thorsten Leemhuis
2024-12-10 10:15 ` [PATCH v1 1/6] docs: more detailed instructions on handling regressions Thorsten Leemhuis
2024-12-13 16:14 ` Jonathan Corbet
2024-12-20 12:17 ` Thorsten Leemhuis
2024-12-10 10:15 ` [PATCH v1 2/6] docs: 6.Followthrough.rst: when to involved Linus in regressions Thorsten Leemhuis
2024-12-13 16:17 ` Jonathan Corbet
2024-12-20 12:41 ` Thorsten Leemhuis
2024-12-10 10:15 ` [PATCH v1 3/6] docs: 6.Followthrough.rst: interaction with stable wrt to regressions Thorsten Leemhuis
2024-12-13 16:20 ` Jonathan Corbet
2024-12-20 13:13 ` Thorsten Leemhuis
2024-12-10 10:15 ` [PATCH v1 4/6] docs: 6.Followthrough.rst: tags to use in regressions fixes Thorsten Leemhuis
2024-12-13 16:24 ` Jonathan Corbet
2024-12-20 13:18 ` Thorsten Leemhuis
2024-12-10 10:15 ` [PATCH v1 5/6] docs: 6.Followthrough.rst: more specific advice on fixing regressions Thorsten Leemhuis
2024-12-13 16:28 ` Jonathan Corbet
2024-12-20 14:21 ` Thorsten Leemhuis
2024-12-10 10:15 ` Thorsten Leemhuis [this message]
2024-12-13 16:30 ` [PATCH v1 6/6] docs: 6.Followthrough.rst: advice on handling regressions fixes Jonathan Corbet
2024-12-20 14:24 ` Thorsten Leemhuis
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=e7344ff7a57b61380152defaa5ec13f06ac5d7d0.1733825632.git.linux@leemhuis.info \
--to=linux@leemhuis.info \
--cc=corbet@lwn.net \
--cc=gregkh@linuxfoundation.org \
--cc=linux-doc@vger.kernel.org \
--cc=regressions@lists.linux.dev \
--cc=torvalds@linux-foundation.org \
--cc=workflows@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox