From owner-svn-src-projects@FreeBSD.ORG Sat Mar 17 08:52:53 2012 Return-Path: Delivered-To: svn-src-projects@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 2ECB9106564A; Sat, 17 Mar 2012 08:52:53 +0000 (UTC) (envelope-from pawel@dawidek.net) Received: from mail.dawidek.net (60.wheelsystems.com [83.12.187.60]) by mx1.freebsd.org (Postfix) with ESMTP id 958DA8FC08; Sat, 17 Mar 2012 08:52:49 +0000 (UTC) Received: from localhost (89-73-195-149.dynamic.chello.pl [89.73.195.149]) by mail.dawidek.net (Postfix) with ESMTPSA id 36007377; Sat, 17 Mar 2012 09:52:40 +0100 (CET) Date: Sat, 17 Mar 2012 09:51:16 +0100 From: Pawel Jakub Dawidek To: Grzegorz Bernacki Message-ID: <20120317085116.GC1340@garage.freebsd.pl> References: <201203170318.q2H3ITdI047893@svn.freebsd.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="W5WqUoFLvi1M7tJE" Content-Disposition: inline In-Reply-To: <201203170318.q2H3ITdI047893@svn.freebsd.org> X-OS: FreeBSD 10.0-CURRENT amd64 User-Agent: Mutt/1.5.21 (2010-09-15) Cc: svn-src-projects@freebsd.org, src-committers@freebsd.org, kib@FreeBSD.org Subject: Re: svn commit: r233072 - projects/nand/sys/kern X-BeenThere: svn-src-projects@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the src " projects" tree" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 17 Mar 2012 08:52:53 -0000 --W5WqUoFLvi1M7tJE Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Mar 17, 2012 at 03:18:29AM +0000, Grzegorz Bernacki wrote: > Author: gber > Date: Sat Mar 17 03:18:28 2012 > New Revision: 233072 > URL: http://svn.freebsd.org/changeset/base/233072 >=20 > Log: > Add VFS changes necessary for NANDFS to work. > =20 > Ignore B_MANAGED buffer by syncer and ignore signal when msleep as it > can cause file system inconsistency. I'd suggest running these changes through kib@. Especially vn_start_write() change below looks ugly, but maybe it is only temporary? > Obtained from: Semihalf > Supported by: FreeBSD Foundation, Juniper Networks >=20 > Modified: > projects/nand/sys/kern/vfs_bio.c > projects/nand/sys/kern/vfs_subr.c > projects/nand/sys/kern/vfs_vnops.c >=20 > Modified: projects/nand/sys/kern/vfs_bio.c > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- projects/nand/sys/kern/vfs_bio.c Sat Mar 17 03:02:10 2012 (r233071) > +++ projects/nand/sys/kern/vfs_bio.c Sat Mar 17 03:18:28 2012 (r233072) > @@ -2672,7 +2672,8 @@ loop: > else if ((bp->b_flags & (B_VMIO | B_INVAL)) =3D=3D 0) > bp->b_flags |=3D B_CACHE; > BO_LOCK(bo); > - bremfree(bp); > + if (!(bp->b_flags & B_MANAGED)) > + bremfree(bp); > BO_UNLOCK(bo); > =20 > /* >=20 > Modified: projects/nand/sys/kern/vfs_subr.c > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- projects/nand/sys/kern/vfs_subr.c Sat Mar 17 03:02:10 2012 (r233071) > +++ projects/nand/sys/kern/vfs_subr.c Sat Mar 17 03:18:28 2012 (r233072) > @@ -2018,7 +2018,8 @@ reassignbuf(struct buf *bp) > * of clean buffers. > */ > if (bp->b_flags & B_DELWRI) { > - if ((bo->bo_flag & BO_ONWORKLST) =3D=3D 0) { > + if (!(bp->b_flags & B_MANAGED) && > + (bo->bo_flag & BO_ONWORKLST) =3D=3D 0) { > switch (vp->v_type) { > case VDIR: > delay =3D dirdelay; > @@ -2034,13 +2035,15 @@ reassignbuf(struct buf *bp) > buf_vlist_add(bp, bo, BX_VNDIRTY); > } else { > buf_vlist_add(bp, bo, BX_VNCLEAN); > - > - if ((bo->bo_flag & BO_ONWORKLST) && bo->bo_dirty.bv_cnt =3D=3D 0) { > - mtx_lock(&sync_mtx); > - LIST_REMOVE(bo, bo_synclist); > - syncer_worklist_len--; > - mtx_unlock(&sync_mtx); > - bo->bo_flag &=3D ~BO_ONWORKLST; > + if (!(bp->b_flags & B_MANAGED)) { > + if ((bo->bo_flag & BO_ONWORKLST) && > + bo->bo_dirty.bv_cnt =3D=3D 0) { > + mtx_lock(&sync_mtx); > + LIST_REMOVE(bo, bo_synclist); > + syncer_worklist_len--; > + mtx_unlock(&sync_mtx); > + bo->bo_flag &=3D ~BO_ONWORKLST; > + } > } > } > #ifdef INVARIANTS >=20 > Modified: projects/nand/sys/kern/vfs_vnops.c > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- projects/nand/sys/kern/vfs_vnops.c Sat Mar 17 03:02:10 2012 (r233071) > +++ projects/nand/sys/kern/vfs_vnops.c Sat Mar 17 03:18:28 2012 (r233072) > @@ -1055,6 +1055,8 @@ vn_start_write(vp, mpp, flags) > error =3D EWOULDBLOCK; > goto unlock; > } > + if (strcmp(mp->mnt_stat.f_fstypename, "nandfs") =3D=3D 0) > + flags &=3D ~PCATCH; > error =3D msleep(&mp->mnt_flag, MNT_MTX(mp), > (PUSER - 1) | (flags & PCATCH), "suspfs", 0); > if (error) --=20 Pawel Jakub Dawidek http://www.wheelsystems.com FreeBSD committer http://www.FreeBSD.org Am I Evil? Yes, I Am! http://tupytaj.pl --W5WqUoFLvi1M7tJE Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.18 (FreeBSD) iEYEARECAAYFAk9kUIQACgkQForvXbEpPzR2ywCgtRN9JkI1s81xZ0AI5BpXTR6c 0LgAoOtNObRl132B1bHxIlZ2Am/AGRGU =TEzT -----END PGP SIGNATURE----- --W5WqUoFLvi1M7tJE--