Re: [Xen-devel] [PATCH 1/12] Add suspend/resume to devices owned by Xen

[Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>]

On 15/5/07 15:14, "Tian, Kevin" <kevin.tian@xxxxxxxxx> wrote:

> +    now = get_cmos_time();
> +    sleep_diff = now - sleep_start;
> +    wc_sec = now;
> +    jiffies += sleep_diff * HZ;
> +    
> +    stime_platform_stamp += SECONDS(sleep_diff);

Is this correct? I think you should either increment wc_sec by sleep_diff,
*or* increment stime by SECONDS(sleep_diff). If you do both, wall clock will
jump by twice the amount it should have done, won't it (because wc_sec is
wallclock time when stime=0)?

 -- Keir



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel