login
Header Space

 
 

Re: new NAPI interface broken

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Jan-Bernd Themann <ossthema@...>
Cc: David Miller <davem@...>, <shemminger@...>, <netdev@...>, <themann@...>, <raisch@...>
Date: Wednesday, September 19, 2007 - 11:33 am

> One other thing I observed is that I can not unload the module as the
 > ref counter of the eth device is too low. I haven't tracked down the 
 > source of this problem yet.

I suspect that this is because netif_rx_reschedule() was missing a
dev_hold() to match the dev_put() in netif_rx_complete().  Dave merged
a fix for that problem yesterday, so current net-2.6.24 should be OK.
Let us know if it's not OK, because then there's another bug...

 - R.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: new NAPI interface broken, David Miller, (Fri Sep 14, 6:12 pm)
Re: new NAPI interface broken, Jan-Bernd Themann, (Tue Sep 18, 12:15 pm)
Re: new NAPI interface broken, Roland Dreier, (Wed Sep 19, 11:33 am)
Re: new NAPI interface broken, Jan-Bernd Themann, (Wed Sep 19, 11:43 am)
Re: new NAPI interface broken, David Miller, (Tue Sep 18, 3:08 pm)
speck-geostationary