Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 11 Mar 2003 15:10:44 -0800 (PST)
From:      Nate Lawson <nate@root.org>
To:        Alexander Kabaev <kan@FreeBSD.org>
Cc:        cvs-all@FreeBSD.org, cvs-src@FreeBSD.org, src-committers@FreeBSD.org
Subject:   Re: cvs commit: src/sys/fs/devfs devfs_vfsops.c src/sys/fs/fdescfs fdesc_vfsops.c src/sys/fs/hpfs hpfs_vfsops.c src/sys/fs/ntfs       ntfs_vfsops.c src/sys/fs/nwfs nwfs_vfsops.c src/sys/fs/portalfs         portal_vfsops.c src/sys/fs/pseudofs pseudofs.h src/sys/fs/smbfs ...
Message-ID:  <Pine.BSF.4.21.0303111508440.96199-100000@root.org>
In-Reply-To: <20030311221518.6B68437B401@hub.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 11 Mar 2003, Alexander Kabaev wrote:
>   Modified files:
>     sys/fs/devfs         devfs_vfsops.c 
>     sys/fs/fdescfs       fdesc_vfsops.c 
>     sys/fs/hpfs          hpfs_vfsops.c 
>     sys/fs/ntfs          ntfs_vfsops.c 
>     sys/fs/nwfs          nwfs_vfsops.c 
>     sys/fs/portalfs      portal_vfsops.c 
>     sys/fs/pseudofs      pseudofs.h 
>     sys/fs/smbfs         smbfs_vfsops.c 
>     sys/fs/udf           udf_vfsops.c 
>     sys/fs/umapfs        umap_vfsops.c 
>     sys/fs/unionfs       union_vfsops.c 
>     sys/isofs/cd9660     cd9660_vfsops.c 
>     sys/kern             vfs_default.c 
>     sys/sys              mount.h 
>   Log:
>   Rename vfs_stdsync function to vfs_stdnosync which matches more
>   closely what function is really doing. Update all existing consumers
>   to use the new name.
>   
>   Introduce a new vfs_stdsync function, which iterates over mount
>   point's vnodes and call FSYNC on each one of them in turn.
>   
>   Make nwfs and smbfs use this new function instead of rolling their
>   own identical sync implementations.
>   
>   Reviewed by:    jeff

Very nice!  It's good to see the default ops doing more useful things
although it is important to keep them from becoming complex in the future
(i.e. if ops == UFS do one thing otherwise do something else).

-Nate


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.BSF.4.21.0303111508440.96199-100000>