[PATCH] Cygwin: sched_setscheduler: allow changes of the priority

[Corinna Vinschen]

Hi Christian,

On Nov 25 15:00, Christian Franke wrote:
> Corinna Vinschen wrote:
> > Fixes: ...?
> 
> ... the very first commit (cgf 2001) of sched.cc :-)
> 
> New patch attached.
> 
> From e95fc1aceb5287f9ad65c6c078125fecba6c6de9 Mon Sep 17 00:00:00 2001
> From: Christian Franke <christian.franke@t-online.de>
> Date: Mon, 25 Nov 2024 14:51:04 +0100
> Subject: [PATCH] Cygwin: sched_setscheduler: allow changes of the priority
> 
> Behave like sched_setparam() if the requested policy is identical
> to the fixed value (SCHED_FIFO) returned by sched_getscheduler().
> 
> Fixes: 6b2a2aa4af1e ("Add missing files.")

Huh, yeah, this is spot on.  I wonder if it would make sense to change
that to 9a08b2c02eea ("* sched.cc: New file.  Implement sched*.")
though, given that was the patch intended to add sched.cc :)))

Sorry, but I have to ask two more questions:

- Isn't returning SCHED_FIFO sched_getscheduler() just as wrong?
  Shouldn't that be SCHED_OTHER, and sched_setscheduler() should check
  for that instead?  Cygwin in a real-time scenario sounds a bit
  far-fetched...

- Don't you want this patch in 3.5.5?  I'd merge the other patch
  into 3.5.5 anyway...


Thanks,
Corinna