On Wednesday 22 April 2009 03:18:28 Aurelien Jarno wrote: > On Mon, Apr 20, 2009 at 01:13:28AM -0400, Mike Frysinger wrote: > > On Sunday 19 April 2009 10:34:10 Aurelien Jarno wrote: > > > SCHED_FIFO, SCHED_OTHER, SCHED_RR are part of POSIX 1003.1b Process > > > Scheduling, so it is correct to assume they always exists. > > > > > > SCHED_BATCH and SCHED_IDLE are Linux specific, we should not assume > > > they exists. > > > > looks like you add a whole lot of ifdef's for no real gain. on non-Linux > > systems, you end up with options that simply dont work. not a big deal. > > on the flip side, these ifdef's are pretty invasive and split up code in > > a pretty bad way. > > Ok, please find a simpler patch below, we only adds the #ifdef to the > locations where we really need the values of SCHED_BATCH and SCHED_IDLE. i still dont get it. why do you need these changes at all ? looks to me like the build should work perfectly fine on any system regardless of SCHED_BATCH or SCHED_IDLE being defined/supported/whatever. -mike
Attachment:
signature.asc
Description: This is a digitally signed message part.
