Commit a2bee066 authored by Peter Zijlstra's avatar Peter Zijlstra

sched,powerclamp: Convert to sched_set_fifo()

Because SCHED_FIFO is a broken scheduler model (see previous patches)
take away the priority field, the kernel can't possibly make an
informed decision.

Effectively no change.

Cc: rafael.j.wysocki@intel.com
Signed-off-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: default avatarIngo Molnar <mingo@kernel.org>
parent 9309de08
...@@ -70,9 +70,6 @@ static unsigned int control_cpu; /* The cpu assigned to collect stat and update ...@@ -70,9 +70,6 @@ static unsigned int control_cpu; /* The cpu assigned to collect stat and update
*/ */
static bool clamping; static bool clamping;
static const struct sched_param sparam = {
.sched_priority = MAX_USER_RT_PRIO / 2,
};
struct powerclamp_worker_data { struct powerclamp_worker_data {
struct kthread_worker *worker; struct kthread_worker *worker;
struct kthread_work balancing_work; struct kthread_work balancing_work;
...@@ -488,7 +485,7 @@ static void start_power_clamp_worker(unsigned long cpu) ...@@ -488,7 +485,7 @@ static void start_power_clamp_worker(unsigned long cpu)
w_data->cpu = cpu; w_data->cpu = cpu;
w_data->clamping = true; w_data->clamping = true;
set_bit(cpu, cpu_clamping_mask); set_bit(cpu, cpu_clamping_mask);
sched_setscheduler(worker->task, SCHED_FIFO, &sparam); sched_set_fifo(worker->task);
kthread_init_work(&w_data->balancing_work, clamp_balancing_func); kthread_init_work(&w_data->balancing_work, clamp_balancing_func);
kthread_init_delayed_work(&w_data->idle_injection_work, kthread_init_delayed_work(&w_data->idle_injection_work,
clamp_idle_injection_func); clamp_idle_injection_func);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment