From owner-freebsd-bugs@FreeBSD.ORG  Sun Jan 21 02:10:22 2007
Return-Path: <owner-freebsd-bugs@FreeBSD.ORG>
X-Original-To: freebsd-bugs@hub.freebsd.org
Delivered-To: freebsd-bugs@hub.freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
	by hub.freebsd.org (Postfix) with ESMTP id 3C67416A401
	for <freebsd-bugs@hub.freebsd.org>;
	Sun, 21 Jan 2007 02:10:22 +0000 (UTC)
	(envelope-from gnats@FreeBSD.org)
Received: from freefall.freebsd.org (freefall.freebsd.org [69.147.83.40])
	by mx1.freebsd.org (Postfix) with ESMTP id 12A7D13C455
	for <freebsd-bugs@hub.freebsd.org>;
	Sun, 21 Jan 2007 02:10:22 +0000 (UTC)
	(envelope-from gnats@FreeBSD.org)
Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1])
	by freefall.freebsd.org (8.13.4/8.13.4) with ESMTP id l0L2ALki044607
	for <freebsd-bugs@freefall.freebsd.org>; Sun, 21 Jan 2007 02:10:21 GMT
	(envelope-from gnats@freefall.freebsd.org)
Received: (from gnats@localhost)
	by freefall.freebsd.org (8.13.4/8.13.4/Submit) id l0L2ALfS044604;
	Sun, 21 Jan 2007 02:10:21 GMT (envelope-from gnats)
Date: Sun, 21 Jan 2007 02:10:21 GMT
Message-Id: <200701210210.l0L2ALfS044604@freefall.freebsd.org>
To: freebsd-bugs@FreeBSD.org
From: Tor Egge <Tor.Egge@cvsup.no.freebsd.org>
Cc: 
Subject: Re: kern/108151: panic: relpbuf with vp
X-BeenThere: freebsd-bugs@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
Reply-To: Tor Egge <Tor.Egge@cvsup.no.freebsd.org>
List-Id: Bug reports <freebsd-bugs.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-bugs>,
	<mailto:freebsd-bugs-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-bugs>
List-Post: <mailto:freebsd-bugs@freebsd.org>
List-Help: <mailto:freebsd-bugs-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-bugs>,
	<mailto:freebsd-bugs-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 21 Jan 2007 02:10:22 -0000

The following reply was made to PR kern/108151; it has been noted by GNATS.

From: Tor Egge <Tor.Egge@cvsup.no.freebsd.org>
To: netchild@freebsd.org
Cc: FreeBSD-gnats-submit@freebsd.org
Subject: Re: kern/108151: panic: relpbuf with vp
Date: Sun, 21 Jan 2007 01:23:52 +0000 (UTC)

 ----Next_Part(Sun_Jan_21_01:23:52_2007_708)--
 Content-Type: Text/Plain; charset=us-ascii
 Content-Transfer-Encoding: 7bit
 
 
 ffs_rawread_main() should call pbgetvp()/pbrelvp() instead of just setting
 b_vp directly.  A suggested fix is enclosed.
 
 - Tor Egge
 
 ----Next_Part(Sun_Jan_21_01:23:52_2007_708)--
 Content-Type: Text/Plain; charset=us-ascii
 Content-Transfer-Encoding: 7bit
 Content-Disposition: inline; filename="rawread.diff"
 
 Index: sys/ufs/ffs/ffs_rawread.c
 ===================================================================
 RCS file: /home/ncvs/src/sys/ufs/ffs/ffs_rawread.c,v
 retrieving revision 1.28
 diff -u -r1.28 ffs_rawread.c
 --- sys/ufs/ffs/ffs_rawread.c	5 May 2006 21:27:31 -0000	1.28
 +++ sys/ufs/ffs/ffs_rawread.c	21 Jan 2007 00:05:47 -0000
 @@ -310,7 +310,7 @@
  			/* XXX: Leave some bufs for swap */
  			bp = getpbuf(&ffsrawbufcnt);
  			sa = bp->b_data;
 -			bp->b_vp = vp; 
 +			pbgetvp(vp, bp);
  			error = ffs_rawread_readahead(vp, udata, offset,
  						     resid, td, bp, sa);
  			if (error != 0)
 @@ -324,7 +324,7 @@
  					nbp = NULL;
  				if (nbp != NULL) {
  					nsa = nbp->b_data;
 -					nbp->b_vp = vp;
 +					pbgetvp(vp, nbp);
  					
  					nerror = ffs_rawread_readahead(vp, 
  								       udata +
 @@ -337,6 +337,7 @@
  								       nbp,
  								       nsa);
  					if (nerror) {
 +						pbrelvp(nbp);
  						relpbuf(nbp, &ffsrawbufcnt);
  						nbp = NULL;
  					}
 @@ -385,6 +386,7 @@
  			nsa = tsa;
  			
  			if (resid <= bp->b_bufsize) { /* No more readaheads */
 +				pbrelvp(nbp);
  				relpbuf(nbp, &ffsrawbufcnt);
  				nbp = NULL;
  			} else { /* Setup next readahead */
 @@ -399,6 +401,7 @@
  							       nbp,
  							       nsa);
  				if (nerror != 0) {
 +					pbrelvp(nbp);
  					relpbuf(nbp, &ffsrawbufcnt);
  					nbp = NULL;
  				}
 @@ -413,13 +416,16 @@
  		}
  	}
  	
 -	if (bp != NULL)
 +	if (bp != NULL) {
 +		pbrelvp(bp);
  		relpbuf(bp, &ffsrawbufcnt);
 +	}
  	if (nbp != NULL) {			/* Run down readahead buffer */
  		spl = splbio();
  		bwait(nbp, PRIBIO, "rawrd");
  		splx(spl);
  		vunmapbuf(nbp);
 +		pbrelvp(nbp);
  		relpbuf(nbp, &ffsrawbufcnt);
  	}
  	
 
 ----Next_Part(Sun_Jan_21_01:23:52_2007_708)----