Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753843AbZLWCJN (ORCPT ); Tue, 22 Dec 2009 21:09:13 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751804AbZLWCJJ (ORCPT ); Tue, 22 Dec 2009 21:09:09 -0500 Received: from mail-px0-f189.google.com ([209.85.216.189]:53454 "EHLO mail-px0-f189.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751373AbZLWCJH convert rfc822-to-8bit (ORCPT ); Tue, 22 Dec 2009 21:09:07 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type :content-transfer-encoding; b=hC7nIRWm2r5ejt6ye24tESiolDr0YmN99Zt+8SCUWUoQYNTmAZDKaNd1eS/osy/nSb WtXbaWzcmLa6TmrsIOCaB0c41/gV7W168TqLrNVlLAdmJGjNOo8ppR1jujbeZ+83JtEx 7k5wIUQOgn0UtP3HQzqSbQup3ihwbV+CLcPj0= MIME-Version: 1.0 In-Reply-To: <4B317491.5050303@kernel.org> References: <86802c440912171728s27dd7108k85a0f1563660c95b@mail.gmail.com> <20091218022112.GA30333@sli10-desk.sh.intel.com> <20091218051457.GB417@elte.hu> <4B309AB9.7030208@kernel.org> <20091223005637.GA16783@sli10-desk.sh.intel.com> <4B317491.5050303@kernel.org> Date: Tue, 22 Dec 2009 18:09:04 -0800 X-Google-Sender-Auth: 461ad69cd1dfe4b8 Message-ID: <86802c440912221809w35dbb1eegcf6133ed1f190069@mail.gmail.com> Subject: Re: [git pull request] ACPI and driver patches for 2.6.33.merge From: Yinghai Lu To: Shaohua Li Cc: Ingo Molnar , Len Brown , "Barnes, Jesse" , Linus Torvalds , Andrew Morton , "linux-acpi@vger.kernel.org" , Linux Kernel Mailing List , "linux-pci@vger.kernel.org" Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 6470 Lines: 125 On Tue, Dec 22, 2009 at 5:38 PM, Yinghai Lu wrote: > Shaohua Li wrote: >> On Tue, Dec 22, 2009 at 06:08:57PM +0800, Yinghai Lu wrote: >>> Ingo Molnar wrote: >>>> * Shaohua Li 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 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; >> Wield BIOS. ACPI spec does mention the return buffer have the same length. >> Does changing the check back make the issue go away? > > change to > if (context->ret.length < context->cap.length) > > make AER work, but pciehp still fail. [ 54.190606] calling aer_service_init+0x0/0x2b @ 1 [ 55.200862] run_acpi_osc: ret length 40 cap length 12 [ 55.203924] aer 0000:00:01.0:pcie02: service driver aer loaded [ 55.208520] aer 0000:00:03.0:pcie02: service driver aer loaded [ 55.222730] aer 0000:00:05.0:pcie02: service driver aer loaded [ 55.238340] aer 0000:00:07.0:pcie02: service driver aer loaded [ 55.242030] aer 0000:00:09.0:pcie02: service driver aer loaded [ 56.270866] run_acpi_osc: ret length 40 cap length 12 [ 56.275370] aer 0000:80:00.0:pcie02: service driver aer loaded [ 56.279645] aer 0000:80:01.0:pcie02: service driver aer loaded [ 56.289378] aer 0000:80:03.0:pcie02: service driver aer loaded [ 56.306406] aer 0000:80:05.0:pcie02: service driver aer loaded [ 56.309489] aer 0000:80:07.0:pcie02: service driver aer loaded [ 56.326848] aer 0000:80:09.0:pcie02: service driver aer loaded [ 56.330392] initcall aer_service_init+0x0/0x2b returned 0 after 2089791 usecs [ 56.347263] calling aer_inject_init+0x0/0x12 @ 1 [ 56.349822] initcall aer_inject_init+0x0/0x12 returned 0 after 290 usecs [ 56.367496] calling ioapic_init+0x0/0x1b @ 1 [ 56.370813] initcall ioapic_init+0x0/0x1b returned 0 after 718 usecs [ 56.387751] calling pci_hotplug_init+0x0/0x4e @ 1 [ 56.391225] pci_hotplug: PCI Hot Plug PCI Core version: 0.5 [ 56.407362] initcall pci_hotplug_init+0x0/0x4e returned 0 after 15785 usecs [ 56.411486] calling pcied_init+0x0/0x9c @ 1 [ 56.427349] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Trying to get hotplug control for \_SB_.PCI0 [ 56.446705] Firmware did not grant requested _OSC control [ 56.450227] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Cannot get control of hotplug hardware for pci 0000:00:03.0 [ 56.469325] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Trying to get hotplug control for \_SB_.PCI0 [ 56.488221] Firmware did not grant requested _OSC control [ 56.491440] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Cannot get control of hotplug hardware for pci 0000:00:05.0 [ 56.511164] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Trying to get hotplug control for \_SB_.PCI0 [ 56.530074] Firmware did not grant requested _OSC control [ 56.546595] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Cannot get control of hotplug hardware for pci 0000:00:1c.0 [ 56.567448] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Trying to get hotplug control for \_SB_.PCIB [ 56.586091] Firmware did not grant requested _OSC control [ 56.589624] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Cannot get control of hotplug hardware for pci 0000:80:03.0 [ 56.611700] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Trying to get hotplug control for \_SB_.PCIB [ 56.627768] Firmware did not grant requested _OSC control [ 56.630734] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Cannot get control of hotplug hardware for pci 0000:80:05.0 [ 56.655403] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Trying to get hotplug control for \_SB_.PCI0 [ 56.671933] Firmware did not grant requested _OSC control [ 56.687138] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Cannot get control of hotplug hardware for pci 0000:00:03.0 [ 56.708671] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Trying to get hotplug control for \_SB_.PCI0 [ 56.726175] Firmware did not grant requested _OSC control [ 56.728791] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Cannot get control of hotplug hardware for pci 0000:00:05.0 [ 56.749652] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Trying to get hotplug control for \_SB_.PCI0 [ 56.768909] Firmware did not grant requested _OSC control [ 56.785642] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Cannot get control of hotplug hardware for pci 0000:00:1c.0 [ 56.805022] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Trying to get hotplug control for \_SB_.PCIB [ 56.810012] Firmware did not grant requested _OSC control [ 56.827406] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Cannot get control of hotplug hardware for pci 0000:80:03.0 [ 56.848214] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Trying to get hotplug control for \_SB_.PCIB [ 56.867509] Firmware did not grant requested _OSC control [ 56.872287] acpi_pcihp: acpi_get_hp_hw_control_from_firmware: Cannot get control of hotplug hardware for pci 0000:80:05.0 [ 56.890273] pciehp: pcie_port_service_register = 0 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/