> Quoting Eric W. Biederman <ebiederman@lnxi.com>: > Subject: Re: [ofa-general] Re: dst_ifdown breaks infiniband? > > "Michael S. Tsirkin" <[EMAIL PROTECTED]> writes: > > >> > Why is neighbour->dev changed here? > >> > >> It holds reference to device and prevents its destruction. > >> If dst is held somewhere, we cannot destroy the device and deadlock > >> while unregister. > > > > BTW, can this ever happen for the loopback device itself? > > Is it ever unregistered? > > Well I don't think the loopback device is currently but as soon > as we get network namespace support we will have multiple loopback > devices and they will get unregistered when we remove the network > namespace.
Hmm. Then the code moving dst->dev to point to the loopback device will have to be fixed too. I'll post a patch a bit later. -- MST - 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
