Re: [PATCH 14/16] ACPICA: Add several repairs for _CST predefined name

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tuesday, June 18, 2013 06:51:45 PM Moore, Robert wrote:
> Correct, additional _CST sanity checks and repairs.

OK, and what AML breakage does it help against, in particular?

Rafael


> > -----Original Message-----
> > From: lenb417@xxxxxxxxx [mailto:lenb417@xxxxxxxxx] On Behalf Of Len Brown
> > Sent: Wednesday, June 12, 2013 10:59 AM
> > To: Zheng, Lv
> > Cc: Wysocki, Rafael J; Moore, Robert; Brown, Len; linux acpi
> > Subject: Re: [PATCH 14/16] ACPICA: Add several repairs for _CST predefined
> > name
> > 
> > On Fri, Jun 7, 2013 at 9:01 PM, Lv Zheng <lv.zheng@xxxxxxxxx> wrote:
> > > Sort list based on the C-state, remove invalid/zero entries.
> > > ACPICA BZ 890. Lv Zheng.
> > 
> > The commit message should not just say what the code does, it should more
> > importantly answer WHY the change was made, and why anybody should care
> > about the patch.
> > 
> > I think the answer here is that we are adding _CST sanity checking --
> > presumably to deal with buggy AML?
> > 
> > What will we see happen after this patch that we did not see before it?
> > 
> > Nacked-by: Len Brown <len.brown@xxxxxxxxx>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux