[PATCH] ntp: fix return value of adjtimex() calling for STA_INS/DEL

From: Seiichi Ikarashi
Date: Tue Nov 20 2012 - 03:20:58 EST


Hi,

Since commit 6b43ae8a619d17c4935c3320d2ef9e92bdeed05d,
adjtimex() system call returns TIME_OK even if STA_INS/DEL calling.
But the man page says it should be TIME_INS/TIME_DEL, respectively.

I have no idea except for such an ad-hoc fix.

Signed-off-by: Seiichi Ikarashi <s.ikarashi@xxxxxxxxxxxxxx>

--- a/kernel/time/ntp.c 2012-11-20 17:09:08.000000000 +0900
+++ b/kernel/time/ntp.c 2012-11-20 17:13:18.000000000 +0900
@@ -684,6 +684,10 @@ int do_adjtimex(struct timex *txc)
}

result = time_state; /* mostly `TIME_OK' */
+ if (time_status & STA_INS)
+ result = TIME_INS;
+ else if (time_status & STA_DEL)
+ result = TIME_DEL;
/* check for errors */
if (is_error_status(time_status))
result = TIME_ERROR;
--
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/