cvs commit: src/sys/fs/hpfs hpfs_vfsops.c hpfs_vnops.c src/sys/fs/msdosfs msdosfs_vfsops.c msdosfs_vnops.c src/sys/fs/ntfs ntfs_vfsops.c ntfs_vnops.c src/sys/fs/nullfs null_vfsops.c null_vnops.c src/sys/fs/udf udf.h udf_vfsops.c ...

Pawel Jakub Dawidek pjd at
Fri Feb 16 09:17:56 UTC 2007

On Fri, Feb 16, 2007 at 07:33:12AM +0000, Robert Watson wrote:
> On Thu, 15 Feb 2007, Pawel Jakub Dawidek wrote:
> > Move vnode-to-file-handle translation from vfs_vptofh to vop_vptofh method.
> > This way we may support multiple structures in v_data vnode field within
> > one file system without using black magic.
> >
> > Vnode-to-file-handle should be VOP in the first place, but was made VFS
> > operation to keep interface as compatible as possible with SUN's VFS.
> > BTW. Now Solaris also implements vnode-to-file-handle as VOP operation.
> >
> > VFS_VPTOFH() was left for API backward compatibility, but is marked for
> > removal before 8.0-RELEASE.
> >
> > Approved by:    mckusick
> > Discussed with: many (on IRC)
> > Tested with:    ufs, msdosfs, cd9660, nullfs and zfs
> Do you think API backward compatibility is actually required in 7.x?  It looks like you've updated all the file systems, in which case the temptation would be to drop it as 
> we already have other VFS changes in 7.x from 6.x.

Those changes break API or only ABI? My change break ABI backward
compatibility, but I thought it will be good to leave API compatibility
so 3rd party file systems (eg. from ports) have time to catch-up.
If this is not necessary, I'll remove it right away.

Pawel Jakub Dawidek             
pjd at                 
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 :

More information about the cvs-src mailing list