From mboxrd@z Thu Jan 1 00:00:00 1970 From: Richard Cochran Subject: Re: [PATCH v4 1/4] Produce system time from correlated clocksource Date: Tue, 20 Oct 2015 16:55:23 +0200 Message-ID: <20151020145523.GA5207@netboy> References: <1444675522-4198-1-git-send-email-christopher.s.hall@intel.com> <1444675522-4198-2-git-send-email-christopher.s.hall@intel.com> <20151020085408.GA2542@netboy> <20151020115113.GA23428@netboy> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: John Stultz , Christopher Hall , Jeff Kirsher , "H. Peter Anvin" , Ingo Molnar , Peter Zijlstra , "x86@kernel.org" , intel-wired-lan@lists.osuosl.org, netdev@vger.kernel.org, lkml , kevin.b.stanton@intel.com To: Thomas Gleixner Return-path: Received: from mail-wi0-f181.google.com ([209.85.212.181]:33383 "EHLO mail-wi0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751142AbbJTOz2 (ORCPT ); Tue, 20 Oct 2015 10:55:28 -0400 Content-Disposition: inline In-Reply-To: <20151020115113.GA23428@netboy> Sender: netdev-owner@vger.kernel.org List-ID: On Tue, Oct 20, 2015 at 01:51:13PM +0200, Richard Cochran wrote: > You can, in fact, achieve "proper" correlation by sampling. As John > said, the question is whether the method in the patch set "measurably > improves the error" over using another, simpler method. Here is a short example to put some numbers on the expected error. Let the driver sample at an interval of 1 ms. If the system time's frequency hasn't changed between two samples, A and B, then the driver may interpolate without introducing any error. If the frequency is changed between the sample times, then the interpolated value will have some error. Because 1 ms is smallest HZ value, the frequency can change at most once during the sample. If the frequency changes near point A or B, then the error is minimal. The worst case occurs when the frequency is changed half way between A and B. Suppose the frequency is changed by 10 PPM, at point C, half way between A and B. This change results in a 5 nanosecond time difference at B (10 PPM over C -> B). The driver will interpolate using line A-B with slope increased by 5 PPM, and the worst case error, found at point C, is then 2.5 nanoseconds. Thanks, Richard