From owner-cvs-all Sun Oct 3 12:36:38 1999 Delivered-To: cvs-all@freebsd.org Received: from apollo.backplane.com (apollo.backplane.com [216.240.41.2]) by hub.freebsd.org (Postfix) with ESMTP id 0750D14DE6; Sun, 3 Oct 1999 12:36:35 -0700 (PDT) (envelope-from dillon@apollo.backplane.com) Received: (from dillon@localhost) by apollo.backplane.com (8.9.3/8.9.1) id MAA60664; Sun, 3 Oct 1999 12:36:35 -0700 (PDT) (envelope-from dillon) Date: Sun, 3 Oct 1999 12:36:35 -0700 (PDT) From: Matthew Dillon Message-Id: <199910031936.MAA60664@apollo.backplane.com> To: Poul-Henning Kamp Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/vn vn.c src/sys/gnu/ext2fs ext2_vfsops.c src/sys/isofs/cd9660 cd9660_vfsops.c src/sys/sys conf.h src/sys/ufs/ffs ffs_vfsops.c References: <199909300711.AAA25094@freefall.freebsd.org> Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk :phk 1999/09/30 00:11:30 PDT : : Modified files: : sys/dev/vn vn.c : sys/gnu/ext2fs ext2_vfsops.c : sys/isofs/cd9660 cd9660_vfsops.c : sys/sys conf.h : sys/ufs/ffs ffs_vfsops.c : Log: : Remove the D_NOCLUSTER[RW] options which were added because vn had : problems. Now that Matt has fixed vn, this can go. The vn driver : should have used d_maxio (now si_iosize_max) anyway. : : Revision Changes Path : 1.100 +1 -2 src/sys/dev/vn/vn.c : 1.55 +1 -15 src/sys/gnu/ext2fs/ext2_vfsops.c : 1.66 +1 -9 src/sys/isofs/cd9660/cd9660_vfsops.c : 1.93 +1 -4 src/sys/sys/conf.h : 1.107 +1 -16 src/sys/ufs/ffs/ffs_vfsops.c Umm... you did test this, right? While I don't see any potential problem that might occur with clustering enabled, I haven't removed the option from VN because I haven't had the time to test it. If you haven't tested it, please put the option back in until either you or I have time to test it. -Matt Matthew Dillon To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message