max_clock of the patch "sched_clock: fix cpu_clock()"

From: Lin Ming
Date: Mon Sep 15 2008 - 23:05:26 EST


Hi, Peter,

I have a question of the max_clock in below patch,
commit 354879bb977e06695993435745f06a0f6d39ce2b
Author: Peter Zijlstra <a.p.zijlstra@xxxxxxxxx>
Date: Mon Aug 25 17:15:34 2008 +0200

sched_clock: fix cpu_clock()

The original code to calculate min_clock and max_clock is,
min_clock = scd->tick_gtod + delta_jiffies * TICK_NSEC;
max_clock = min_clock + TICK_NSEC;

Now it's changed to,
min_clock = wrap_max(scd->tick_gtod, scd->clock);
max_clock = scd->tick_gtod + TICK_NSEC;

Is this max_clock still correct if, in a rare case, irq is disabled for
many ticks?

Thanks,
Lin Ming


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