On Tue, Jan 11, 2011 at 05:47:13AM +0900, Kuwahara,T. wrote: > On Mon, Jan 10, 2011 at 4:17 PM, Richard Cochran > <richardcochran@xxxxxxxxx> wrote: > > the PTP Hardware Clocks for which this whole patch > > set was created in the first place will keep their time as TAI. > > Are you sure of that? Yes. > I don't have the standard handy (it's non-free, right?) but it seems > that the Annex B states differently. IEEE Std 1588-2008, Page 41: The timescale PTP: In normal operation, the epoch is the PTP epoch and the timescale is continuous; see 7.2.4. The unit of measure of time is the SI second as realized on the rotating geoid. IEEE Std 1588-2008, Page 42: The PTP epoch is 1 January 1970 00:00:00 TAI, which is 31 December 1969 23:59:51.999918 UTC. The standard does permit using some other, arbitrary timescale, "set by an administrative procedure." But no other timescales are standardized by 1588. > But that's not the point anyway. My concern is that your patch not > only adds the useless (and broken) feature to the existing syscall > but also makes a permanent change to the public interface for your > own use. It is neither useless or broken. Using the new interface, you can estimate offset and frequency for a few seconds and jump directly into a closely synchronized state (like within 100 nanoseconds) relative to another clock on the network. That is pretty useful. Your issue with leap seconds is really a non-issue. The time value of the ADJ_SETOFFSET mode is a time interval, *not* an absolute time. > That's what I'm against. So if you stop touching the struct timex, > I won't complain anymore. I have no love for the timex interface. However, I used it for the new syscall because Arnd Bergmann and John Stultz specifically asked for it. Also, several other people have reviewed the new call without objecting to the new mode. Richard -- To unsubscribe from this list: send the line "unsubscribe linux-api" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html