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

From: Tilman Schmidt
Date: Wed Dec 20 2006 - 15:01:45 EST


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@xxxxxxx>
>
> fix bug in hrtimer_is_queued(), introduced by a cleanup during
> the recent refactoring.
>
> Signed-off-by: Ingo Molnar <mingo@xxxxxxx>

Acked-by: Tilman Schmidt <tilman@xxxxxxx>

> ---
> 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@xxxxxxx
Bonn, Germany
Diese Nachricht besteht zu 100% aus wiederverwerteten Bits.
Ungeöffnet mindestens haltbar bis: (siehe Rückseite)

Attachment: signature.asc
Description: OpenPGP digital signature