diff options
| author |   Peter Zijlstra <peterz@infradead.org> | 2020-04-21 12:09:13 +0200 |
|---|---|---|
| committer | Peter Zijlstra <peterz@infradead.org> | 2020-06-15 14:10:23 +0200 |
| commit | a2bee0662f72fcbd248a424a1453dd946dcdaf3d (patch) | |
| tree | b9c2f37ea16a9e23e72cae0b55126573f2f5ff76 /drivers/thermal/intel/intel_powerclamp.c | |
| parent | sched,ion: Convert to sched_set_normal() (diff) | |
| download | linux-dev-a2bee0662f72fcbd248a424a1453dd946dcdaf3d.tar.xz linux-dev-a2bee0662f72fcbd248a424a1453dd946dcdaf3d.zip | |
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: Peter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'drivers/thermal/intel/intel_powerclamp.c')
| -rw-r--r-- | drivers/thermal/intel/intel_powerclamp.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/thermal/intel/intel_powerclamp.c b/drivers/thermal/intel/intel_powerclamp.c index f74b2473440d..b0eb5ece9243 100644 --- a/drivers/thermal/intel/intel_powerclamp.c +++ b/drivers/thermal/intel/intel_powerclamp.c @@ -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); |