Skip to contentSkip to conversation

adjtimex.2: add fields in struct timex description

PATCH In review Laurent Georget 2 participants 2 messages Nov 30, 2014 – Dec 30, 2014 Man-pages: manual pages for Linux -- Sections 2, 3, 4, 5, and 7

lore
Share & tools
lore

Laurent Georget

author
#0 lore raw

PATCH v3 1/2 adjtimex.2: add fields in struct timex description

Hello,

this is a refactored series of patches for adjtimex(2) man page. Please ignore all previous patches I sent for adjtimex.2.

The first patch updates the man page with the new fields added in struct timex since last edition of the man page.

Thank you for your time. These are my first patches so you may have to bear with me being a little inexpert and clumsy for some time. I apologize for that.

Laurent

man2/adjtimex.2 +9 -0
index eb3fda1..0b919c5 100644
@@ -73,6 +73,15 @@ struct timex {
                             (read-only) */
     struct timeval time; /* current time (read-only) */
     long tick;           /* usecs between clock ticks */
+    long ppsfreq;        /* pps frequency (scaled ppm) (read-only) */
+    long jitter;         /* pps jitter (usec) (read-only) */
+    int shift;           /* interval duration (sec) (read-only) */
+    long stabil;         /* pps stability (scaled ppm) (read-only) */
+    long jitcnt;         /* jitter limit exceeded (read-only) */
+    long calcnt;         /* calibration intervals (read-only) */
+    long errcnt;         /* calibration errors (read-only) */
+    long stbcnt;         /* stability limit exceeded (read-only) */
+    int tai;             /* TAI offset (s) (read-only) */
 };
 .fi
 .in

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html

#1 lore raw

Hello Laurent,

Sorry for the delayed follow up...

Quoted 4 lines · Laurent Georget

Hello,

this is a refactored series of patches for adjtimex(2) man page. Please ignore all previous patches I sent for adjtimex.2.

Thanks for the clear note about disregarding previous versions. That makes things much simpler for me.

Quoted 6 lines

The first patch updates the man page with the new fields added in struct timex since last edition of the man page.

Thank you for your time. These are my first patches so you may have to bear with me being a little inexpert and clumsy for some time. I apologize for that.

I've applied the patch. Note one point/question below.

Quoted 8 lines
man2/adjtimex.2 +0 -0
index eb3fda1..0b919c5 100644
@@ -73,6 +73,15 @@ struct timex {
                             (read-only) */
     struct timeval time; /* current time (read-only) */
     long tick;           /* usecs between clock ticks */

man2/adjtimex.2 struct timex { · line 85

Quoted 10 lines
man2/adjtimex.2 +9 -0
+    long ppsfreq;        /* pps frequency (scaled ppm) (read-only) */
+    long jitter;         /* pps jitter (usec) (read-only) */
+    int shift;           /* interval duration (sec) (read-only) */
+    long stabil;         /* pps stability (scaled ppm) (read-only) */
+    long jitcnt;         /* jitter limit exceeded (read-only) */
+    long calcnt;         /* calibration intervals (read-only) */
+    long errcnt;         /* calibration errors (read-only) */
+    long stbcnt;         /* stability limit exceeded (read-only) */
+    int tai;             /* TAI offset (s) (read-only) */
 };

Presumably, '(s)' means seconds, right?

I changed this to to '(sec)' for consistency with the other comments above.

Thanks,

Michael`

Signature · 7 lines
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-man" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html