Re: [PATCH 4/8] time: Condense timekeeper.xtime into xtime_sec

From: John Stultz
Date: Mon Aug 20 2012 - 15:58:12 EST


On 08/20/2012 12:45 PM, Andreas Schwab wrote:
John Stultz <john.stultz@xxxxxxxxxx> writes:

I'm not very familiar w/ the iBook hardware, but does it use a
clocksource, or does it use arch_gettimeoffset()?
clocksource: timebase mult[3640e38e] shift[24] registered

I suspect that the casting has avoided clipping some strange values from
the persistent clock.
That's my guess as well.

Could you try with the following patch against Linus' HEAD? I suspect it
will let the box resume (although it will seem as though no time was
spent in resume) and then let me know what the JDB lines print out?
JDB: suspend_time: 1345491706:0 resume_time: 1345491737:0
JDB: Trying to add: 31:0

(Looks reasonable.)
Huh. Yea, that looks fine. And without the __timekeeping_inject_sleeptime() call, the system resumed ok?

Thanks for the testing! I'll keep looking here.
-john





--
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/