On Mon, Oct 22, 2012 at 09:47:06AM -0700, Kevin Hilman wrote: > Peter Zijlstra <peterz@xxxxxxxxxxxxx> writes: > > > On Fri, 2012-10-19 at 16:54 -0700, Kevin Hilman wrote: > > > >> So I did the same thing for my ARM SoC, and it definitley stops the RT > >> throttling. > >> > >> However, it has the undesriable (IMO) side effect of making timed printk > >> output rather unhelpful for debugging suspend/resume since printk time > >> stays constant throughout suspend/resume no matter how long you > >> sleep. :( > >> > >> So does that mean we have to choose between useful printk times during > >> suspend/resume or functioning IRQ threads during suspend/resume ? > > > > Urgh.. this was not something I considered. This being primarily the > > sched_clock infrastructure and such. > > > > So what exactly is the problem with the suspend resume thing (its not > > something I've ever debugged), is all you need a clean break between pre > > and post suspend, or do you need the actual time the machine was gone? > > I think it's more a question of what people are used to. I think folks > used to debugging suspend/resume (at least on ARM) are used to having > the printk timestamps reflect the amount of time the machine was gone. > > With a sched_clock() that counts during suspend, that feature doesn't > work anymore. I'm not sure that this feature is a deal breaker, but it > has been convenient. IMHO, this isn't a deal breaker, it's nothing more than cosmetic issue. The big hint about the sched_clock() behaviour is partly in the name, and the behaviour you get from the scheduler if you don't give it what it wants. The scheduler sets the requirements for sched_clock(), not printk, so if we have to fix sched_clock() to get correct behaviour from the scheduler, that's what we have to do irrespective of cosmetic printk issues. And there are many other ways to measure time off in suspend... we have at least three other functions which return time, and which will return updated time after a resume event. -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
