On Dec 16 08:48, Christian Franke wrote:
> Christian Franke wrote:
> 
> A slighty changed version is attached. SCHED_IDLE now sets the lowest
> prioritity.
> 
> 
> > 
> > Linux also provides (40) rt_priority and (41) policy. Adding these would
> > require to determine (or fake) fields (26) to (39) first :-)
> > 
> 
> This would also require to decide whether the values of (41) policy should
> be Cygwin (SCHED_OTHER=3) or Linux (...=0) compatible...

Cygwin-compatible.  A tool actually using this feature should check
against SCHED_OTHER of the target system, not the value of a foreign
system or, worse, the constant 0.

> --- a/winsup/cygwin/sigproc.cc
> +++ b/winsup/cygwin/sigproc.cc
> @@ -1348,8 +1348,8 @@ wait_sig (VOID *)
>    /* GetTickCount() here is enough because GetTickCount() - t0 does
>       not overflow until 49 days psss. Even if GetTickCount() overflows,
>       GetTickCount() - t0 returns correct value, since underflow in
> -     unsigned wraps correctly. Pending a signal for more than 49
> -     days makes no sense. */
> +     unsigned wraps correctly. Pending a signal for more thtn 49
> +     days would be noncense. */
>    DWORD t0 = GetTickCount ();
>    for (;;)
>      {

This hunk isn't supposed to be part of the patch, I guess...


Corinna

Reply via email to