From owner-cvs-all@FreeBSD.ORG Sat Nov 3 23:18:02 2007 Return-Path: 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 7EFAE16A46D; Sat, 3 Nov 2007 23:18:02 +0000 (UTC) (envelope-from pjd@garage.freebsd.pl) Received: from mail.garage.freebsd.pl (arm132.internetdsl.tpnet.pl [83.17.198.132]) by mx1.freebsd.org (Postfix) with ESMTP id 0363513C4B0; Sat, 3 Nov 2007 23:17:57 +0000 (UTC) (envelope-from pjd@garage.freebsd.pl) Received: by mail.garage.freebsd.pl (Postfix, from userid 65534) id 0693145EEB; Sat, 3 Nov 2007 22:05:23 +0100 (CET) Received: from localhost (154.81.datacomsa.pl [195.34.81.154]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mail.garage.freebsd.pl (Postfix) with ESMTP id 7B134456AB; Sat, 3 Nov 2007 22:05:14 +0100 (CET) Date: Sat, 3 Nov 2007 22:05:03 +0100 From: Pawel Jakub Dawidek To: Kris Kennaway Message-ID: <20071103210503.GH37519@garage.freebsd.pl> References: <200711011104.lA1B4LeH036010@repoman.freebsd.org> <472CDC0E.6000805@FreeBSD.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="aT9PWwzfKXlsBJM1" Content-Disposition: inline In-Reply-To: <472CDC0E.6000805@FreeBSD.org> User-Agent: Mutt/1.4.2.3i X-PGP-Key-URL: http://people.freebsd.org/~pjd/pjd.asc X-OS: FreeBSD 7.0-CURRENT i386 X-Spam-Checker-Version: SpamAssassin 3.0.4 (2005-06-05) on mail.garage.freebsd.pl X-Spam-Level: X-Spam-Status: No, score=-2.6 required=3.0 tests=BAYES_00 autolearn=ham version=3.0.4 Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/contrib/opensolaris/uts/common/fs/zfs zvol.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 03 Nov 2007 23:18:02 -0000 --aT9PWwzfKXlsBJM1 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Nov 03, 2007 at 09:37:34PM +0100, Kris Kennaway wrote: > Pawel Jakub Dawidek wrote: > >pjd 2007-11-01 11:04:21 UTC > > > > FreeBSD src repository > > > > Modified files: > > sys/contrib/opensolaris/uts/common/fs/zfs zvol.c=20 > > Log: > > Call zil_commit() (if ZIL is not disabled) after every non-read request > > (BIO_WRITE and BIO_FLUSH) as it is done is Solaris. The difference is > > that Solaris calls it only for sync requests, but we can't say in GEOM > > is the request is sync or async, so we do it for every request. > > =20 > > MFC after: 1 week > > =20 > > Revision Changes Path > > 1.4 +11 -11 src/sys/contrib/opensolaris/uts/common/fs/zfs/zvo= l.c > > > > >=20 > Does this add significant overhead? ZVOLs are replacement for disks and file system put on top of that should handle delaying write requests, etc. Even if there is overhead, this is the right thing to do, but I don't think it's significant. I'll do some tests a bit later, now I need to make ZFS compile again in my perforce after last integration:) --=20 Pawel Jakub Dawidek http://www.wheel.pl pjd@FreeBSD.org http://www.FreeBSD.org FreeBSD committer Am I Evil? Yes, I Am! --aT9PWwzfKXlsBJM1 Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.4 (FreeBSD) iD8DBQFHLOJ/ForvXbEpPzQRAkmhAJ0XJX0NCBN96fVhlrvCH7mOSe/hxQCg390v 6Y+tDUd0MN0biyViET/utFc= =BJg+ -----END PGP SIGNATURE----- --aT9PWwzfKXlsBJM1--