Home » Mailing list archives » linux-kernel » 2007 » March » 12

Re: [patch] change futex_wait() to hrtimers

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Ingo Molnar <mingo@...>
To: Nick Piggin <npiggin@...>
Cc: Roland McGrath <roland@...>, <akpm@...>, <mm-commits@...>, <drepper@...>, <oleg@...>, <sebastien.dugue@...>, <linux-kernel@...>, Thomas Gleixner <tglx@...>
Subject: Re: [patch] change futex_wait() to hrtimers
Date: Monday, March 12, 2007 - 7:19 am

* Nick Piggin <npiggin@suse.de> wrote:


the issue is this: your fix reduces the effects of the bug but it is 
still fundamentally incomplete because of the use of timer_list. So 
instead of trying to fix the bug the wrong way, please try to fix it the 
right way, ontop of an already existing and tested patch, ok? That also 
enables the other neat stuff Thomas talked about.


hm. I'm wondering how this wasnt noticed sooner - this futex_wait 
behavior has been there for like forever.

	Ingo
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[patch] change futex_wait() to hrtimers, Ingo Molnar, (Mon Mar 12, 5:10 am)
Re: [patch] change futex_wait() to hrtimers, Andi Kleen, (Mon Mar 12, 7:27 am)
Re: [patch] change futex_wait() to hrtimers, Thomas Gleixner, (Mon Mar 12, 7:00 am)
Re: [patch] change futex_wait() to hrtimers, Andi Kleen, (Mon Mar 12, 6:58 am)
Re: [patch] change futex_wait() to hrtimers, Theodore Tso, (Mon Mar 12, 10:12 am)
Re: [patch] change futex_wait() to hrtimers, Nick Piggin, (Mon Mar 12, 10:32 am)
Re: [patch] change futex_wait() to hrtimers, Ingo Molnar, (Mon Mar 12, 10:31 am)
Re: [patch] change futex_wait() to hrtimers, Andi Kleen, (Mon Mar 12, 10:22 am)
Re: [patch] change futex_wait() to hrtimers, Ingo Molnar, (Mon Mar 12, 7:04 am)
Re: [patch] change futex_wait() to hrtimers, Andi Kleen, (Mon Mar 12, 7:20 am)
Re: [patch] change futex_wait() to hrtimers, Ingo Molnar, (Mon Mar 12, 7:28 am)
Re: [patch] change futex_wait() to hrtimers, Nick Piggin, (Mon Mar 12, 5:16 am)
Re: [patch] change futex_wait() to hrtimers, Ingo Molnar, (Mon Mar 12, 7:02 am)
Re: [patch] change futex_wait() to hrtimers, Nick Piggin, (Mon Mar 12, 7:13 am)
Re: [patch] change futex_wait() to hrtimers, Ingo Molnar, (Mon Mar 12, 7:19 am)
Re: [patch] change futex_wait() to hrtimers, Nick Piggin, (Mon Mar 12, 7:29 am)
Re: [patch] change futex_wait() to hrtimers, Ingo Molnar, (Mon Mar 12, 7:38 am)
Re: [patch] change futex_wait() to hrtimers, Nick Piggin, (Mon Mar 12, 7:52 am)
Re: [patch] change futex_wait() to hrtimers, Ingo Molnar, (Mon Mar 12, 8:21 am)
Re: [patch] change futex_wait() to hrtimers, Nick Piggin, (Mon Mar 12, 8:36 am)
Re: [patch] change futex_wait() to hrtimers, Thomas Gleixner, (Mon Mar 12, 7:19 am)