Re: [PATCH 1/1] LinuxPPS core support.

[Ingo Molnar]

* Rodolfo Giometti <giometti@xxxxxxxx> wrote:

> This patch adds the kernel side of the PPS support currently 
> named "LinuxPPS".
> 
> PPS means "pulse per second" and a PPS source is just a device 
> which provides a high precision signal each second so that an 
> application can use it to adjust system clock time.
> 
> Common use is the combination of the NTPD as userland program 
> with a GPS receiver as PPS source to obtain a wallclock-time 
> with sub-millisecond synchronisation to UTC.

Hm. I was looking at this stuff with the prospect of adding it 
to the timer tree, but i'm really struggling with a few 
fundamental questions.

The most basic one is: why do we need this?

The main purpose of your current patchset seems to be to deliver 
interrupt timestamps to user-space, where it will in essence be 
used to feed new adjtimex adjustments via ntpd.

I.e. the whole thing comes around in a circle in the end, but 
via user-space, where jitter will only increase.

Why not cut out the jittery middle man and add some intelligent 
API to register PPS interrupt sources straight with the NTP 
code, and let those IRQ timestamps be fed _directly_ into our 
time adjustment code?

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/