Re: Oops from local semaphore race condition

Bob Miller (rem@osdl.org)
Tue, 21 May 2002 14:16:14 -0700


On Tue, May 21, 2002 at 03:41:19PM -0400, Ron Niles wrote:
>
>
> Ron.Niles@falconstor.com said:
> >> Then I realized it can possibly go corrupt, due to a race condition
> >> which lets down() continue before up() is complete:
>
> From: David Woodhouse [mailto:dwmw2@infradead.org]
>
> >This is what completions were added for.
>
> Thanks, struct completion is the best way; it's gonna be tough to maintain
> backward compatibility though.
>
> One comment; it looks like the implementation in sched.c should more
> properly be using wq_write_lock_irqsave on the lock.
>
I sent patches to Linus to fix this back in February. Dave Jones picked
them up and they are still in his tree. I don't know when/if he is going
to forward them onto Linus for inclusion into his tree.

-- 
Bob Miller					Email: rem@osdl.org
Open Source Development Lab			Phone: 503.626.2455 Ext. 17
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/