From: Stephen Hemminger <[EMAIL PROTECTED]> Date: Tue, 16 Oct 2007 09:10:35 -0700
> On Tue, 16 Oct 2007 15:47:13 +1000 > Benjamin Herrenschmidt <[EMAIL PROTECTED]> wrote: > > > The way it's currently implemented, there's a list of NAPI's attached to > > the netdev, so obviously, that won't work for my usage scenario. > > sky2 driver has potentially 2 ports for 1 NAPI instance. And it's unfortunately buggy. For example, the sky2_suspend() code iterates over the ports and calls sky2_down() on each one that is up. That will call napi_disable() on the same NAPI instance if multiple ports are up, and thus hang the machine. This is why, Stephen, for the second time, I'm telling you that perhaps you should put the NAPI instance into the sky2_port struct. - 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
