Re: [PATCH] hwclock: flush stdout in hwclock -c

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

 




On 04/20/2015 03:28 AM, Alexey Galakhov wrote:
> Hello William,
> 
> Am Sun, 19 Apr 2015 20:34:19 -0400
> schrieb J William Piggott <elseifthen@xxxxxxx>:
> 
>>  I've been advocating for the removal of the compare function. One of
>> my justifications for doing so is that it has been broken from day
>> one and therefore nobody can be using for anything.
>>
>>  So could you tell me how you are using it, and what useful purpose
>> it is serving for you?
> 
> We use it in an automated test of the Linux system time on an embedded
> system. Recently we fixed a kernel bug introduced by one of the 3rd
> party patches that caused a huge (0.01%-0.1%) drift of the
> CLOCK_REALTIME due to incorrect PLL initialization. The corresponding
> test based on hwclock's compare is now added to our automated test
> system. The test runs hwclock -c forever and fails if any drift
> anomality is discovered.
>

So you are using the Hardware Clock as a reference to detect anomalies
in the System Clock? You must have an exceptional Hardware Clock. It
sounds as though you are using NTP, why not compare NTP time directly to
your System time?

> Looks like hwclock -c works correctly at least in our case.

I assume then, that you are only using columns one and two, because
columns 3 and 4 are invalid output. If yes, then you are not using
hwclock -c for its actual 'compare' functionality, but only as a source
for timestamps. Your test could capture timestamps without hwclock's
compare function, yes?


> I think it
> could be rewritten using clock_gettime() instead of gettimeofday().
> 
> Regards,
> Alexey
> 
--
To unsubscribe from this list: send the line "unsubscribe util-linux" 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