Re: [git pull request] ACPI and driver patches for 2.6.33.merge

Yinghai Lu
2009-12-22T10:32:50Z

Yinghai Lu wrote:
> Ingo Molnar wrote:
>> * Shaohua Li <shaohua.li@intel.com> wrote:
>>
>>> On Fri, Dec 18, 2009 at 09:28:50AM +0800, Yinghai Lu wrote:
>>>> On Wed, Dec 16, 2009 at 12:06 PM, Len Brown <lenb@kernel.org> wrote:
>>>>> Hi Linus,
>>>>>
>>>>> please pull from:
>>>>>
>>>>> git://git.kernel.org/pub/scm/linux/kernel/git/lenb/linux-acpi-2.6.git release
>>>>>
>>>> ;..
>>>>> Shaohua Li (3):
>>>>> ? ? ?ACPI: Add a generic API for _OSC -v2
>>>>> ? ? ?ACPI: cleanup pci_root _OSC code.
>>>>> ? ? ?ACPI: Add platform-wide _OSC support.
>>>> it seems these three patches broke the _OSC on my intel new systems.
>>>>
>>>> revert them fix the problem with AER and pciehp and etc
>>> can you give more details? I just cleaned up the _OSC code for AER and
>>> pciehp, no function changes.
>> Famous last words ;-)
>>
>> Yinghai, i suspect Shaohua needs the kind of info you'd need if you tried to
>> fix it: acpidump, before/after debug boot log, a description of what goes bad,
>> etc.
>
> the so called clean up, change the ret length checking.
>
> - if (!output.length)
> - return AE_NULL_OBJECT;
> -
>
> + /* return buffer should have the same length as cap buffer */
> + if (context->ret.length != context->cap.length)
> + return AE_NULL_OBJECT;
>
printk said:

[ 54.709165] calling aer_service_init+0x0/0x2b @ 1
[ 56.121190] run_acpi_osc: ret length 40 cap length 12
[ 56.124996] aer 0000:00:01.0:pcie02: AER service couldn't init device: _OSC failed

YH