Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756746Ab0BKSdP (ORCPT ); Thu, 11 Feb 2010 13:33:15 -0500 Received: from g1t0026.austin.hp.com ([15.216.28.33]:11110 "EHLO g1t0026.austin.hp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755894Ab0BKSdM (ORCPT ); Thu, 11 Feb 2010 13:33:12 -0500 From: Bjorn Helgaas To: Gary Hade Subject: Re: [PATCH 8/9] PCI / ACPI / PM: Platform support for PCI PME wake-up (rev. 7) Date: Thu, 11 Feb 2010 11:33:09 -0700 User-Agent: KMail/1.9.10 Cc: "Rafael J. Wysocki" , linux-pm@lists.linux-foundation.org, Linux PCI , LKML , Jesse Barnes , "Moore, Robert" , Matthew Garrett References: <201001101431.38630.rjw@sisk.pl> <201002111427.17275.rjw@sisk.pl> <20100211182910.GA9766@us.ibm.com> In-Reply-To: <20100211182910.GA9766@us.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <201002111133.10174.bjorn.helgaas@hp.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2846 Lines: 60 On Thursday 11 February 2010 11:29:10 am Gary Hade wrote: > On Thu, Feb 11, 2010 at 02:27:17PM +0100, Rafael J. Wysocki wrote: > > On Thursday 11 February 2010, Gary Hade wrote: > > > On Thu, Feb 11, 2010 at 01:56:54AM +0100, Rafael J. Wysocki wrote: > > > > On Thursday 11 February 2010, Rafael J. Wysocki wrote: > > > > > On Thursday 11 February 2010, Gary Hade wrote: > > > > > > On Wed, Feb 10, 2010 at 11:58:00PM +0100, Rafael J. Wysocki wrote: > > > > > > > On Wednesday 10 February 2010, Gary Hade wrote: > > > > > ... > > > > > > > > Yes, 6/9 also appeared to be the most likely suspect to me and > > > > > > > > I was already doing what you asked except in the opposite order. > > > > > > > > With 1/9 through 6/9 both the hot-add and hot-remove issues still > > > > > > > > reproduced. After removing 6/9 both issues disappeared. > > > > > > > > > > > > > > Thanks for verifying, I've already started to look for bugs in it. > > > > > > > > > > > > Thanks! > > > > > > > > > > > > > > > > > > > > What's your kernel command line, BTW? > > > > > > > > > > > > root=/dev/disk/by-id/scsi-35000c5000036ffcb-part7 ip=9.47.66.9:9.47.67.50:9.47.66.1:255.255.254.0 resume=/dev/disk/by-id/scsi-35000c50000370247-part3 crashkernel=256M-:128M console=tty0 console=ttyS0,115200 pci=use_crs pci=norom > > > > > > > > > > Thanks. We found a bug in the patch, but it would require you to use a > > > > > specific command line switch to trigger, which you don't use. > > > > > > > > > > I'll let you know if I have anything to test. > > > > > > > > OK, please try the patch below on top of [1-6/9]. > > > > > > Sorry, it didn't help. The hot-add and hot-remove behaviors > > > still look the same. > > > > Thanks for testing. > > > > Well, this time I found another real bug, which is that we should add > > gpe_block->block_base_number to the GPE index within the block in order to > > compute the GPE number. If there are GPE devices other than the FADT ones in > > your system, that may explain the symptoms. > > > > Please try the patch below (on top of [1-6/9]). In case it doesn't help, please > > look for "Failed to ebable GPE" messages in dmesg and let me know if there are > > any. > > That one worked! It corrected both the blinking amber LED > after hot-remove and acpiphp's failure to see the hot-add event. > > Now when you have patch(es) ready for Jesse's linux-next branch > I can get back to my original mission of checking out Bjorn's > _CRS enablement changes. I'm hoping to repost my series today with some fixes, so don't bother trying anything before that. Bjorn -- 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/