Subject: Re: CVS commit: src/sys
To: YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
From: Andrew Doran <ad@netbsd.org>
List: source-changes
Date: 03/23/2007 18:49:28
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