On Tue, Dec 11, 2012 at 6:32 PM, Huang Ying <ying.huang@xxxxxxxxx> wrote: > On Fri, 2012-12-07 at 22:00 +0100, Rafael J. Wysocki wrote: >> On Thursday, December 06, 2012 04:28:08 PM Sarah Sharp wrote: >> > On Thu, Dec 06, 2012 at 01:43:32AM +0100, Rafael J. Wysocki wrote: >> > > On Wednesday, December 05, 2012 04:33:44 PM Sarah Sharp wrote: >> > > > Wakeup from D3 works fine on the 3.5.0 kernel, but fails on 3.6.2. I >> > > > haven't fully bisected yet. >> > > > >> > > > In debugging, I found that if you only enable runtime suspend for the >> > > > NEC host controller, the host successfully comes out of D3 when you plug >> > > > in a USB device. However, if you enable runtime PM for the parent PCIe root >> > > > port, it stops working. Disabling D3cold for both devices did not help. >> > > > >> > > > It looks like a PCI issue, so what sort of debugging info do you need >> > > > from me? >> > > >> > > It looks like this is related to one of the following commits: >> > >> > > Generally, please try to bisect changes in drivers/pci between v3.5 and v3.6. >> > >> > Ok, I ran git bisect with only the drivers/pci directory as a target. >> > >> > > ee85f54 ACPI/PM: specify lowest allowed state for device sleep state >> > >> > git bisect ended up identifying this as the bad patch, although >> > reverting just that patch after the bisect finished didn't seem to help. >> > However, it does make sense that this would be the culprit patch, if >> > Huang Ying's theory about the PME polling is correct. >> >> Well, we surely don't handle this particular case correctly, and it shouldn't >> be very difficult to fix, so let's hope that this is the culprit. :-) >> > > Please try the patch attached. With it, USB host controller can be > waken up with USB card reader plugging in on my test machine. Potentially silly question: is this better than waking up the PCIe port when polling the device (and putting it back to sleep again)? --Andy > > Best Regards, > Huang Ying > -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
