From owner-cvs-all@FreeBSD.ORG  Fri May 23 11:14:04 2008
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 117661065679;
	Fri, 23 May 2008 11:14:04 +0000 (UTC) (envelope-from kib@FreeBSD.org)
Received: from repoman.freebsd.org (repoman.freebsd.org
	[IPv6:2001:4f8:fff6::29])
	by mx1.freebsd.org (Postfix) with ESMTP id 0410E8FC0C;
	Fri, 23 May 2008 11:14:04 +0000 (UTC) (envelope-from kib@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.14.1/8.14.1) with ESMTP id m4NBE3gL070078;
	Fri, 23 May 2008 11:14:03 GMT (envelope-from kib@repoman.freebsd.org)
Received: (from kib@localhost)
	by repoman.freebsd.org (8.14.1/8.14.1/Submit) id m4NBE32n070077;
	Fri, 23 May 2008 11:14:03 GMT (envelope-from kib)
Message-Id: <200805231114.m4NBE32n070077@repoman.freebsd.org>
From: Konstantin Belousov <kib@FreeBSD.org>
Date: Fri, 23 May 2008 11:14:03 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Cc: 
Subject: cvs commit: src/sys/kern sys_pipe.c src/sys/sys pipe.h
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 23 May 2008 11:14:04 -0000

kib         2008-05-23 11:14:03 UTC

  FreeBSD src repository

  Modified files:
    sys/kern             sys_pipe.c 
    sys/sys              pipe.h 
  Log:
  Another problem caused by the knlist_cleardel() potentially dropping
  PIPE_MTX().
  
  Since the pipe_present is cleared before (potentially) sleeping, the
  second thread may enter the pipeclose() for the reciprocal pipe end.
  The test at the end of the pipeclose() for the pipe_present == 0 would
  succeed, allowing the second thread to free the pipe memory. First
  threads then accesses the freed memory after being woken up.
  
  Properly track the closing state of the pipe in the pipe_present.
  Introduce the intermediate state that marks the pipe as mostly
  dismantled but might be sleeping waiting for the knote list to be
  cleared. Free the pipe pair memory only when both ends pass that point.
  
  Debugging help and tested by:   pho
  Discussed with: jmg
  MFC after:      2 weeks
  
  Revision  Changes    Path
  1.198     +22 -11    src/sys/kern/sys_pipe.c
  1.30      +7 -0      src/sys/sys/pipe.h