From: "Michael C. Pratt" <mcpratt@pm.me>
To: Linus Torvalds <torvalds@linux-foundation.org>,
Andrew Morton <akpm@linux-foundation.org>,
Thomas Gleixner <tglx@linutronix.de>,
Peter Zijlstra <peterz@infradead.org>,
Ingo Molnar <mingo@redhat.com>,
Juri Lelli <juri.lelli@redhat.com>,
Vincent Guittot <vincent.guittot@linaro.org>,
Dietmar Eggemann <dietmar.eggemann@arm.com>,
Steven Rostedt <rostedt@goodmis.org>,
Ben Segall <bsegall@google.com>, Mel Gorman <mgorman@suse.de>,
Valentin Schneider <vschneid@redhat.com>
Cc: "Michael C . Pratt" <mcpratt@pm.me>, linux-kernel@vger.kernel.org
Subject: [PATCH RESEND 2 0/1] sched/syscalls: Allow setting niceness using sched_param struct
Date: Mon, 11 Nov 2024 07:02:12 +0000 [thread overview]
Message-ID: <20241111070152.9781-1-mcpratt@pm.me> (raw)
Dear Linus and devs,
I don't want to take much of your time, this is a very simple patch.
I am concerned that it is being ignored without a good reason.
This is my third time sending it, the first being within the merge window
for 6.12 which I was hoping this would be included in.
This patch is a short and safe way to expand the current ABI regarding
the ability to set a process priority at the moment of process creation.
For example, there currently is no way to set a process priority
for normal processes made with the posix_spawn*() family of functions.
Since it is unsafe to directly change the static priority
for non-RT processes, the niceness value is modified instead
in order to result in the same effective priority as what is requested.
Any requested priority outside the effect of the niceness range is invalid.
Peter had responded with a concern that the value passed in
to sched_priority should be the niceness value itself in this case,
however, for the functions used to query the accepted range,
sched_get_priority_max() and sched_get_priority_min(),
as defined by POSIX, a negative value indicates failure.
This would also be inconsistent with other usage of the value.
Therefore, I believe this remains to be the correct way to implement it.
I replied as such and got no reply back.
I still think I have a good idea here, but for whatever reason,
this being in my opinion one of the simplest and most sensible patches
that I have tried to send so far, is being met with
mostly dead silence or misunderstanding of the situation...
So I am opening up the conversation to a wider audience.
Thanks for reading
next reply other threads:[~2024-11-11 7:02 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-11-11 7:02 Michael C. Pratt [this message]
2024-11-11 7:03 ` [PATCH RESEND 2 1/1] sched/syscalls: Allow setting niceness using sched_param struct Michael C. Pratt
2024-11-12 15:34 ` Steven Rostedt
2024-11-13 0:13 ` Michael Pratt
2024-11-13 0:36 ` Steven Rostedt
2024-11-13 6:04 ` Michael Pratt
2024-11-13 14:58 ` Steven Rostedt
2024-11-25 14:33 ` kernel test robot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20241111070152.9781-1-mcpratt@pm.me \
--to=mcpratt@pm.me \
--cc=akpm@linux-foundation.org \
--cc=bsegall@google.com \
--cc=dietmar.eggemann@arm.com \
--cc=juri.lelli@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mgorman@suse.de \
--cc=mingo@redhat.com \
--cc=peterz@infradead.org \
--cc=rostedt@goodmis.org \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.org \
--cc=vincent.guittot@linaro.org \
--cc=vschneid@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox