linux-kernel - Re: [patch 44/45] PCI: Unhide the SMBus on the Compaq Evo D510 USDT

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: <20090917095823.12682531@hyperion.delvare>
Date:	Thu, 17 Sep 2009 09:58:23 +0200
From:	Jean Delvare <khali@...ux-fr.org>
To:	Greg KH <gregkh@...e.de>
Cc:	linux-kernel@...r.kernel.org, stable@...nel.org,
	stable-review@...nel.org, torvalds@...ux-foundation.org,
	akpm@...ux-foundation.org, alan@...rguk.ukuu.org.uk,
	Jesse Barnes <jbarnes@...tuousgeek.org>
Subject: Re: [patch 44/45] PCI: Unhide the SMBus on the Compaq Evo D510 USDT

Hi Greg,

On Wed, 16 Sep 2009 15:36:57 -0700, Greg KH wrote:
> 2.6.31-stable review patch.  If anyone has any objections, please let us know.
> 
> ------------------
> From: Jean Delvare <khali@...ux-fr.org>
> 
> commit 6b5096e4d4496e185cd1ada5d1b8e1d941c805ed upstream.
> 
> One more form factor for Compaq Evo D510, which needs the same quirk
> as the other form factors. Apparently there's no hardware monitoring
> chip on that one, but SPD EEPROMs, so it's still worth unhiding the
> SMBus.

FWIW, this one doesn't strike me as being needed in -stable. While it
should be safe (otherwise it wouldn't have gone upstream in the first
place), it hardly fulfills -stable's requirement that "it must fix a
real bug that bothers people". There's no hardware monitoring chip on
the SMBus on this machine, and people can easily live without detailed
memory module information.

So it's up to you, but I won't cry if you drop this patch from -stable.

> 
> Signed-off-by: Jean Delvare <khali@...ux-fr.org>
> Tested-by: Nuzhna Pomoshch <nuzhna_pomoshch@...oo.com>
> Signed-off-by: Jesse Barnes <jbarnes@...tuousgeek.org>
> Signed-off-by: Greg Kroah-Hartman <gregkh@...e.de>
> 
> ---
>  drivers/pci/quirks.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -1201,6 +1201,7 @@ static void __init asus_hides_smbus_host
>  			switch(dev->subsystem_device) {
>  			case 0x00b8: /* Compaq Evo D510 CMT */
>  			case 0x00b9: /* Compaq Evo D510 SFF */
> +			case 0x00ba: /* Compaq Evo D510 USDT */
>  				/* Motherboard doesn't have Host bridge
>  				 * subvendor/subdevice IDs and on-board VGA
>  				 * controller is disabled if an AGP card is
> 
> 


-- 
Jean Delvare
--
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/