Your CVS fix 1.109 to union_vnops.c

takawata at takawata at
Sun Oct 3 07:30:46 PDT 2004

In message <415FC1A1.3020502 at>, Uwe Doering wrote:
>Hi there,
>with regard to your above mentioned fix you may be interested in reading 
>this short discussion, especially my answer to the original article:

Thank you for your comment.

The code is what nullfs do.

static int
        struct vop_getattr_args /* {
                struct vnode *a_vp;
                struct vattr *a_vap;
                struct ucred *a_cred;
                struct thread *a_td;
        } */ *ap;
        int error;

        if ((error = null_bypass((struct vop_generic_args *)ap)) != 0)
                return (error);

        ap->a_vap->va_fsid = ap->a_vp->v_mount->mnt_stat.f_fsid.val[0];
        return (0);

I'm pleased if you explain why it is done
for nullfs and not for unionfs, if any.
IMHO, there are not so much advantage in assuming 
exactly same file exists in different filesystem,
if the entity is same.

But I want to hear from FS gurus. 
I found that I reverted the change at CVS rev  1.62.

More information about the freebsd-current mailing list