From owner-cvs-all Thu Aug 13 01:31:52 1998 Return-Path: Received: (from daemon@localhost) by hub.freebsd.org (8.8.8/8.8.8) id BAA26450 for cvs-all-outgoing; Thu, 13 Aug 1998 01:31:52 -0700 (PDT) (envelope-from owner-cvs-all) Received: from godzilla.zeta.org.au (godzilla.zeta.org.au [203.15.68.22]) by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id BAA26444; Thu, 13 Aug 1998 01:31:49 -0700 (PDT) (envelope-from bde@godzilla.zeta.org.au) Received: (from bde@localhost) by godzilla.zeta.org.au (8.8.7/8.8.7) id SAA09075; Thu, 13 Aug 1998 18:31:22 +1000 Date: Thu, 13 Aug 1998 18:31:22 +1000 From: Bruce Evans Message-Id: <199808130831.SAA09075@godzilla.zeta.org.au> To: cvs-all@FreeBSD.ORG, cvs-committers@FreeBSD.ORG, dfr@FreeBSD.ORG Subject: Re: cvs commit: src/sys/kern subr_diskslice.c vfs_bio.c vfs_cluster.c src/sys/nfs nfs_vnops.c Sender: owner-cvs-all@FreeBSD.ORG X-Loop: FreeBSD.org Precedence: bulk > Modified files: > sys/kern subr_diskslice.c vfs_bio.c vfs_cluster.c > sys/nfs nfs_vnops.c > Log: > Protect all modifications to v_numoutput with splbio(). > > Revision Changes Path > 1.57 +7 -3 src/sys/kern/subr_diskslice.c > 1.169 +4 -2 src/sys/kern/vfs_bio.c > 1.68 +3 -1 src/sys/kern/vfs_cluster.c > 1.106 +2 -2 src/sys/nfs/nfs_vnops.c This can't be the right fix. vwakeup() also modifies v_flag, so zillions of of modifications of v_flag need similar changes. Perhaps keep the VBWAIT flag and a pending v_numoutput decrement in separate variables to reduce contention. Bruce