18 Mar
2013
18 Mar
'13
3:57 p.m.
2013/3/18 Viresh Kumar viresh.kumar@linaro.org:
On Mon, Mar 18, 2013 at 9:09 PM, Frederic Weisbecker fweisbec@gmail.com wrote:
2013/3/18 Viresh Kumar viresh.kumar@linaro.org:
+static inline int sched_select_cpu(unsigned int sd_flags) +{
return raw_smp_processor_id();I feel this should be symetric with the requirement of having preemption disabled as in the CONFIG_NO_HZ version. This should be smp_processor_id().
Yes, my fault.
+int sched_select_cpu(unsigned int sd_flags)
It would be nice to have some more precise naming. sched_select_busy_cpu() ?
You are correct that name can be improved but busy may give it a different meaning. Maybe sched_select_non_idle_cpu()?
That works too yeah.
Thanks.