Re: [PATCH] futex null pointer timeout

2007-01-18 Thread Ingo Molnar
* Pierre Peiffer <[EMAIL PROTECTED]> wrote: > Ingo Molnar a écrit : > >* Daniel Walker <[EMAIL PROTECTED]> wrote: > > > [...] > >>The patch reworks do_futex, and futex_wait* so a NULL pointer in the > >>timeout position is infinite, and anything else is evaluated as a real > >>timeout. > > >

Re: [PATCH] futex null pointer timeout

2007-01-18 Thread Daniel Walker
On Thu, 2007-01-18 at 13:26 +0100, Pierre Peiffer wrote: > Ingo Molnar a écrit : > > * Daniel Walker <[EMAIL PROTECTED]> wrote: > > > [...] > >> The patch reworks do_futex, and futex_wait* so a NULL pointer in the > >> timeout position is infinite, and anything else is evaluated as a real > >>

Re: [PATCH] futex null pointer timeout

2007-01-18 Thread Pierre Peiffer
Ingo Molnar a écrit : * Daniel Walker <[EMAIL PROTECTED]> wrote: [...] The patch reworks do_futex, and futex_wait* so a NULL pointer in the timeout position is infinite, and anything else is evaluated as a real timeout. thanks, applied. On top of this patch, you will need the following

Re: [PATCH] futex null pointer timeout

2007-01-18 Thread Pierre Peiffer
Ingo Molnar a écrit : * Daniel Walker [EMAIL PROTECTED] wrote: [...] The patch reworks do_futex, and futex_wait* so a NULL pointer in the timeout position is infinite, and anything else is evaluated as a real timeout. thanks, applied. On top of this patch, you will need the following

Re: [PATCH] futex null pointer timeout

2007-01-18 Thread Daniel Walker
On Thu, 2007-01-18 at 13:26 +0100, Pierre Peiffer wrote: Ingo Molnar a écrit : * Daniel Walker [EMAIL PROTECTED] wrote: [...] The patch reworks do_futex, and futex_wait* so a NULL pointer in the timeout position is infinite, and anything else is evaluated as a real timeout.

Re: [PATCH] futex null pointer timeout

2007-01-18 Thread Ingo Molnar
* Pierre Peiffer [EMAIL PROTECTED] wrote: Ingo Molnar a écrit : * Daniel Walker [EMAIL PROTECTED] wrote: [...] The patch reworks do_futex, and futex_wait* so a NULL pointer in the timeout position is infinite, and anything else is evaluated as a real timeout. thanks, applied.

Re: [PATCH] futex null pointer timeout

2007-01-17 Thread Ingo Molnar
* Daniel Walker <[EMAIL PROTECTED]> wrote: > This fix is mostly from Thomas .. > > The problem was that a futex can be called with a zero timeout (0 > seconds, 0 nanoseconds) and it's a valid expired timeout. However, the > current futex in -rt assumes a zero timeout is an infinite timeout. >

Re: [PATCH] futex null pointer timeout

2007-01-17 Thread Thomas Gleixner
On Wed, 2007-01-17 at 16:25 -0800, Daniel Walker wrote: > The patch reworks do_futex, and futex_wait* so a NULL pointer in the timeout > position is infinite, and anything else is evaluated as a real timeout. > > Signed-Off-By: Daniel Walker <[EMAIL PROTECTED]> Ack. tglx - To

[PATCH] futex null pointer timeout

2007-01-17 Thread Daniel Walker
This fix is mostly from Thomas .. The problem was that a futex can be called with a zero timeout (0 seconds, 0 nanoseconds) and it's a valid expired timeout. However, the current futex in -rt assumes a zero timeout is an infinite timeout. Kevin Hilman found this using LTP's nptl01 test case

[PATCH] futex null pointer timeout

2007-01-17 Thread Daniel Walker
This fix is mostly from Thomas .. The problem was that a futex can be called with a zero timeout (0 seconds, 0 nanoseconds) and it's a valid expired timeout. However, the current futex in -rt assumes a zero timeout is an infinite timeout. Kevin Hilman found this using LTP's nptl01 test case

Re: [PATCH] futex null pointer timeout

2007-01-17 Thread Thomas Gleixner
On Wed, 2007-01-17 at 16:25 -0800, Daniel Walker wrote: The patch reworks do_futex, and futex_wait* so a NULL pointer in the timeout position is infinite, and anything else is evaluated as a real timeout. Signed-Off-By: Daniel Walker [EMAIL PROTECTED] Ack. tglx - To unsubscribe

Re: [PATCH] futex null pointer timeout

2007-01-17 Thread Ingo Molnar
* Daniel Walker [EMAIL PROTECTED] wrote: This fix is mostly from Thomas .. The problem was that a futex can be called with a zero timeout (0 seconds, 0 nanoseconds) and it's a valid expired timeout. However, the current futex in -rt assumes a zero timeout is an infinite timeout. Kevin