Re: [patch] hrtimers: add state tracking, fix

[Tilman Schmidt <tilman@imap.cc>]

[-- Attachment #1: Type: text/plain, Size: 1325 bytes --]

Am 19.12.2006 20:56 schrieb Ingo Molnar:
> Could you try the fix below, does it fix your problem?

The system has been running for a whole day now without freezing,
convincing me that your patch does indeed fix my problem.

> -------------------------->
> Subject: [patch] hrtimers: add state tracking, fix
> From: Ingo Molnar <mingo@elte.hu>
> 
> fix bug in hrtimer_is_queued(), introduced by a cleanup during
> the recent refactoring.
> 
> Signed-off-by: Ingo Molnar <mingo@elte.hu>

Acked-by: Tilman Schmidt <tilman@imap.cc>

> ---
>  kernel/hrtimer.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Index: linux/kernel/hrtimer.c
> ===================================================================
> --- linux.orig/kernel/hrtimer.c
> +++ linux/kernel/hrtimer.c
> @@ -157,7 +157,7 @@ static void hrtimer_get_softirq_time(str
>  static inline int hrtimer_is_queued(struct hrtimer *timer)
>  {
>  	return timer->state &
> -		(HRTIMER_STATE_ENQUEUED || HRTIMER_STATE_PENDING);
> +		(HRTIMER_STATE_ENQUEUED | HRTIMER_STATE_PENDING);
>  }
>  
>  /*

Thanks again
Tilman

-- 
Tilman Schmidt                          E-Mail: tilman@imap.cc
Bonn, Germany
Diese Nachricht besteht zu 100% aus wiederverwerteten Bits.
Ungeöffnet mindestens haltbar bis: (siehe Rückseite)


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 253 bytes --]