On Thu, 2012-01-19 at 11:35 -0600, Larry Finger wrote: > This patch converts all rtlwifi-based drivers to use the asynchronous firmware > loading mechanism. A completion queue entry is inititialized and the firmware > load is started with the device initialization. The firmware-loaded callback routine > issues a complete() call. The routine that uploads firmware to the device calls > wait_for_completion_interruptible_timeout() with a 5 sec timeout. > > The current patch is working, but as this is my first usage of the completion > mechanism, I would like an expert to look it over. In particular, is it > sufficient and prudent to issue a complete() call when the driver is unloaded > so as to not leave a queue entry when the underlying struct is destroyed? I don't think this makes a lot of sense. You're already registering with mac80211, and then you wait inside start -- but that would be holding the RTNL etc. The way we do this in iwlwifi (and some others) makes more sense I think: we only register with mac80211 after we actually load firmware, i.e. in your rtl_fw_cb() you'd continue with ieee80211_register_hw() etc instead of doing it in pci_probe(). If you never get firmware, you obviously don't do that -- you unbind instead by calling device_release_driver(). Doing that not only saves you the complexity of the completion but also the potential deadlock (until timeout) when the tool that uploads the firmware needs to acquire the rtnl for some reason and ifup is waiting for the firmware. johannes -- 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
