Re: SATA resume slowness, e1000 MSI warning

[Kok, Auke]

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@xxxxxxxxx>


Cheers,

Auke
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/