On 1 August 2013 20:54, Srivatsa S. Bhat srivatsa.bhat@linux.vnet.ibm.com wrote:
Well, I now agree that we don't have to keep the module refcount non-zero as long as CPUs are being managed (that was just my misunderstanding, sorry for the noise). However, I think the _get() and _put() used in the existing code is for synchronization: that is, to avoid races between trying to unload the cpufreq driver module and running a hotplug notifier (and calling the driver module's ->init() or ->exit() function).
With that being the case, I think we can retain the module refcounts and use them only for synchronization. And naturally the refcount should drop to zero after the critical section; no point keeping it incremented until the CPU is taken offline.
Or, am I confused again?
No, you aren't.
But, for synchronization we need some blocking stuff, so that when user tries to rmmod the module, it should wait until other critical sections are over and then unload the module. But here, we are simply returning from rmmod, saying that we are busy :)
So, for that kind of synchronization we better use locks available inside cpufreq core of if required another variable which can be used for refcount. But now this.