On 03/27/2012 01:11 AM, Peter Zijlstra wrote:
> On Mon, 2012-03-26 at 19:33 -0700, Boaz Harrosh wrote:
>
>
> So I'm fine with adding wait_for_completion_timeout_state(), but make it
> look and smell like wait_for_completion_timeout() and use a proper
> state, like wake_up_state().
>
> IOW:
>
> unsigned long __sched
> wait_for_completion_timeout_state(struct completion *x,
> unsigned long timeout,
> unsigned int state)
> {
> return wait_for_common(x, timeout, state);
> }
> EXPORT_SYMBOL(wait_for_completion_timeout_state);
>
Looks good thanks It's what I had at first, and went
overboard with it. KISS right?
I'll resend and use it as is? Should I add a BUG_ON
if state is not one of:
TASK_UNINTERRUPTIBLE
TASK_KILLABLE
TASK_INTERRUPTIBLE
Thanks Peter, this will help me a lot
Boaz
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
