Re: [PATCH v2] Add suspend/resume for HPET

2007-03-31 Thread Daniel Walker
On Sat, 2007-03-31 at 19:17 +0200, Ingo Molnar wrote: > yeah. There's some practical problems that need to be sorted out: much > of the current GTOD code is irq-driven (and all GTOD locks are > irq-safe), while the sysfs code needs to run in process-context level. > > Clocksources 'arrive' and

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-31 Thread Ingo Molnar
* Linus Torvalds <[EMAIL PROTECTED]> wrote: > Umm.. WHy not make the device tree look like this: > > -- "clocksource" -- +-- HPET > | > +-- TSC > | > +-- i8259 >

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-31 Thread Linus Torvalds
On Sat, 31 Mar 2007, Maxim Levitsky wrote: > > So maybe I was right afrer all, > Maybe it is better to add a suspend/resume hook to each clock source and call > it from timekeeping_resume() ? Umm.. WHy not make the device tree look like this: -- "clocksource" -- +-- HPET

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-31 Thread Greg KH
On Sat, Mar 31, 2007 at 09:53:43AM -0700, Linus Torvalds wrote: > Make them be at the top of the device tree by adding them early. That's > the whole point of the device tree after all - we have an ordering that is > enforced by its topology, and that we sort by when things were added. > > Right

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-31 Thread Ingo Molnar
* Linus Torvalds <[EMAIL PROTECTED]> wrote: > On Sat, 31 Mar 2007, Thomas Gleixner wrote: > > > > Right, but clock - sources/events need to be extremly late suspended and > > early resumed. How can we ensure this ? [...] > So the only thing that needs to be done is to make sure that we add > t

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-31 Thread Linus Torvalds
On Sat, 31 Mar 2007, Thomas Gleixner wrote: > > Right, but clock - sources/events need to be extremly late suspended and > early resumed. How can we ensure this ? Make them be at the top of the device tree by adding them early. That's the whole point of the device tree after all - we have an o

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-31 Thread Maxim Levitsky
On Saturday 31 March 2007 18:51:11 Thomas Gleixner wrote: > On Thu, 2007-03-29 at 15:46 +0200, Maxim Levitsky wrote: > > Subject: Add suspend/resume for HPET > > This adds support of suspend/resume on i386 for HPET > > Signed-off-by: Maxim Levitsky <[EMAIL PROTECTED]> > > > > +static struct sysdev_

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-31 Thread Greg KH
On Sat, Mar 31, 2007 at 06:33:20PM +0200, Thomas Gleixner wrote: > On Sat, 2007-03-31 at 09:09 -0700, Linus Torvalds wrote: > > > > On Sat, 31 Mar 2007, Thomas Gleixner wrote: > > > > > > While I agree in principle with the patch, I'm a bit uncomfortable. The > > > sys device suspend / resume ord

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-31 Thread Thomas Gleixner
On Sat, 2007-03-31 at 09:09 -0700, Linus Torvalds wrote: > > On Sat, 31 Mar 2007, Thomas Gleixner wrote: > > > > While I agree in principle with the patch, I'm a bit uncomfortable. The > > sys device suspend / resume ordering is not guaranteed and relies on the > > registering order. > > Well, t

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-31 Thread Linus Torvalds
On Sat, 31 Mar 2007, Thomas Gleixner wrote: > > While I agree in principle with the patch, I'm a bit uncomfortable. The > sys device suspend / resume ordering is not guaranteed and relies on the > registering order. Well, this is why we probably should try to get away from the "system device"

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-31 Thread Thomas Gleixner
On Sun, 2007-04-01 at 00:01 +0800, Jeff Chua wrote: > On 3/31/07, Thomas Gleixner <[EMAIL PROTECTED]> wrote: > > > Jeff still seems to have problems with CONFIG_NO_HZ=n and it might be > > caused by time keeping / tick management resume happening before the > > HPET resume. > > > me>Confirmed th

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-31 Thread Jeff Chua
On 3/31/07, Thomas Gleixner <[EMAIL PROTECTED]> wrote: Jeff still seems to have problems with CONFIG_NO_HZ=n and it might be caused by time keeping / tick management resume happening before the HPET resume. me>Confirmed that suspend/resume disk/ram works on X60s with me>CONFIG_HPET_TIMER=y an

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-31 Thread Thomas Gleixner
On Thu, 2007-03-29 at 15:46 +0200, Maxim Levitsky wrote: > Subject: Add suspend/resume for HPET > This adds support of suspend/resume on i386 for HPET > Signed-off-by: Maxim Levitsky <[EMAIL PROTECTED]> > > +static struct sysdev_class hpet_class = { > + set_kset_name("hpet"), > + .suspend

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-29 Thread Andi Kleen
Ingo Molnar <[EMAIL PROTECTED]> writes: > > there's no fundamental reason. x86_64 COW-ed hpet_timer.c and > time_hpet.c years ago and drifted off into different areas. Not quite -- x86-64 did HPET long before i386; the only stuff cowed was the character driver support code. But the core HPET cod

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-29 Thread Jeff Chua
On 3/29/07, Maxim Levitsky <[EMAIL PROTECTED]> wrote: Subject: Add suspend/resume for HPET This adds support of suspend/resume on i386 for HPET Signed-off-by: Maxim Levitsky <[EMAIL PROTECTED]> Confirmed that suspend/resume disk/ram works on X60s with CONFIG_HPET_TIMER=y and CONFIG_NO_HZ unset.

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-29 Thread Ingo Molnar
* Linus Torvalds <[EMAIL PROTECTED]> wrote: > Btw, what about arch/x86_64/kernel/hpet.c? at least wrt. suspend/resume it should be fine, because in arch/x86_64/kernel/time.c it does this upon resume: static int timer_resume(struct sys_device *dev) { if (hpet_address)

Re: [patch, v2] add suspend/resume for HPET

2007-03-29 Thread Ingo Molnar
update: i've tested Maxim's v2 patch on both a hpet-capable and a hpet-less system, and it works fine here. on a dual-core hpet-capable system, running a NO_HZ+!HIGH_RES_TIMERS kernel: europe:~> grep Clock /proc/timer_list Clock Event Device: hpet Clock Event Device: lapic Clock Event

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-29 Thread Maxim Levitsky
On Thursday 29 March 2007 18:53:37 Linus Torvalds wrote: > > On Thu, 29 Mar 2007, Maxim Levitsky wrote: > > > > Subject: Add suspend/resume for HPET > > > > This adds support of suspend/resume on i386 for HPET > > > > Signed-off-by: Maxim Levitsky <[EMAIL PROTECTED]> > > > > --- > > arch/i386/ke

Re: [PATCH v2] Add suspend/resume for HPET

2007-03-29 Thread Linus Torvalds
On Thu, 29 Mar 2007, Maxim Levitsky wrote: > > Subject: Add suspend/resume for HPET > > This adds support of suspend/resume on i386 for HPET > > Signed-off-by: Maxim Levitsky <[EMAIL PROTECTED]> > > --- > arch/i386/kernel/hpet.c | 68 > +++ Btw, wh