From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Kok, Auke" Subject: Re: SATA resume slowness, e1000 MSI warning Date: Tue, 06 Mar 2007 09:09:30 -0800 Message-ID: <45EDA04A.1060500@intel.com> References: <20070225175559.GC12392@stusta.de> <20070227100202.GV3822@kernel.dk> <20070227102109.GG6745@elf.ucw.cz> <20070227103021.GA2250@kernel.dk> <20070227103407.GA17819@elte.hu> <20070227105922.GD2250@kernel.dk> <20070227111515.GA4271@kernel.dk> <20070301093450.GA8508@elte.hu> <20070302100704.GB2293@elf.ucw.cz> <20070305084257.GA4464@mellanox.co.il> <20070305101120.GA23032@elte.hu> <1173198401.24738.340.camel@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-pm-bounces@lists.osdl.org Errors-To: linux-pm-bounces@lists.osdl.org To: Linus Torvalds Cc: Michal Piotrowski , Ingo Molnar , linux-pm@lists.osdl.org, Linux Kernel Mailing List , Adrian Bunk , Pavel Machek , Jens Axboe , "Michael S. Tsirkin" , Andrew Morton , Jeff Garzik , Thomas Gleixner List-Id: linux-pm@vger.kernel.org Linus Torvalds wrote: > = > On Tue, 6 Mar 2007, Thomas Gleixner wrote: >> SATA has another nice feature. Somehow there is an interrupt pending on >> the SATA controller, which comes in somewhere in the middle of resume. >> If it happens before the SATA code resumed, the SATA code ignores the >> interrupt and the interrupt is disabled due to "nobody cared", which in >> turn prevents SATA to ever become functional again. > > And if you don't want to do any of these things (or are unable to, becaus= e = > of some ordering constraint or bad design), then you simply need to = > unregister and re-register the interrupt handler over sleep. > = >> Any idea on that one ? > = > Jeff, Auke, does this ring any bells? For the e1000 issue, the problem is solved with Eric Biederman's 3-patch ms= i = cleanups. You should have another message in your mailbox confirming that I = tested his patches and the MSI warning for e1000 suspend-resume is gone wit= h them. Cheers, Auke