diff options
author | 2018-04-28 03:13:04 +0000 | |
---|---|---|
committer | 2018-04-28 03:13:04 +0000 | |
commit | 36bb23f12acb06197346657619c449dc8e0cae8e (patch) | |
tree | c24cd9be1c65672fca95ccca71d5542d91c3c25f | |
parent | rasops: implement scrollback (diff) | |
download | wireguard-openbsd-36bb23f12acb06197346657619c449dc8e0cae8e.tar.xz wireguard-openbsd-36bb23f12acb06197346657619c449dc8e0cae8e.zip |
Clean up the parameters of VOP_LOCK() and VOP_UNLOCK(). It is always
curproc that does the locking or unlocking, so the proc parameter
is pointless and can be dropped.
OK mpi@, deraadt@
58 files changed, 253 insertions, 262 deletions
diff --git a/share/man/man9/VOP_LOOKUP.9 b/share/man/man9/VOP_LOOKUP.9 index 32276fa1b7a..060bfa05a93 100644 --- a/share/man/man9/VOP_LOOKUP.9 +++ b/share/man/man9/VOP_LOOKUP.9 @@ -1,4 +1,4 @@ -.\" $OpenBSD: VOP_LOOKUP.9,v 1.40 2017/11/28 12:06:10 jmc Exp $ +.\" $OpenBSD: VOP_LOOKUP.9,v 1.41 2018/04/28 03:13:04 visa Exp $ .\" .\" Copyright (c) 2003 Ted Unangst .\" Copyright (c) 2000, 2001 The NetBSD Foundation, Inc. @@ -24,7 +24,7 @@ .\" (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF .\" THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. .\" -.Dd $Mdocdate: November 28 2017 $ +.Dd $Mdocdate: April 28 2018 $ .Dt VOP_LOOKUP 9 .Os .Sh NAME @@ -159,7 +159,6 @@ .Fo VOP_LOCK .Fa "struct vnode *vp" .Fa "int flags" -.Fa "struct proc *p" .Fc .Ft int .Fo VOP_LOOKUP @@ -278,7 +277,6 @@ .Ft int .Fo VOP_UNLOCK .Fa "struct vnode *vp" -.Fa "struct proc *p" .Fc .Ft int .Fo VOP_WRITE @@ -555,8 +553,8 @@ Upon success, zero is returned; otherwise, an appropriate error code is returned. .Pp .It Fn VOP_ISLOCKED vp -.It Fn VOP_LOCK vp flags p -.It Fn VOP_UNLOCK vp p +.It Fn VOP_LOCK vp flags +.It Fn VOP_UNLOCK vp .Fn VOP_LOCK is used internally by .Xr vn_lock 9 diff --git a/sys/arch/sparc64/dev/vdsp.c b/sys/arch/sparc64/dev/vdsp.c index 762ab872ee5..a08778cc7f1 100644 --- a/sys/arch/sparc64/dev/vdsp.c +++ b/sys/arch/sparc64/dev/vdsp.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vdsp.c,v 1.43 2018/01/17 15:52:34 stsp Exp $ */ +/* $OpenBSD: vdsp.c,v 1.44 2018/04/28 03:13:04 visa Exp $ */ /* * Copyright (c) 2009, 2011, 2014 Mark Kettenis * @@ -941,7 +941,7 @@ vdsp_open(void *arg1) sc->sc_vdisk_size = va.va_size / DEV_BSIZE; } - VOP_UNLOCK(nd.ni_vp, p); + VOP_UNLOCK(nd.ni_vp); sc->sc_vp = nd.ni_vp; vdsp_readlabel(sc); @@ -1013,7 +1013,7 @@ vdsp_readlabel(struct vdsp_softc *sc) vn_lock(sc->sc_vp, LK_EXCLUSIVE | LK_RETRY, p); err = VOP_READ(sc->sc_vp, &uio, 0, p->p_ucred); - VOP_UNLOCK(sc->sc_vp, p); + VOP_UNLOCK(sc->sc_vp); if (err) { free(sc->sc_label, M_DEVBUF, 0); sc->sc_label = NULL; @@ -1043,7 +1043,7 @@ vdsp_writelabel(struct vdsp_softc *sc) vn_lock(sc->sc_vp, LK_EXCLUSIVE | LK_RETRY, p); err = VOP_WRITE(sc->sc_vp, &uio, 0, p->p_ucred); - VOP_UNLOCK(sc->sc_vp, p); + VOP_UNLOCK(sc->sc_vp); return (err); } @@ -1074,7 +1074,7 @@ vdsp_is_iso(struct vdsp_softc *sc) vn_lock(sc->sc_vp, LK_EXCLUSIVE | LK_RETRY, p); err = VOP_READ(sc->sc_vp, &uio, 0, p->p_ucred); - VOP_UNLOCK(sc->sc_vp, p); + VOP_UNLOCK(sc->sc_vp); if (err == 0 && memcmp(vdp->id, ISO_STANDARD_ID, sizeof(vdp->id))) err = ENOENT; @@ -1153,7 +1153,7 @@ vdsp_read_desc(struct vdsp_softc *sc, struct vdsk_desc_msg *dm) vn_lock(sc->sc_vp, LK_EXCLUSIVE | LK_RETRY, p); dm->status = VOP_READ(sc->sc_vp, &uio, 0, p->p_ucred); - VOP_UNLOCK(sc->sc_vp, p); + VOP_UNLOCK(sc->sc_vp); KERNEL_UNLOCK(); if (dm->status == 0) { @@ -1227,7 +1227,7 @@ vdsp_read_dring(void *arg1, void *arg2) vn_lock(sc->sc_vp, LK_EXCLUSIVE | LK_RETRY, p); vd->status = VOP_READ(sc->sc_vp, &uio, 0, p->p_ucred); - VOP_UNLOCK(sc->sc_vp, p); + VOP_UNLOCK(sc->sc_vp); KERNEL_UNLOCK(); if (vd->status == 0) { @@ -1326,7 +1326,7 @@ vdsp_write_dring(void *arg1, void *arg2) vn_lock(sc->sc_vp, LK_EXCLUSIVE | LK_RETRY, p); vd->status = VOP_WRITE(sc->sc_vp, &uio, 0, p->p_ucred); - VOP_UNLOCK(sc->sc_vp, p); + VOP_UNLOCK(sc->sc_vp); fail: free(buf, M_DEVBUF, 0); diff --git a/sys/dev/diskmap.c b/sys/dev/diskmap.c index a14f03d4689..a61644f7810 100644 --- a/sys/dev/diskmap.c +++ b/sys/dev/diskmap.c @@ -1,4 +1,4 @@ -/* $OpenBSD: diskmap.c,v 1.17 2018/01/02 06:38:45 guenther Exp $ */ +/* $OpenBSD: diskmap.c,v 1.18 2018/04/28 03:13:04 visa Exp $ */ /* * Copyright (c) 2009, 2010 Joel Sing <jsing@openbsd.org> @@ -114,7 +114,7 @@ diskmapioctl(dev_t dev, u_long cmd, caddr_t addr, int flag, struct proc *p) fp->f_rbytes = 0; fp->f_wbytes = 0; - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); FRELE(fp, p); fdpunlock(fdp); diff --git a/sys/dev/usb/uvideo.c b/sys/dev/usb/uvideo.c index 3711ee24949..56856a9d8d2 100644 --- a/sys/dev/usb/uvideo.c +++ b/sys/dev/usb/uvideo.c @@ -1,4 +1,4 @@ -/* $OpenBSD: uvideo.c,v 1.197 2018/04/24 17:22:33 landry Exp $ */ +/* $OpenBSD: uvideo.c,v 1.198 2018/04/28 03:13:04 visa Exp $ */ /* * Copyright (c) 2008 Robert Nagy <robert@openbsd.org> @@ -2741,7 +2741,7 @@ uvideo_debug_file_open(struct uvideo_softc *sc) } sc->sc_vp = nd.ni_vp; - VOP_UNLOCK(sc->sc_vp, p); + VOP_UNLOCK(sc->sc_vp); if (nd.ni_vp->v_type != VREG) { vn_close(nd.ni_vp, FWRITE, p->p_ucred, p); return (EIO); diff --git a/sys/dev/vnd.c b/sys/dev/vnd.c index d9559ddda91..c7ecc9be87e 100644 --- a/sys/dev/vnd.c +++ b/sys/dev/vnd.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vnd.c,v 1.167 2018/02/19 08:59:52 mpi Exp $ */ +/* $OpenBSD: vnd.c,v 1.168 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: vnd.c,v 1.26 1996/03/30 23:06:11 christos Exp $ */ /* @@ -461,14 +461,14 @@ vndioctl(dev_t dev, u_long cmd, caddr_t addr, int flag, struct proc *p) else { error = VOP_GETATTR(nd.ni_vp, &vattr, p->p_ucred, p); if (error) { - VOP_UNLOCK(nd.ni_vp, p); + VOP_UNLOCK(nd.ni_vp); vn_close(nd.ni_vp, VNDRW(sc), p->p_ucred, p); disk_unlock(&sc->sc_dk); return (error); } sc->sc_size = vattr.va_size / sc->sc_secsize; } - VOP_UNLOCK(nd.ni_vp, p); + VOP_UNLOCK(nd.ni_vp); sc->sc_vp = nd.ni_vp; if ((error = vndsetcred(sc, p->p_ucred)) != 0) { (void) vn_close(nd.ni_vp, VNDRW(sc), p->p_ucred, p); diff --git a/sys/isofs/cd9660/cd9660_lookup.c b/sys/isofs/cd9660/cd9660_lookup.c index afdfb3dd6ea..c4ec9abfd51 100644 --- a/sys/isofs/cd9660/cd9660_lookup.c +++ b/sys/isofs/cd9660/cd9660_lookup.c @@ -1,4 +1,4 @@ -/* $OpenBSD: cd9660_lookup.c,v 1.25 2017/12/30 20:47:00 guenther Exp $ */ +/* $OpenBSD: cd9660_lookup.c,v 1.26 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: cd9660_lookup.c,v 1.18 1997/05/08 16:19:59 mycroft Exp $ */ /*- @@ -376,7 +376,7 @@ found: */ if (flags & ISDOTDOT) { brelse(bp); - VOP_UNLOCK(pdp, p); /* race to get the inode */ + VOP_UNLOCK(pdp); /* race to get the inode */ cnp->cn_flags |= PDIRUNLOCK; error = cd9660_vget_internal(vdp->v_mount, dp->i_ino, &tdp, dp->i_ino != ino, NULL); @@ -404,7 +404,7 @@ found: if (error) return (error); if (!lockparent || !(flags & ISLASTCN)) { - VOP_UNLOCK(pdp, p); + VOP_UNLOCK(pdp); cnp->cn_flags |= PDIRUNLOCK; } *vpp = tdp; diff --git a/sys/isofs/cd9660/cd9660_node.c b/sys/isofs/cd9660/cd9660_node.c index 07eaa182408..c8839dfee64 100644 --- a/sys/isofs/cd9660/cd9660_node.c +++ b/sys/isofs/cd9660/cd9660_node.c @@ -1,4 +1,4 @@ -/* $OpenBSD: cd9660_node.c,v 1.32 2017/12/30 20:47:00 guenther Exp $ */ +/* $OpenBSD: cd9660_node.c,v 1.33 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: cd9660_node.c,v 1.17 1997/05/05 07:13:57 mycroft Exp $ */ /*- @@ -193,7 +193,7 @@ cd9660_inactive(v) #endif ip->i_flag = 0; - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); /* * If we are done with the inode, reclaim it * so that it can be reused immediately. diff --git a/sys/isofs/cd9660/cd9660_vfsops.c b/sys/isofs/cd9660/cd9660_vfsops.c index be84d998b74..94c8c56c3d0 100644 --- a/sys/isofs/cd9660/cd9660_vfsops.c +++ b/sys/isofs/cd9660/cd9660_vfsops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: cd9660_vfsops.c,v 1.88 2018/03/28 16:34:28 visa Exp $ */ +/* $OpenBSD: cd9660_vfsops.c,v 1.89 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: cd9660_vfsops.c,v 1.26 1997/06/13 15:38:58 pk Exp $ */ /*- @@ -244,7 +244,7 @@ iso_mountfs(devvp, mp, p, argp) return (EBUSY); vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, p); error = vinvalbuf(devvp, V_SAVE, p->p_ucred, p, 0, 0); - VOP_UNLOCK(devvp, p); + VOP_UNLOCK(devvp); if (error) return (error); @@ -438,7 +438,7 @@ out: vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, p); VOP_CLOSE(devvp, ronly ? FREAD : FREAD|FWRITE, NOCRED, p); - VOP_UNLOCK(devvp, p); + VOP_UNLOCK(devvp); if (isomp) { free((caddr_t)isomp, M_ISOFSMNT, 0); diff --git a/sys/isofs/udf/udf_vfsops.c b/sys/isofs/udf/udf_vfsops.c index 7e917a7fe86..83d8a6f8e9a 100644 --- a/sys/isofs/udf/udf_vfsops.c +++ b/sys/isofs/udf/udf_vfsops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: udf_vfsops.c,v 1.63 2018/03/28 16:34:28 visa Exp $ */ +/* $OpenBSD: udf_vfsops.c,v 1.64 2018/04/28 03:13:04 visa Exp $ */ /* * Copyright (c) 2001, 2002 Scott Long <scottl@freebsd.org> @@ -240,7 +240,7 @@ udf_mountfs(struct vnode *devvp, struct mount *mp, uint32_t lb, struct proc *p) return (EBUSY); vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, p); error = vinvalbuf(devvp, V_SAVE, p->p_ucred, p, 0, 0); - VOP_UNLOCK(devvp, p); + VOP_UNLOCK(devvp); if (error) return (error); @@ -443,7 +443,7 @@ bail: vn_lock(devvp, LK_EXCLUSIVE|LK_RETRY, p); VOP_CLOSE(devvp, FREAD, FSCRED, p); - VOP_UNLOCK(devvp, p); + VOP_UNLOCK(devvp); return (error); } @@ -467,7 +467,7 @@ udf_unmount(struct mount *mp, int mntflags, struct proc *p) vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, p); vinvalbuf(devvp, V_SAVE, NOCRED, p, 0, 0); (void)VOP_CLOSE(devvp, FREAD, NOCRED, p); - VOP_UNLOCK(devvp, p); + VOP_UNLOCK(devvp); devvp->v_specmountpoint = NULL; vrele(devvp); diff --git a/sys/isofs/udf/udf_vnops.c b/sys/isofs/udf/udf_vnops.c index 95fcb12c10e..54876a84077 100644 --- a/sys/isofs/udf/udf_vnops.c +++ b/sys/isofs/udf/udf_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: udf_vnops.c,v 1.64 2016/06/19 11:54:33 natano Exp $ */ +/* $OpenBSD: udf_vnops.c,v 1.65 2018/04/28 03:13:04 visa Exp $ */ /* * Copyright (c) 2001, 2002 Scott Long <scottl@freebsd.org> @@ -1104,7 +1104,7 @@ lookloop: nchstats.ncs_pass2++; if (!(flags & LOCKPARENT) || !(flags & ISLASTCN)) { ap->a_cnp->cn_flags |= PDIRUNLOCK; - VOP_UNLOCK(dvp, p); + VOP_UNLOCK(dvp); } *vpp = tdp; @@ -1146,12 +1146,11 @@ udf_inactive(void *v) { struct vop_inactive_args *ap = v; struct vnode *vp = ap->a_vp; - struct proc *p = ap->a_p; /* * No need to sync anything, so just unlock the vnode and return. */ - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); return (0); } diff --git a/sys/kern/exec_script.c b/sys/kern/exec_script.c index 2e51357dbe6..4a659c80867 100644 --- a/sys/kern/exec_script.c +++ b/sys/kern/exec_script.c @@ -1,4 +1,4 @@ -/* $OpenBSD: exec_script.c,v 1.42 2018/01/02 06:38:45 guenther Exp $ */ +/* $OpenBSD: exec_script.c,v 1.43 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: exec_script.c,v 1.13 1996/02/04 02:15:06 christos Exp $ */ /* @@ -160,7 +160,7 @@ check_shell: */ vn_lock(scriptvp, LK_EXCLUSIVE|LK_RETRY, p); error = VOP_ACCESS(scriptvp, VREAD, p->p_ucred, p); - VOP_UNLOCK(scriptvp, p); + VOP_UNLOCK(scriptvp); if (error == EACCES || script_sbits) { struct file *fp; diff --git a/sys/kern/init_main.c b/sys/kern/init_main.c index 87604c5b501..f5a4984070a 100644 --- a/sys/kern/init_main.c +++ b/sys/kern/init_main.c @@ -1,4 +1,4 @@ -/* $OpenBSD: init_main.c,v 1.276 2018/04/12 17:13:44 deraadt Exp $ */ +/* $OpenBSD: init_main.c,v 1.277 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: init_main.c,v 1.84.4.1 1996/06/02 09:08:06 mrg Exp $ */ /* @@ -493,7 +493,7 @@ main(void *framep) panic("cannot find root vnode"); p->p_fd->fd_cdir = rootvnode; vref(p->p_fd->fd_cdir); - VOP_UNLOCK(rootvnode, p); + VOP_UNLOCK(rootvnode); p->p_fd->fd_rdir = NULL; /* diff --git a/sys/kern/kern_acct.c b/sys/kern/kern_acct.c index 9951a2a2789..3ce6831734d 100644 --- a/sys/kern/kern_acct.c +++ b/sys/kern/kern_acct.c @@ -1,4 +1,4 @@ -/* $OpenBSD: kern_acct.c,v 1.35 2018/02/19 08:59:52 mpi Exp $ */ +/* $OpenBSD: kern_acct.c,v 1.36 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: kern_acct.c,v 1.42 1996/02/04 02:15:12 christos Exp $ */ /*- @@ -116,7 +116,7 @@ sys_acct(struct proc *p, void *v, register_t *retval) p); if ((error = vn_open(&nd, FWRITE|O_APPEND, 0)) != 0) return (error); - VOP_UNLOCK(nd.ni_vp, p); + VOP_UNLOCK(nd.ni_vp); if (nd.ni_vp->v_type != VREG) { vn_close(nd.ni_vp, FWRITE, p->p_ucred, p); return (EACCES); diff --git a/sys/kern/kern_descrip.c b/sys/kern/kern_descrip.c index f5e98872df6..40966c4df66 100644 --- a/sys/kern/kern_descrip.c +++ b/sys/kern/kern_descrip.c @@ -1,4 +1,4 @@ -/* $OpenBSD: kern_descrip.c,v 1.154 2018/04/27 10:13:37 mpi Exp $ */ +/* $OpenBSD: kern_descrip.c,v 1.155 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: kern_descrip.c,v 1.42 1996/03/30 22:24:38 christos Exp $ */ /* @@ -785,7 +785,7 @@ sys_fpathconf(struct proc *p, void *v, register_t *retval) vp = fp->f_data; vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, p); error = VOP_PATHCONF(vp, SCARG(uap, name), retval); - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); break; default: diff --git a/sys/kern/kern_exec.c b/sys/kern/kern_exec.c index 0cee0b3a6d8..10d2eb1bbbc 100644 --- a/sys/kern/kern_exec.c +++ b/sys/kern/kern_exec.c @@ -1,4 +1,4 @@ -/* $OpenBSD: kern_exec.c,v 1.194 2018/04/27 10:13:37 mpi Exp $ */ +/* $OpenBSD: kern_exec.c,v 1.195 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: kern_exec.c,v 1.75 1996/02/09 18:59:28 christos Exp $ */ /*- @@ -163,7 +163,7 @@ check_exec(struct proc *p, struct exec_package *epp) goto bad1; /* unlock vp, we need it unlocked from here */ - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); /* now we have the file, get the exec header */ error = vn_rdwr(UIO_READ, vp, epp->ep_hdr, epp->ep_hdrlen, 0, diff --git a/sys/kern/kern_ktrace.c b/sys/kern/kern_ktrace.c index 5bd55d71d5f..8a14c70d1e1 100644 --- a/sys/kern/kern_ktrace.c +++ b/sys/kern/kern_ktrace.c @@ -1,4 +1,4 @@ -/* $OpenBSD: kern_ktrace.c,v 1.95 2018/02/19 08:59:52 mpi Exp $ */ +/* $OpenBSD: kern_ktrace.c,v 1.96 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: kern_ktrace.c,v 1.23 1996/02/09 18:59:36 christos Exp $ */ /* @@ -511,7 +511,7 @@ sys_ktrace(struct proc *p, void *v, register_t *retval) return error; vp = nd.ni_vp; - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); } error = doktrace(vp, SCARG(uap, ops), SCARG(uap, facs), diff --git a/sys/kern/kern_sig.c b/sys/kern/kern_sig.c index cb448f48280..db7adf619ba 100644 --- a/sys/kern/kern_sig.c +++ b/sys/kern/kern_sig.c @@ -1,4 +1,4 @@ -/* $OpenBSD: kern_sig.c,v 1.219 2018/04/12 17:13:44 deraadt Exp $ */ +/* $OpenBSD: kern_sig.c,v 1.220 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: kern_sig.c,v 1.54 1996/04/22 01:38:32 christos Exp $ */ /* @@ -1569,7 +1569,7 @@ coredump(struct proc *p) */ vp = nd.ni_vp; if ((error = VOP_GETATTR(vp, &vattr, cred, p)) != 0) { - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); vn_close(vp, FWRITE, cred, p); goto out; } @@ -1577,7 +1577,7 @@ coredump(struct proc *p) vattr.va_mode & ((VREAD | VWRITE) >> 3 | (VREAD | VWRITE) >> 6) || vattr.va_uid != cred->cr_uid) { error = EACCES; - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); vn_close(vp, FWRITE, cred, p); goto out; } @@ -1590,7 +1590,7 @@ coredump(struct proc *p) io.io_vp = vp; io.io_cred = cred; io.io_offset = 0; - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); vref(vp); error = vn_close(vp, FWRITE, cred, p); if (error == 0) diff --git a/sys/kern/spec_vnops.c b/sys/kern/spec_vnops.c index db115daf7ca..f40ba0430de 100644 --- a/sys/kern/spec_vnops.c +++ b/sys/kern/spec_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: spec_vnops.c,v 1.90 2017/12/30 23:08:29 guenther Exp $ */ +/* $OpenBSD: spec_vnops.c,v 1.91 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: spec_vnops.c,v 1.29 1996/04/22 01:42:38 christos Exp $ */ /* @@ -154,7 +154,7 @@ spec_open(void *v) vp->v_flag |= VISTTY; if (cdevsw[maj].d_flags & D_CLONE) return (spec_open_clone(ap)); - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); error = (*cdevsw[maj].d_open)(dev, ap->a_mode, S_IFCHR, p); vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, p); return (error); @@ -219,7 +219,7 @@ spec_read(void *v) switch (vp->v_type) { case VCHR: - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); error = (*cdevsw[major(vp->v_rdev)].d_read) (vp->v_rdev, uio, ap->a_ioflag); vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, p); @@ -273,7 +273,7 @@ spec_inactive(void *v) { struct vop_inactive_args *ap = v; - VOP_UNLOCK(ap->a_vp, ap->a_p); + VOP_UNLOCK(ap->a_vp); return (0); } @@ -306,7 +306,7 @@ spec_write(void *v) switch (vp->v_type) { case VCHR: - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); error = (*cdevsw[major(vp->v_rdev)].d_write) (vp->v_rdev, uio, ap->a_ioflag); vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, p); @@ -525,7 +525,7 @@ spec_close(void *v) vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, p); error = vinvalbuf(vp, V_SAVE, ap->a_cred, p, 0, 0); if (!(vp->v_flag & VXLOCK)) - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); if (error) return (error); /* @@ -550,7 +550,7 @@ spec_close(void *v) /* release lock if held and this isn't coming from vclean() */ relock = VOP_ISLOCKED(vp) && !(vp->v_flag & VXLOCK); if (relock) - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); error = (*devclose)(dev, ap->a_fflag, mode, p); if (relock) vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, p); @@ -582,7 +582,7 @@ spec_setattr(void *v) vn_lock(vp->v_specparent, LK_EXCLUSIVE|LK_RETRY, p); error = VOP_SETATTR(vp->v_specparent, ap->a_vap, ap->a_cred, p); - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); return (error); } @@ -723,7 +723,7 @@ spec_open_clone(struct vop_open_args *ap) return (error); /* out of vnodes */ } - VOP_UNLOCK(vp, ap->a_p); + VOP_UNLOCK(vp); error = cdevsw[major(vp->v_rdev)].d_open(cvp->v_rdev, ap->a_mode, S_IFCHR, ap->a_p); diff --git a/sys/kern/tty.c b/sys/kern/tty.c index b9fec5b6bb9..1a562b50f99 100644 --- a/sys/kern/tty.c +++ b/sys/kern/tty.c @@ -1,4 +1,4 @@ -/* $OpenBSD: tty.c,v 1.137 2018/02/19 08:59:52 mpi Exp $ */ +/* $OpenBSD: tty.c,v 1.138 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: tty.c,v 1.68.4.2 1996/06/06 16:04:52 thorpej Exp $ */ /*- @@ -797,7 +797,7 @@ ttioctl(struct tty *tp, u_long cmd, caddr_t data, int flag, struct proc *p) return (error); vn_lock(nid.ni_vp, LK_EXCLUSIVE | LK_RETRY, p); error = VOP_ACCESS(nid.ni_vp, VREAD, p->p_ucred, p); - VOP_UNLOCK(nid.ni_vp, p); + VOP_UNLOCK(nid.ni_vp); vrele(nid.ni_vp); if (error) return (error); diff --git a/sys/kern/tty_pty.c b/sys/kern/tty_pty.c index 8babd283dcc..4e5f75aa1b9 100644 --- a/sys/kern/tty_pty.c +++ b/sys/kern/tty_pty.c @@ -1,4 +1,4 @@ -/* $OpenBSD: tty_pty.c,v 1.83 2018/02/19 08:59:52 mpi Exp $ */ +/* $OpenBSD: tty_pty.c,v 1.84 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: tty_pty.c,v 1.33.4.1 1996/06/02 09:08:11 mrg Exp $ */ /* @@ -1103,7 +1103,7 @@ retry: cfp->f_type = DTYPE_VNODE; cfp->f_ops = &vnops; cfp->f_data = (caddr_t) cnd.ni_vp; - VOP_UNLOCK(cnd.ni_vp, p); + VOP_UNLOCK(cnd.ni_vp); /* * Open the slave. @@ -1137,7 +1137,7 @@ retry: goto bad; } } - VOP_UNLOCK(snd.ni_vp, p); + VOP_UNLOCK(snd.ni_vp); if (snd.ni_vp->v_usecount > 1 || (snd.ni_vp->v_flag & (VALIASED))) VOP_REVOKE(snd.ni_vp, REVOKEALL); @@ -1158,7 +1158,7 @@ retry: sfp->f_type = DTYPE_VNODE; sfp->f_ops = &vnops; sfp->f_data = (caddr_t) snd.ni_vp; - VOP_UNLOCK(snd.ni_vp, p); + VOP_UNLOCK(snd.ni_vp); /* now, put the indexen and names into struct ptmget */ ptm->cfd = cindx; diff --git a/sys/kern/tty_tty.c b/sys/kern/tty_tty.c index 0f88b8d9b27..6e3df463d6a 100644 --- a/sys/kern/tty_tty.c +++ b/sys/kern/tty_tty.c @@ -1,4 +1,4 @@ -/* $OpenBSD: tty_tty.c,v 1.22 2018/02/19 08:59:52 mpi Exp $ */ +/* $OpenBSD: tty_tty.c,v 1.23 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: tty_tty.c,v 1.13 1996/03/30 22:24:46 christos Exp $ */ /*- @@ -59,7 +59,7 @@ cttyopen(dev_t dev, int flag, int mode, struct proc *p) return (ENXIO); vn_lock(ttyvp, LK_EXCLUSIVE | LK_RETRY, p); error = VOP_OPEN(ttyvp, flag, NOCRED, p); - VOP_UNLOCK(ttyvp, p); + VOP_UNLOCK(ttyvp); return (error); } @@ -74,7 +74,7 @@ cttyread(dev_t dev, struct uio *uio, int flag) return (EIO); vn_lock(ttyvp, LK_EXCLUSIVE | LK_RETRY, p); error = VOP_READ(ttyvp, uio, flag, NOCRED); - VOP_UNLOCK(ttyvp, p); + VOP_UNLOCK(ttyvp); return (error); } @@ -89,7 +89,7 @@ cttywrite(dev_t dev, struct uio *uio, int flag) return (EIO); vn_lock(ttyvp, LK_EXCLUSIVE | LK_RETRY, p); error = VOP_WRITE(ttyvp, uio, flag, NOCRED); - VOP_UNLOCK(ttyvp, p); + VOP_UNLOCK(ttyvp); return (error); } diff --git a/sys/kern/uipc_usrreq.c b/sys/kern/uipc_usrreq.c index 8151c653110..b92f37d06fd 100644 --- a/sys/kern/uipc_usrreq.c +++ b/sys/kern/uipc_usrreq.c @@ -1,4 +1,4 @@ -/* $OpenBSD: uipc_usrreq.c,v 1.125 2018/04/27 10:13:37 mpi Exp $ */ +/* $OpenBSD: uipc_usrreq.c,v 1.126 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: uipc_usrreq.c,v 1.18 1996/02/09 19:00:50 christos Exp $ */ /* @@ -468,7 +468,7 @@ unp_bind(struct unpcb *unp, struct mbuf *nam, struct proc *p) unp->unp_connid.gid = p->p_ucred->cr_gid; unp->unp_connid.pid = p->p_p->ps_pid; unp->unp_flags |= UNP_FEIDSBIND; - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); return (0); } diff --git a/sys/kern/vfs_cache.c b/sys/kern/vfs_cache.c index 6a38ab19b4d..39f2b21d393 100644 --- a/sys/kern/vfs_cache.c +++ b/sys/kern/vfs_cache.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vfs_cache.c,v 1.53 2017/02/09 19:02:34 bluhm Exp $ */ +/* $OpenBSD: vfs_cache.c,v 1.54 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: vfs_cache.c,v 1.13 1996/02/04 02:18:09 christos Exp $ */ /* @@ -209,7 +209,7 @@ cache_lookup(struct vnode *dvp, struct vnode **vpp, vref(dvp); error = 0; } else if (cnp->cn_flags & ISDOTDOT) { - VOP_UNLOCK(dvp, p); + VOP_UNLOCK(dvp); cnp->cn_flags |= PDIRUNLOCK; error = vget(vp, LK_EXCLUSIVE, p); /* @@ -230,7 +230,7 @@ cache_lookup(struct vnode *dvp, struct vnode **vpp, * ISLASTCN is set, unlock the directory vnode. */ if (error || (~cnp->cn_flags & (LOCKPARENT|ISLASTCN)) != 0) { - VOP_UNLOCK(dvp, p); + VOP_UNLOCK(dvp); cnp->cn_flags |= PDIRUNLOCK; } } diff --git a/sys/kern/vfs_getcwd.c b/sys/kern/vfs_getcwd.c index dbe40f62508..6ada9e640f6 100644 --- a/sys/kern/vfs_getcwd.c +++ b/sys/kern/vfs_getcwd.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vfs_getcwd.c,v 1.28 2017/09/06 12:36:24 bluhm Exp $ */ +/* $OpenBSD: vfs_getcwd.c,v 1.29 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: vfs_getcwd.c,v 1.3.2.3 1999/07/11 10:24:09 sommerfeld Exp $ */ /* @@ -233,7 +233,7 @@ vfs_getcwd_getcache(struct vnode **lvpp, struct vnode **uvpp, char **bpp, /* Release current lock before acquiring the parent lock */ - VOP_UNLOCK(lvp, p); + VOP_UNLOCK(lvp); error = vget(uvp, LK_EXCLUSIVE | LK_RETRY, p); if (error) diff --git a/sys/kern/vfs_lookup.c b/sys/kern/vfs_lookup.c index 2c63ae415b4..fbf5d73499a 100644 --- a/sys/kern/vfs_lookup.c +++ b/sys/kern/vfs_lookup.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vfs_lookup.c,v 1.67 2018/04/27 10:13:37 mpi Exp $ */ +/* $OpenBSD: vfs_lookup.c,v 1.68 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: vfs_lookup.c,v 1.17 1996/02/09 19:00:59 christos Exp $ */ /* @@ -222,7 +222,7 @@ fail: return(0); } if ((cnp->cn_flags & LOCKPARENT) && (cnp->cn_flags & ISLASTCN)) - VOP_UNLOCK(ndp->ni_dvp, p); + VOP_UNLOCK(ndp->ni_dvp); if (ndp->ni_loopcnt++ >= SYMLOOP_MAX) { error = ELOOP; break; @@ -553,7 +553,7 @@ dirloop: (cnp->cn_flags & NOCROSSMOUNT) == 0) { if (vfs_busy(mp, VB_READ|VB_WAIT)) continue; - VOP_UNLOCK(dp, p); + VOP_UNLOCK(dp); error = VFS_ROOT(mp, &tdp); vfs_unbusy(mp); if (error) { @@ -620,13 +620,13 @@ terminal: vrele(ndp->ni_dvp); } if ((cnp->cn_flags & LOCKLEAF) == 0) - VOP_UNLOCK(dp, p); + VOP_UNLOCK(dp); return (0); bad2: if ((cnp->cn_flags & LOCKPARENT) && (cnp->cn_flags & ISLASTCN) && ((cnp->cn_flags & PDIRUNLOCK) == 0)) - VOP_UNLOCK(ndp->ni_dvp, p); + VOP_UNLOCK(ndp->ni_dvp); vrele(ndp->ni_dvp); bad: if (dpunlocked) @@ -754,12 +754,12 @@ vfs_relookup(struct vnode *dvp, struct vnode **vpp, struct componentname *cnp) if (!wantparent) vrele(dvp); if ((cnp->cn_flags & LOCKLEAF) == 0) - VOP_UNLOCK(dp, p); + VOP_UNLOCK(dp); return (0); bad2: if ((cnp->cn_flags & LOCKPARENT) && (cnp->cn_flags & ISLASTCN)) - VOP_UNLOCK(dvp, p); + VOP_UNLOCK(dvp); vrele(dvp); bad: vput(dp); diff --git a/sys/kern/vfs_subr.c b/sys/kern/vfs_subr.c index 72ce218967b..5b9bd48a46d 100644 --- a/sys/kern/vfs_subr.c +++ b/sys/kern/vfs_subr.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vfs_subr.c,v 1.267 2018/03/07 18:30:23 bluhm Exp $ */ +/* $OpenBSD: vfs_subr.c,v 1.268 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: vfs_subr.c,v 1.53 1996/04/22 01:39:13 christos Exp $ */ /* @@ -587,7 +587,7 @@ loop: * The vnodes created by bdevvp should not be aliased (why?). */ - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); vclean(vp, 0, p); vp->v_op = nvp->v_op; vp->v_tag = nvp->v_tag; @@ -718,7 +718,7 @@ vput(struct vnode *vp) #endif vp->v_usecount--; if (vp->v_usecount > 0) { - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); return; } @@ -970,7 +970,7 @@ vclean(struct vnode *vp, int flags, struct proc *p) * For active vnodes, it ensures that no other activity can * occur while the underlying object is being cleaned out. */ - VOP_LOCK(vp, LK_DRAIN | LK_EXCLUSIVE, p); + VOP_LOCK(vp, LK_DRAIN | LK_EXCLUSIVE); /* * Clean out any VM data associated with the vnode. @@ -995,7 +995,7 @@ vclean(struct vnode *vp, int flags, struct proc *p) * Any other processes trying to obtain this lock must first * wait for VXLOCK to clear, then call the new lock operation. */ - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); } /* diff --git a/sys/kern/vfs_sync.c b/sys/kern/vfs_sync.c index bb8ee505e81..755ca7411cb 100644 --- a/sys/kern/vfs_sync.c +++ b/sys/kern/vfs_sync.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vfs_sync.c,v 1.57 2018/02/10 05:24:23 deraadt Exp $ */ +/* $OpenBSD: vfs_sync.c,v 1.58 2018/04/28 03:13:05 visa Exp $ */ /* * Portions of this code are: @@ -360,7 +360,7 @@ sync_inactive(void *v) int s; if (vp->v_usecount == 0) { - VOP_UNLOCK(vp, ap->a_p); + VOP_UNLOCK(vp); return (0); } diff --git a/sys/kern/vfs_syscalls.c b/sys/kern/vfs_syscalls.c index 1051bc271b4..978e932d571 100644 --- a/sys/kern/vfs_syscalls.c +++ b/sys/kern/vfs_syscalls.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vfs_syscalls.c,v 1.280 2018/04/27 10:13:37 mpi Exp $ */ +/* $OpenBSD: vfs_syscalls.c,v 1.281 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: vfs_syscalls.c,v 1.71 1996/04/23 10:29:02 mycroft Exp $ */ /* @@ -297,7 +297,7 @@ update: TAILQ_INSERT_TAIL(&mountlist, mp, mnt_list); checkdirs(vp); vfs_unbusy(vp->v_mount); - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); if ((mp->mnt_flag & MNT_RDONLY) == 0) error = vfs_allocate_syncvnode(mp); vfs_unbusy(mp); @@ -769,7 +769,7 @@ sys_fchdir(struct proc *p, void *v, register_t *retval) vput(vp); return (error); } - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); old_cdir = fdp->fd_cdir; fdp->fd_cdir = vp; vrele(old_cdir); @@ -856,7 +856,7 @@ change_dir(struct nameidata *ndp, struct proc *p) if (error) vput(vp); else - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); return (error); } @@ -966,7 +966,7 @@ doopenat(struct proc *p, int fd, const char *path, int oflags, mode_t mode, type = F_FLOCK; if ((flags & FNONBLOCK) == 0) type |= F_WAIT; - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); error = VOP_ADVLOCK(vp, (caddr_t)fp, F_SETLK, &lf, type); if (error) { /* closef will vn_close the file for us. */ @@ -990,14 +990,14 @@ doopenat(struct proc *p, int fd, const char *path, int oflags, mode_t mode, error = VOP_SETATTR(vp, &vattr, fp->f_cred, p); } if (error) { - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); /* closef will close the file for us. */ fdremove(fdp, indx); closef(fp, p); goto out; } } - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); *retval = indx; FILE_SET_MATURE(fp, p); out: @@ -1149,7 +1149,7 @@ sys_fhopen(struct proc *p, void *v, register_t *retval) type = F_FLOCK; if ((flags & FNONBLOCK) == 0) type |= F_WAIT; - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); error = VOP_ADVLOCK(vp, (caddr_t)fp, F_SETLK, &lf, type); if (error) { vp = NULL; /* closef will vn_close the file */ @@ -1158,7 +1158,7 @@ sys_fhopen(struct proc *p, void *v, register_t *retval) vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, p); fp->f_iflags |= FIF_HASLOCK; } - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); *retval = indx; FILE_SET_MATURE(fp, p); @@ -2106,7 +2106,7 @@ sys_fchmod(struct proc *p, void *v, register_t *retval) vattr.va_mode = mode & ALLPERMS; error = VOP_SETATTR(vp, &vattr, p->p_ucred, p); } - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); FRELE(fp, p); return (error); } @@ -2287,7 +2287,7 @@ sys_fchown(struct proc *p, void *v, register_t *retval) error = VOP_SETATTR(vp, &vattr, p->p_ucred, p); } out: - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); FRELE(fp, p); return (error); } @@ -2557,7 +2557,7 @@ sys_ftruncate(struct proc *p, void *v, register_t *retval) vattr.va_size = len; error = VOP_SETATTR(vp, &vattr, fp->f_cred, p); } - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); bad: FRELE(fp, p); return (error); @@ -2586,7 +2586,7 @@ sys_fsync(struct proc *p, void *v, register_t *retval) error = softdep_fsync(vp); #endif - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); FRELE(fp, p); return (error); } @@ -2822,7 +2822,7 @@ sys_getdents(struct proc *p, void *v, register_t *retval) auio.uio_offset = fp->f_offset; error = VOP_READDIR(vp, &auio, fp->f_cred, &eofflag); fp->f_offset = auio.uio_offset; - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); if (error) goto bad; *retval = buflen - auio.uio_resid; diff --git a/sys/kern/vfs_vnops.c b/sys/kern/vfs_vnops.c index c05cff1908f..0946c8aa0cd 100644 --- a/sys/kern/vfs_vnops.c +++ b/sys/kern/vfs_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vfs_vnops.c,v 1.90 2018/04/10 09:17:45 mpi Exp $ */ +/* $OpenBSD: vfs_vnops.c,v 1.91 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: vfs_vnops.c,v 1.20 1996/02/04 02:18:41 christos Exp $ */ /* @@ -178,7 +178,7 @@ vn_open(struct nameidata *ndp, int fmode, int cmode) ndp->ni_vp = cip->ci_vp; /* return cloned vnode */ vp->v_data = cip->ci_data; /* restore v_data */ - VOP_UNLOCK(vp, p); /* keep a reference */ + VOP_UNLOCK(vp); /* keep a reference */ vp = ndp->ni_vp; /* for the increment below */ free(cip, M_TEMP, sizeof(*cip)); @@ -318,7 +318,7 @@ vn_rdwr(enum uio_rw rw, struct vnode *vp, caddr_t base, int len, off_t offset, error = VOP_WRITE(vp, &auio, ioflg, cred); } if ((ioflg & IO_NODELOCKED) == 0) - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); if (aresid) *aresid = auio.uio_resid; @@ -351,7 +351,7 @@ vn_read(struct file *fp, off_t *poff, struct uio *uio, struct ucred *cred) error = VOP_READ(vp, uio, (fp->f_flag & FNONBLOCK) ? IO_NDELAY : 0, cred); *poff += count - uio->uio_resid; - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); return (error); } @@ -383,7 +383,7 @@ vn_write(struct file *fp, off_t *poff, struct uio *uio, struct ucred *cred) *poff = uio->uio_offset; else *poff += count - uio->uio_resid; - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); return (error); } @@ -528,7 +528,7 @@ vn_lock(struct vnode *vp, int flags, struct proc *p) tsleep(vp, PINOD, "vn_lock", 0); error = ENOENT; } else { - error = VOP_LOCK(vp, flags, p); + error = VOP_LOCK(vp, flags); if (error == 0) return (error); } diff --git a/sys/kern/vfs_vops.c b/sys/kern/vfs_vops.c index 42ba4eecf00..fb2d89f5895 100644 --- a/sys/kern/vfs_vops.c +++ b/sys/kern/vfs_vops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vfs_vops.c,v 1.17 2018/02/10 05:24:23 deraadt Exp $ */ +/* $OpenBSD: vfs_vops.c,v 1.18 2018/04/28 03:13:05 visa Exp $ */ /* * Copyright (c) 2010 Thordur I. Bjornsson <thib@openbsd.org> * @@ -584,12 +584,11 @@ VOP_RECLAIM(struct vnode *vp, struct proc *p) } int -VOP_LOCK(struct vnode *vp, int flags, struct proc *p) +VOP_LOCK(struct vnode *vp, int flags) { struct vop_lock_args a; a.a_vp = vp; a.a_flags = flags; - a.a_p = p; if (vp->v_op->vop_lock == NULL) return (EOPNOTSUPP); @@ -598,12 +597,11 @@ VOP_LOCK(struct vnode *vp, int flags, struct proc *p) } int -VOP_UNLOCK(struct vnode *vp, struct proc *p) +VOP_UNLOCK(struct vnode *vp) { int r; struct vop_unlock_args a; a.a_vp = vp; - a.a_p = p; if (vp->v_op->vop_unlock == NULL) return (EOPNOTSUPP); diff --git a/sys/miscfs/fifofs/fifo_vnops.c b/sys/miscfs/fifofs/fifo_vnops.c index 1f45487fabb..6fb53cb2f22 100644 --- a/sys/miscfs/fifofs/fifo_vnops.c +++ b/sys/miscfs/fifofs/fifo_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: fifo_vnops.c,v 1.64 2018/04/08 18:57:39 guenther Exp $ */ +/* $OpenBSD: fifo_vnops.c,v 1.65 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: fifo_vnops.c,v 1.18 1996/03/16 23:52:42 christos Exp $ */ /* @@ -183,7 +183,7 @@ fifo_open(void *v) sounlock(s); if ((ap->a_mode & O_NONBLOCK) == 0) { if ((ap->a_mode & FREAD) && fip->fi_writers == 0) { - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); error = tsleep(&fip->fi_readers, PCATCH | PSOCK, "fifor", 0); vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, p); @@ -191,7 +191,7 @@ fifo_open(void *v) goto bad; } if ((ap->a_mode & FWRITE) && fip->fi_readers == 0) { - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); error = tsleep(&fip->fi_writers, PCATCH | PSOCK, "fifow", 0); vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, p); @@ -226,7 +226,7 @@ fifo_read(void *v) return (0); if (ap->a_ioflag & IO_NDELAY) rso->so_state |= SS_NBIO; - VOP_UNLOCK(ap->a_vp, p); + VOP_UNLOCK(ap->a_vp); error = soreceive(rso, NULL, uio, NULL, NULL, NULL, 0); vn_lock(ap->a_vp, LK_EXCLUSIVE | LK_RETRY, p); if (ap->a_ioflag & IO_NDELAY) { @@ -257,7 +257,7 @@ fifo_write(void *v) /* XXXSMP changing state w/o lock isn't safe. */ if (ap->a_ioflag & IO_NDELAY) wso->so_state |= SS_NBIO; - VOP_UNLOCK(ap->a_vp, p); + VOP_UNLOCK(ap->a_vp); error = sosend(wso, NULL, ap->a_uio, NULL, NULL, 0); vn_lock(ap->a_vp, LK_EXCLUSIVE | LK_RETRY, p); if (ap->a_ioflag & IO_NDELAY) @@ -346,7 +346,7 @@ fifo_inactive(void *v) { struct vop_inactive_args *ap = v; - VOP_UNLOCK(ap->a_vp, ap->a_p); + VOP_UNLOCK(ap->a_vp); return (0); } diff --git a/sys/miscfs/fuse/fuse_lookup.c b/sys/miscfs/fuse/fuse_lookup.c index 5ccc9fe8121..5bc23fc63d1 100644 --- a/sys/miscfs/fuse/fuse_lookup.c +++ b/sys/miscfs/fuse/fuse_lookup.c @@ -1,4 +1,4 @@ -/* $OpenBSD: fuse_lookup.c,v 1.17 2017/11/27 13:15:56 helg Exp $ */ +/* $OpenBSD: fuse_lookup.c,v 1.18 2018/04/28 03:13:05 visa Exp $ */ /* * Copyright (c) 2012-2013 Sylvestre Gallon <ccna.syl@gmail.com> * @@ -101,7 +101,7 @@ fusefs_lookup(void *v) cnp->cn_flags |= SAVENAME; if (!lockparent) { - VOP_UNLOCK(vdp, p); + VOP_UNLOCK(vdp); cnp->cn_flags |= PDIRUNLOCK; } @@ -149,7 +149,7 @@ fusefs_lookup(void *v) } if (flags & ISDOTDOT) { - VOP_UNLOCK(vdp, p); /* race to get the inode */ + VOP_UNLOCK(vdp); /* race to get the inode */ cnp->cn_flags |= PDIRUNLOCK; error = VFS_VGET(fmp->mp, nid, &tdp); @@ -184,7 +184,7 @@ fusefs_lookup(void *v) tdp->v_type = nvtype; if (!lockparent || !(flags & ISLASTCN)) { - VOP_UNLOCK(vdp, p); + VOP_UNLOCK(vdp); cnp->cn_flags |= PDIRUNLOCK; } diff --git a/sys/miscfs/fuse/fuse_vnops.c b/sys/miscfs/fuse/fuse_vnops.c index 81146b99b76..cecbac68760 100644 --- a/sys/miscfs/fuse/fuse_vnops.c +++ b/sys/miscfs/fuse/fuse_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: fuse_vnops.c,v 1.37 2017/11/30 11:29:03 helg Exp $ */ +/* $OpenBSD: fuse_vnops.c,v 1.38 2018/04/28 03:13:05 visa Exp $ */ /* * Copyright (c) 2012-2013 Sylvestre Gallon <ccna.syl@gmail.com> * @@ -601,7 +601,7 @@ fusefs_link(void *v) out1: if (dvp != vp) - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); out2: vput(dvp); return (error); @@ -773,7 +773,7 @@ fusefs_inactive(void *v) error = VOP_GETATTR(vp, &vattr, cred, p); - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); if (error) vrecycle(vp, p); @@ -1211,7 +1211,7 @@ abortit: dp == ip || (fcnp->cn_flags & ISDOTDOT) || (tcnp->cn_flags & ISDOTDOT)) { - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); error = EINVAL; goto abortit; } @@ -1220,13 +1220,13 @@ abortit: if (!fmp->sess_init) { error = ENXIO; - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); goto abortit; } if (fmp->undef_op & UNDEF_RENAME) { error = ENOSYS; - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); goto abortit; } @@ -1248,14 +1248,14 @@ abortit: } fb_delete(fbuf); - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); goto abortit; } fb_delete(fbuf); VN_KNOTE(fvp, NOTE_RENAME); - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); if (tdvp == tvp) vrele(tdvp); else diff --git a/sys/msdosfs/msdosfs_denode.c b/sys/msdosfs/msdosfs_denode.c index c2755607dbc..9ff5d6086ec 100644 --- a/sys/msdosfs/msdosfs_denode.c +++ b/sys/msdosfs/msdosfs_denode.c @@ -1,4 +1,4 @@ -/* $OpenBSD: msdosfs_denode.c,v 1.60 2018/03/28 16:34:28 visa Exp $ */ +/* $OpenBSD: msdosfs_denode.c,v 1.61 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: msdosfs_denode.c,v 1.23 1997/10/17 11:23:58 ws Exp $ */ /*- @@ -661,7 +661,7 @@ msdosfs_inactive(void *v) deupdat(dep, 0); out: - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); /* * If we are done with the denode, reclaim it * so that it can be reused immediately. diff --git a/sys/msdosfs/msdosfs_lookup.c b/sys/msdosfs/msdosfs_lookup.c index fcb7f2d7a7e..dab91299e80 100644 --- a/sys/msdosfs/msdosfs_lookup.c +++ b/sys/msdosfs/msdosfs_lookup.c @@ -1,4 +1,4 @@ -/* $OpenBSD: msdosfs_lookup.c,v 1.30 2016/03/19 12:04:16 natano Exp $ */ +/* $OpenBSD: msdosfs_lookup.c,v 1.31 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: msdosfs_lookup.c,v 1.34 1997/10/18 22:12:27 ws Exp $ */ /*- @@ -400,7 +400,7 @@ notfound:; */ cnp->cn_flags |= SAVENAME; if (!lockparent) { - VOP_UNLOCK(vdp, p); + VOP_UNLOCK(vdp); cnp->cn_flags |= PDIRUNLOCK; } return (EJUSTRETURN); @@ -491,7 +491,7 @@ foundroot:; return (error); *vpp = DETOV(tdp); if (!lockparent) { - VOP_UNLOCK(vdp, p); + VOP_UNLOCK(vdp); cnp->cn_flags |= PDIRUNLOCK; } return (0); @@ -524,7 +524,7 @@ foundroot:; *vpp = DETOV(tdp); cnp->cn_flags |= SAVENAME; if (!lockparent) - VOP_UNLOCK(vdp, p); + VOP_UNLOCK(vdp); return (0); } @@ -549,7 +549,7 @@ foundroot:; */ pdp = vdp; if (flags & ISDOTDOT) { - VOP_UNLOCK(pdp, p); /* race to get the inode */ + VOP_UNLOCK(pdp); /* race to get the inode */ cnp->cn_flags |= PDIRUNLOCK; if ((error = deget(pmp, cluster, blkoff, &tdp)) != 0) { if (vn_lock(pdp, LK_EXCLUSIVE | LK_RETRY, p) == 0) @@ -572,7 +572,7 @@ foundroot:; if ((error = deget(pmp, cluster, blkoff, &tdp)) != 0) return (error); if (!lockparent || !(flags & ISLASTCN)) { - VOP_UNLOCK(pdp, p); + VOP_UNLOCK(pdp); cnp->cn_flags |= PDIRUNLOCK; } *vpp = DETOV(tdp); diff --git a/sys/msdosfs/msdosfs_vfsops.c b/sys/msdosfs/msdosfs_vfsops.c index f2a7a5de845..e8b7fac2c5c 100644 --- a/sys/msdosfs/msdosfs_vfsops.c +++ b/sys/msdosfs/msdosfs_vfsops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: msdosfs_vfsops.c,v 1.87 2018/02/10 05:24:23 deraadt Exp $ */ +/* $OpenBSD: msdosfs_vfsops.c,v 1.88 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: msdosfs_vfsops.c,v 1.48 1997/10/18 02:54:57 briggs Exp $ */ /*- @@ -283,7 +283,7 @@ msdosfs_mountfs(struct vnode *devvp, struct mount *mp, struct proc *p, return (EBUSY); vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, p); error = vinvalbuf(devvp, V_SAVE, p->p_ucred, p, 0, 0); - VOP_UNLOCK(devvp, p); + VOP_UNLOCK(devvp); if (error) return (error); @@ -566,7 +566,7 @@ error_exit: vn_lock(devvp, LK_EXCLUSIVE|LK_RETRY, p); (void) VOP_CLOSE(devvp, ronly ? FREAD : FREAD|FWRITE, NOCRED, p); - VOP_UNLOCK(devvp, p); + VOP_UNLOCK(devvp); if (pmp) { if (pmp->pm_inusemap) @@ -681,7 +681,7 @@ msdosfs_sync_vnode(struct vnode *vp, void *arg) if ((error = VOP_FSYNC(vp, msa->cred, msa->waitfor, msa->p)) != 0) msa->allerror = error; - VOP_UNLOCK(vp, msa->p); + VOP_UNLOCK(vp); vrele(vp); return (0); @@ -724,7 +724,7 @@ msdosfs_sync(struct mount *mp, int waitfor, int stall, struct ucred *cred, vn_lock(pmp->pm_devvp, LK_EXCLUSIVE | LK_RETRY, p); if ((error = VOP_FSYNC(pmp->pm_devvp, cred, waitfor, p)) != 0) msa.allerror = error; - VOP_UNLOCK(pmp->pm_devvp, p); + VOP_UNLOCK(pmp->pm_devvp); } return (msa.allerror); diff --git a/sys/msdosfs/msdosfs_vnops.c b/sys/msdosfs/msdosfs_vnops.c index 3735e6edbf8..f5d5e5f34f4 100644 --- a/sys/msdosfs/msdosfs_vnops.c +++ b/sys/msdosfs/msdosfs_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: msdosfs_vnops.c,v 1.117 2018/01/02 06:38:45 guenther Exp $ */ +/* $OpenBSD: msdosfs_vnops.c,v 1.118 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: msdosfs_vnops.c,v 1.63 1997/10/17 11:24:19 ws Exp $ */ /*- @@ -1013,7 +1013,7 @@ abortit: (fcnp->cn_flags & ISDOTDOT) || (tcnp->cn_flags & ISDOTDOT) || (ip->de_flag & DE_RENAME)) { - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); error = EINVAL; goto abortit; } @@ -1045,7 +1045,7 @@ abortit: * call to doscheckpath(). */ error = VOP_ACCESS(fvp, VWRITE, tcnp->cn_cred, tcnp->cn_proc); - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); if (VTODE(fdvp)->de_StartCluster != VTODE(tdvp)->de_StartCluster) newparent = 1; vrele(fdvp); @@ -1114,7 +1114,7 @@ abortit: if ((fcnp->cn_flags & SAVESTART) == 0) panic("msdosfs_rename: lost from startdir"); if (!newparent) - VOP_UNLOCK(tdvp, p); + VOP_UNLOCK(tdvp); (void) vfs_relookup(fdvp, &fvp, fcnp); if (fvp == NULL) { /* @@ -1124,7 +1124,7 @@ abortit: panic("rename: lost dir entry"); vrele(ap->a_fvp); if (newparent) - VOP_UNLOCK(tdvp, p); + VOP_UNLOCK(tdvp); vrele(tdvp); return 0; } @@ -1145,7 +1145,7 @@ abortit: panic("rename: lost dir entry"); vrele(ap->a_fvp); if (newparent) - VOP_UNLOCK(fdvp, p); + VOP_UNLOCK(fdvp); xp = NULL; } else { vrele(fvp); @@ -1167,7 +1167,7 @@ abortit: if (error) { bcopy(oldname, ip->de_Name, 11); if (newparent) - VOP_UNLOCK(fdvp, p); + VOP_UNLOCK(fdvp); goto bad; } ip->de_refcnt++; @@ -1175,7 +1175,7 @@ abortit: if ((error = removede(zp, ip)) != 0) { /* XXX should really panic here, fs is corrupt */ if (newparent) - VOP_UNLOCK(fdvp, p); + VOP_UNLOCK(fdvp); goto bad; } @@ -1187,7 +1187,7 @@ abortit: if (error) { /* XXX should really panic here, fs is corrupt */ if (newparent) - VOP_UNLOCK(fdvp, p); + VOP_UNLOCK(fdvp); goto bad; } ip->de_diroffset = to_diroffset; @@ -1196,7 +1196,7 @@ abortit: } reinsert(ip); if (newparent) - VOP_UNLOCK(fdvp, p); + VOP_UNLOCK(fdvp); } /* @@ -1235,7 +1235,7 @@ abortit: VN_KNOTE(fvp, NOTE_RENAME); bad: - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); vrele(fdvp); bad1: if (xp) diff --git a/sys/nfs/nfs_node.c b/sys/nfs/nfs_node.c index 74b05b6f387..40383760a08 100644 --- a/sys/nfs/nfs_node.c +++ b/sys/nfs/nfs_node.c @@ -1,4 +1,4 @@ -/* $OpenBSD: nfs_node.c,v 1.67 2018/04/09 09:39:53 mpi Exp $ */ +/* $OpenBSD: nfs_node.c,v 1.68 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: nfs_node.c,v 1.16 1996/02/18 11:53:42 fvdl Exp $ */ /* @@ -190,7 +190,7 @@ nfs_inactive(void *v) } np->n_flag &= (NMODIFIED | NFLUSHINPROG | NFLUSHWANT); - VOP_UNLOCK(ap->a_vp, ap->a_p); + VOP_UNLOCK(ap->a_vp); return (0); } diff --git a/sys/nfs/nfs_serv.c b/sys/nfs/nfs_serv.c index ee4188b2604..0c72f1c00c5 100644 --- a/sys/nfs/nfs_serv.c +++ b/sys/nfs/nfs_serv.c @@ -1,4 +1,4 @@ -/* $OpenBSD: nfs_serv.c,v 1.112 2017/12/30 20:47:00 guenther Exp $ */ +/* $OpenBSD: nfs_serv.c,v 1.113 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: nfs_serv.c,v 1.34 1997/05/12 23:37:12 fvdl Exp $ */ /* @@ -2105,7 +2105,7 @@ nfsrv_readdir(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp, error = 0; goto nfsmout; } - VOP_UNLOCK(vp, procp); + VOP_UNLOCK(vp); rbuf = malloc(fullsiz, M_TEMP, M_WAITOK); again: iv.iov_base = rbuf; @@ -2129,7 +2129,7 @@ again: error = getret; } - VOP_UNLOCK(vp, procp); + VOP_UNLOCK(vp); if (error) { vrele(vp); free(rbuf, M_TEMP, fullsiz); @@ -2304,7 +2304,7 @@ nfsrv_readdirplus(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp, error = 0; goto nfsmout; } - VOP_UNLOCK(vp, procp); + VOP_UNLOCK(vp); rbuf = malloc(fullsiz, M_TEMP, M_WAITOK); again: @@ -2325,7 +2325,7 @@ again: off = (u_quad_t)io.uio_offset; getret = VOP_GETATTR(vp, &at, cred, procp); - VOP_UNLOCK(vp, procp); + VOP_UNLOCK(vp); if (!error) error = getret; diff --git a/sys/nfs/nfs_subs.c b/sys/nfs/nfs_subs.c index 65bb908ab52..6b141a1b4f4 100644 --- a/sys/nfs/nfs_subs.c +++ b/sys/nfs/nfs_subs.c @@ -1,4 +1,4 @@ -/* $OpenBSD: nfs_subs.c,v 1.135 2017/08/29 02:51:27 deraadt Exp $ */ +/* $OpenBSD: nfs_subs.c,v 1.136 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: nfs_subs.c,v 1.27.4.3 1996/07/08 20:34:24 jtc Exp $ */ /* @@ -1432,7 +1432,6 @@ nfsrv_fhtovp(fhandle_t *fhp, int lockflag, struct vnode **vpp, struct ucred *cred, struct nfssvc_sock *slp, struct mbuf *nam, int *rdonlyp) { - struct proc *p = curproc; /* XXX */ struct mount *mp; int i; struct ucred *credanon; @@ -1472,7 +1471,7 @@ nfsrv_fhtovp(fhandle_t *fhp, int lockflag, struct vnode **vpp, else *rdonlyp = 0; if (!lockflag) - VOP_UNLOCK(*vpp, p); + VOP_UNLOCK(*vpp); return (0); } diff --git a/sys/nfs/nfs_vfsops.c b/sys/nfs/nfs_vfsops.c index bd1074d76c2..628a381f3cc 100644 --- a/sys/nfs/nfs_vfsops.c +++ b/sys/nfs/nfs_vfsops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: nfs_vfsops.c,v 1.117 2018/04/09 09:39:53 mpi Exp $ */ +/* $OpenBSD: nfs_vfsops.c,v 1.118 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: nfs_vfsops.c,v 1.46.4.1 1996/05/25 22:40:35 fvdl Exp $ */ /* @@ -698,7 +698,7 @@ mountnfs(struct nfs_args *argp, struct mount *mp, struct mbuf *nam, if (vp->v_type == VNON) vp->v_type = VDIR; vp->v_flag = VROOT; - VOP_UNLOCK(vp, curproc); + VOP_UNLOCK(vp); *vpp = vp; return (0); diff --git a/sys/nfs/nfs_vnops.c b/sys/nfs/nfs_vnops.c index aacaed26dac..90ecf936bef 100644 --- a/sys/nfs/nfs_vnops.c +++ b/sys/nfs/nfs_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: nfs_vnops.c,v 1.173 2018/04/25 15:53:08 visa Exp $ */ +/* $OpenBSD: nfs_vnops.c,v 1.174 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: nfs_vnops.c,v 1.62.4.1 1996/07/08 20:26:52 jtc Exp $ */ /* @@ -838,7 +838,7 @@ nfs_lookup(void *v) cnp->cn_flags |= SAVENAME; if ((!lockparent || !(flags & ISLASTCN)) && newvp != dvp) { - VOP_UNLOCK(dvp, p); + VOP_UNLOCK(dvp); cnp->cn_flags |= PDIRUNLOCK; } return (0); @@ -897,7 +897,7 @@ dorpc: m_freem(info.nmi_mrep); cnp->cn_flags |= SAVENAME; if (!lockparent) { - VOP_UNLOCK(dvp, p); + VOP_UNLOCK(dvp); cnp->cn_flags |= PDIRUNLOCK; } return (0); @@ -918,7 +918,7 @@ dorpc: } else nfsm_loadattr(newvp, NULL); } else if (flags & ISDOTDOT) { - VOP_UNLOCK(dvp, p); + VOP_UNLOCK(dvp); cnp->cn_flags |= PDIRUNLOCK; error = nfs_nget(dvp->v_mount, fhp, fhsize, &np); @@ -958,7 +958,7 @@ dorpc: } else nfsm_loadattr(newvp, NULL); if (!lockparent || !(flags & ISLASTCN)) { - VOP_UNLOCK(dvp, p); + VOP_UNLOCK(dvp); cnp->cn_flags |= PDIRUNLOCK; } } diff --git a/sys/ntfs/ntfs_vfsops.c b/sys/ntfs/ntfs_vfsops.c index 9ad5ac1e01d..4abef47d94b 100644 --- a/sys/ntfs/ntfs_vfsops.c +++ b/sys/ntfs/ntfs_vfsops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ntfs_vfsops.c,v 1.58 2018/02/10 05:24:23 deraadt Exp $ */ +/* $OpenBSD: ntfs_vfsops.c,v 1.59 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: ntfs_vfsops.c,v 1.7 2003/04/24 07:50:19 christos Exp $ */ /*- @@ -280,7 +280,7 @@ ntfs_mountfs(struct vnode *devvp, struct mount *mp, struct ntfs_args *argsp, return (EBUSY); vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, p); error = vinvalbuf(devvp, V_SAVE, p->p_ucred, p, 0, 0); - VOP_UNLOCK(devvp, p); + VOP_UNLOCK(devvp); if (error) return (error); @@ -445,7 +445,7 @@ out: /* lock the device vnode before calling VOP_CLOSE() */ vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, p); (void)VOP_CLOSE(devvp, FREAD, NOCRED, p); - VOP_UNLOCK(devvp, p); + VOP_UNLOCK(devvp); return (error); } diff --git a/sys/ntfs/ntfs_vnops.c b/sys/ntfs/ntfs_vnops.c index b63268151ce..55edd6de2c6 100644 --- a/sys/ntfs/ntfs_vnops.c +++ b/sys/ntfs/ntfs_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ntfs_vnops.c,v 1.42 2016/09/01 08:40:39 natano Exp $ */ +/* $OpenBSD: ntfs_vnops.c,v 1.43 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: ntfs_vnops.c,v 1.6 2003/04/10 21:57:26 jdolecek Exp $ */ /* @@ -179,7 +179,6 @@ ntfs_inactive(void *v) { struct vop_inactive_args *ap = v; struct vnode *vp = ap->a_vp; - struct proc *p = ap->a_p; #ifdef NTFS_DEBUG struct ntnode *ip = VTONT(vp); #endif @@ -191,7 +190,7 @@ ntfs_inactive(void *v) vprint("ntfs_inactive: pushing active", vp); #endif - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); /* XXX since we don't support any filesystem changes * right now, nothing more needs to be done @@ -572,7 +571,7 @@ ntfs_lookup(void *v) DPRINTF("ntfs_lookup: faking .. directory in %u\n", dip->i_number); - VOP_UNLOCK(dvp, p); + VOP_UNLOCK(dvp); cnp->cn_flags |= PDIRUNLOCK; error = ntfs_ntvattrget(ntmp, dip, NTFS_A_NAME, NULL, 0, &vap); @@ -609,7 +608,7 @@ ntfs_lookup(void *v) VTONT(*ap->a_vpp)->i_number); if(!lockparent || (cnp->cn_flags & ISLASTCN) == 0) { - VOP_UNLOCK(dvp, p); + VOP_UNLOCK(dvp); cnp->cn_flags |= PDIRUNLOCK; } } diff --git a/sys/sys/vnode.h b/sys/sys/vnode.h index f113739395e..b8c508544e7 100644 --- a/sys/sys/vnode.h +++ b/sys/sys/vnode.h @@ -1,4 +1,4 @@ -/* $OpenBSD: vnode.h,v 1.143 2018/02/10 05:24:23 deraadt Exp $ */ +/* $OpenBSD: vnode.h,v 1.144 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: vnode.h,v 1.38 1996/02/29 20:59:05 cgd Exp $ */ /* @@ -514,15 +514,13 @@ int VOP_RECLAIM(struct vnode *, struct proc *); struct vop_lock_args { struct vnode *a_vp; int a_flags; - struct proc *a_p; }; -int VOP_LOCK(struct vnode *, int, struct proc *); +int VOP_LOCK(struct vnode *, int); struct vop_unlock_args { struct vnode *a_vp; - struct proc *a_p; }; -int VOP_UNLOCK(struct vnode *, struct proc *); +int VOP_UNLOCK(struct vnode *); struct vop_bmap_args { struct vnode *a_vp; diff --git a/sys/tmpfs/tmpfs_vnops.c b/sys/tmpfs/tmpfs_vnops.c index 53408ef281d..c675ec07f48 100644 --- a/sys/tmpfs/tmpfs_vnops.c +++ b/sys/tmpfs/tmpfs_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: tmpfs_vnops.c,v 1.27 2016/06/19 11:54:33 natano Exp $ */ +/* $OpenBSD: tmpfs_vnops.c,v 1.28 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: tmpfs_vnops.c,v 1.100 2012/11/05 17:27:39 dholland Exp $ */ /* @@ -182,7 +182,7 @@ tmpfs_lookup(void *v) * and thus prevents parent from disappearing. */ rw_enter_write(&pnode->tn_nlock); - VOP_UNLOCK(dvp, curproc); + VOP_UNLOCK(dvp); /* * Get a vnode of the '..' entry and re-acquire the lock. @@ -295,7 +295,7 @@ out: if ((error == 0 || error == EJUSTRETURN) && /* (1) */ *vpp != dvp && /* (2) */ (!lockparent || !lastcn)) { /* (3) */ - VOP_UNLOCK(dvp, curproc); + VOP_UNLOCK(dvp); cnp->cn_flags |= PDIRUNLOCK; } else KASSERT(VOP_ISLOCKED(dvp)); @@ -832,7 +832,7 @@ tmpfs_link(void *v) error = 0; out: pool_put(&namei_pool, cnp->cn_pnbuf); - VOP_UNLOCK(vp, curproc); + VOP_UNLOCK(vp); vput(dvp); return error; } @@ -1046,7 +1046,7 @@ tmpfs_inactive(void *v) if (vp->v_type == VREG && tmpfs_uio_cached(node)) tmpfs_uio_uncache(node); - VOP_UNLOCK(vp, curproc); + VOP_UNLOCK(vp); /* * If we are done with the node, reclaim it so that it can be reused @@ -1354,9 +1354,9 @@ tmpfs_rename(void *v) * the caller does reject rename("x/.", "y"). Go figure.) */ - VOP_UNLOCK(tdvp, curproc); + VOP_UNLOCK(tdvp); if ((tvp != NULL) && (tvp != tdvp)) - VOP_UNLOCK(tvp, curproc); + VOP_UNLOCK(tvp); vrele(fvp); if (tvp != NULL) @@ -1782,7 +1782,7 @@ fail3: if (tvp != NULL) { } fail2: vput(fvp); -fail1: VOP_UNLOCK(dvp, curproc); +fail1: VOP_UNLOCK(dvp); fail0: return error; } @@ -1885,10 +1885,10 @@ tmpfs_rename_exit(struct tmpfs_mount *tmpfs, else vrele(tvp); } - VOP_UNLOCK(tdvp, curproc); + VOP_UNLOCK(tdvp); vput(fvp); if (fdvp != tdvp) - VOP_UNLOCK(fdvp, curproc); + VOP_UNLOCK(fdvp); #if 0 /* XXX */ if (fdvp != tdvp) @@ -1912,7 +1912,7 @@ tmpfs_rename_lock_directory(struct vnode *vp, struct tmpfs_node *node) vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, curproc); if (node->tn_spec.tn_dir.tn_parent == NULL) { - VOP_UNLOCK(vp, curproc); + VOP_UNLOCK(vp); return ENOENT; } @@ -1982,7 +1982,7 @@ tmpfs_rename_genealogy(struct tmpfs_node *fdnode, struct tmpfs_node *tdnode, node = parent; } - VOP_UNLOCK(tdnode->tn_vnode, curproc); + VOP_UNLOCK(tdnode->tn_vnode); return 0; } @@ -2170,7 +2170,7 @@ fail4: if (b_vp != NULL) { } fail3: KASSERT(VOP_ISLOCKED(b_dvp) == LK_EXCLUSIVE); - VOP_UNLOCK(b_dvp, curproc); + VOP_UNLOCK(b_dvp); fail2: if (a_vp != NULL) { KASSERT(VOP_ISLOCKED(a_vp) == LK_EXCLUSIVE); @@ -2178,7 +2178,7 @@ fail2: if (a_vp != NULL) { } fail1: KASSERT(VOP_ISLOCKED(a_dvp) == LK_EXCLUSIVE); - VOP_UNLOCK(a_dvp, curproc); + VOP_UNLOCK(a_dvp); fail0: /* KASSERT(VOP_ISLOCKED(a_dvp) != LK_EXCLUSIVE); */ /* KASSERT(VOP_ISLOCKED(b_dvp) != LK_EXCLUSIVE); */ diff --git a/sys/ufs/ext2fs/ext2fs_inode.c b/sys/ufs/ext2fs/ext2fs_inode.c index 714a040d8ca..467a2ccbc56 100644 --- a/sys/ufs/ext2fs/ext2fs_inode.c +++ b/sys/ufs/ext2fs/ext2fs_inode.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ext2fs_inode.c,v 1.58 2016/03/19 12:04:16 natano Exp $ */ +/* $OpenBSD: ext2fs_inode.c,v 1.59 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: ext2fs_inode.c,v 1.24 2001/06/19 12:59:18 wiz Exp $ */ /* @@ -132,7 +132,7 @@ ext2fs_inactive(void *v) ext2fs_update(ip, 0); } out: - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); /* * If we are done with the inode, reclaim it * so that it can be reused immediately. diff --git a/sys/ufs/ext2fs/ext2fs_lookup.c b/sys/ufs/ext2fs/ext2fs_lookup.c index 6e17fdeaa03..6f90182b61b 100644 --- a/sys/ufs/ext2fs/ext2fs_lookup.c +++ b/sys/ufs/ext2fs/ext2fs_lookup.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ext2fs_lookup.c,v 1.42 2016/03/19 12:04:16 natano Exp $ */ +/* $OpenBSD: ext2fs_lookup.c,v 1.43 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: ext2fs_lookup.c,v 1.16 2000/08/03 20:29:26 thorpej Exp $ */ /* @@ -429,7 +429,7 @@ searchloop: */ cnp->cn_flags |= SAVENAME; if (!lockparent) { - VOP_UNLOCK(vdp, p); + VOP_UNLOCK(vdp); cnp->cn_flags |= PDIRUNLOCK; } return (EJUSTRETURN); @@ -512,7 +512,7 @@ found: } *vpp = tdp; if (!lockparent) { - VOP_UNLOCK(vdp, p); + VOP_UNLOCK(vdp); cnp->cn_flags |= PDIRUNLOCK; } return (0); @@ -538,7 +538,7 @@ found: *vpp = tdp; cnp->cn_flags |= SAVENAME; if (!lockparent) { - VOP_UNLOCK(vdp, p); + VOP_UNLOCK(vdp); cnp->cn_flags |= PDIRUNLOCK; } return (0); @@ -565,7 +565,7 @@ found: */ pdp = vdp; if (flags & ISDOTDOT) { - VOP_UNLOCK(pdp, p); /* race to get the inode */ + VOP_UNLOCK(pdp); /* race to get the inode */ cnp->cn_flags |= PDIRUNLOCK; if ((error = VFS_VGET(vdp->v_mount, dp->i_ino, &tdp)) != 0) { if (vn_lock(pdp, LK_EXCLUSIVE | LK_RETRY, p) == 0) @@ -587,7 +587,7 @@ found: if ((error = VFS_VGET(vdp->v_mount, dp->i_ino, &tdp)) != 0) return (error); if (!lockparent || !(flags & ISLASTCN)) { - VOP_UNLOCK(pdp, p); + VOP_UNLOCK(pdp); cnp->cn_flags |= PDIRUNLOCK; } *vpp = tdp; diff --git a/sys/ufs/ext2fs/ext2fs_vfsops.c b/sys/ufs/ext2fs/ext2fs_vfsops.c index 84c9305b1c0..754263e598d 100644 --- a/sys/ufs/ext2fs/ext2fs_vfsops.c +++ b/sys/ufs/ext2fs/ext2fs_vfsops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ext2fs_vfsops.c,v 1.104 2018/03/28 09:37:42 mpi Exp $ */ +/* $OpenBSD: ext2fs_vfsops.c,v 1.105 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: ext2fs_vfsops.c,v 1.1 1997/06/11 09:34:07 bouyer Exp $ */ /* @@ -572,7 +572,7 @@ out: brelse(bp); vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, p); (void)VOP_CLOSE(devvp, ronly ? FREAD : FREAD|FWRITE, cred, p); - VOP_UNLOCK(devvp, p); + VOP_UNLOCK(devvp); if (ump) { free(ump->um_e2fs, M_UFSMNT, sizeof *ump->um_e2fs); free(ump, M_UFSMNT, sizeof *ump); @@ -641,7 +641,7 @@ ext2fs_flushfiles(struct mount *mp, int flags, struct proc *p) */ vn_lock(ump->um_devvp, LK_EXCLUSIVE | LK_RETRY, p); error = VOP_FSYNC(ump->um_devvp, p->p_ucred, MNT_WAIT, p); - VOP_UNLOCK(ump->um_devvp, p); + VOP_UNLOCK(ump->um_devvp); return (error); } @@ -780,7 +780,7 @@ ext2fs_sync(struct mount *mp, int waitfor, int stall, vn_lock(ump->um_devvp, LK_EXCLUSIVE | LK_RETRY, p); if ((error = VOP_FSYNC(ump->um_devvp, cred, waitfor, p)) != 0) allerror = error; - VOP_UNLOCK(ump->um_devvp, p); + VOP_UNLOCK(ump->um_devvp); } /* * Write back modified superblock. diff --git a/sys/ufs/ext2fs/ext2fs_vnops.c b/sys/ufs/ext2fs/ext2fs_vnops.c index 9a795ff81b4..f40cf7d5ff1 100644 --- a/sys/ufs/ext2fs/ext2fs_vnops.c +++ b/sys/ufs/ext2fs/ext2fs_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ext2fs_vnops.c,v 1.79 2018/01/08 16:15:34 millert Exp $ */ +/* $OpenBSD: ext2fs_vnops.c,v 1.80 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: ext2fs_vnops.c,v 1.1 1997/06/11 09:34:09 bouyer Exp $ */ /* @@ -477,7 +477,7 @@ ext2fs_link(void *v) pool_put(&namei_pool, cnp->cn_pnbuf); out1: if (dvp != vp) - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); out2: vput(dvp); return (error); @@ -585,13 +585,13 @@ abortit: dp = VTOI(fdvp); ip = VTOI(fvp); if ((nlink_t)ip->i_e2fs_nlink >= LINK_MAX) { - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); error = EMLINK; goto abortit; } if ((ip->i_e2fs_flags & (EXT2_IMMUTABLE | EXT2_APPEND)) || (dp->i_e2fs_flags & EXT2_APPEND)) { - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); error = EPERM; goto abortit; } @@ -601,7 +601,7 @@ abortit: error = VOP_ACCESS(tvp, VWRITE, tcnp->cn_cred, tcnp->cn_proc); if (error) { - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); error = EACCES; goto abortit; } @@ -613,7 +613,7 @@ abortit: (fcnp->cn_flags&ISDOTDOT) || (tcnp->cn_flags & ISDOTDOT) || (ip->i_flag & IN_RENAME)) { - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); error = EINVAL; goto abortit; } @@ -641,7 +641,7 @@ abortit: ip->i_e2fs_nlink++; ip->i_flag |= IN_CHANGE; if ((error = ext2fs_update(ip, 1)) != 0) { - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); goto bad; } @@ -656,7 +656,7 @@ abortit: * call to checkpath(). */ error = VOP_ACCESS(fvp, VWRITE, tcnp->cn_cred, tcnp->cn_proc); - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); if (oldparent != dp->i_number) newparent = dp->i_number; if (doingdirectory && newparent) { diff --git a/sys/ufs/ffs/ffs_softdep.c b/sys/ufs/ffs/ffs_softdep.c index 4955e9edb76..cea2d15e1d4 100644 --- a/sys/ufs/ffs/ffs_softdep.c +++ b/sys/ufs/ffs/ffs_softdep.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ffs_softdep.c,v 1.140 2018/04/01 12:02:00 dhill Exp $ */ +/* $OpenBSD: ffs_softdep.c,v 1.141 2018/04/28 03:13:05 visa Exp $ */ /* * Copyright 1998, 2000 Marshall Kirk McKusick. All Rights Reserved. @@ -868,7 +868,7 @@ softdep_flushworklist(struct mount *oldmnt, int *countp, struct proc *p) *countp += count; vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, p); error = VOP_FSYNC(devvp, p->p_ucred, MNT_WAIT, p); - VOP_UNLOCK(devvp, p); + VOP_UNLOCK(devvp); if (error) break; } @@ -4578,7 +4578,7 @@ softdep_fsync(struct vnode *vp) * ufs_lookup for details on possible races. */ FREE_LOCK(&lk); - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); error = VFS_VGET(mnt, parentino, &pvp); vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, p); if (error != 0) diff --git a/sys/ufs/ffs/ffs_vfsops.c b/sys/ufs/ffs/ffs_vfsops.c index 50fc5ef6e59..2800d995412 100644 --- a/sys/ufs/ffs/ffs_vfsops.c +++ b/sys/ufs/ffs/ffs_vfsops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ffs_vfsops.c,v 1.174 2018/03/30 17:35:20 dhill Exp $ */ +/* $OpenBSD: ffs_vfsops.c,v 1.175 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: ffs_vfsops.c,v 1.19 1996/02/09 22:22:26 christos Exp $ */ /* @@ -564,7 +564,7 @@ ffs_reload(struct mount *mountp, struct ucred *cred, struct proc *p) devvp = VFSTOUFS(mountp)->um_devvp; vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, p); error = vinvalbuf(devvp, 0, cred, p, 0, 0); - VOP_UNLOCK(devvp, p); + VOP_UNLOCK(devvp); if (error) panic("ffs_reload: dirty1"); @@ -710,7 +710,7 @@ ffs_mountfs(struct vnode *devvp, struct mount *mp, struct proc *p) return (EBUSY); vn_lock(devvp, LK_EXCLUSIVE | LK_RETRY, p); error = vinvalbuf(devvp, V_SAVE, cred, p, 0, 0); - VOP_UNLOCK(devvp, p); + VOP_UNLOCK(devvp); if (error) return (error); @@ -933,7 +933,7 @@ out: vn_lock(devvp, LK_EXCLUSIVE|LK_RETRY, p); (void)VOP_CLOSE(devvp, ronly ? FREAD : FREAD|FWRITE, cred, p); - VOP_UNLOCK(devvp, p); + VOP_UNLOCK(devvp); if (ump) { free(ump->um_fs, M_UFSMNT, ump->um_fs->fs_sbsize); @@ -1098,7 +1098,7 @@ ffs_flushfiles(struct mount *mp, int flags, struct proc *p) */ vn_lock(ump->um_devvp, LK_EXCLUSIVE | LK_RETRY, p); error = VOP_FSYNC(ump->um_devvp, p->p_ucred, MNT_WAIT, p); - VOP_UNLOCK(ump->um_devvp, p); + VOP_UNLOCK(ump->um_devvp); return (error); } @@ -1186,7 +1186,7 @@ ffs_sync_vnode(struct vnode *vp, void *arg) if ((error = VOP_FSYNC(vp, fsa->cred, fsa->waitfor, fsa->p))) fsa->allerror = error; - VOP_UNLOCK(vp, fsa->p); + VOP_UNLOCK(vp); vrele(vp); end: @@ -1252,7 +1252,7 @@ ffs_sync(struct mount *mp, int waitfor, int stall, struct ucred *cred, struct pr vn_lock(ump->um_devvp, LK_EXCLUSIVE | LK_RETRY, p); if ((error = VOP_FSYNC(ump->um_devvp, cred, waitfor, p)) != 0) allerror = error; - VOP_UNLOCK(ump->um_devvp, p); + VOP_UNLOCK(ump->um_devvp); } qsync(mp); /* diff --git a/sys/ufs/mfs/mfs_vnops.c b/sys/ufs/mfs/mfs_vnops.c index 2895d74b473..f1e995f5c09 100644 --- a/sys/ufs/mfs/mfs_vnops.c +++ b/sys/ufs/mfs/mfs_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: mfs_vnops.c,v 1.50 2018/04/02 14:23:04 dhill Exp $ */ +/* $OpenBSD: mfs_vnops.c,v 1.51 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: mfs_vnops.c,v 1.8 1996/03/17 02:16:32 christos Exp $ */ /* @@ -225,7 +225,7 @@ mfs_inactive(void *v) if (mfsp->mfs_shutdown && bufq_peek(&mfsp->mfs_bufq)) panic("mfs_inactive: not inactive"); #endif - VOP_UNLOCK(ap->a_vp, ap->a_p); + VOP_UNLOCK(ap->a_vp); return (0); } diff --git a/sys/ufs/ufs/ufs_inode.c b/sys/ufs/ufs/ufs_inode.c index 7f8688913fb..7ac354e9316 100644 --- a/sys/ufs/ufs/ufs_inode.c +++ b/sys/ufs/ufs/ufs_inode.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ufs_inode.c,v 1.42 2016/03/19 12:04:16 natano Exp $ */ +/* $OpenBSD: ufs_inode.c,v 1.43 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: ufs_inode.c,v 1.7 1996/05/11 18:27:52 mycroft Exp $ */ /* @@ -111,7 +111,7 @@ ufs_inactive(void *v) UFS_UPDATE(ip, 0); } out: - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); /* * If we are done with the inode, reclaim it diff --git a/sys/ufs/ufs/ufs_lookup.c b/sys/ufs/ufs/ufs_lookup.c index be26d52e781..e27b0cb600e 100644 --- a/sys/ufs/ufs/ufs_lookup.c +++ b/sys/ufs/ufs/ufs_lookup.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ufs_lookup.c,v 1.50 2016/09/10 16:53:30 natano Exp $ */ +/* $OpenBSD: ufs_lookup.c,v 1.51 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: ufs_lookup.c,v 1.7 1996/02/09 22:36:06 christos Exp $ */ /* @@ -422,7 +422,7 @@ notfound: */ cnp->cn_flags |= SAVENAME; if (!lockparent) { - VOP_UNLOCK(vdp, p); + VOP_UNLOCK(vdp); cnp->cn_flags |= PDIRUNLOCK; } return (EJUSTRETURN); @@ -504,7 +504,7 @@ found: } *vpp = tdp; if (!lockparent) { - VOP_UNLOCK(vdp, p); + VOP_UNLOCK(vdp); cnp->cn_flags |= PDIRUNLOCK; } return (0); @@ -533,7 +533,7 @@ found: *vpp = tdp; cnp->cn_flags |= SAVENAME; if (!lockparent) { - VOP_UNLOCK(vdp, p); + VOP_UNLOCK(vdp); cnp->cn_flags |= PDIRUNLOCK; } return (0); @@ -560,7 +560,7 @@ found: */ pdp = vdp; if (flags & ISDOTDOT) { - VOP_UNLOCK(pdp, p); /* race to get the inode */ + VOP_UNLOCK(pdp); /* race to get the inode */ cnp->cn_flags |= PDIRUNLOCK; error = VFS_VGET(vdp->v_mount, dp->i_ino, &tdp); if (error) { @@ -584,7 +584,7 @@ found: if (error) return (error); if (!lockparent || !(flags & ISLASTCN)) { - VOP_UNLOCK(pdp, p); + VOP_UNLOCK(pdp); cnp->cn_flags |= PDIRUNLOCK; } *vpp = tdp; @@ -779,7 +779,7 @@ ufs_direnter(struct vnode *dvp, struct vnode *tvp, struct direct *dirp, if ((error = VOP_BWRITE(bp))) return (error); if (tvp != NULL) - VOP_UNLOCK(tvp, p); + VOP_UNLOCK(tvp); error = VOP_FSYNC(dvp, p->p_ucred, MNT_WAIT, p); if (tvp != NULL) vn_lock(tvp, LK_EXCLUSIVE | LK_RETRY, p); @@ -919,7 +919,7 @@ ufs_direnter(struct vnode *dvp, struct vnode *tvp, struct direct *dirp, if (error == 0 && dp->i_endoff && dp->i_endoff < DIP(dp, size)) { if (tvp != NULL) - VOP_UNLOCK(tvp, p); + VOP_UNLOCK(tvp); error = UFS_TRUNCATE(dp, (off_t)dp->i_endoff, IO_SYNC, cr); #ifdef UFS_DIRHASH if (error == 0 && dp->i_dirhash != NULL) diff --git a/sys/ufs/ufs/ufs_quota.c b/sys/ufs/ufs/ufs_quota.c index 54c68a53f06..37c3af267bd 100644 --- a/sys/ufs/ufs/ufs_quota.c +++ b/sys/ufs/ufs/ufs_quota.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ufs_quota.c,v 1.41 2018/02/19 08:59:53 mpi Exp $ */ +/* $OpenBSD: ufs_quota.c,v 1.42 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: ufs_quota.c,v 1.8 1996/02/09 22:36:09 christos Exp $ */ /* @@ -499,7 +499,7 @@ quotaon(struct proc *p, struct mount *mp, int type, caddr_t fname) if ((error = vn_open(&nd, FREAD|FWRITE, 0)) != 0) return (error); vp = nd.ni_vp; - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); if (vp->v_type != VREG) { (void) vn_close(vp, FREAD|FWRITE, p->p_ucred, p); return (EACCES); @@ -921,7 +921,7 @@ dqget(struct vnode *vp, u_long id, struct ufsmount *ump, int type, if (auio.uio_resid == sizeof(struct dqblk) && error == 0) memset(&dq->dq_dqb, 0, sizeof(struct dqblk)); if (vp != dqvp) - VOP_UNLOCK(dqvp, p); + VOP_UNLOCK(dqvp); if (dq->dq_flags & DQ_WANT) wakeup(dq); dq->dq_flags = 0; @@ -998,7 +998,7 @@ dqsync(struct vnode *vp, struct dquot *dq) (void) tsleep(dq, PINOD+2, "dqsync", 0); if ((dq->dq_flags & DQ_MOD) == 0) { if (vp != dqvp) - VOP_UNLOCK(dqvp, p); + VOP_UNLOCK(dqvp); return (0); } } @@ -1019,7 +1019,7 @@ dqsync(struct vnode *vp, struct dquot *dq) wakeup(dq); dq->dq_flags &= ~(DQ_MOD|DQ_LOCK|DQ_WANT); if (vp != dqvp) - VOP_UNLOCK(dqvp, p); + VOP_UNLOCK(dqvp); return (error); } diff --git a/sys/ufs/ufs/ufs_vnops.c b/sys/ufs/ufs/ufs_vnops.c index ad25d038876..4b6c210c131 100644 --- a/sys/ufs/ufs/ufs_vnops.c +++ b/sys/ufs/ufs/ufs_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ufs_vnops.c,v 1.136 2018/01/08 16:15:34 millert Exp $ */ +/* $OpenBSD: ufs_vnops.c,v 1.137 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: ufs_vnops.c,v 1.18 1996/05/11 18:28:04 mycroft Exp $ */ /* @@ -685,7 +685,7 @@ ufs_link(void *v) VN_KNOTE(dvp, NOTE_WRITE); out1: if (dvp != vp) - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); out2: vput(dvp); return (error); @@ -810,13 +810,13 @@ abortit: dp = VTOI(fdvp); ip = VTOI(fvp); if ((nlink_t) DIP(ip, nlink) >= LINK_MAX) { - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); error = EMLINK; goto abortit; } if ((DIP(ip, flags) & (IMMUTABLE | APPEND)) || (DIP(dp, flags) & APPEND)) { - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); error = EPERM; goto abortit; } @@ -825,7 +825,7 @@ abortit: if (!error && tvp) error = VOP_ACCESS(tvp, VWRITE, tcnp->cn_cred, tcnp->cn_proc); if (error) { - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); error = EACCES; goto abortit; } @@ -837,7 +837,7 @@ abortit: (fcnp->cn_flags & ISDOTDOT) || (tcnp->cn_flags & ISDOTDOT) || (ip->i_flag & IN_RENAME)) { - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); error = EINVAL; goto abortit; } @@ -868,7 +868,7 @@ abortit: if (DOINGSOFTDEP(fvp)) softdep_change_linkcnt(ip, 0); if ((error = UFS_UPDATE(ip, !DOINGSOFTDEP(fvp))) != 0) { - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); goto bad; } @@ -883,7 +883,7 @@ abortit: * call to checkpath(). */ error = VOP_ACCESS(fvp, VWRITE, tcnp->cn_cred, tcnp->cn_proc); - VOP_UNLOCK(fvp, p); + VOP_UNLOCK(fvp); /* tdvp and tvp locked */ if (oldparent != dp->i_number) diff --git a/sys/uvm/uvm_vnode.c b/sys/uvm/uvm_vnode.c index cc1a566a5ce..c626b4082dc 100644 --- a/sys/uvm/uvm_vnode.c +++ b/sys/uvm/uvm_vnode.c @@ -1,4 +1,4 @@ -/* $OpenBSD: uvm_vnode.c,v 1.100 2018/03/30 11:22:58 mpi Exp $ */ +/* $OpenBSD: uvm_vnode.c,v 1.101 2018/04/28 03:13:05 visa Exp $ */ /* $NetBSD: uvm_vnode.c,v 1.36 2000/11/24 20:34:01 chs Exp $ */ /* @@ -1197,7 +1197,7 @@ uvn_io(struct uvm_vnode *uvn, vm_page_t *pps, int npages, int flags, int rw) curproc->p_ucred); if ((uvn->u_flags & UVM_VNODE_VNISLOCKED) == 0) - VOP_UNLOCK(vn, curproc); + VOP_UNLOCK(vn); } @@ -1323,7 +1323,7 @@ uvm_vnp_uncache(struct vnode *vp) * unlocked causing us to return TRUE when we should not. we ignore * this as a false-positive return value doesn't hurt us. */ - VOP_UNLOCK(vp, curproc); + VOP_UNLOCK(vp); uvn_detach(&uvn->u_obj); vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, curproc); |