Re: pm_trace displays the wrong time from the RTC

[Dave Jones <davej@redhat.com>]

The patch below seems to have got dropped on the floor.
We've had it in Fedora since it was posted, and it does the
right thing, so I'm happy to give this an

ACKed-by: Dave Jones <davej@redhat.com>

if it helps it into upstream.

	Dave

On Wed, Aug 01, 2007 at 11:04:34PM +0200, Rafael J. Wysocki wrote:

 > > On 08/01/2007 11:47 AM, Rafael J. Wysocki wrote:
 > > > Sorry for slow response, I've just noticed this thread.
 > > > 
 > > > On Tuesday, 31 July 2007 23:20, Chuck Ebbert wrote:
 > > >> When resuming, systems print this when booted on 07/31/07:
 > > >>
 > > >> Date: 06/31/107
 > > >>
 > > >> /drivers/base/power/trace.c::read_magic_time():
 > > >> 114	get_rtc_time(&time);
 > > >> 115	printk("Time: %2d:%02d:%02d Date: %02d/%02d/%02d\n",
 > > >> 116		time.tm_hour, time.tm_min, time.tm_sec,
 > > >> 117		time.tm_mon, time.tm_mday, time.tm_year);
 > > >>
 > > >> include/asm-generic/rtc.h:
 > > >> 102         /*
 > > >> 103          * Account for differences between how the RTC uses the values
 > > >> 104          * and how they are defined in a struct rtc_time;
 > > >> 105          */
 > > >> 106         if (time->tm_year <= 69)
 > > >> 107                 time->tm_year += 100;
 > > >> 108 
 > > >> 109         time->tm_mon--;
 > > > 
 > > > Do you mean that systems display bad date after resuming if PM_TRACE is set,
 > > > even if /sys/power/pm_trace contains 1 ?
 > 
 > s/1/0/
 > 
 > > > If that's the case, are they i386 or x86_64?
 > > > 
 > > 
 > > i386:
 > > 
 > > https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=250238
 > 
 > OK, I think the appended patch is needed.
 > 
 > Greetings,
 > Rafael
 > 
 > 
 > ---
 > The way in which read_magic_time() displays the date read from the RTC is
 > apparently confusing to the users
 > (cf. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=250238).  Make it
 > print dates in the standard way.
 > 
 > 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.23-rc1/drivers/base/power/trace.c
 > ===================================================================
 > --- linux-2.6.23-rc1.orig/drivers/base/power/trace.c	2007-07-23 22:06:42.000000000 +0200
 > +++ linux-2.6.23-rc1/drivers/base/power/trace.c	2007-08-01 22:51:16.000000000 +0200
 > @@ -114,7 +114,7 @@ static unsigned int read_magic_time(void
 >  	get_rtc_time(&time);
 >  	printk("Time: %2d:%02d:%02d  Date: %02d/%02d/%02d\n",
 >  		time.tm_hour, time.tm_min, time.tm_sec,
 > -		time.tm_mon, time.tm_mday, time.tm_year);
 > +		time.tm_mon + 1, time.tm_mday, time.tm_year % 100);
 >  	val = time.tm_year;				/* 100 years */
 >  	if (val > 100)
 >  		val -= 100;
 > -
 > 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/
---end quoted text---

-- 
http://www.codemonkey.org.uk