linux-kernel - Re: [patch] hrtimers: add state tracking, fix

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4588940B.2020604@imap.cc>
Date:	Wed, 20 Dec 2006 02:38:19 +0100
From:	Tilman Schmidt <tilman@...p.cc>
To:	Ingo Molnar <mingo@...e.hu>
CC:	Andrew Morton <akpm@...l.org>, linux-kernel@...r.kernel.org,
	Thomas Gleixner <tglx@...utronix.de>
Subject: Re: [patch] hrtimers: add state tracking, fix

Am 19.12.2006 20:56 schrieb Ingo Molnar:
> thanks for the report - this made me review the hrtimer state engine 
> logic, and bingo, it indeed has a nasty typo! Could you try the fix 
> below, does it fix your problem? It might explain the crash you are 
> seeing, because the typo means we'd ignore HRTIMER_STATE_PENDING state 
> (which is rare but possible).

Ok, the machine has been running for a couple of hours with that patch
and so far hasn't frozen again. I'll watch it some more but it looks
like your patch did indeed fix my problem.

Thanks
Tilman

> -------------------------->
> Subject: [patch] hrtimers: add state tracking, fix
> From: Ingo Molnar <mingo@...e.hu>
> 
> fix bug in hrtimer_is_queued(), introduced by a cleanup during
> the recent refactoring.
> 
> Signed-off-by: Ingo Molnar <mingo@...e.hu>
> ---
>  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);
>  }
>  
>  /*

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


Download attachment "signature.asc" of type "application/pgp-signature" (254 bytes)