Re: [PATCH] RFC: nomadik: expand timesource to 63 bits

Nicolas Pitre
2010-11-11T19:19:04Z

On Thu, 11 Nov 2010, Thomas Gleixner wrote:

> On Thu, 11 Nov 2010, Nicolas Pitre wrote:
> > On Thu, 11 Nov 2010, Thomas Gleixner wrote:
> > > Well, you don't need the keepwarm() timer, if you just call
> > > sched_clock() in the idle wakeup path, so the magic in the NOHZ code
> > > will ensure that you reread the thing timely.
> >
> > You still have to make sure you get out of idle before 16 secs has
> > passed.
>
> No, I meant to put it into the idle loop, right after mach_idle() or
> whatever is called to put the CPU into deep power sleep. The
> clockevents core code makes sure that you get interrupted in time.

OK, if the clockevents core code takes care of that then fine.

Of course one could claim that the sched_clock() code could be called
needlessly when frequently entering and leaving idle mode. That's why
I've used a simple timer for this purpose so far, which also makes the
implementation and its algorithm constraint workaround close together.

Nicolas