lkml.org 
[lkml]   [2004]   [Nov]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH]time run too fast after S3
    From
    Date
    Hi.

    On Tue, 2004-11-23 at 05:33, john stultz wrote:
    > I'm not all that familiar w/ the suspend code, but yea, this looks like
    > an improvement. The previous code was wrong because they are setting
    > xtime themselves, and then updating only jiffies. At the next timer
    > interrupt, the difference between jiffies and wall_jiffies would then be
    > added to xtime again.

    That makes a lot of sense to me :>

    It also happens with suspend to disk now that Pavel and I have added
    sysdev support to our implementations.

    I was doing some looking in this area last week, but ran out of time. I
    was seeing the clock being out - sometimes - by 1 hour+.

    That section of code could also be improved by reusing the value of the
    first call to get_cmos_time(). The way that function works, two
    consecutive calls will cause a one second delay for the second call. A
    __get_cmos_time function that doesn't wait for the start of a second was
    suggested for where we only care about consistency and not about getting
    the start of the second. I'll send a patch shortly.

    > Why they don't just use do_settimeofday() for all of this is a mystery
    > to me. Are we wanting to pretend timer ticks arrived while we were
    > suspended?

    I think that was Pavel's idea; something about getting process times
    right. Speaking for myself, I might be being short sighted, but I just
    want to save the user having to run /sbin/hwclock --hctosys afterwards.

    Regards,

    Nigel
    --
    Nigel Cunningham
    Pastoral Worker
    Christian Reformed Church of Tuggeranong
    PO Box 1004, Tuggeranong, ACT 2901

    You see, at just the right time, when we were still powerless, Christ
    died for the ungodly. -- Romans 5:6

    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2005-03-22 14:08    [W:2.976 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site