Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753131AbZL2WlV (ORCPT ); Tue, 29 Dec 2009 17:41:21 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753110AbZL2WlU (ORCPT ); Tue, 29 Dec 2009 17:41:20 -0500 Received: from ogre.sisk.pl ([217.79.144.158]:40872 "EHLO ogre.sisk.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753107AbZL2WlT (ORCPT ); Tue, 29 Dec 2009 17:41:19 -0500 From: "Rafael J. Wysocki" To: OGAWA Hirofumi Subject: Re: [Bug #14730] sky2 won't work after suspend/resume cycle Date: Tue, 29 Dec 2009 23:41:55 +0100 User-Agent: KMail/1.12.3 (Linux/2.6.33-rc2-tst; KDE/4.3.3; x86_64; ; ) Cc: Linux Kernel Mailing List , Kernel Testers List , "Maciej J. Woloszyk" References: <200912292229.47644.rjw@sisk.pl> <874on9tov5.fsf@devron.myhome.or.jp> In-Reply-To: <874on9tov5.fsf@devron.myhome.or.jp> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <200912292341.56040.rjw@sisk.pl> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2188 Lines: 52 On Tuesday 29 December 2009, OGAWA Hirofumi wrote: > "Rafael J. Wysocki" writes: > > > On Tuesday 29 December 2009, OGAWA Hirofumi wrote: > >> "Rafael J. Wysocki" writes: > >> > >> > This message has been generated automatically as a part of a report > >> > of regressions introduced between 2.6.31 and 2.6.32. > >> > > >> > The following bug entry is on the current list of known regressions > >> > introduced between 2.6.31 and 2.6.32. Please verify if it still should > >> > be listed and let me know (either way). > >> > > >> > > >> > Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=14730 > >> > Subject : sky2 won't work after suspend/resume cycle > >> > Submitter : Maciej J. Woloszyk > >> > Date : 2009-12-04 09:02 (26 days old) > >> > >> This seems to be related to commit 4b77b0a2ba27d64f58f16d8d4d48d8319dda36ff. > >> > >> It clears dev->state_saved blindly, but pci_pm_resume() calls > >> pci_restore_standard_config() before pm->resume() (by probably > >> historical reason). > > > > The other way around. pci_restore_standard_config() is the new thing and the > > pci_restore_state() in ->resume() is supposed to be unnecessary. Unfortunately > > in this case the restoration of PCI config registers in > > pci_restore_standard_config() doesn't work. > > > > I'd like to understand the reason, but temporarily we can just set > > pdev->state_saved to 1 in sky2_resume(), as proposed in the bug entry. > > Sure. However, it might not be only sky2. That's correct. It more likely is a chipset issue. > Well, anyway, another likely one is - restore() should be called after > pci_pm_default_resume()? No, because we want the PCI standard config spaces to be restored before the device interrupts are enabled to avoid intermittent resume failures related to interrupt mishandling. That's why it is important to understand the nature of the sky2 failure here. Rafael -- 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/