Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752665AbaL3Myh (ORCPT ); Tue, 30 Dec 2014 07:54:37 -0500 Received: from mail-wi0-f170.google.com ([209.85.212.170]:41079 "EHLO mail-wi0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751977AbaL3Myg (ORCPT ); Tue, 30 Dec 2014 07:54:36 -0500 Message-ID: <54A2A088.3030602@gmail.com> Date: Tue, 30 Dec 2014 13:54:32 +0100 From: "Michael Kerrisk (man-pages)" User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: Laurent Georget , linux-man CC: mtk.manpages@gmail.com, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2 v3] adjtimex.2: add explanation on ADJ_TAI mode References: <547AEE19.4040901@supelec.fr> In-Reply-To: <547AEE19.4040901@supelec.fr> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Laurent, On 11/30/2014 11:14 AM, Laurent Georget wrote: > This patch includes the adjtimex mode ADJ_TAI, intended to update the > TAI offset. Thanks. Applied, with a few rewordings (which you can find in Git). One point/question below. > There are other FIXMEs in the man page and it's a bit > outdated. I'll be on it once those first patches are accepted (to be > sure to do the right thing). I would be happy to get more such patches. > diff --git a/man2/adjtimex.2 b/man2/adjtimex.2 > index 0b919c5..396b8cc 100644 > --- a/man2/adjtimex.2 > +++ b/man2/adjtimex.2 > @@ -24,9 +24,7 @@ > .\" Modified 1997-01-31 by Eric S. Raymond > .\" Modified 1997-07-30 by Paul Slootman > .\" Modified 2004-05-27 by Michael Kerrisk > -.\" > -.\" FIXME Document ADJ_TAI (added in Linux 2.6.26) > -.\" commit 153b5d054ac2d98ea0d86504884326b6777f683d > +.\" Modified 2014-11-30 by Laurent Georget > .\" > .\" FIXME Document ADJ_MICRO and ADJ_NANO (added in Linux 2.6.26) > .\" commit eea83d896e318bda54be2d2770d2c5d6668d11db > @@ -101,6 +99,7 @@ combination of zero or more of the following bits: > #define ADJ_ESTERROR 0x0008 /* estimated time error */ > #define ADJ_STATUS 0x0010 /* clock status */ > #define ADJ_TIMECONST 0x0020 /* pll time constant */ > +#define ADJ_TAI 0x0080 /* TAI offset */ > #define ADJ_TICK 0x4000 /* tick value */ > #define ADJ_OFFSET_SINGLESHOT 0x8001 /* old-fashioned adjtime() */ > .fi > @@ -110,6 +109,24 @@ Ordinary users are restricted to a zero value for > .IR modes . > Only the superuser may set any parameters. > .br > +In the case of > +.BR ADJ_TAI , > +as > +.I buf->tai > +is read-only, the new value is passed through > +.IR buf->constant , > +therefore > +.B ADJ_TIMECONST > +and > +.B ADJ_TAI > +should probably not be used at the same time. "should probably not" seems a little weak to me. Surely it is rather the case that they *must not* be used at the same time. I changed it to "must not". Thanks, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/