* [PATCH]drivers:base:power:trace.c Add "UTC" Coordinated Universal Time to the printk. @ 2011-06-09 6:01 Justin P. Mattock 2011-06-12 12:12 ` Rafael J. Wysocki 0 siblings, 1 reply; 12+ messages in thread From: Justin P. Mattock @ 2011-06-09 6:01 UTC (permalink / raw) To: len.brown; +Cc: linux-pm, linux-kernel, Justin P. Mattock From: "Justin P. Mattock" <justinmattock@gmail.com> sounds stupid, but taking a glance at the time, and seeing the wrong time, or what seemed wrong in dmesg, caused me to go into total check the time clock panic mode.. So the patch below adds: "UTC" Coordinated Universal Time abreviation to the printk so people like me dont flip out over the time! before: [ 0.114915] Time: 1:47:03 Date: 06/09/11 after: [ 0.114728] Time: 5:46:02 UTC Date: 06/09/11 Signed-off-by: Justin P. Mattock <justinmattock@gmail.com> --- drivers/base/power/trace.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/base/power/trace.c b/drivers/base/power/trace.c index c80e138..009f459 100644 --- a/drivers/base/power/trace.c +++ b/drivers/base/power/trace.c @@ -112,7 +112,7 @@ static unsigned int read_magic_time(void) unsigned int val; get_rtc_time(&time); - pr_info("Time: %2d:%02d:%02d Date: %02d/%02d/%02d\n", + pr_info("Time: %2d:%02d:%02d UTC Date: %02d/%02d/%02d\n", time.tm_hour, time.tm_min, time.tm_sec, time.tm_mon + 1, time.tm_mday, time.tm_year % 100); val = time.tm_year; /* 100 years */ -- 1.7.5.2 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH]drivers:base:power:trace.c Add "UTC" Coordinated Universal Time to the printk. 2011-06-09 6:01 [PATCH]drivers:base:power:trace.c Add "UTC" Coordinated Universal Time to the printk Justin P. Mattock @ 2011-06-12 12:12 ` Rafael J. Wysocki 2011-06-12 15:52 ` Justin P. Mattock 0 siblings, 1 reply; 12+ messages in thread From: Rafael J. Wysocki @ 2011-06-12 12:12 UTC (permalink / raw) To: Justin P. Mattock; +Cc: linux-pm, linux-kernel, Len Brown Hi, On Thursday, June 09, 2011, Justin P. Mattock wrote: > From: "Justin P. Mattock" <justinmattock@gmail.com> > > sounds stupid, but taking a glance at the time, and seeing the wrong time, or what seemed > wrong in dmesg, caused me to go into total check the time clock panic mode.. So the patch below adds: > "UTC" Coordinated Universal Time abreviation to the printk so people like me dont flip out over the time! > > before: > [ 0.114915] Time: 1:47:03 Date: 06/09/11 > > after: > [ 0.114728] Time: 5:46:02 UTC Date: 06/09/11 > > Signed-off-by: Justin P. Mattock <justinmattock@gmail.com> I suspect the goal is to mark messages printed by the PM trace code so that they can be easily distinguished from messages from other sources to avoid confusion. Why do you think it's a good idea to use the "UTC" string for this purpose? The time printed in those messages need not be UTC. It would be better to simply print "RTC time: ..., date: ..." IMO. Thanks, Rafael > --- > drivers/base/power/trace.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/base/power/trace.c b/drivers/base/power/trace.c > index c80e138..009f459 100644 > --- a/drivers/base/power/trace.c > +++ b/drivers/base/power/trace.c > @@ -112,7 +112,7 @@ static unsigned int read_magic_time(void) > unsigned int val; > > get_rtc_time(&time); > - pr_info("Time: %2d:%02d:%02d Date: %02d/%02d/%02d\n", > + pr_info("Time: %2d:%02d:%02d UTC Date: %02d/%02d/%02d\n", > time.tm_hour, time.tm_min, time.tm_sec, > time.tm_mon + 1, time.tm_mday, time.tm_year % 100); > val = time.tm_year; /* 100 years */ > ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH]drivers:base:power:trace.c Add "UTC" Coordinated Universal Time to the printk. 2011-06-12 12:12 ` Rafael J. Wysocki @ 2011-06-12 15:52 ` Justin P. Mattock 2011-06-12 18:35 ` Rafael J. Wysocki 0 siblings, 1 reply; 12+ messages in thread From: Justin P. Mattock @ 2011-06-12 15:52 UTC (permalink / raw) To: Rafael J. Wysocki; +Cc: linux-pm, linux-kernel, Len Brown On 06/12/2011 05:12 AM, Rafael J. Wysocki wrote: > Hi, > > On Thursday, June 09, 2011, Justin P. Mattock wrote: >> From: "Justin P. Mattock"<justinmattock@gmail.com> >> >> sounds stupid, but taking a glance at the time, and seeing the wrong time, or what seemed >> wrong in dmesg, caused me to go into total check the time clock panic mode.. So the patch below adds: >> "UTC" Coordinated Universal Time abreviation to the printk so people like me dont flip out over the time! >> >> before: >> [ 0.114915] Time: 1:47:03 Date: 06/09/11 >> >> after: >> [ 0.114728] Time: 5:46:02 UTC Date: 06/09/11 >> >> Signed-off-by: Justin P. Mattock<justinmattock@gmail.com> > > I suspect the goal is to mark messages printed by the PM trace code so that > they can be easily distinguished from messages from other sources to avoid > confusion. Why do you think it's a good idea to use the "UTC" string for > this purpose? The time printed in those messages need not be UTC. > > It would be better to simply print "RTC time: ..., date: ..." IMO. > > Thanks, > Rafael > well.. if thats better, then thats better.. over here(people that dont know what RTC time is) would not get so confused with a simple UTC or PDT or whatever the time zone is but if RTC is bettr, then its better. > >> --- >> drivers/base/power/trace.c | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/drivers/base/power/trace.c b/drivers/base/power/trace.c >> index c80e138..009f459 100644 >> --- a/drivers/base/power/trace.c >> +++ b/drivers/base/power/trace.c >> @@ -112,7 +112,7 @@ static unsigned int read_magic_time(void) >> unsigned int val; >> >> get_rtc_time(&time); >> - pr_info("Time: %2d:%02d:%02d Date: %02d/%02d/%02d\n", >> + pr_info("Time: %2d:%02d:%02d UTC Date: %02d/%02d/%02d\n", >> time.tm_hour, time.tm_min, time.tm_sec, >> time.tm_mon + 1, time.tm_mday, time.tm_year % 100); >> val = time.tm_year; /* 100 years */ >> > > ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH]drivers:base:power:trace.c Add "UTC" Coordinated Universal Time to the printk. 2011-06-12 15:52 ` Justin P. Mattock @ 2011-06-12 18:35 ` Rafael J. Wysocki 2011-06-12 19:34 ` Justin P. Mattock 0 siblings, 1 reply; 12+ messages in thread From: Rafael J. Wysocki @ 2011-06-12 18:35 UTC (permalink / raw) To: Justin P. Mattock; +Cc: linux-pm, linux-kernel, Len Brown On Sunday, June 12, 2011, Justin P. Mattock wrote: > On 06/12/2011 05:12 AM, Rafael J. Wysocki wrote: > > Hi, > > > > On Thursday, June 09, 2011, Justin P. Mattock wrote: > >> From: "Justin P. Mattock"<justinmattock@gmail.com> > >> > >> sounds stupid, but taking a glance at the time, and seeing the wrong time, or what seemed > >> wrong in dmesg, caused me to go into total check the time clock panic mode.. So the patch below adds: > >> "UTC" Coordinated Universal Time abreviation to the printk so people like me dont flip out over the time! > >> > >> before: > >> [ 0.114915] Time: 1:47:03 Date: 06/09/11 > >> > >> after: > >> [ 0.114728] Time: 5:46:02 UTC Date: 06/09/11 > >> > >> Signed-off-by: Justin P. Mattock<justinmattock@gmail.com> > > > > I suspect the goal is to mark messages printed by the PM trace code so that > > they can be easily distinguished from messages from other sources to avoid > > confusion. Why do you think it's a good idea to use the "UTC" string for > > this purpose? The time printed in those messages need not be UTC. > > > > It would be better to simply print "RTC time: ..., date: ..." IMO. > > > > Thanks, > > Rafael > > > > well.. if thats better, then thats better.. over here(people that dont > know what RTC time is) would not get so confused with a simple UTC or > PDT or whatever the time zone is but if RTC is bettr, then its better. My point is we don't know this time is always UTC, so we rather shouldn't label it as UTC unconditionally, should we? Rafael ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH]drivers:base:power:trace.c Add "UTC" Coordinated Universal Time to the printk. 2011-06-12 18:35 ` Rafael J. Wysocki @ 2011-06-12 19:34 ` Justin P. Mattock 2011-06-12 20:27 ` Rafael J. Wysocki 0 siblings, 1 reply; 12+ messages in thread From: Justin P. Mattock @ 2011-06-12 19:34 UTC (permalink / raw) To: Rafael J. Wysocki; +Cc: linux-pm, linux-kernel, Len Brown On 06/12/2011 11:35 AM, Rafael J. Wysocki wrote: > On Sunday, June 12, 2011, Justin P. Mattock wrote: >> On 06/12/2011 05:12 AM, Rafael J. Wysocki wrote: >>> Hi, >>> >>> On Thursday, June 09, 2011, Justin P. Mattock wrote: >>>> From: "Justin P. Mattock"<justinmattock@gmail.com> >>>> >>>> sounds stupid, but taking a glance at the time, and seeing the wrong time, or what seemed >>>> wrong in dmesg, caused me to go into total check the time clock panic mode.. So the patch below adds: >>>> "UTC" Coordinated Universal Time abreviation to the printk so people like me dont flip out over the time! >>>> >>>> before: >>>> [ 0.114915] Time: 1:47:03 Date: 06/09/11 >>>> >>>> after: >>>> [ 0.114728] Time: 5:46:02 UTC Date: 06/09/11 >>>> >>>> Signed-off-by: Justin P. Mattock<justinmattock@gmail.com> >>> >>> I suspect the goal is to mark messages printed by the PM trace code so that >>> they can be easily distinguished from messages from other sources to avoid >>> confusion. Why do you think it's a good idea to use the "UTC" string for >>> this purpose? The time printed in those messages need not be UTC. >>> >>> It would be better to simply print "RTC time: ..., date: ..." IMO. >>> >>> Thanks, >>> Rafael >>> >> >> well.. if thats better, then thats better.. over here(people that dont >> know what RTC time is) would not get so confused with a simple UTC or >> PDT or whatever the time zone is but if RTC is bettr, then its better. > > My point is we don't know this time is always UTC, so we rather shouldn't > label it as UTC unconditionally, should we? > > Rafael > sounds good to me!! ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH]drivers:base:power:trace.c Add "UTC" Coordinated Universal Time to the printk. 2011-06-12 19:34 ` Justin P. Mattock @ 2011-06-12 20:27 ` Rafael J. Wysocki 2011-06-12 22:29 ` Justin P. Mattock 0 siblings, 1 reply; 12+ messages in thread From: Rafael J. Wysocki @ 2011-06-12 20:27 UTC (permalink / raw) To: Justin P. Mattock; +Cc: linux-pm, linux-kernel, Len Brown On Sunday, June 12, 2011, Justin P. Mattock wrote: > On 06/12/2011 11:35 AM, Rafael J. Wysocki wrote: > > On Sunday, June 12, 2011, Justin P. Mattock wrote: > >> On 06/12/2011 05:12 AM, Rafael J. Wysocki wrote: > >>> Hi, > >>> > >>> On Thursday, June 09, 2011, Justin P. Mattock wrote: > >>>> From: "Justin P. Mattock"<justinmattock@gmail.com> > >>>> > >>>> sounds stupid, but taking a glance at the time, and seeing the wrong time, or what seemed > >>>> wrong in dmesg, caused me to go into total check the time clock panic mode.. So the patch below adds: > >>>> "UTC" Coordinated Universal Time abreviation to the printk so people like me dont flip out over the time! > >>>> > >>>> before: > >>>> [ 0.114915] Time: 1:47:03 Date: 06/09/11 > >>>> > >>>> after: > >>>> [ 0.114728] Time: 5:46:02 UTC Date: 06/09/11 > >>>> > >>>> Signed-off-by: Justin P. Mattock<justinmattock@gmail.com> > >>> > >>> I suspect the goal is to mark messages printed by the PM trace code so that > >>> they can be easily distinguished from messages from other sources to avoid > >>> confusion. Why do you think it's a good idea to use the "UTC" string for > >>> this purpose? The time printed in those messages need not be UTC. > >>> > >>> It would be better to simply print "RTC time: ..., date: ..." IMO. > >>> > >>> Thanks, > >>> Rafael > >>> > >> > >> well.. if thats better, then thats better.. over here(people that dont > >> know what RTC time is) would not get so confused with a simple UTC or > >> PDT or whatever the time zone is but if RTC is bettr, then its better. > > > > My point is we don't know this time is always UTC, so we rather shouldn't > > label it as UTC unconditionally, should we? > > > > Rafael > > > > sounds good to me!! I'm not sure what you mean? ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH]drivers:base:power:trace.c Add "UTC" Coordinated Universal Time to the printk. 2011-06-12 20:27 ` Rafael J. Wysocki @ 2011-06-12 22:29 ` Justin P. Mattock 2011-06-13 21:47 ` Rafael J. Wysocki 0 siblings, 1 reply; 12+ messages in thread From: Justin P. Mattock @ 2011-06-12 22:29 UTC (permalink / raw) To: Rafael J. Wysocki; +Cc: linux-pm, linux-kernel, Len Brown On 06/12/2011 01:27 PM, Rafael J. Wysocki wrote: > On Sunday, June 12, 2011, Justin P. Mattock wrote: >> On 06/12/2011 11:35 AM, Rafael J. Wysocki wrote: >>> On Sunday, June 12, 2011, Justin P. Mattock wrote: >>>> On 06/12/2011 05:12 AM, Rafael J. Wysocki wrote: >>>>> Hi, >>>>> >>>>> On Thursday, June 09, 2011, Justin P. Mattock wrote: >>>>>> From: "Justin P. Mattock"<justinmattock@gmail.com> >>>>>> >>>>>> sounds stupid, but taking a glance at the time, and seeing the wrong time, or what seemed >>>>>> wrong in dmesg, caused me to go into total check the time clock panic mode.. So the patch below adds: >>>>>> "UTC" Coordinated Universal Time abreviation to the printk so people like me dont flip out over the time! >>>>>> >>>>>> before: >>>>>> [ 0.114915] Time: 1:47:03 Date: 06/09/11 >>>>>> >>>>>> after: >>>>>> [ 0.114728] Time: 5:46:02 UTC Date: 06/09/11 >>>>>> >>>>>> Signed-off-by: Justin P. Mattock<justinmattock@gmail.com> >>>>> >>>>> I suspect the goal is to mark messages printed by the PM trace code so that >>>>> they can be easily distinguished from messages from other sources to avoid >>>>> confusion. Why do you think it's a good idea to use the "UTC" string for >>>>> this purpose? The time printed in those messages need not be UTC. >>>>> >>>>> It would be better to simply print "RTC time: ..., date: ..." IMO. >>>>> >>>>> Thanks, >>>>> Rafael >>>>> >>>> >>>> well.. if thats better, then thats better.. over here(people that dont >>>> know what RTC time is) would not get so confused with a simple UTC or >>>> PDT or whatever the time zone is but if RTC is bettr, then its better. >>> >>> My point is we don't know this time is always UTC, so we rather shouldn't >>> label it as UTC unconditionally, should we? >>> >>> Rafael >>> >> >> sounds good to me!! > > I'm not sure what you mean? > with what you are saying "RTC" if anything.. rather than "UTC" ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH]drivers:base:power:trace.c Add "UTC" Coordinated Universal Time to the printk. 2011-06-12 22:29 ` Justin P. Mattock @ 2011-06-13 21:47 ` Rafael J. Wysocki 2011-06-13 22:19 ` Justin P. Mattock 0 siblings, 1 reply; 12+ messages in thread From: Rafael J. Wysocki @ 2011-06-13 21:47 UTC (permalink / raw) To: Justin P. Mattock; +Cc: linux-pm, linux-kernel, Len Brown On Monday, June 13, 2011, Justin P. Mattock wrote: > On 06/12/2011 01:27 PM, Rafael J. Wysocki wrote: > > On Sunday, June 12, 2011, Justin P. Mattock wrote: > >> On 06/12/2011 11:35 AM, Rafael J. Wysocki wrote: > >>> On Sunday, June 12, 2011, Justin P. Mattock wrote: > >>>> On 06/12/2011 05:12 AM, Rafael J. Wysocki wrote: > >>>>> Hi, > >>>>> > >>>>> On Thursday, June 09, 2011, Justin P. Mattock wrote: > >>>>>> From: "Justin P. Mattock"<justinmattock@gmail.com> > >>>>>> > >>>>>> sounds stupid, but taking a glance at the time, and seeing the wrong time, or what seemed > >>>>>> wrong in dmesg, caused me to go into total check the time clock panic mode.. So the patch below adds: > >>>>>> "UTC" Coordinated Universal Time abreviation to the printk so people like me dont flip out over the time! > >>>>>> > >>>>>> before: > >>>>>> [ 0.114915] Time: 1:47:03 Date: 06/09/11 > >>>>>> > >>>>>> after: > >>>>>> [ 0.114728] Time: 5:46:02 UTC Date: 06/09/11 > >>>>>> > >>>>>> Signed-off-by: Justin P. Mattock<justinmattock@gmail.com> > >>>>> > >>>>> I suspect the goal is to mark messages printed by the PM trace code so that > >>>>> they can be easily distinguished from messages from other sources to avoid > >>>>> confusion. Why do you think it's a good idea to use the "UTC" string for > >>>>> this purpose? The time printed in those messages need not be UTC. > >>>>> > >>>>> It would be better to simply print "RTC time: ..., date: ..." IMO. > >>>>> > >>>>> Thanks, > >>>>> Rafael > >>>>> > >>>> > >>>> well.. if thats better, then thats better.. over here(people that dont > >>>> know what RTC time is) would not get so confused with a simple UTC or > >>>> PDT or whatever the time zone is but if RTC is bettr, then its better. > >>> > >>> My point is we don't know this time is always UTC, so we rather shouldn't > >>> label it as UTC unconditionally, should we? > >>> > >>> Rafael > >>> > >> > >> sounds good to me!! > > > > I'm not sure what you mean? > > > > with what you are saying "RTC" if anything.. rather than "UTC" So, it looks like you'd like the appended patch to be appiled? Rafael --- From: Rafael J. Wysocki <rjw@sisk.pl> Subject: PM: Add "RTC" to PM trace time stamps to avoid confusion Some users are apparently confused by dmesg output from read_magic_time(), which looks like "real" time and date. Add the "RTC" string to time stamps printed by read_magic_time() to avoid that confusion. Reported-by: Justin P. Mattock <justinmattock@gmail.com> Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl> --- drivers/base/power/trace.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: linux-2.6/drivers/base/power/trace.c =================================================================== --- linux-2.6.orig/drivers/base/power/trace.c +++ linux-2.6/drivers/base/power/trace.c @@ -112,7 +112,7 @@ static unsigned int read_magic_time(void unsigned int val; get_rtc_time(&time); - pr_info("Time: %2d:%02d:%02d Date: %02d/%02d/%02d\n", + pr_info("RTC time: %2d:%02d:%02d, date: %02d/%02d/%02d\n", time.tm_hour, time.tm_min, time.tm_sec, time.tm_mon + 1, time.tm_mday, time.tm_year % 100); val = time.tm_year; /* 100 years */ ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH]drivers:base:power:trace.c Add "UTC" Coordinated Universal Time to the printk. 2011-06-13 21:47 ` Rafael J. Wysocki @ 2011-06-13 22:19 ` Justin P. Mattock 2011-06-14 20:02 ` Rafael J. Wysocki 0 siblings, 1 reply; 12+ messages in thread From: Justin P. Mattock @ 2011-06-13 22:19 UTC (permalink / raw) To: Rafael J. Wysocki; +Cc: linux-pm, linux-kernel, Len Brown On 06/13/2011 02:47 PM, Rafael J. Wysocki wrote: > On Monday, June 13, 2011, Justin P. Mattock wrote: >> On 06/12/2011 01:27 PM, Rafael J. Wysocki wrote: >>> On Sunday, June 12, 2011, Justin P. Mattock wrote: >>>> On 06/12/2011 11:35 AM, Rafael J. Wysocki wrote: >>>>> On Sunday, June 12, 2011, Justin P. Mattock wrote: >>>>>> On 06/12/2011 05:12 AM, Rafael J. Wysocki wrote: >>>>>>> Hi, >>>>>>> >>>>>>> On Thursday, June 09, 2011, Justin P. Mattock wrote: >>>>>>>> From: "Justin P. Mattock"<justinmattock@gmail.com> >>>>>>>> >>>>>>>> sounds stupid, but taking a glance at the time, and seeing the wrong time, or what seemed >>>>>>>> wrong in dmesg, caused me to go into total check the time clock panic mode.. So the patch below adds: >>>>>>>> "UTC" Coordinated Universal Time abreviation to the printk so people like me dont flip out over the time! >>>>>>>> >>>>>>>> before: >>>>>>>> [ 0.114915] Time: 1:47:03 Date: 06/09/11 >>>>>>>> >>>>>>>> after: >>>>>>>> [ 0.114728] Time: 5:46:02 UTC Date: 06/09/11 >>>>>>>> >>>>>>>> Signed-off-by: Justin P. Mattock<justinmattock@gmail.com> >>>>>>> >>>>>>> I suspect the goal is to mark messages printed by the PM trace code so that >>>>>>> they can be easily distinguished from messages from other sources to avoid >>>>>>> confusion. Why do you think it's a good idea to use the "UTC" string for >>>>>>> this purpose? The time printed in those messages need not be UTC. >>>>>>> >>>>>>> It would be better to simply print "RTC time: ..., date: ..." IMO. >>>>>>> >>>>>>> Thanks, >>>>>>> Rafael >>>>>>> >>>>>> >>>>>> well.. if thats better, then thats better.. over here(people that dont >>>>>> know what RTC time is) would not get so confused with a simple UTC or >>>>>> PDT or whatever the time zone is but if RTC is bettr, then its better. >>>>> >>>>> My point is we don't know this time is always UTC, so we rather shouldn't >>>>> label it as UTC unconditionally, should we? >>>>> >>>>> Rafael >>>>> >>>> >>>> sounds good to me!! >>> >>> I'm not sure what you mean? >>> >> >> with what you are saying "RTC" if anything.. rather than "UTC" > > So, it looks like you'd like the appended patch to be appiled? > > Rafael whatever works best with you guys... I just tripped out on that, and created what I had sent out.. > > --- > From: Rafael J. Wysocki<rjw@sisk.pl> > Subject: PM: Add "RTC" to PM trace time stamps to avoid confusion > > Some users are apparently confused by dmesg output from > read_magic_time(), which looks like "real" time and date. > Add the "RTC" string to time stamps printed by read_magic_time() to > avoid that confusion. > > Reported-by: Justin P. Mattock<justinmattock@gmail.com> > Signed-off-by: Rafael J. Wysocki<rjw@sisk.pl> > --- > drivers/base/power/trace.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > Index: linux-2.6/drivers/base/power/trace.c > =================================================================== > --- linux-2.6.orig/drivers/base/power/trace.c > +++ linux-2.6/drivers/base/power/trace.c > @@ -112,7 +112,7 @@ static unsigned int read_magic_time(void > unsigned int val; > > get_rtc_time(&time); > - pr_info("Time: %2d:%02d:%02d Date: %02d/%02d/%02d\n", > + pr_info("RTC time: %2d:%02d:%02d, date: %02d/%02d/%02d\n", > time.tm_hour, time.tm_min, time.tm_sec, > time.tm_mon + 1, time.tm_mday, time.tm_year % 100); > val = time.tm_year; /* 100 years */ > Justin P. Mattock ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH]drivers:base:power:trace.c Add "UTC" Coordinated Universal Time to the printk. 2011-06-13 22:19 ` Justin P. Mattock @ 2011-06-14 20:02 ` Rafael J. Wysocki 2011-06-14 20:03 ` Justin P. Mattock 2011-06-19 6:28 ` Justin P. Mattock 0 siblings, 2 replies; 12+ messages in thread From: Rafael J. Wysocki @ 2011-06-14 20:02 UTC (permalink / raw) To: Justin P. Mattock; +Cc: linux-pm, linux-kernel, Len Brown On Tuesday, June 14, 2011, Justin P. Mattock wrote: > On 06/13/2011 02:47 PM, Rafael J. Wysocki wrote: > > On Monday, June 13, 2011, Justin P. Mattock wrote: > >> On 06/12/2011 01:27 PM, Rafael J. Wysocki wrote: > >>> On Sunday, June 12, 2011, Justin P. Mattock wrote: > >>>> On 06/12/2011 11:35 AM, Rafael J. Wysocki wrote: > >>>>> On Sunday, June 12, 2011, Justin P. Mattock wrote: > >>>>>> On 06/12/2011 05:12 AM, Rafael J. Wysocki wrote: > >>>>>>> Hi, > >>>>>>> > >>>>>>> On Thursday, June 09, 2011, Justin P. Mattock wrote: > >>>>>>>> From: "Justin P. Mattock"<justinmattock@gmail.com> > >>>>>>>> > >>>>>>>> sounds stupid, but taking a glance at the time, and seeing the wrong time, or what seemed > >>>>>>>> wrong in dmesg, caused me to go into total check the time clock panic mode.. So the patch below adds: > >>>>>>>> "UTC" Coordinated Universal Time abreviation to the printk so people like me dont flip out over the time! > >>>>>>>> > >>>>>>>> before: > >>>>>>>> [ 0.114915] Time: 1:47:03 Date: 06/09/11 > >>>>>>>> > >>>>>>>> after: > >>>>>>>> [ 0.114728] Time: 5:46:02 UTC Date: 06/09/11 > >>>>>>>> > >>>>>>>> Signed-off-by: Justin P. Mattock<justinmattock@gmail.com> > >>>>>>> > >>>>>>> I suspect the goal is to mark messages printed by the PM trace code so that > >>>>>>> they can be easily distinguished from messages from other sources to avoid > >>>>>>> confusion. Why do you think it's a good idea to use the "UTC" string for > >>>>>>> this purpose? The time printed in those messages need not be UTC. > >>>>>>> > >>>>>>> It would be better to simply print "RTC time: ..., date: ..." IMO. > >>>>>>> > >>>>>>> Thanks, > >>>>>>> Rafael > >>>>>>> > >>>>>> > >>>>>> well.. if thats better, then thats better.. over here(people that dont > >>>>>> know what RTC time is) would not get so confused with a simple UTC or > >>>>>> PDT or whatever the time zone is but if RTC is bettr, then its better. > >>>>> > >>>>> My point is we don't know this time is always UTC, so we rather shouldn't > >>>>> label it as UTC unconditionally, should we? > >>>>> > >>>>> Rafael > >>>>> > >>>> > >>>> sounds good to me!! > >>> > >>> I'm not sure what you mean? > >>> > >> > >> with what you are saying "RTC" if anything.. rather than "UTC" > > > > So, it looks like you'd like the appended patch to be appiled? > > > > Rafael > > > whatever works best with you guys... OK, I'll queue up the one below for 3.1, then. > I just tripped out on that, and created what I had sent out.. > > > > --- > > From: Rafael J. Wysocki<rjw@sisk.pl> > > Subject: PM: Add "RTC" to PM trace time stamps to avoid confusion > > > > Some users are apparently confused by dmesg output from > > read_magic_time(), which looks like "real" time and date. > > Add the "RTC" string to time stamps printed by read_magic_time() to > > avoid that confusion. > > > > Reported-by: Justin P. Mattock<justinmattock@gmail.com> > > Signed-off-by: Rafael J. Wysocki<rjw@sisk.pl> > > --- > > drivers/base/power/trace.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > Index: linux-2.6/drivers/base/power/trace.c > > =================================================================== > > --- linux-2.6.orig/drivers/base/power/trace.c > > +++ linux-2.6/drivers/base/power/trace.c > > @@ -112,7 +112,7 @@ static unsigned int read_magic_time(void > > unsigned int val; > > > > get_rtc_time(&time); > > - pr_info("Time: %2d:%02d:%02d Date: %02d/%02d/%02d\n", > > + pr_info("RTC time: %2d:%02d:%02d, date: %02d/%02d/%02d\n", > > time.tm_hour, time.tm_min, time.tm_sec, > > time.tm_mon + 1, time.tm_mday, time.tm_year % 100); > > val = time.tm_year; /* 100 years */ > > > > > Justin P. Mattock > > ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH]drivers:base:power:trace.c Add "UTC" Coordinated Universal Time to the printk. 2011-06-14 20:02 ` Rafael J. Wysocki @ 2011-06-14 20:03 ` Justin P. Mattock 2011-06-19 6:28 ` Justin P. Mattock 1 sibling, 0 replies; 12+ messages in thread From: Justin P. Mattock @ 2011-06-14 20:03 UTC (permalink / raw) To: Rafael J. Wysocki; +Cc: linux-pm, linux-kernel, Len Brown On 06/14/2011 01:02 PM, Rafael J. Wysocki wrote: > On Tuesday, June 14, 2011, Justin P. Mattock wrote: >> On 06/13/2011 02:47 PM, Rafael J. Wysocki wrote: >>> On Monday, June 13, 2011, Justin P. Mattock wrote: >>>> On 06/12/2011 01:27 PM, Rafael J. Wysocki wrote: >>>>> On Sunday, June 12, 2011, Justin P. Mattock wrote: >>>>>> On 06/12/2011 11:35 AM, Rafael J. Wysocki wrote: >>>>>>> On Sunday, June 12, 2011, Justin P. Mattock wrote: >>>>>>>> On 06/12/2011 05:12 AM, Rafael J. Wysocki wrote: >>>>>>>>> Hi, >>>>>>>>> >>>>>>>>> On Thursday, June 09, 2011, Justin P. Mattock wrote: >>>>>>>>>> From: "Justin P. Mattock"<justinmattock@gmail.com> >>>>>>>>>> >>>>>>>>>> sounds stupid, but taking a glance at the time, and seeing the wrong time, or what seemed >>>>>>>>>> wrong in dmesg, caused me to go into total check the time clock panic mode.. So the patch below adds: >>>>>>>>>> "UTC" Coordinated Universal Time abreviation to the printk so people like me dont flip out over the time! >>>>>>>>>> >>>>>>>>>> before: >>>>>>>>>> [ 0.114915] Time: 1:47:03 Date: 06/09/11 >>>>>>>>>> >>>>>>>>>> after: >>>>>>>>>> [ 0.114728] Time: 5:46:02 UTC Date: 06/09/11 >>>>>>>>>> >>>>>>>>>> Signed-off-by: Justin P. Mattock<justinmattock@gmail.com> >>>>>>>>> >>>>>>>>> I suspect the goal is to mark messages printed by the PM trace code so that >>>>>>>>> they can be easily distinguished from messages from other sources to avoid >>>>>>>>> confusion. Why do you think it's a good idea to use the "UTC" string for >>>>>>>>> this purpose? The time printed in those messages need not be UTC. >>>>>>>>> >>>>>>>>> It would be better to simply print "RTC time: ..., date: ..." IMO. >>>>>>>>> >>>>>>>>> Thanks, >>>>>>>>> Rafael >>>>>>>>> >>>>>>>> >>>>>>>> well.. if thats better, then thats better.. over here(people that dont >>>>>>>> know what RTC time is) would not get so confused with a simple UTC or >>>>>>>> PDT or whatever the time zone is but if RTC is bettr, then its better. >>>>>>> >>>>>>> My point is we don't know this time is always UTC, so we rather shouldn't >>>>>>> label it as UTC unconditionally, should we? >>>>>>> >>>>>>> Rafael >>>>>>> >>>>>> >>>>>> sounds good to me!! >>>>> >>>>> I'm not sure what you mean? >>>>> >>>> >>>> with what you are saying "RTC" if anything.. rather than "UTC" >>> >>> So, it looks like you'd like the appended patch to be appiled? >>> >>> Rafael >> >> >> whatever works best with you guys... > > OK, I'll queue up the one below for 3.1, then. cool.. > >> I just tripped out on that, and created what I had sent out.. >>> >>> --- >>> From: Rafael J. Wysocki<rjw@sisk.pl> >>> Subject: PM: Add "RTC" to PM trace time stamps to avoid confusion >>> >>> Some users are apparently confused by dmesg output from >>> read_magic_time(), which looks like "real" time and date. >>> Add the "RTC" string to time stamps printed by read_magic_time() to >>> avoid that confusion. >>> >>> Reported-by: Justin P. Mattock<justinmattock@gmail.com> >>> Signed-off-by: Rafael J. Wysocki<rjw@sisk.pl> >>> --- >>> drivers/base/power/trace.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> Index: linux-2.6/drivers/base/power/trace.c >>> =================================================================== >>> --- linux-2.6.orig/drivers/base/power/trace.c >>> +++ linux-2.6/drivers/base/power/trace.c >>> @@ -112,7 +112,7 @@ static unsigned int read_magic_time(void >>> unsigned int val; >>> >>> get_rtc_time(&time); >>> - pr_info("Time: %2d:%02d:%02d Date: %02d/%02d/%02d\n", >>> + pr_info("RTC time: %2d:%02d:%02d, date: %02d/%02d/%02d\n", >>> time.tm_hour, time.tm_min, time.tm_sec, >>> time.tm_mon + 1, time.tm_mday, time.tm_year % 100); >>> val = time.tm_year; /* 100 years */ >>> >> >> >> Justin P. Mattock >> >> > > ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH]drivers:base:power:trace.c Add "UTC" Coordinated Universal Time to the printk. 2011-06-14 20:02 ` Rafael J. Wysocki 2011-06-14 20:03 ` Justin P. Mattock @ 2011-06-19 6:28 ` Justin P. Mattock 1 sibling, 0 replies; 12+ messages in thread From: Justin P. Mattock @ 2011-06-19 6:28 UTC (permalink / raw) To: Rafael J. Wysocki; +Cc: linux-pm, linux-kernel, Len Brown On 06/14/2011 01:02 PM, Rafael J. Wysocki wrote: > On Tuesday, June 14, 2011, Justin P. Mattock wrote: >> On 06/13/2011 02:47 PM, Rafael J. Wysocki wrote: >>> On Monday, June 13, 2011, Justin P. Mattock wrote: >>>> On 06/12/2011 01:27 PM, Rafael J. Wysocki wrote: >>>>> On Sunday, June 12, 2011, Justin P. Mattock wrote: >>>>>> On 06/12/2011 11:35 AM, Rafael J. Wysocki wrote: >>>>>>> On Sunday, June 12, 2011, Justin P. Mattock wrote: >>>>>>>> On 06/12/2011 05:12 AM, Rafael J. Wysocki wrote: >>>>>>>>> Hi, >>>>>>>>> >>>>>>>>> On Thursday, June 09, 2011, Justin P. Mattock wrote: >>>>>>>>>> From: "Justin P. Mattock"<justinmattock@gmail.com> >>>>>>>>>> >>>>>>>>>> sounds stupid, but taking a glance at the time, and seeing the wrong time, or what seemed >>>>>>>>>> wrong in dmesg, caused me to go into total check the time clock panic mode.. So the patch below adds: >>>>>>>>>> "UTC" Coordinated Universal Time abreviation to the printk so people like me dont flip out over the time! >>>>>>>>>> >>>>>>>>>> before: >>>>>>>>>> [ 0.114915] Time: 1:47:03 Date: 06/09/11 >>>>>>>>>> >>>>>>>>>> after: >>>>>>>>>> [ 0.114728] Time: 5:46:02 UTC Date: 06/09/11 >>>>>>>>>> >>>>>>>>>> Signed-off-by: Justin P. Mattock<justinmattock@gmail.com> >>>>>>>>> >>>>>>>>> I suspect the goal is to mark messages printed by the PM trace code so that >>>>>>>>> they can be easily distinguished from messages from other sources to avoid >>>>>>>>> confusion. Why do you think it's a good idea to use the "UTC" string for >>>>>>>>> this purpose? The time printed in those messages need not be UTC. >>>>>>>>> >>>>>>>>> It would be better to simply print "RTC time: ..., date: ..." IMO. >>>>>>>>> >>>>>>>>> Thanks, >>>>>>>>> Rafael >>>>>>>>> >>>>>>>> >>>>>>>> well.. if thats better, then thats better.. over here(people that dont >>>>>>>> know what RTC time is) would not get so confused with a simple UTC or >>>>>>>> PDT or whatever the time zone is but if RTC is bettr, then its better. >>>>>>> >>>>>>> My point is we don't know this time is always UTC, so we rather shouldn't >>>>>>> label it as UTC unconditionally, should we? >>>>>>> >>>>>>> Rafael >>>>>>> >>>>>> >>>>>> sounds good to me!! >>>>> >>>>> I'm not sure what you mean? >>>>> >>>> >>>> with what you are saying "RTC" if anything.. rather than "UTC" >>> >>> So, it looks like you'd like the appended patch to be appiled? >>> >>> Rafael >> >> >> whatever works best with you guys... > > OK, I'll queue up the one below for 3.1, then. before anything happens, I just noticed down dmesg: [ 1.931771] Magic number: 15:676:165 [ 1.940022] rtc_cmos 00:08: setting system clock to 2011-06-19 06:11:22 UTC (1308463882) [ 1.948456] Initializing network drop monitor service see how there is a UTC after the time. should this be for the above? (keep in mind I just noticed this, so I could be wrong) > >> I just tripped out on that, and created what I had sent out.. >>> >>> --- >>> From: Rafael J. Wysocki<rjw@sisk.pl> >>> Subject: PM: Add "RTC" to PM trace time stamps to avoid confusion >>> >>> Some users are apparently confused by dmesg output from >>> read_magic_time(), which looks like "real" time and date. >>> Add the "RTC" string to time stamps printed by read_magic_time() to >>> avoid that confusion. >>> >>> Reported-by: Justin P. Mattock<justinmattock@gmail.com> >>> Signed-off-by: Rafael J. Wysocki<rjw@sisk.pl> >>> --- >>> drivers/base/power/trace.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> Index: linux-2.6/drivers/base/power/trace.c >>> =================================================================== >>> --- linux-2.6.orig/drivers/base/power/trace.c >>> +++ linux-2.6/drivers/base/power/trace.c >>> @@ -112,7 +112,7 @@ static unsigned int read_magic_time(void >>> unsigned int val; >>> >>> get_rtc_time(&time); >>> - pr_info("Time: %2d:%02d:%02d Date: %02d/%02d/%02d\n", >>> + pr_info("RTC time: %2d:%02d:%02d, date: %02d/%02d/%02d\n", >>> time.tm_hour, time.tm_min, time.tm_sec, >>> time.tm_mon + 1, time.tm_mday, time.tm_year % 100); >>> val = time.tm_year; /* 100 years */ >>> >> >> >> Justin P. Mattock >> >> > > ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2011-06-19 6:28 UTC | newest] Thread overview: 12+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2011-06-09 6:01 [PATCH]drivers:base:power:trace.c Add "UTC" Coordinated Universal Time to the printk Justin P. Mattock 2011-06-12 12:12 ` Rafael J. Wysocki 2011-06-12 15:52 ` Justin P. Mattock 2011-06-12 18:35 ` Rafael J. Wysocki 2011-06-12 19:34 ` Justin P. Mattock 2011-06-12 20:27 ` Rafael J. Wysocki 2011-06-12 22:29 ` Justin P. Mattock 2011-06-13 21:47 ` Rafael J. Wysocki 2011-06-13 22:19 ` Justin P. Mattock 2011-06-14 20:02 ` Rafael J. Wysocki 2011-06-14 20:03 ` Justin P. Mattock 2011-06-19 6:28 ` Justin P. Mattock
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox