Page MenuHomeFreeBSD
Files F142403161

D43815.id134138.diff
No OneTemporary
Actions

D43815.id134138.diff
View Options

diff --git a/sys/fs/unionfs/union.h b/sys/fs/unionfs/union.h
--- a/sys/fs/unionfs/union.h
+++ b/sys/fs/unionfs/union.h
@@ -51,6 +51,8 @@
} unionfs_whitemode;
struct unionfs_mount {
+ struct mount *um_lowermp; /* MNT_REFed lower mount object */
+ struct mount *um_uppermp; /* MNT_REFed upper mount object */
struct vnode *um_lowervp; /* VREFed once */
struct vnode *um_uppervp; /* VREFed once */
struct vnode *um_rootvp; /* ROOT vnode */
diff --git a/sys/fs/unionfs/union_vfsops.c b/sys/fs/unionfs/union_vfsops.c
--- a/sys/fs/unionfs/union_vfsops.c
+++ b/sys/fs/unionfs/union_vfsops.c
@@ -71,7 +71,6 @@
static int
unionfs_domount(struct mount *mp)
{
- struct mount *lowermp, *uppermp;
struct vnode *lowerrootvp;
struct vnode *upperrootvp;
struct unionfs_mount *ump;
@@ -303,18 +302,18 @@
* reused until that happens. We assume the caller holds a reference
* to lowerrootvp as it is the mount's covered vnode.
*/
- lowermp = vfs_register_upper_from_vp(ump->um_lowervp, mp,
+ ump->um_lowermp = vfs_register_upper_from_vp(ump->um_lowervp, mp,
&ump->um_lower_link);
- uppermp = vfs_register_upper_from_vp(ump->um_uppervp, mp,
+ ump->um_uppermp = vfs_register_upper_from_vp(ump->um_uppervp, mp,
&ump->um_upper_link);
vrele(upperrootvp);
- if (lowermp == NULL || uppermp == NULL) {
- if (lowermp != NULL)
- vfs_unregister_upper(lowermp, &ump->um_lower_link);
- if (uppermp != NULL)
- vfs_unregister_upper(uppermp, &ump->um_upper_link);
+ if (ump->um_lowermp == NULL || ump->um_uppermp == NULL) {
+ if (ump->um_lowermp != NULL)
+ vfs_unregister_upper(ump->um_lowermp, &ump->um_lower_link);
+ if (ump->um_uppermp != NULL)
+ vfs_unregister_upper(ump->um_uppermp, &ump->um_upper_link);
vflush(mp, 1, FORCECLOSE, curthread);
free(ump, M_UNIONFSMNT);
mp->mnt_data = NULL;
@@ -346,8 +345,8 @@
}
MNT_ILOCK(mp);
- if ((lowermp->mnt_flag & MNT_LOCAL) != 0 &&
- (uppermp->mnt_flag & MNT_LOCAL) != 0)
+ if ((ump->um_lowermp->mnt_flag & MNT_LOCAL) != 0 &&
+ (ump->um_uppermp->mnt_flag & MNT_LOCAL) != 0)
mp->mnt_flag |= MNT_LOCAL;
mp->mnt_kern_flag |= MNTK_NOMSYNC | MNTK_UNIONFS;
MNT_IUNLOCK(mp);
@@ -400,8 +399,8 @@
vn_lock(mp->mnt_vnodecovered, LK_EXCLUSIVE | LK_RETRY | LK_CANRECURSE);
mp->mnt_vnodecovered->v_vflag &= ~VV_CROSSLOCK;
VOP_UNLOCK(mp->mnt_vnodecovered);
- vfs_unregister_upper(ump->um_lowervp->v_mount, &ump->um_lower_link);
- vfs_unregister_upper(ump->um_uppervp->v_mount, &ump->um_upper_link);
+ vfs_unregister_upper(ump->um_lowermp, &ump->um_lower_link);
+ vfs_unregister_upper(ump->um_uppermp, &ump->um_upper_link);
free(ump, M_UNIONFSMNT);
mp->mnt_data = NULL;
@@ -433,13 +432,11 @@
unionfs_quotactl(struct mount *mp, int cmd, uid_t uid, void *arg,
bool *mp_busy)
{
- struct mount *uppermp;
struct unionfs_mount *ump;
int error;
bool unbusy;
ump = MOUNTTOUNIONFSMOUNT(mp);
- uppermp = atomic_load_ptr(&ump->um_uppervp->v_mount);
KASSERT(*mp_busy == true, ("upper mount not busy"));
/*
* See comment in sys_quotactl() for an explanation of why the
@@ -452,14 +449,14 @@
vfs_unbusy(mp);
*mp_busy = false;
unbusy = true;
- error = vfs_busy(uppermp, 0);
+ error = vfs_busy(ump->um_uppermp, 0);
/*
* Writing is always performed to upper vnode.
*/
if (error == 0)
- error = VFS_QUOTACTL(uppermp, cmd, uid, arg, &unbusy);
+ error = VFS_QUOTACTL(ump->um_uppermp, cmd, uid, arg, &unbusy);
if (unbusy)
- vfs_unbusy(uppermp);
+ vfs_unbusy(ump->um_uppermp);
return (error);
}
@@ -479,7 +476,7 @@
mstat = malloc(sizeof(struct statfs), M_STATFS, M_WAITOK | M_ZERO);
- error = VFS_STATFS(ump->um_lowervp->v_mount, mstat);
+ error = VFS_STATFS(ump->um_lowermp, mstat);
if (error) {
free(mstat, M_STATFS);
return (error);
@@ -491,7 +488,7 @@
lbsize = mstat->f_bsize;
- error = VFS_STATFS(ump->um_uppervp->v_mount, mstat);
+ error = VFS_STATFS(ump->um_uppermp, mstat);
if (error) {
free(mstat, M_STATFS);
return (error);
@@ -558,10 +555,10 @@
unp = VTOUNIONFS(filename_vp);
if (unp->un_uppervp != NULLVP) {
- return (VFS_EXTATTRCTL(ump->um_uppervp->v_mount, cmd,
+ return (VFS_EXTATTRCTL(ump->um_uppermp, cmd,
unp->un_uppervp, namespace, attrname));
} else {
- return (VFS_EXTATTRCTL(ump->um_lowervp->v_mount, cmd,
+ return (VFS_EXTATTRCTL(ump->um_lowermp, cmd,
unp->un_lowervp, namespace, attrname));
}
}
diff --git a/sys/fs/unionfs/union_vnops.c b/sys/fs/unionfs/union_vnops.c
--- a/sys/fs/unionfs/union_vnops.c
+++ b/sys/fs/unionfs/union_vnops.c
@@ -764,7 +764,7 @@
if (lvp != NULLVP) {
if (accmode & VWRITE) {
- if (ump->um_uppervp->v_mount->mnt_flag & MNT_RDONLY) {
+ if ((ump->um_uppermp->mnt_flag & MNT_RDONLY) != 0) {
switch (ap->a_vp->v_type) {
case VREG:
case VDIR:
@@ -835,7 +835,7 @@
error = VOP_GETATTR(lvp, ap->a_vap, ap->a_cred);
- if (error == 0 && !(ump->um_uppervp->v_mount->mnt_flag & MNT_RDONLY)) {
+ if (error == 0 && (ump->um_uppermp->mnt_flag & MNT_RDONLY) == 0) {
/* correct the attr toward shadow file/dir. */
if (ap->a_vp->v_type == VREG || ap->a_vp->v_type == VDIR) {
unionfs_create_uppervattr_core(ump, ap->a_vap, &va, td);

File Metadata

Mime Type
text/plain
Expires
Tue, Jan 20, 4:13 PM (19 h, 50 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
27765273
Default Alt Text
D43815.id134138.diff (5 KB)

Event Timeline