Provide a concrete example of how to specify what stable series should be targeted for change inclusion. Looking around on the stable mailing list this seems like a common practice already, so let mention that in the documentation as well (but worded so it is not interpreted as the only way to do so).
Reviewed-by: Paul Barker paul.barker.ct@bp.renesas.com Signed-off-by: Shung-Hsi Yu shung-hsi.yu@suse.com --- Change from v1: - "asks that the patch to be included in..." is edit to "asks that the patch is included in..." for better wording (Paul) --- Documentation/process/stable-kernel-rules.rst | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/Documentation/process/stable-kernel-rules.rst b/Documentation/process/stable-kernel-rules.rst index edf90bbe30f4..d22aa2280f6e 100644 --- a/Documentation/process/stable-kernel-rules.rst +++ b/Documentation/process/stable-kernel-rules.rst @@ -57,10 +57,13 @@ options for cases where a mainlined patch needs adjustments to apply in older series (for example due to API changes).
When using option 2 or 3 you can ask for your change to be included in specific -stable series. When doing so, ensure the fix or an equivalent is applicable, -submitted, or already present in all newer stable trees still supported. This is -meant to prevent regressions that users might later encounter on updating, if -e.g. a fix merged for 5.19-rc1 would be backported to 5.10.y, but not to 5.15.y. +stable series, one way to do so is by specifying the target series in the +subject prefix (e.g. '[PATCH stable 5.15 5.10]' asks that the patch is +included in both 5.10.y and 5.15.y). When doing so, ensure the fix or an +equivalent is applicable, submitted, or already present in all newer stable +trees still supported. This is meant to prevent regressions that users might +later encounter on updating, if e.g. a fix merged for 5.19-rc1 would be +backported to 5.10.y, but not to 5.15.y.
.. _option_1:
When sending patch authored by someone else to stable, it is quite easy for the sender to forget adding the Developer's Certification of Origin (DCO, i.e. Signed-off-by). An example of such can be seen in the link below. Mention DCO explicitly so senders are less likely to forget to do so and cause another round-trip.
Add a label in submitting-patches.rst so we can directly link to the DCO section.
Link: https://lore.kernel.org/stable/2024051500-underage-unfixed-5d28@gregkh/ Signed-off-by: Shung-Hsi Yu shung-hsi.yu@suse.com --- Change from v1: - explicitly refer to the link as an example in the 1st paragraph (Paul) - commit message typo fix s/explicilty/explicitly/ (Paul) --- Documentation/process/stable-kernel-rules.rst | 4 ++++ Documentation/process/submitting-patches.rst | 1 + 2 files changed, 5 insertions(+)
diff --git a/Documentation/process/stable-kernel-rules.rst b/Documentation/process/stable-kernel-rules.rst index d22aa2280f6e..85a91fd40da9 100644 --- a/Documentation/process/stable-kernel-rules.rst +++ b/Documentation/process/stable-kernel-rules.rst @@ -168,6 +168,10 @@ If the submitted patch deviates from the original upstream patch (for example because it had to be adjusted for the older API), this must be very clearly documented and justified in the patch description.
+Be sure to also include a :ref:`Developer's Certificate of Origin +<sign_your_work>` (i.e. ``Signed-off-by``) when sending patches that you did +not author yourself. +
Following the submission ------------------------ diff --git a/Documentation/process/submitting-patches.rst b/Documentation/process/submitting-patches.rst index 66029999b587..98f1c8d8b429 100644 --- a/Documentation/process/submitting-patches.rst +++ b/Documentation/process/submitting-patches.rst @@ -394,6 +394,7 @@ e-mail discussions.
``git send-email`` will do this for you automatically.
+.. _sign_your_work:
Sign your work - the Developer's Certificate of Origin ------------------------------------------------------
On 15/06/2024 03:03, Shung-Hsi Yu wrote:
When sending patch authored by someone else to stable, it is quite easy for the sender to forget adding the Developer's Certification of Origin (DCO, i.e. Signed-off-by). An example of such can be seen in the link below. Mention DCO explicitly so senders are less likely to forget to do so and cause another round-trip.
Add a label in submitting-patches.rst so we can directly link to the DCO section.
Link: https://lore.kernel.org/stable/2024051500-underage-unfixed-5d28@gregkh/ Signed-off-by: Shung-Hsi Yu shung-hsi.yu@suse.com
Change from v1:
- explicitly refer to the link as an example in the 1st paragraph (Paul)
- commit message typo fix s/explicilty/explicitly/ (Paul)
Documentation/process/stable-kernel-rules.rst | 4 ++++ Documentation/process/submitting-patches.rst | 1 + 2 files changed, 5 insertions(+)
diff --git a/Documentation/process/stable-kernel-rules.rst b/Documentation/process/stable-kernel-rules.rst index d22aa2280f6e..85a91fd40da9 100644 --- a/Documentation/process/stable-kernel-rules.rst +++ b/Documentation/process/stable-kernel-rules.rst @@ -168,6 +168,10 @@ If the submitted patch deviates from the original upstream patch (for example because it had to be adjusted for the older API), this must be very clearly documented and justified in the patch description. +Be sure to also include a :ref:`Developer's Certificate of Origin +<sign_your_work>` (i.e. ``Signed-off-by``) when sending patches that you did +not author yourself.
Following the submission
diff --git a/Documentation/process/submitting-patches.rst b/Documentation/process/submitting-patches.rst index 66029999b587..98f1c8d8b429 100644 --- a/Documentation/process/submitting-patches.rst +++ b/Documentation/process/submitting-patches.rst @@ -394,6 +394,7 @@ e-mail discussions. ``git send-email`` will do this for you automatically. +.. _sign_your_work: Sign your work - the Developer's Certificate of Origin
Reviewed-by: Paul Barker paul.barker.ct@bp.renesas.com
Hi,
Thanks for your patch.
FYI: kernel test robot notices the stable kernel rule is not satisfied.
The check is based on https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html#opti...
Rule: add the tag "Cc: stable@vger.kernel.org" in the sign-off area to have the patch automatically included in the stable tree. Subject: [PATCH v2 1/2] docs: stable-kernel-rules: provide example of specifing target series Link: https://lore.kernel.org/stable/20240615020356.5595-1-shung-hsi.yu%40suse.com
linux-stable-mirror@lists.linaro.org