linux-kernel - Re: [stable] patch hwmon-applesmc-add-support-for-macbook-air.patch added to 2.6.27-stable tree

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-next>] [day] [month] [year] [list]

Message-ID: <20081121163219.601e68be@redhat.com>
Date:	Fri, 21 Nov 2008 16:32:19 -0500
From:	Chuck Ebbert <cebbert@...hat.com>
To:	linux-kernel@...r.kernel.org
Cc:	gregkh@...e.de, Jean Delvare <khali@...ux-fr.org>,
	Henrik Rydberg <rydberg@...omail.se>, Greg KH <greg@...ah.com>,
	rkoktarianto@...il.com, mhoffman@...htlink.com, nicolas@...chat.ch,
	stable-commits@...r.kernel.org, akpm@...ux-foundation.org,
	torvalds@...ux-foundation.org, stable@...nel.org
Subject: Re: [stable] patch hwmon-applesmc-add-support-for-macbook-air.patch
 added to 2.6.27-stable tree

On Fri, 21 Nov 2008 11:01:42 -0800
Greg KH <gregkh@...e.de> wrote:

> On Thu, Nov 20, 2008 at 09:01:21AM +0100, Jean Delvare wrote:
> > On Thu, 20 Nov 2008 02:07:17 +0100, Henrik Rydberg wrote:
> > > Greg KH wrote:
> > > > On Wed, Nov 19, 2008 at 02:26:41PM +0100, Henrik Rydberg wrote:
> > > >> Greg KH wrote:
> > > >> [...]
> > > >>>>> Heh, ok, then it really doesn't make sense to include any of these in
> > > >>>>> the 2.6.27-stable tree, as it isn't just a simple "add a new device id
> > > >>>>> to a table" type of thing.
> > > >>>>>
> > > >>>>> Should I drop them all?
> > > >>>> Either that, or add all the patches. From this end, with an eye at the
> > > >>>> Ubuntu user community, none of the patches has caused troubles, but a
> > > >>>> stable release is a stable release... I say drop them all.
> > > >>> Ok, consider them dropped :)
> > > >> On second thought, why not add all applesmc patches to -stable? I put
> > > >> together a small table, showing what machines work in the stable version
> > > >> and which ones, by testimony, work in either Ubuntu 8.10, which includes
> > > >> all patches but a few trival ones, or in a fully patched dkms version.
> > > >> I have left out machines that have not been fully tested.
> > > > 
> > > > Ok, care to send us patches to add all of this to the tree so we can see
> > > > what this entails?
> > > 
> > > Absolutely. However, since the commits are all in Linus' tree, cherry-picking
> > > sounds like the easiest thing, does it not? Here are the commits to applesmc,
> > > in reverse chronological order, since the 2.6.27 release:
> > > 
> > > 468cc0320ed083e26364d9febde2679d981ed6a6 hwmon: applesmc: add support for Macbook 4
> > > eefc488f96cdde6e152b45675b50bf380b95d99f hwmon: applesmc: add support for iMac 8
> > > a66603257bf88bbe2c9fd6a97ee5dc24de15d196 hwmon: applesmc: add support for Macbook Pro 5
> > > 181209a1d91756bfd83b1d6ce2008cea3ca225b6 hwmon: applesmc: add support for Macbook 5
> > > 6e3530fa241ae759313496f67295c9252691ed04 hwmon: applesmc: add support for iMac 5
> > > 8c9398d1e9766e3659e277acb2e8ca1c17684139 hwmon: applesmc: lighter wait mechanism, drastic improvement
> > > 07e8dbd3ebbdedc71335049dd4b0d542cb038d7d hwmon: applesmc: Add support for Macbook Pro 3
> > > d7549905f157f217b888edb0f2055f5090eaf1d8 hwmon: applesmc: Add support for Macbook Pro 4
> > > 7b5e3cb28f52f42d8cf2b36771be580f33bbc7ae drivers/hwmon/applesmc.c: remove unneeded casts
> > > f5274c972bac2d14c092a9c634e03f51785c7b76 hwmon: applesmc: add support for Macbook Air
> > > 8bd1a12a51871f0c870612d8220b1485d6aefc73 hwmon: applesmc: allow for variable ALV0 and ALV1 package length
> > > 02fcbd144d684167aa67b1d3ad68f18d265f2d08 hwmon: applesmc: prolong status wait
> > > 84d2d7f2ee98dc9bd799790c89b5eb8280af2d37 hwmon: applesmc: fix the 'wait status failed: c != 8' problem
> > > 05224091af06177c0ce7c1fae1f498455b47a6be hwmon: applesmc: specified number of bytes to read should match actual
> > > 
> > > The non-trivial ones are 8c9398, 8bd1a1 and 84d2d7.
> > 
> > This is too much for -stable IMHO.
> 
> In looking at the non-trivial ones, I agree, it's not just a "simple"
> addition, sorry.
> 

OTOH the driver is basically broken without all those patches.

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