Re: [PATCH] schedutils: don't assume SCHED_BATCH and SCHED_IDLE exist

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.

 Exactly.
 
 It'd be better to add a NOTES section to the chrt.1 man page where
 you can describe the difference between Linux and non-linux systems. 

    Karel

-- 
 Karel Zak  <kzak@xxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe util-linux-ng" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux