linux-kernel - Re: [PATCHv4 01/30] x86/mm: Fix warning on build with X86_MEM

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:   Mon, 28 Feb 2022 09:11:59 -0800
From:   Josh Poimboeuf <jpoimboe@...hat.com>
To:     Dave Hansen <dave.hansen@...el.com>
Cc:     "Kirill A. Shutemov" <kirill.shutemov@...ux.intel.com>,
        tglx@...utronix.de, mingo@...hat.com, bp@...en8.de,
        luto@...nel.org, peterz@...radead.org,
        sathyanarayanan.kuppuswamy@...ux.intel.com, aarcange@...hat.com,
        ak@...ux.intel.com, dan.j.williams@...el.com, david@...hat.com,
        hpa@...or.com, jgross@...e.com, jmattson@...gle.com,
        joro@...tes.org, knsathya@...nel.org, pbonzini@...hat.com,
        sdeep@...are.com, seanjc@...gle.com, tony.luck@...el.com,
        vkuznets@...hat.com, wanpengli@...cent.com,
        thomas.lendacky@....com, brijesh.singh@....com, x86@...nel.org,
        linux-kernel@...r.kernel.org, David Rientjes <rientjes@...gle.com>
Subject: Re: [PATCHv4 01/30] x86/mm: Fix warning on build with
 X86_MEM_ENCRYPT=y

On Mon, Feb 28, 2022 at 08:51:20AM -0800, Dave Hansen wrote:
> On 2/28/22 08:40, Josh Poimboeuf wrote:
> >> maintainer-tip.rst seems disagree with you:
> >>
> >>    A Fixes tag should be added even for changes which do not need to be
> >>    backported to stable kernels, i.e. when addressing a recently introduced
> >>    issue which only affects tip or the current head of mainline.
> >>
> >> I will leave it as is.
> > How does that disagree with me?
> > 
> > The "Fixes" tag is for bug fixes.  If it's not possible to trigger the
> > warning and there's no user impact, it's not a bug.
> 
> Does having Fixes: *break* anything?

People rely on the "Fixes:" tag for actual bug fixes.  Using it here --
along with the rest of the "this is fixing a bug" tone of the title and
description -- is guaranteed to confuse stable maintainers and distros
doing backports.

Again, if nothing's broken from the standpoint of the user then it's not
a bug and shouldn't be reported that way.

> If not, I think I'd generally rather have the metadata with more
> information as opposed to less information.

I would call it misinformation.  How is that useful?

It's ok to reference a related commit in the patch description itself.
Just don't abuse the "Fixes" tag.

But IMO, for the least amount of confusion, it makes more sense to
squash this with the patch which actually requires it.

-- 
Josh