cvs commit: src/sys/contrib/opensolaris/uts/common/fs/zfs zvol.c

[Pawel Jakub Dawidek]

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
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-src/attachments/20071103/7284828a/attachment.pgp