cvs commit: src/sys/contrib/opensolaris/uts/common/fs/zfs zvol.c
Pawel Jakub Dawidek
pjd at FreeBSD.org
Sat Nov 3 16:18:02 PDT 2007
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
> > 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.
> > MFC after: 1 week
> > Revision Changes Path
> > 1.4 +11 -11 src/sys/contrib/opensolaris/uts/common/fs/zfs/zvol.c
> 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:)
Pawel Jakub Dawidek http://www.wheel.pl
pjd at FreeBSD.org http://www.FreeBSD.org
FreeBSD committer Am I Evil? Yes, I Am!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-src/attachments/20071103/7284828a/attachment.pgp
More information about the cvs-src