Re: [PATCH 17/23] sched,irq: Convert to sched_set_fifo()

From: Peter Zijlstra
Date: Wed Apr 22 2020 - 07:39:36 EST


On Wed, Apr 22, 2020 at 01:27:36PM +0200, Peter Zijlstra wrote:
> 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.

That's ofcourse a flat out lie; I wrote this before I noticed that -1
thing. It most definitely changes that. I also only noticed this while
reading through my own copy after sending :-/

> @@ -1281,13 +1278,12 @@ setup_irq_thread(struct irqaction *new,
> } else {
> t = kthread_create(irq_thread, new, "irq/%d-s-%s", irq,
> new->name);
> - param.sched_priority -= 1;
> }