[PATCH] use CLOCK_MONOTONIC_COARSE posix timer instead of CLOCK_MONOTONIC in Xorg

Alan Coopersmith alan.coopersmith at oracle.com
Mon Aug 23 17:46:02 PDT 2010


ykzhao wrote:
> On Mon, 2010-08-23 at 23:25 +0800, Adam Jackson wrote:
>> On Mon, 2010-08-23 at 10:23 +0200, Mark Kettenis wrote:
>>>> From: yakui.zhao at intel.com
>>>> Date: Mon, 23 Aug 2010 15:20:05 +0800
>>>> From: Zhao Yakui <yakui.zhao at intel.com>
>>>>
>>>> ---
>>>>  os/utils.c |   14 +++++++++++++-
>>>>  1 files changed, 13 insertions(+), 1 deletions(-)
>>>>
>>>> diff --git a/os/utils.c b/os/utils.c
>>>> index 51455cc..a08d591 100644
>>>> --- a/os/utils.c
>>>> +++ b/os/utils.c
>>>> @@ -242,6 +242,10 @@ OsSignal(int sig, OsSigHandlerPtr handler)
>>>>  #endif
>>>>  #endif
>>>>  
>>>> +#ifndef CLOCK_MONOTONIC_COARSE
>>>> +#define CLOCK_MONOTONIC_COARSE 6
>>>> +#endif
>>> What if an OS doesn't have CLOCK_MONOTONIC_COARSE, but uses the clock
>>> ID 6 for some other purpose?
>> Then this patch would be wrong.
>>
>> NAK on that basis.
> 
> Yes. Agree.
> 
> How about using the constant value(6) directly? 

That's the part that's wrong, assuming 6 will always mean
CLOCK_MONOTONIC_COARSE.   Perhaps if it's now part of the Linux ABI,
you could do it inside an #ifdef linux, but you can't assume that
value means anything useful on a non-linux kernel.

-- 
	-Alan Coopersmith-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System



More information about the xorg-devel mailing list