lkml.org 
[lkml]   [2005]   [Aug]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[PATCH 1/3] Updated dynamic tick patches - Fix lost tick calculation in timer_pm.c
    On Wed, Aug 31, 2005 at 10:28:43PM +0530, Srivatsa Vaddagiri wrote:
    > Following patches related to dynamic tick are posted in separate mails,
    > for convenience of review. The first patch probably applies w/o dynamic
    > tick consideration also.
    >
    > Patch 1/3 -> Fixup lost tick calculation in timer_pm.c

    Currently, lost tick calculation in timer_pm.c is based on number
    of microseconds that has elapsed since the last tick. Calculating
    the number of microseconds is approximated by cyc2us, which
    basically does :

    microsec = (cycles * 286) / 1024

    Consider 10 ticks lost. This amounts to 14319*10 = 143190 cycles
    (14319 = PMTMR_EXPECTED_RATE/(CALIBRATE_LATCH/LATCH)).
    This amount to 39992 microseconds as per the above equation
    or 39992 / 4000 = 9 lost ticks, which is incorrect.

    I feel lost ticks can be based on cycles difference directly
    rather than being based on microseconds that has elapsed.

    Following patch is in that direction.

    With this patch, time had kept up really well on one particular
    machine (Intel 4way Pentium 3 box) overnight, while
    on another newer machine (Intel 4way Xeon with HT) it didnt do so
    well (time sped up after 3 or 4 hours). Hence I consider this
    particular patch will need more review/work.

    Patch is against 2.6.13-rc6-mm2.



    Fix lost tick calculation in timer_pm.c

    ---

    linux-2.6.13-rc6-mm2-root/arch/i386/kernel/timers/timer_pm.c | 44 +++++------
    1 files changed, 20 insertions(+), 24 deletions(-)

    diff -puN arch/i386/kernel/timers/timer_pm.c~pm_timer_fix arch/i386/kernel/timers/timer_pm.c
    --- linux-2.6.13-rc6-mm2/arch/i386/kernel/timers/timer_pm.c~pm_timer_fix 2005-08-31 16:31:52.000000000 +0530
    +++ linux-2.6.13-rc6-mm2-root/arch/i386/kernel/timers/timer_pm.c 2005-08-31 16:32:51.000000000 +0530
    @@ -30,6 +30,8 @@
    ((CALIBRATE_LATCH * (PMTMR_TICKS_PER_SEC >> 10)) / (CLOCK_TICK_RATE>>10))


    +static int pm_ticks_per_jiffy = PMTMR_EXPECTED_RATE / (CALIBRATE_LATCH/LATCH);
    +
    /* The I/O port the PMTMR resides at.
    * The location is detected during setup_arch(),
    * in arch/i386/acpi/boot.c */
    @@ -37,8 +39,7 @@ u32 pmtmr_ioport = 0;


    /* value of the Power timer at last timer interrupt */
    -static u32 offset_tick;
    -static u32 offset_delay;
    +static u32 offset_last;

    static unsigned long long monotonic_base;
    static seqlock_t monotonic_lock = SEQLOCK_UNLOCKED;
    @@ -127,6 +128,11 @@ pm_good:
    if (verify_pmtmr_rate() != 0)
    return -ENODEV;

    + printk ("Using %u PM timer ticks per jiffy \n", pm_ticks_per_jiffy);
    +
    + offset_last = read_pmtmr();
    + setup_pit_timer();
    +
    init_cpu_khz();
    return 0;
    }
    @@ -150,47 +156,37 @@ static inline u32 cyc2us(u32 cycles)
    */
    static void mark_offset_pmtmr(void)
    {
    - u32 lost, delta, last_offset;
    - static int first_run = 1;
    - last_offset = offset_tick;
    + u32 lost, delta, deltaus, offset_now;

    write_seqlock(&monotonic_lock);

    - offset_tick = read_pmtmr();
    + offset_now = read_pmtmr();

    /* calculate tick interval */
    - delta = (offset_tick - last_offset) & ACPI_PM_MASK;
    + delta = (offset_now - offset_last) & ACPI_PM_MASK;

    /* convert to usecs */
    - delta = cyc2us(delta);
    + deltaus = cyc2us(delta);

    /* update the monotonic base value */
    - monotonic_base += delta * NSEC_PER_USEC;
    + monotonic_base += deltaus * NSEC_PER_USEC;
    write_sequnlock(&monotonic_lock);

    /* convert to ticks */
    - delta += offset_delay;
    - lost = delta / (USEC_PER_SEC / HZ);
    - offset_delay = delta % (USEC_PER_SEC / HZ);
    -
    + lost = delta / pm_ticks_per_jiffy;
    + offset_last += lost * pm_ticks_per_jiffy;
    + offset_last &= ACPI_PM_MASK;

    /* compensate for lost ticks */
    if (lost >= 2)
    jiffies_64 += lost - 1;
    -
    - /* don't calculate delay for first run,
    - or if we've got less then a tick */
    - if (first_run || (lost < 1)) {
    - first_run = 0;
    - offset_delay = 0;
    - }
    }

    static int pmtmr_resume(void)
    {
    write_seqlock(&monotonic_lock);
    /* Assume this is the last mark offset time */
    - offset_tick = read_pmtmr();
    + offset_last = read_pmtmr();
    write_sequnlock(&monotonic_lock);
    return 0;
    }
    @@ -205,7 +201,7 @@ static unsigned long long monotonic_cloc
    /* atomically read monotonic base & last_offset */
    do {
    seq = read_seqbegin(&monotonic_lock);
    - last_offset = offset_tick;
    + last_offset = offset_last;
    base = monotonic_base;
    } while (read_seqretry(&monotonic_lock, seq));

    @@ -239,11 +235,11 @@ static unsigned long get_offset_pmtmr(vo
    {
    u32 now, offset, delta = 0;

    - offset = offset_tick;
    + offset = offset_last;
    now = read_pmtmr();
    delta = (now - offset)&ACPI_PM_MASK;

    - return (unsigned long) offset_delay + cyc2us(delta);
    + return (unsigned long) cyc2us(delta);
    }


    _

    --


    Thanks and Regards,
    Srivatsa Vaddagiri,
    Linux Technology Center,
    IBM Software Labs,
    Bangalore, INDIA - 560017
    -
    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-08-31 19:29    [W:4.901 / U:0.064 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site