Before, if cb->start() failed, the module reference would never be put,because cb->cb_running is intentionally false at this point. Users aregenerally annoyed by this because they can no longer unload modules thatleak references. Also, it may be possible to tediously wrap a referencecounter back to zero, especially since module.c still uses atomic_incinstead of refcount_inc.