Re: [PATCH] clock_nanosleep(2)

2011-07-20 Thread Corinna Vinschen
Hi Yaakov, On Jul 19 20:54, Yaakov (Cygwin/X) wrote: This patchset implements the POSIX clock_nanosleep(2) function: http://pubs.opengroup.org/onlinepubs/9699919799/functions/clock_nanosleep.html http://www.kernel.org/doc/man-pages/online/pages/man2/clock_nanosleep.2.html In summary,

Re: [PATCH] clock_nanosleep(2)

2011-07-20 Thread Yaakov (Cygwin/X)
On Wed, 2011-07-20 at 09:56 +0200, Corinna Vinschen wrote: This doesn't look right. In contrast to nanosleep, clock_nanosleep is not subsumed under the _POSIX_TIMERS option. In fact it's the only function under the _POSIX_CLOCK_SELECTION option. I did some searching, and there are actually

Re: [PATCH] clock_nanosleep(2)

2011-07-20 Thread Yaakov (Cygwin/X)
On Wed, 2011-07-20 at 04:16 -0500, Yaakov (Cygwin/X) wrote: On Wed, 2011-07-20 at 09:56 +0200, Corinna Vinschen wrote: This doesn't look right. In contrast to nanosleep, clock_nanosleep is not subsumed under the _POSIX_TIMERS option. In fact it's the only function under the

Re: [PATCH] clock_nanosleep(2)

2011-07-20 Thread Corinna Vinschen
On Jul 20 04:50, Yaakov (Cygwin/X) wrote: On Wed, 2011-07-20 at 04:16 -0500, Yaakov (Cygwin/X) wrote: On Wed, 2011-07-20 at 09:56 +0200, Corinna Vinschen wrote: This doesn't look right. In contrast to nanosleep, clock_nanosleep is not subsumed under the _POSIX_TIMERS option. In fact

Re: [PATCH] clock_nanosleep(2)

2011-07-20 Thread Christopher Faylor
On Wed, Jul 20, 2011 at 09:56:54AM +0200, Corinna Vinschen wrote: I haven't much time right now. If cgf doesn't beat me to it, I'll review the function later. I have a couple of PriA problems to look into at work so I don't have much time either. cgf

Re: [PATCH] clock_nanosleep(2), pthread_condattr_[gs]etclock(3)

2011-07-20 Thread Yaakov (Cygwin/X)
On Wed, 2011-07-20 at 16:11 +0200, Corinna Vinschen wrote: On Jul 20 04:50, Yaakov (Cygwin/X) wrote: Actually, no need to panic, I took a closer look at this, and it's not all that hard at all, so I'll go ahead and implement pthread_condattr_[gs]etclock() as well. Just give me a day or two

Re: [PATCH] clock_nanosleep(2), pthread_condattr_[gs]etclock(3)

2011-07-20 Thread Yaakov (Cygwin/X)
On Wed, 2011-07-20 at 17:03 -0500, Yaakov (Cygwin/X) wrote: On Wed, 2011-07-20 at 16:11 +0200, Corinna Vinschen wrote: The only problem I see is the fact that a call to clock_settime influences calls to clock_nanosleep with absolute timeouts(*). However, clock_settime() can set only