From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757201Ab3FSQPf (ORCPT ); Wed, 19 Jun 2013 12:15:35 -0400 Received: from qmta02.emeryville.ca.mail.comcast.net ([76.96.30.24]:39656 "EHLO qmta02.emeryville.ca.mail.comcast.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757177Ab3FSQPd (ORCPT ); Wed, 19 Jun 2013 12:15:33 -0400 From: Shuah Khan To: pavel@ucw.cz, rjw@sisk.pl, len.brown@intel.com Cc: Shuah Khan , linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, shuahkhan@gmail.com Subject: [PATCH 2/2] power: Add warning message about incorrect system date and time after resume when pm_trace is enabled Date: Wed, 19 Jun 2013 10:15:18 -0600 Message-Id: <1371658518-2878-1-git-send-email-shuah.kh@samsung.com> X-Mailer: git-send-email 1.7.10.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org pm_trace uses the system's Real Time Clock (RTC) to save the magic number. Reason for this is that the RTC is the only reliably available piece of hardware during resume operations where a value can be set that will survive a reboot. Consequence is that after a resume (even if it is successful) your system clock will have a value corresponding to the magic number instead of the correct date/time! It is therefore advisable to use a program like ntp-date or rdate to reset the correct date/time from an external time source when using this trace option. There is no run-time message to warn users of the consequences of enabling pm_trace. Adding a warning message to pm_trace_store() will serve as a reminder to users to set the system date and time after resume. Signed-off-by: Shuah Khan --- kernel/power/main.c | 1 + 1 file changed, 1 insertion(+) diff --git a/kernel/power/main.c b/kernel/power/main.c index d77663b..4f769a6 100644 --- a/kernel/power/main.c +++ b/kernel/power/main.c @@ -528,6 +528,7 @@ pm_trace_store(struct kobject *kobj, struct kobj_attribute *attr, if (sscanf(buf, "%d", &val) == 1) { pm_trace_enabled = !!val; + pr_warn("CAUTION: this option will cause your machine's real-time clock to be set to an invalid time after a resume. Remember to set correct time after resume\n"); return n; } return -EINVAL; -- 1.7.10.4