Re: 2.6.17-rc3-mm1

[Thomas Gleixner]

Benoit,

On Fri, 2006-05-05 at 17:05 +0200, Benoit Boissinot wrote:
> @@ -437,6 +438,12 @@ hrtimer_start(struct hrtimer *timer, kti
>  
>  	if (mode == HRTIMER_REL) {
- 		tim = ktime_add(tim, new_base->get_time());
+		ktime_t curr = new_base->get_time();
+		
+		tim = ktime_add(tim, curr);
+

Can you change the debug that way? So we have the values which are
added. Please print out new_base->id too.

> and when urxvtd hanged I had the following in dmesg:
> [  356.696000] urxvtd: empty nanosleep 356726124322 17948911854451
> 
> So I suppose something is wrong in ktime_add()

Well, ktime_add is adding two 64 bit values.

The delta between the two values is 0xFFFFFFB3451. That looks like the
timekeeping on your box is screwed by 0x100000000000. 

John, any idea ?

	tglx


-
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/