提交 b9db4273 编写于 作者: R Rafael J. Wysocki

cpufreq: governor: Avoid irq_work_queue_on() crash on non-SMP ARM

As it turns out, irq_work_queue_on() will crash if invoked on
non-SMP ARM platforms, but in fact it is not necessary to use that
function in the cpufreq governor code (as it doesn't matter to that
code which CPU will handle the irq_work), so change it to always use
irq_work_queue().

Fixes: 8fb47ff100af (cpufreq: governor: Replace timers with utilization update callbacks)
Reported-and-tested-by: NGuenter Roeck <linux@roeck-us.net>
Reported-and-tested-by: NTony Lindgren <tony@atomide.com>
Signed-off-by: NRafael J. Wysocki <rafael.j.wysocki@intel.com>
上级 a23d6d18
...@@ -350,15 +350,6 @@ static void dbs_irq_work(struct irq_work *irq_work) ...@@ -350,15 +350,6 @@ static void dbs_irq_work(struct irq_work *irq_work)
schedule_work(&policy_dbs->work); schedule_work(&policy_dbs->work);
} }
static inline void gov_queue_irq_work(struct policy_dbs_info *policy_dbs)
{
#ifdef CONFIG_SMP
irq_work_queue_on(&policy_dbs->irq_work, smp_processor_id());
#else
irq_work_queue(&policy_dbs->irq_work);
#endif
}
static void dbs_update_util_handler(struct update_util_data *data, u64 time, static void dbs_update_util_handler(struct update_util_data *data, u64 time,
unsigned long util, unsigned long max) unsigned long util, unsigned long max)
{ {
...@@ -378,7 +369,7 @@ static void dbs_update_util_handler(struct update_util_data *data, u64 time, ...@@ -378,7 +369,7 @@ static void dbs_update_util_handler(struct update_util_data *data, u64 time,
delta_ns = time - policy_dbs->last_sample_time; delta_ns = time - policy_dbs->last_sample_time;
if ((s64)delta_ns >= policy_dbs->sample_delay_ns) { if ((s64)delta_ns >= policy_dbs->sample_delay_ns) {
policy_dbs->last_sample_time = time; policy_dbs->last_sample_time = time;
gov_queue_irq_work(policy_dbs); irq_work_queue(&policy_dbs->irq_work);
return; return;
} }
} }
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册