Messages in this thread | | | Date | Wed, 19 Mar 2014 10:50:44 +0530 | Subject | Re: [PATCH v2 2/2] intel_pstate: Set core to min P state during core offline | From | Viresh Kumar <> |
| |
On 19 March 2014 01:14, Dirk Brandewie <dirk.brandewie@gmail.com> wrote: > There was no problem per se. In stop() all I really needed to do is stop > the > timer and set the P state to MIN. > > At init time I need to allocate memory and start timer. If stopping the > timer > and deallocating memory are separated then I need code in init() to detect > this case.
Sorry, I didn't understood what exactly is special here :(
If we return failure from CPU_POST_DEAD for some reason without calling exit() then you will have memory leak in your init() as we are allocating memory without checking if we already have that (nothing wrong in it though as other parts of kernel should handle things properly here).
Probably the situation would be exactly same if we divide the exit path into stop and exit routines, which I still feel is the right way forward. Because ideally cpufreq shouldn't call init() if it hasn't called exit() (If it is doing that right now then its wrong and can be fixed). And so you must do the cleanup in exit()..
| |