sched,powerclamp: Convert to sched_set_fifo()
authorPeter Zijlstra <peterz@infradead.org>
Tue, 21 Apr 2020 10:09:13 +0000 (12:09 +0200)
committerPeter Zijlstra <peterz@infradead.org>
Mon, 15 Jun 2020 12:10:23 +0000 (14:10 +0200)
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: Peter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: Ingo Molnar <mingo@kernel.org>
drivers/thermal/intel/intel_powerclamp.c

index f74b2473440d26f6f51be525b1be09f556c020a0..b0eb5ece9243b2007002ed6beb5e76888348e049 100644 (file)
@@ -70,9 +70,6 @@ static unsigned int control_cpu; /* The cpu assigned to collect stat and update
                                  */
 static bool clamping;
 
-static const struct sched_param sparam = {
-       .sched_priority = MAX_USER_RT_PRIO / 2,
-};
 struct powerclamp_worker_data {
        struct kthread_worker *worker;
        struct kthread_work balancing_work;
@@ -488,7 +485,7 @@ static void start_power_clamp_worker(unsigned long cpu)
        w_data->cpu = cpu;
        w_data->clamping = true;
        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_delayed_work(&w_data->idle_injection_work,
                                  clamp_idle_injection_func);