From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Kok, Auke" Subject: Re: SATA resume slowness, e1000 MSI warning Date: Mon, 05 Mar 2007 22:35:43 -0800 Message-ID: <45ED0BBF.1050000@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> <45ECFC5F.7000102@garzik.org> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <45ECFC5F.7000102@garzik.org> 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: Jeff Garzik , Linus Torvalds Cc: Michal Piotrowski , linux-pm@lists.osdl.org, Linux Kernel Mailing List , Andrew Morton , Adrian Bunk , "Eric W. Biederman" , Pavel Machek , Jens Axboe , "Michael S. Tsirkin" , Thomas Gleixner , Ingo Molnar List-Id: linux-pm@vger.kernel.org Jeff Garzik wrote: > Ingo Molnar wrote: >> i'm also getting this WARN_ON() from e1000: >> >> BUG: at drivers/pci/msi.c:611 pci_enable_msi() >> [] show_trace_log_lvl+0x19/0x2e >> [] show_trace+0x12/0x14 >> [] dump_stack+0x14/0x16 >> [] pci_enable_msi+0x6d/0x203 >> [] e1000_request_irq+0x2e/0xe2 >> [] e1000_resume+0x7f/0xef >> [] pci_device_resume+0x1a/0x44 >> [] resume_device+0xf7/0x16f >> [] dpm_resume+0x77/0xcb >> [] device_resume+0x3a/0x51 >> [] enter_state+0x193/0x1bb >> [] state_store+0x81/0x97 >> [] subsys_attr_store+0x20/0x25 >> [] sysfs_write_file+0xce/0xf6 >> [] vfs_write+0xb1/0x13a >> [] sys_write+0x3d/0x61 >> [] syscall_call+0x7/0xb >> >> seems harmless because it seems to work fine. > = > I would poke Eric Biederman(sp?) about this one. Maybe its even solved = > by the MSI-enable-related patch he posted in the past 24-48 hours. Eric, Linus, I tried the 3-patch series "[PATCH 0/3] Basic msi bug fixes.." and they fix= this = problem for me. Were you expecting the OOPS in the first place? In any case= , it = survived several suspend/resume cycles on both enabled (irq alloc'd and ena= bled) = and disabled devices (only initialized). Jens Axboe was seeing the same problem, perhaps he can confirm the fix as w= ell. In any case, the patches have my blessing :) Please add my: Signed-off-by: Auke Kok Cheers, Auke From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932222AbXCFGfs (ORCPT ); Tue, 6 Mar 2007 01:35:48 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932672AbXCFGfs (ORCPT ); Tue, 6 Mar 2007 01:35:48 -0500 Received: from mga01.intel.com ([192.55.52.88]:22576 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932222AbXCFGfr (ORCPT ); Tue, 6 Mar 2007 01:35:47 -0500 X-ExtLoop1: 1 X-IronPort-AV: i="4.14,253,1170662400"; d="scan'208"; a="207115810:sNHT66957898" Message-ID: <45ED0BBF.1050000@intel.com> Date: Mon, 05 Mar 2007 22:35:43 -0800 From: "Kok, Auke" User-Agent: Mail/News 1.5.0.9 (X11/20061228) MIME-Version: 1.0 To: Jeff Garzik , Linus Torvalds CC: Ingo Molnar , "Michael S. Tsirkin" , Pavel Machek , Jens Axboe , Adrian Bunk , Andrew Morton , Linux Kernel Mailing List , Thomas Gleixner , linux-pm@lists.osdl.org, Michal Piotrowski , "Eric W. Biederman" Subject: Re: SATA resume slowness, e1000 MSI warning 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> <45ECFC5F.7000102@garzik.org> In-Reply-To: <45ECFC5F.7000102@garzik.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Jeff Garzik wrote: > Ingo Molnar wrote: >> i'm also getting this WARN_ON() from e1000: >> >> BUG: at drivers/pci/msi.c:611 pci_enable_msi() >> [] show_trace_log_lvl+0x19/0x2e >> [] show_trace+0x12/0x14 >> [] dump_stack+0x14/0x16 >> [] pci_enable_msi+0x6d/0x203 >> [] e1000_request_irq+0x2e/0xe2 >> [] e1000_resume+0x7f/0xef >> [] pci_device_resume+0x1a/0x44 >> [] resume_device+0xf7/0x16f >> [] dpm_resume+0x77/0xcb >> [] device_resume+0x3a/0x51 >> [] enter_state+0x193/0x1bb >> [] state_store+0x81/0x97 >> [] subsys_attr_store+0x20/0x25 >> [] sysfs_write_file+0xce/0xf6 >> [] vfs_write+0xb1/0x13a >> [] sys_write+0x3d/0x61 >> [] syscall_call+0x7/0xb >> >> seems harmless because it seems to work fine. > > I would poke Eric Biederman(sp?) about this one. Maybe its even solved > by the MSI-enable-related patch he posted in the past 24-48 hours. Eric, Linus, I tried the 3-patch series "[PATCH 0/3] Basic msi bug fixes.." and they fix this problem for me. Were you expecting the OOPS in the first place? In any case, it survived several suspend/resume cycles on both enabled (irq alloc'd and enabled) and disabled devices (only initialized). Jens Axboe was seeing the same problem, perhaps he can confirm the fix as well. In any case, the patches have my blessing :) Please add my: Signed-off-by: Auke Kok Cheers, Auke