Jeff Garzik wrote:
Auke Kok wrote:
Herbert Xu wrote:
"netif_poll_enable can only be called if you've previously called
netif_poll_disable. Otherwise a poll might already be in action
and you may get a crash like this."
Removing the call to netif_poll_enable in e1000_open should fix this issue,
the only other call to netif_poll_enable is in e1000_up() which is only
reached after a device reset or resume.
Bugzilla: http://bugzilla.kernel.org/show_bug.cgi?id=8455
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=240339
Tested by Doug Chapman <[EMAIL PROTECTED]>
Signed-off-by: Auke Kok <[EMAIL PROTECTED]>
Acked-by: Herbert Xu <[EMAIL PROTECTED]>
---
drivers/net/e1000/e1000_main.c | 4 ----
1 files changed, 0 insertions(+), 4 deletions(-)
applied, though as a poster (DaveJ?) noted, I'm not sure it completely
fixes the bug
It fixes the HP IA64 issue reported by Doug Chapman, but indeed does not even
apply to the FC bugzilla. I was mistaken due to the fact that I sent the code
that introduced the bug a while ago, but it's not in 2.6.21 (it was in the
e1000-fixes tree you had and merged after 2.6.21).
So, this patch doesn't apply for -stable.
Auke
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
