linux-kernel - Re: [Bugfix] x86, irq: Fix a regression caused by commit b5dc8e6c21e7

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]

Message-ID: <alpine.DEB.2.11.1508172301350.3873@nanos>
Date:	Mon, 17 Aug 2015 23:11:41 +0200 (CEST)
From:	Thomas Gleixner <tglx@...utronix.de>
To:	Jiang Liu <jiang.liu@...ux.intel.com>
cc:	Alexander Holler <holler@...oftware.de>,
	Mark Rustad <mark.d.rustad@...el.com>,
	Alex Deucher <alexdeucher@...il.com>,
	Ingo Molnar <mingo@...hat.com>,
	"H. Peter Anvin" <hpa@...or.com>, x86@...nel.org,
	Tony Luck <tony.luck@...el.com>, linux-kernel@...r.kernel.org
Subject: Re: [Bugfix] x86, irq: Fix a regression caused by commit
 b5dc8e6c21e7

On Sun, 9 Aug 2015, Jiang Liu wrote:

> Alex Deucher, Mark Rustad and Alexander Holler reported a regression
> with the latest v4.2-rc4 kernel, which breaks some SATA controllers.
> With multi-MSI capable SATA controllers, only the first port works,
> all other ports times out when executing SATA commands. This regression
> bisects to 52f518a3a7c2 ("x86/MSI: Use hierarchical irqdomains to manage
> MSI interrupts"), but it's not the root cause, it just triggers a bug
> caused by b5dc8e6c21e7 ("x86/irq: Use hierarchical irqdomain to manage
> CPU interrupt vectors").
> 
> With this patch applied, the affected SATA controllers work as expected.

This changelog including the subject line is horrible.

1) The subject line should describe the change in a short and precise form

       x86/irq: Fix a regression caused by commit b5dc8e6c21e7

   fits the short category, but completely fails to be precise. It's
   not interesting for the subject line which commit caused the
   problem and whether it's a regression or not. We want a to see a
   proper description of the change itself.

2) The changelog should describe the bug itself.

     ... but it's not the root cause, it just triggers a bug caused by
     b5dc8e6c21e7 ("x86/irq: Use hierarchical irqdomain to manage CPU
     interrupt vectors").
   
   does not tell what the actual bug in the code is.

3) The changelog should describe the solution.

     With this patch applied, the affected SATA controllers work as
     expected.

   is describing the desired effect of the change, but not the change
   itself.
 
Thanks,

	tglx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/