cvs commit: src/sys/fs/nullfs null_vfsops.c src/sys/fs/nwfs nwfs_vfsops.c src/sys/fs/smbfs smbfs_vfsops.c src/sys/ufs/ufs quota.h ufs_quota.c ufs_vfsops.c src/sys/kern vfs_default.c vfs_vnops.c vnode_if.src src/sys/sys mount.h vnode.h

Dag-Erling Smørgrav des at des.no
Tue Feb 26 13:43:16 UTC 2008


Robert Watson <rwatson at FreeBSD.org> writes:
> I'm fine with us discussing how committed we are to VFS being a KPI we
> don't break in -STABLE branches, but I'm not fine with the claim that
> the KPI wasn't broken by this change.  If pushed on the breaking the
> KPI point, I would come down on the side that says 6.x is a pretty
> mature branch at this point, and that breaking the KPI for purely
> cosmetic reasons (i.e., caddr_t -> void *) doesn't cut it.  I would be
> more swayed by an argument involving a necessary KPI change to address
> a critical bugfix, such as a race condition that results from a poor
> existing interface.

As far as I can tell, David MFCed that patch to avoid merge conflicts
with some other patches, but you'll have to ask him for the details.

DES
-- 
Dag-Erling Smørgrav - des at des.no


More information about the cvs-all mailing list