Re: CVS commit: src/sys

[Andrew Doran <ad%netbsd.org@localhost>]

On Fri, Mar 23, 2007 at 10:38:01PM +0900, YAMAMOTO Takashi wrote:

> > Module Name:        src
> > Committed By:       ad
> > Date:               Mon Mar 12 21:31:15 UTC 2007
> > 
> > Modified Files:
> >     src/sys/kern: init_main.c sys_pipe.c
> >     src/sys/sys: pipe.h
> > 
> > Log Message:
> > Put a lock around pipe->pipe_peer.
> > 
> > 
> > To generate a diff of this commit:
> > cvs rdiff -r1.299 -r1.300 src/sys/kern/init_main.c
> > cvs rdiff -r1.81 -r1.82 src/sys/kern/sys_pipe.c
> > cvs rdiff -r1.20 -r1.21 src/sys/sys/pipe.h
> > 
> > Please note that diffs are not public domain; they are subject to the
> > copyright notices on the relevant files.
> 
> i think this can deadlock.
> 
> pipeclose -> pipeselwakeup -> selwakeup -> knote -> filt_pipewrite

Well spotted, thanks.

Andrew