linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [2.6.10][Suspend] - Time problems
@ 2005-01-11  7:35 Shawn Starr
  2005-01-12 22:24 ` Pavel Machek
  0 siblings, 1 reply; 8+ messages in thread
From: Shawn Starr @ 2005-01-11  7:35 UTC (permalink / raw)
  To: linux-kernel


When resuming from suspend, I noticed the clock is waay off (its 10:16pm, it 
shows 2:34AM EST time). This is even after a reboot the bios now shows wrong 
time?

Anyone else notice this recently?

Shawn.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [2.6.10][Suspend] - Time problems
  2005-01-11  7:35 [2.6.10][Suspend] - Time problems Shawn Starr
@ 2005-01-12 22:24 ` Pavel Machek
  2005-01-12 22:44   ` Rafael J. Wysocki
  0 siblings, 1 reply; 8+ messages in thread
From: Pavel Machek @ 2005-01-12 22:24 UTC (permalink / raw)
  To: Shawn Starr; +Cc: linux-kernel

Hi!

> When resuming from suspend, I noticed the clock is waay off (its 10:16pm, it 
> shows 2:34AM EST time). This is even after a reboot the bios now shows wrong 
> time?

Yes, see for example thread "2.6.10-mm2: swsusp regression
[update]". Nigel has some patch that should fix it...

									Pavel

-- 
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [2.6.10][Suspend] - Time problems
  2005-01-12 22:24 ` Pavel Machek
@ 2005-01-12 22:44   ` Rafael J. Wysocki
  2005-01-12 22:50     ` Pavel Machek
  0 siblings, 1 reply; 8+ messages in thread
From: Rafael J. Wysocki @ 2005-01-12 22:44 UTC (permalink / raw)
  To: Pavel Machek; +Cc: Shawn Starr, linux-kernel

On Wednesday, 12 of January 2005 23:24, Pavel Machek wrote:
> Hi!
> 
> > When resuming from suspend, I noticed the clock is waay off (its 10:16pm, 
it 
> > shows 2:34AM EST time). This is even after a reboot the bios now shows 
wrong 
> > time?
> 
> Yes, see for example thread "2.6.10-mm2: swsusp regression
> [update]". Nigel has some patch that should fix it...

Do you mean patches in the "[RFC] Patches to reduce delay in 
arch/kernel/time.c" thread?

RJW

-- 
- Would you tell me, please, which way I ought to go from here?
- That depends a good deal on where you want to get to.
		-- Lewis Carroll "Alice's Adventures in Wonderland"

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [2.6.10][Suspend] - Time problems
  2005-01-12 22:44   ` Rafael J. Wysocki
@ 2005-01-12 22:50     ` Pavel Machek
  2005-01-13  0:31       ` Shawn Starr
  0 siblings, 1 reply; 8+ messages in thread
From: Pavel Machek @ 2005-01-12 22:50 UTC (permalink / raw)
  To: Rafael J. Wysocki; +Cc: Shawn Starr, linux-kernel

Hi!

> > > When resuming from suspend, I noticed the clock is waay off (its 10:16pm, 
> it 
> > > shows 2:34AM EST time). This is even after a reboot the bios now shows 
> wrong 
> > > time?
> > 
> > Yes, see for example thread "2.6.10-mm2: swsusp regression
> > [update]". Nigel has some patch that should fix it...
> 
> Do you mean patches in the "[RFC] Patches to reduce delay in 
> arch/kernel/time.c" thread?

I meant this one... (cut&pasted, apply by hand). But it seems to be
included in 2.6.11-rc1. I'm now confused.
								Pavel

diff -ruNp 910-original-time-patch-old/arch/i386/kernel/time.c
910-original-time-patch-new/arch/i386/kernel/time.c
--- 910-original-time-patch-old/arch/i386/kernel/time.c 2004-12-27
+++ 910-original-time-patch-new/arch/i386/kernel/time.c 2005-01-08
@@ -343,12 +343,13 @@ static int timer_resume(struct sys_devic
                hpet_reenable();
 #endif
        sec = get_cmos_time() + clock_cmos_diff;
-       sleep_length = get_cmos_time() - sleep_start;
+       sleep_length = (get_cmos_time() - sleep_start) * HZ;
        write_seqlock_irqsave(&xtime_lock, flags);
        xtime.tv_sec = sec;
        xtime.tv_nsec = 0;
        write_sequnlock_irqrestore(&xtime_lock, flags);
-       jiffies += sleep_length * HZ;
+       jiffies += sleep_length;
+       wall_jiffies += sleep_length;
        return 0;
 }


-- 
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [2.6.10][Suspend] - Time problems
  2005-01-12 22:50     ` Pavel Machek
@ 2005-01-13  0:31       ` Shawn Starr
  2005-03-28  7:51         ` [2.6.12-rc1] suspend to Disk success - T42 laptop Shawn Starr
  0 siblings, 1 reply; 8+ messages in thread
From: Shawn Starr @ 2005-01-13  0:31 UTC (permalink / raw)
  To: Pavel Machek; +Cc: Rafael J. Wysocki, linux-kernel

I'll just bump to 2.6.11-rc1, Rafael, not that one

In either case, it won't be a problem in a few moments. :)

Shawn.

On January 12, 2005 17:50, Pavel Machek wrote:
> Hi!
>
> > > > When resuming from suspend, I noticed the clock is waay off (its
> > > > 10:16pm,
> >
> > it
> >
> > > > shows 2:34AM EST time). This is even after a reboot the bios now
> > > > shows
> >
> > wrong
> >
> > > > time?
> > >
> > > Yes, see for example thread "2.6.10-mm2: swsusp regression
> > > [update]". Nigel has some patch that should fix it...
> >
> > Do you mean patches in the "[RFC] Patches to reduce delay in
> > arch/kernel/time.c" thread?
>
> I meant this one... (cut&pasted, apply by hand). But it seems to be
> included in 2.6.11-rc1. I'm now confused.
>         Pavel
>
> diff -ruNp 910-original-time-patch-old/arch/i386/kernel/time.c
> 910-original-time-patch-new/arch/i386/kernel/time.c
> --- 910-original-time-patch-old/arch/i386/kernel/time.c 2004-12-27
> +++ 910-original-time-patch-new/arch/i386/kernel/time.c 2005-01-08
> @@ -343,12 +343,13 @@ static int timer_resume(struct sys_devic
>                 hpet_reenable();
>  #endif
>         sec = get_cmos_time() + clock_cmos_diff;
> -       sleep_length = get_cmos_time() - sleep_start;
> +       sleep_length = (get_cmos_time() - sleep_start) * HZ;
>         write_seqlock_irqsave(&xtime_lock, flags);
>         xtime.tv_sec = sec;
>         xtime.tv_nsec = 0;
>         write_sequnlock_irqrestore(&xtime_lock, flags);
> -       jiffies += sleep_length * HZ;
> +       jiffies += sleep_length;
> +       wall_jiffies += sleep_length;
>         return 0;
>  }

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [2.6.12-rc1] suspend to Disk success - T42 laptop
  2005-01-13  0:31       ` Shawn Starr
@ 2005-03-28  7:51         ` Shawn Starr
  2005-03-29 11:38           ` Pavel Machek
  0 siblings, 1 reply; 8+ messages in thread
From: Shawn Starr @ 2005-03-28  7:51 UTC (permalink / raw)
  To: Pavel Machek; +Cc: LKML


Hello Pavel, I can now suspend to disk on the laptop with 2.6.12-rc1. There is 
no failures anymore. It resumes perfectly.

Thank you.

Shawn.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [2.6.12-rc1] suspend to Disk success - T42 laptop
  2005-03-28  7:51         ` [2.6.12-rc1] suspend to Disk success - T42 laptop Shawn Starr
@ 2005-03-29 11:38           ` Pavel Machek
  2005-03-31 16:59             ` Stefan Seyfried
  0 siblings, 1 reply; 8+ messages in thread
From: Pavel Machek @ 2005-03-29 11:38 UTC (permalink / raw)
  To: Shawn Starr; +Cc: LKML

Hi!

> Hello Pavel, I can now suspend to disk on the laptop with 2.6.12-rc1. There is 
> no failures anymore. It resumes perfectly.

No video hacks needed? Good.
									Pavel
-- 
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [2.6.12-rc1] suspend to Disk success - T42 laptop
  2005-03-29 11:38           ` Pavel Machek
@ 2005-03-31 16:59             ` Stefan Seyfried
  0 siblings, 0 replies; 8+ messages in thread
From: Stefan Seyfried @ 2005-03-31 16:59 UTC (permalink / raw)
  To: Pavel Machek; +Cc: LKML

Pavel Machek wrote:
> Hi!
> 
>> Hello Pavel, I can now suspend to disk on the laptop with 2.6.12-rc1. There is 
>> no failures anymore. It resumes perfectly.
> 
> No video hacks needed? Good.

suspend to disk. !(suspend to RAM).

:-)

Stefan

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2005-03-31 17:00 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-01-11  7:35 [2.6.10][Suspend] - Time problems Shawn Starr
2005-01-12 22:24 ` Pavel Machek
2005-01-12 22:44   ` Rafael J. Wysocki
2005-01-12 22:50     ` Pavel Machek
2005-01-13  0:31       ` Shawn Starr
2005-03-28  7:51         ` [2.6.12-rc1] suspend to Disk success - T42 laptop Shawn Starr
2005-03-29 11:38           ` Pavel Machek
2005-03-31 16:59             ` Stefan Seyfried

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).