On 24-11-16, 05:53, Ingo Molnar wrote: > > Firstly, please start changes to scheduler code with a verb. This title: > > Subject: Re: [PATCH V2 4/4] cpufreq: schedutil: irq-work and mutex are only > used in slow path > > is totally inadequate as it's a statement that says nothing about the > _change_. > > What does the patch do? Does it add, remove, modify, fix or clean up?
Thanks for the tip. I have sometimes seen similar subjects-line in patches from core developers and so thought it might be right. But yes I understand what you are saying and will take care of this in future across all subsystems. > * Viresh Kumar <viresh.ku...@linaro.org> wrote: > > > Execute the irq-work specific initialization/exit code only when the > > fast path isn't available. > > Is this an optimization? A correctness fix? Its an optimization but yeah I will try to explain a bit more next time. -- viresh
