[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <45ED0BBF.1050000@intel.com>
Date: Mon, 05 Mar 2007 22:35:43 -0800
From: "Kok, Auke" <auke-jan.h.kok@...el.com>
To: Jeff Garzik <jeff@...zik.org>,
Linus Torvalds <torvalds@...ux-foundation.org>
CC: Ingo Molnar <mingo@...e.hu>,
"Michael S. Tsirkin" <mst@...lanox.co.il>,
Pavel Machek <pavel@....cz>,
Jens Axboe <jens.axboe@...cle.com>,
Adrian Bunk <bunk@...sta.de>,
Andrew Morton <akpm@...ux-foundation.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Thomas Gleixner <tglx@...utronix.de>, linux-pm@...ts.osdl.org,
Michal Piotrowski <michal.k.k.piotrowski@...il.com>,
"Eric W. Biederman" <ebiederm@...ssion.com>
Subject: Re: SATA resume slowness, e1000 MSI warning
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()
>> [<c01061bd>] show_trace_log_lvl+0x19/0x2e
>> [<c01062b6>] show_trace+0x12/0x14
>> [<c01062cc>] dump_stack+0x14/0x16
>> [<c024fcc4>] pci_enable_msi+0x6d/0x203
>> [<c02b709e>] e1000_request_irq+0x2e/0xe2
>> [<c02bb742>] e1000_resume+0x7f/0xef
>> [<c0249a68>] pci_device_resume+0x1a/0x44
>> [<c02b39ec>] resume_device+0xf7/0x16f
>> [<c02b3adb>] dpm_resume+0x77/0xcb
>> [<c02b3b69>] device_resume+0x3a/0x51
>> [<c014e669>] enter_state+0x193/0x1bb
>> [<c014e712>] state_store+0x81/0x97
>> [<c01b68bc>] subsys_attr_store+0x20/0x25
>> [<c01b6feb>] sysfs_write_file+0xce/0xf6
>> [<c017e16b>] vfs_write+0xb1/0x13a
>> [<c017e899>] sys_write+0x3d/0x61
>> [<c0105220>] 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 <auke-jan.h.kok@...el.com>
Cheers,
Auke
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists