PERFORCE change 133320 for review
John Birrell
jb at FreeBSD.org
Tue Jan 15 02:22:08 PST 2008
http://perforce.freebsd.org/chv.cgi?CH=133320
Change 133320 by jb at jb_freebsd1 on 2008/01/15 10:21:14
Catch up with the vn_lock/VOP_UNLOCK changes.
Affected files ...
.. //depot/projects/dtrace/src/sys/contrib/opensolaris/uts/common/fs/gfs.c#3 edit
.. //depot/projects/dtrace/src/sys/contrib/opensolaris/uts/common/fs/zfs/zfs_ctldir.c#3 edit
.. //depot/projects/dtrace/src/sys/contrib/opensolaris/uts/common/fs/zfs/zfs_replay.c#3 edit
.. //depot/projects/dtrace/src/sys/contrib/opensolaris/uts/common/fs/zfs/zfs_vfsops.c#3 edit
.. //depot/projects/dtrace/src/sys/contrib/opensolaris/uts/common/fs/zfs/zfs_vnops.c#3 edit
.. //depot/projects/dtrace/src/sys/contrib/opensolaris/uts/common/fs/zfs/zfs_znode.c#4 edit
Differences ...
==== //depot/projects/dtrace/src/sys/contrib/opensolaris/uts/common/fs/gfs.c#3 (text+ko) ====
@@ -321,7 +321,7 @@
VN_HOLD(pvp);
*vpp = pvp;
}
- vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY, curthread);
+ vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY);
return (0);
}
==== //depot/projects/dtrace/src/sys/contrib/opensolaris/uts/common/fs/zfs/zfs_ctldir.c#3 (text+ko) ====
@@ -391,7 +391,7 @@
if (strcmp(nm, "..") == 0) {
err = VFS_ROOT(dvp->v_vfsp, LK_EXCLUSIVE, vpp, curthread);
if (err == 0)
- VOP_UNLOCK(*vpp, 0, curthread);
+ VOP_UNLOCK(*vpp, 0);
} else {
err = gfs_dir_lookup(dvp, nm, vpp);
}
@@ -429,7 +429,7 @@
err = zfsctl_root_lookup(dvp, nm, vpp, NULL, 0, NULL, cr);
if (err == 0 && (nm[0] != '.' || nm[1] != '\0'))
- vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY, ap->a_cnp->cn_thread);
+ vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY);
return (err);
}
@@ -692,7 +692,7 @@
*/
goto domount;
}
- vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY, ap->a_cnp->cn_thread);
+ vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY);
mutex_exit(&sdp->sd_lock);
ZFS_EXIT(zfsvfs);
return (0);
@@ -732,7 +732,7 @@
kmem_free(mountpoint, mountpoint_len);
/* FreeBSD: This line was moved from below to avoid a lock recursion. */
if (err == 0)
- vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY, curthread);
+ vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY);
mutex_exit(&sdp->sd_lock);
/*
@@ -882,7 +882,7 @@
VERIFY(gfs_dir_lookup(vp, "..", &dvp) == 0);
sdp = dvp->v_data;
- VOP_UNLOCK(dvp, 0, ap->a_td);
+ VOP_UNLOCK(dvp, 0);
if (!(locked = MUTEX_HELD(&sdp->sd_lock)))
mutex_enter(&sdp->sd_lock);
==== //depot/projects/dtrace/src/sys/contrib/opensolaris/uts/common/fs/zfs/zfs_replay.c#3 (text+ko) ====
@@ -109,7 +109,7 @@
cn.cn_thread = curthread;
cn.cn_flags = SAVENAME;
- vn_lock(ZTOV(dzp), LK_EXCLUSIVE | LK_RETRY, curthread);
+ vn_lock(ZTOV(dzp), LK_EXCLUSIVE | LK_RETRY);
switch ((int)lr->lr_common.lrc_txtype) {
case TX_CREATE:
error = VOP_CREATE(ZTOV(dzp), &vp, &cn, &va);
@@ -127,10 +127,10 @@
default:
error = ENOTSUP;
}
- VOP_UNLOCK(ZTOV(dzp), 0, curthread);
+ VOP_UNLOCK(ZTOV(dzp), 0);
if (error == 0 && vp != NULL) {
- VOP_UNLOCK(vp, 0, curthread);
+ VOP_UNLOCK(vp, 0);
VN_RELE(vp);
}
@@ -162,10 +162,10 @@
cn.cn_lkflags = LK_EXCLUSIVE | LK_RETRY;
cn.cn_cred = kcred;
cn.cn_thread = curthread;
- vn_lock(ZTOV(dzp), LK_EXCLUSIVE | LK_RETRY, curthread);
+ vn_lock(ZTOV(dzp), LK_EXCLUSIVE | LK_RETRY);
error = VOP_LOOKUP(ZTOV(dzp), &vp, &cn);
if (error != 0) {
- VOP_UNLOCK(ZTOV(dzp), 0, curthread);
+ VOP_UNLOCK(ZTOV(dzp), 0);
goto fail;
}
@@ -180,7 +180,7 @@
error = ENOTSUP;
}
vput(vp);
- VOP_UNLOCK(ZTOV(dzp), 0, curthread);
+ VOP_UNLOCK(ZTOV(dzp), 0);
fail:
VN_RELE(ZTOV(dzp));
@@ -211,11 +211,11 @@
cn.cn_thread = curthread;
cn.cn_flags = SAVENAME;
- vn_lock(ZTOV(dzp), LK_EXCLUSIVE | LK_RETRY, curthread);
- vn_lock(ZTOV(zp), LK_EXCLUSIVE | LK_RETRY, curthread);
+ vn_lock(ZTOV(dzp), LK_EXCLUSIVE | LK_RETRY);
+ vn_lock(ZTOV(zp), LK_EXCLUSIVE | LK_RETRY);
error = VOP_LINK(ZTOV(dzp), ZTOV(zp), &cn);
- VOP_UNLOCK(ZTOV(zp), 0, curthread);
- VOP_UNLOCK(ZTOV(dzp), 0, curthread);
+ VOP_UNLOCK(ZTOV(zp), 0);
+ VOP_UNLOCK(ZTOV(dzp), 0);
VN_RELE(ZTOV(zp));
VN_RELE(ZTOV(dzp));
@@ -255,12 +255,12 @@
scn.cn_lkflags = LK_EXCLUSIVE | LK_RETRY;
scn.cn_cred = kcred;
scn.cn_thread = td;
- vn_lock(ZTOV(sdzp), LK_EXCLUSIVE | LK_RETRY, td);
+ vn_lock(ZTOV(sdzp), LK_EXCLUSIVE | LK_RETRY);
error = VOP_LOOKUP(ZTOV(sdzp), &svp, &scn);
- VOP_UNLOCK(ZTOV(sdzp), 0, td);
+ VOP_UNLOCK(ZTOV(sdzp), 0);
if (error != 0)
goto fail;
- VOP_UNLOCK(svp, 0, td);
+ VOP_UNLOCK(svp, 0);
bzero(&tcn, sizeof(tcn));
tcn.cn_nameptr = tname;
@@ -270,12 +270,12 @@
tcn.cn_lkflags = LK_EXCLUSIVE | LK_RETRY;
tcn.cn_cred = kcred;
tcn.cn_thread = td;
- vn_lock(ZTOV(tdzp), LK_EXCLUSIVE | LK_RETRY, td);
+ vn_lock(ZTOV(tdzp), LK_EXCLUSIVE | LK_RETRY);
error = VOP_LOOKUP(ZTOV(tdzp), &tvp, &tcn);
if (error == EJUSTRETURN)
tvp = NULL;
else if (error != 0) {
- VOP_UNLOCK(ZTOV(tdzp), 0, td);
+ VOP_UNLOCK(ZTOV(tdzp), 0);
goto fail;
}
@@ -360,9 +360,9 @@
ZFS_TIME_DECODE(&va.va_mtime, lr->lr_mtime);
vp = ZTOV(zp);
- vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, curthread);
+ vn_lock(vp, LK_EXCLUSIVE | LK_RETRY);
error = VOP_SETATTR(vp, &va, kcred, curthread);
- VOP_UNLOCK(vp, 0, curthread);
+ VOP_UNLOCK(vp, 0);
VN_RELE(vp);
return (error);
==== //depot/projects/dtrace/src/sys/contrib/opensolaris/uts/common/fs/zfs/zfs_vfsops.c#3 (text+ko) ====
@@ -663,7 +663,7 @@
error = zfs_zget(zfsvfs, zfsvfs->z_root, &rootzp);
if (error == 0) {
*vpp = ZTOV(rootzp);
- error = vn_lock(*vpp, flags, td);
+ error = vn_lock(*vpp, flags);
(*vpp)->v_vflag |= VV_ROOT;
}
@@ -763,7 +763,7 @@
*vpp = NULL;
else {
*vpp = ZTOV(zp);
- vn_lock(*vpp, flags, curthread);
+ vn_lock(*vpp, flags);
}
ZFS_EXIT(zfsvfs);
return (err);
@@ -830,7 +830,7 @@
}
ZFS_EXIT(zfsvfs);
/* XXX: LK_RETRY? */
- vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY, td);
+ vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY);
return (0);
}
@@ -853,7 +853,7 @@
*vpp = ZTOV(zp);
/* XXX: LK_RETRY? */
- vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY, td);
+ vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY);
vnode_create_vobject(*vpp, zp->z_phys->zp_size, td);
ZFS_EXIT(zfsvfs);
return (0);
==== //depot/projects/dtrace/src/sys/contrib/opensolaris/uts/common/fs/zfs/zfs_vnops.c#3 (text+ko) ====
@@ -1114,11 +1114,11 @@
if (cnp->cn_flags & ISDOTDOT) {
ltype = VOP_ISLOCKED(dvp, td);
- VOP_UNLOCK(dvp, 0, td);
+ VOP_UNLOCK(dvp, 0);
}
- error = vn_lock(*vpp, cnp->cn_lkflags, td);
+ error = vn_lock(*vpp, cnp->cn_lkflags);
if (cnp->cn_flags & ISDOTDOT)
- vn_lock(dvp, ltype | LK_RETRY, td);
+ vn_lock(dvp, ltype | LK_RETRY);
if (error != 0) {
VN_RELE(*vpp);
*vpp = NULL;
@@ -1302,7 +1302,7 @@
if (error == 0) {
*vpp = ZTOV(zp);
- vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY, td);
+ vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY);
}
if (dl)
@@ -1584,7 +1584,7 @@
zfs_log_create(zilog, tx, TX_MKDIR, dzp, zp, dirname);
dmu_tx_commit(tx);
- vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY, curthread);
+ vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY);
zfs_dirent_unlock(dl);
@@ -2769,7 +2769,7 @@
if (error == 0) {
zfs_log_symlink(zilog, tx, TX_SYMLINK, dzp, zp, name, link);
*vpp = ZTOV(zp);
- vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY, td);
+ vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY);
}
dmu_tx_commit(tx);
==== //depot/projects/dtrace/src/sys/contrib/opensolaris/uts/common/fs/zfs/zfs_znode.c#4 (text+ko) ====
@@ -85,9 +85,9 @@
ZTOV(zp) = NULL;
vhold(vp);
mutex_exit(&zp->z_lock);
- vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, curthread);
+ vn_lock(vp, LK_EXCLUSIVE | LK_RETRY);
vrecycle(vp, curthread);
- VOP_UNLOCK(vp, 0, curthread);
+ VOP_UNLOCK(vp, 0);
vdrop(vp);
zfs_znode_free(zp);
} else {
More information about the p4-projects
mailing list