Re: [patch] change futex_wait() to hrtimers

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Theodore Tso <tytso@...>, Andi Kleen <andi@...>, Thomas Gleixner <tglx@...>, Ingo Molnar <mingo@...>, Roland McGrath <roland@...>, <akpm@...>, <mm-commits@...>, <drepper@...>, <oleg@...>, <sebastien.dugue@...>, <linux-kernel@...>
Cc: Michael Kerrisk <michael.kerrisk@...>
Date: Monday, March 12, 2007 - 10:32 am

On Mon, Mar 12, 2007 at 10:12:14AM -0400, Theodore Tso wrote:

Would this work, or will it just create more confusion for the API user?
I mean, all sleeps can only guarantee "no less than".

Would it be enough for a binary (exact as possible / relaxed if needed)
flag? Or perhaps ternary (exact/relaxed/batched), where relaxed could
add an extra jiffy or so, and batched is really relaxed that may delay
up to double the value of the timeout.


But given that we know most userspace API timeouts are broadly just an
"equal to or greater", then we could add another timeout flag to specify
it is a userspace timeout, and make that controllable by sysctl.

Sure it isn't ideal, but for those who really want power / hypervisor
savings, it could be useful.

BTW. my futex man page says timeout's contents "describe the maximum duration
of the wait". Surely that should be *minimum*? Michael cc'ed.
-
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)