From mboxrd@z Thu Jan 1 00:00:00 1970 From: mark gross Subject: Re: mmotm 2010-05-11 - dies in pm_qos_update_request() Date: Fri, 14 May 2010 16:43:53 -0700 Message-ID: <20100514234353.GA14494@linux.intel.com> References: <201005120149.o4C1n7P4002637@imap1.linux-foundation.org> <201005122307.20860.rjw@sisk.pl> <4793.1273761278@localhost> <201005140024.50279.rjw@sisk.pl> Reply-To: mgross@linux.intel.com Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Valdis.Kletnieks@vt.edu, Andrew Morton , "David S. Miller" , linux-kernel@vger.kernel.org, e1000-devel@lists.sourceforge.net, netdev@vger.kernel.org To: "Rafael J. Wysocki" Return-path: Content-Disposition: inline In-Reply-To: <201005140024.50279.rjw@sisk.pl> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Fri, May 14, 2010 at 12:24:50AM +0200, Rafael J. Wysocki wrote: > On Thursday 13 May 2010, Valdis.Kletnieks@vt.edu wrote: > > On Wed, 12 May 2010 23:07:20 +0200, "Rafael J. Wysocki" said: > > > On Wednesday 12 May 2010, Valdis.Kletnieks@vt.edu wrote: > > > > On Tue, 11 May 2010 18:21:22 PDT, akpm@linux-foundation.org said: > > > > > The mm-of-the-moment snapshot 2010-05-11-18-20 has been uploaded to > > > > > > > > > > http://userweb.kernel.org/~akpm/mmotm/ > > > > > > > > Dell Latitude E6500, x86_64 kernel. > > > > > > > > Died a horrid death at boot in the e1000e driver. Seems to be > > > > something in linux-next.patch. Didn't get a netconsole trace for obvious > > > > reasons... > > > > > > > > Copied-by-hand traceback: > > > > pm_qos_update_request()+0x22 > > > > e1000_configure+0x478 > > > > e1000_open_device+0xee > > > > ? _raw_notifier_call_chain+0xf > > > > __dev_open+0xec > > > > dev_open+0x1b > > > > netpoll_setup+0x28b > > > > init_netconsole+0xbc > > > > > > > > I suspect this commit: > > > > > > > > commit 23606cf5d1192c2b17912cb2ef6e62f9b11de133 > > > > Author: Rafael J. Wysocki > > > > Date: Sun Mar 14 14:35:17 2010 +0000 > > > > > > > > e1000e / PCI / PM: Add basic runtime PM support (rev. 4) > > > > > > No, I don't think so. I'm running -rc6 with this patch applied on a box with > > > e1000e and it works just fine. > > > > > > Please try to revert this one instead: > > > > > > http://git.kernel.org/?p=linux/kernel/git/rafael/suspend-2.6.git;a=patch;h=ed77134bfccf5e75b6cbadab268e559dbe6a4ebb > > > > Confirming - reverting that patch and doing the build fixup results in a > > kernel that doesn't blow up in the e1000e driver... > > Then I guess there's an initializations problem somewhere. > > Mark, any chance to look into that any time soon? If we don't resolve this > before the merge window opens, I'm afraid I'll have to revert that commit > from my tree. > I'll look at it right away. I think I have an e1000e in my home box. --mgross