Thread (7 messages) 7 messages, 2 authors, 2019-08-02

Re: [PATCH 1/1] psi: do not require setsched permission from the trigger creator

From: Suren Baghdasaryan <surenb@google.com>
Date: 2019-08-02 01:19:28
Also in: linux-mm, lkml

On Thu, Aug 1, 2019 at 2:59 PM Peter Zijlstra [off-list ref] wrote:
On Thu, Aug 01, 2019 at 11:28:30AM -0700, Suren Baghdasaryan wrote:
quoted
quoted
By marking it FIFO-99 you're in effect saying that your stupid
statistics gathering is more important than your life. It will preempt
the task that's in control of the band-saw emergency break, it will
preempt the task that's adjusting the electromagnetic field containing
this plasma flow.

That's insane.
IMHO an opt-in feature stops being "stupid" as soon as the user opted
in to use it, therefore explicitly indicating interest in it. However
I assume you are using "stupid" here to indicate that it's "less
important" rather than it's "useless".
Quite; PSI does have its uses. RT just isn't one of them.
Sorry about messing it up in the first place.
If you don't see any issues with my patch replacing
sched_setscheduler() with sched_setscheduler_nocheck(), would you mind
taking it too? I applied it over your patch onto Linus' ToT with no
merge conflicts.
Thanks,
Suren.
--
To unsubscribe from this group and stop receiving emails from it, send an email to kernel-team+unsubscribe@android.com.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help