On Friday 26 February 2010 21:42:29 Linus Torvalds wrote: > > On Fri, 26 Feb 2010, Linus Torvalds wrote: > > > > So send me a patch. I'll try it. But I have no hardware docs, nor any > > information about how that SSB bridge is supposed to work, or why DMA > > might be failing. > > Btw, I also object to your argument that > > "Well, my original plan was to get rid of controller_restart and not add > yet another user of it, because it is extremely broken and racy. The > locking in the whole driver is completely braindead due to the mere > existence of this function." > > is a reason to not apply the patch. Blah, I never said that. I just said what my original plan was and that the workaround is not as sound as it looks like. In fact, it kind of depends on luck that it works at all. The controller_restart _never_ worked correctly, because it does not notify the stack about the restart. So your code implicitly relies on getting a DMA error before the stack does any significant negotiation with the AP. (Or that the stack detects the device fuckup and starts over again). > The fact that the driver locking is odd is _not_ a reason to not fix other > issues that are totally unrelated to locking. To say it again: I _never_ said that. -- Greetings, Michael. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html