linux-kernel - Re: [GIT PULL] irqchip fixes for 6.6, take #1

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 28 Sep 2023 23:17:23 +0200
From:   Ingo Molnar <mingo@...nel.org>
To:     Marc Zyngier <maz@...nel.org>
Cc:     Thomas Gleixner <tglx@...utronix.de>,
        Biju Das <biju.das.jz@...renesas.com>,
        Claudiu Beznea <claudiu.beznea.uj@...renesas.com>,
        Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
        Geert Uytterhoeven <geert+renesas@...der.be>,
        Konrad Dybcio <konrad.dybcio@...aro.org>,
        Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>,
        Maulik Shah <quic_mkshah@...cinc.com>,
        Neil Armstrong <neil.armstrong@...aro.org>,
        linux-kernel@...r.kernel.org
Subject: Re: [GIT PULL] irqchip fixes for 6.6, take #1


* Marc Zyngier <maz@...nel.org> wrote:

> Dmitry Baryshkov (1):
>       arm64: dts: qcom: sm8150: extend the size of the PDC resource

This SOB chain, while valid, is a hodgepodge with no rhyme and reason:

  Fixes: 397ad94668c1 ("arm64: dts: qcom: sm8150: Add pdc interrupt controller node")
  Reviewed-by: Konrad Dybcio <konrad.dybcio@...aro.org>
  Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
  Reviewed-by: Neil Armstrong <neil.armstrong@...aro.org>
  Signed-off-by: Neil Armstrong <neil.armstrong@...aro.org>
  Signed-off-by: Marc Zyngier <maz@...nel.org>
  Link: https://lore.kernel.org/r/20230905-topic-sm8x50-upstream-pdc-ver-v4-2-fc633c7df84b@linaro.org

Besides the ordering, people in the SOB chain should not have a separate
Reviewed-by tag: *of course* anyone who forwards a patch with a SOB has
reviewed it, it's basically implicit.

So the tags should be something like:

  Fixes: 397ad94668c1 ("arm64: dts: qcom: sm8150: Add pdc interrupt controller node")
  Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
  Signed-off-by: Neil Armstrong <neil.armstrong@...aro.org>
  Signed-off-by: Marc Zyngier <maz@...nel.org>
  Reviewed-by: Konrad Dybcio <konrad.dybcio@...aro.org>
  Link: https://lore.kernel.org/r/20230905-topic-sm8x50-upstream-pdc-ver-v4-2-fc633c7df84b@linaro.org

Ie. we generally sort the tags not chronologically (or whatever the original idea
was), but group them topically, sort the groups by importance, and only within
the group is there chronological order.

Thanks,

	Ingo