[PATCH 0/2] ACPI: processor / EC enablement for core i7 platforms
Alex Chiang
alex at chizang.net
Tue Jan 26 20:12:57 UTC 2010
* Stefan Bader <stefan.bader at canonical.com>:
> Alex Chiang wrote:
> >
> > Len told me on irc that he plans on marking it for -stable, but
> > hasn't gotten around to it yet.
>
> The first one I think is ok to have in Karmic. Depending on the
> upstream stable submission it might land in Lucid without further
> effort (Greg is queuing up for 2.6.32.7 but I had no time to look into
> those), so maybe its there or not. But given the current speed it
> would work if This is forwarded to stable soon.
I poked lenb today to remind him to send the patch onto stable, so it
should appear soon.
> To include it into Karmic, we have to open a Launchpad bug about it,
> so we can link the SRU request and patch to it.
Ok, I can do that. Anything special I should mark in Launchpad? Or just
reply here with the id after filing the bug?
> > To make a long story short(er), I cherry-picked the 3 patches (out
> > of the 14 upstream ones) that actually enable the platforms, and
> > left out all the code cleanup stuff.
>
> I only glanced over the patch and so this is not the last word. But
> generally its size and the fact that it moves around code at the same
> time make me a bit reluctant to pick it for Karmic. Even tough it
> allows running Karmic on additional hardware. With the other changes
> it is harder to tell whether the code does the same without opting in.
I agree that the patch is hard to read unless you're intimately familiar
with it. Sorry about that; like I said, I really wasn't thinking
backport at the time, so what I sent you was the best I could do. :-/
> So for Karmic at least I would rather see a patch that is minimalistic
> in change. OTOH I am not sure this is worth effort on Karmic as the
> window for non-critical and non-security fixes is closing mid-Feb.
Fair enough. I'm less concerned about Karmic for this patch than I am
with Lucid.
> But probably Andy thinks the same for Lucid and we should try to get a
> minimal patch suitable for stable out of this. [Again this should get
> its own bug report to track things]
After talking with lenb today, we decided to mark the early _PDC patches
for stable too, on the justification that:
a) it enables real machines
b) the opt-in mechanism limits the amount of potential regressions
So lenb will send these patches for -stable rather soon. I can open
another bug for the _PDC patches.
Anything else I should do?
Thanks,
/ac
More information about the kernel-team
mailing list