Module Name: src
Committed By: riastradh
Date: Tue Apr 21 03:19:03 UTC 2015
Modified Files:
src/sys/kern: vfs_lookup.c vfs_syscalls.c
src/sys/nfs: nfs_serv.c
src/sys/rump/include/rump: rump_namei.h
src/sys/sys: namei.h
Log Message:
Cull unused INRENAME and INRELOOKUP from callers.
To generate a diff of this commit:
cvs rdiff -u -r1.201 -r1.202 src/sys/kern/vfs_lookup.c
cvs rdiff -u -r1.496 -r1.497 src/sys/kern/vfs_syscalls.c
cvs rdiff -u -r1.171 -r1.172 src/sys/nfs/nfs_serv.c
cvs rdiff -u -r1.26 -r1.27 src/sys/rump/include/rump/rump_namei.h
cvs rdiff -u -r1.92 -r1.93 src/sys/sys/namei.h
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/sys/kern/vfs_lookup.c
diff -u src/sys/kern/vfs_lookup.c:1.201 src/sys/kern/vfs_lookup.c:1.202
--- src/sys/kern/vfs_lookup.c:1.201 Fri Feb 7 15:29:22 2014
+++ src/sys/kern/vfs_lookup.c Tue Apr 21 03:19:03 2015
@@ -1,4 +1,4 @@
-/* $NetBSD: vfs_lookup.c,v 1.201 2014/02/07 15:29:22 hannken Exp $ */
+/* $NetBSD: vfs_lookup.c,v 1.202 2015/04/21 03:19:03 riastradh Exp $ */
/*
* Copyright (c) 1982, 1986, 1989, 1993
@@ -37,7 +37,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: vfs_lookup.c,v 1.201 2014/02/07 15:29:22 hannken Exp $");
+__KERNEL_RCSID(0, "$NetBSD: vfs_lookup.c,v 1.202 2015/04/21 03:19:03 riastradh Exp $");
#include "opt_magiclinks.h"
@@ -482,9 +482,9 @@ struct namei_state {
static void
namei_init(struct namei_state *state, struct nameidata *ndp)
{
+
state->ndp = ndp;
state->cnp = &ndp->ni_cnd;
- KASSERT((state->cnp->cn_flags & INRELOOKUP) == 0);
state->docache = 0;
state->rdonly = 0;
@@ -1736,9 +1736,7 @@ relookup(struct vnode *dvp, struct vnode
* We now have a segment name to search for, and a directory to search.
*/
*vpp = NULL;
- cnp->cn_flags |= INRELOOKUP;
error = VOP_LOOKUP(dvp, vpp, cnp);
- cnp->cn_flags &= ~INRELOOKUP;
if ((error) != 0) {
#ifdef DIAGNOSTIC
if (*vpp != NULL)
Index: src/sys/kern/vfs_syscalls.c
diff -u src/sys/kern/vfs_syscalls.c:1.496 src/sys/kern/vfs_syscalls.c:1.497
--- src/sys/kern/vfs_syscalls.c:1.496 Mon Apr 20 23:03:08 2015
+++ src/sys/kern/vfs_syscalls.c Tue Apr 21 03:19:03 2015
@@ -1,4 +1,4 @@
-/* $NetBSD: vfs_syscalls.c,v 1.496 2015/04/20 23:03:08 riastradh Exp $ */
+/* $NetBSD: vfs_syscalls.c,v 1.497 2015/04/21 03:19:03 riastradh Exp $ */
/*-
* Copyright (c) 2008, 2009 The NetBSD Foundation, Inc.
@@ -70,7 +70,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: vfs_syscalls.c,v 1.496 2015/04/20 23:03:08 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: vfs_syscalls.c,v 1.497 2015/04/21 03:19:03 riastradh Exp $");
#ifdef _KERNEL_OPT
#include "opt_fileassoc.h"
@@ -4198,7 +4198,7 @@ do_sys_renameat(struct lwp *l, int fromf
* locked yet, but (a) namei is insane, and (b) VOP_RENAME is
* insane, so for the time being we need to leave it like this.
*/
- NDINIT(&fnd, DELETE, (LOCKPARENT | TRYEMULROOT | INRENAME), fpb);
+ NDINIT(&fnd, DELETE, (LOCKPARENT | TRYEMULROOT), fpb);
if ((error = fd_nameiat(l, fromfd, &fnd)) != 0)
goto out2;
@@ -4251,7 +4251,7 @@ do_sys_renameat(struct lwp *l, int fromf
* XXX Why not pass CREATEDIR always?
*/
NDINIT(&tnd, RENAME,
- (LOCKPARENT | NOCACHE | TRYEMULROOT | INRENAME |
+ (LOCKPARENT | NOCACHE | TRYEMULROOT |
((fvp->v_type == VDIR)? CREATEDIR : 0)),
tpb);
if ((error = fd_nameiat(l, tofd, &tnd)) != 0)
Index: src/sys/nfs/nfs_serv.c
diff -u src/sys/nfs/nfs_serv.c:1.171 src/sys/nfs/nfs_serv.c:1.172
--- src/sys/nfs/nfs_serv.c:1.171 Mon Apr 20 23:03:09 2015
+++ src/sys/nfs/nfs_serv.c Tue Apr 21 03:19:03 2015
@@ -1,4 +1,4 @@
-/* $NetBSD: nfs_serv.c,v 1.171 2015/04/20 23:03:09 riastradh Exp $ */
+/* $NetBSD: nfs_serv.c,v 1.172 2015/04/21 03:19:03 riastradh Exp $ */
/*
* Copyright (c) 1989, 1993
@@ -55,7 +55,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: nfs_serv.c,v 1.171 2015/04/20 23:03:09 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: nfs_serv.c,v 1.172 2015/04/21 03:19:03 riastradh Exp $");
#include <sys/param.h>
#include <sys/systm.h>
@@ -1930,7 +1930,7 @@ nfsrv_rename(struct nfsrv_descript *nfsd
saved_uid = kauth_cred_geteuid(cred);
fromnd.ni_cnd.cn_cred = cred;
fromnd.ni_cnd.cn_nameiop = DELETE;
- fromnd.ni_cnd.cn_flags = LOCKPARENT | INRENAME;
+ fromnd.ni_cnd.cn_flags = LOCKPARENT;
error = nfs_namei(&fromnd, &fnsfh, len, slp, nam, &md,
&dpos, &fdirp, lwp, (nfsd->nd_flag & ND_KERBAUTH), false);
if (error == 0 && fdirp && v3) {
@@ -2006,7 +2006,7 @@ nfsrv_rename(struct nfsrv_descript *nfsd
kauth_cred_seteuid(cred, saved_uid);
tond.ni_cnd.cn_cred = cred;
tond.ni_cnd.cn_nameiop = RENAME;
- tond.ni_cnd.cn_flags = LOCKPARENT | LOCKLEAF | NOCACHE | INRENAME;
+ tond.ni_cnd.cn_flags = LOCKPARENT | LOCKLEAF | NOCACHE;
error = nfs_namei(&tond, &tnsfh, len2, slp, nam, &md,
&dpos, &tdirp, lwp, (nfsd->nd_flag & ND_KERBAUTH), false);
if (tdirp && v3) {
Index: src/sys/rump/include/rump/rump_namei.h
diff -u src/sys/rump/include/rump/rump_namei.h:1.26 src/sys/rump/include/rump/rump_namei.h:1.27
--- src/sys/rump/include/rump/rump_namei.h:1.26 Fri Jan 16 21:41:07 2015
+++ src/sys/rump/include/rump/rump_namei.h Tue Apr 21 03:19:03 2015
@@ -1,11 +1,11 @@
-/* $NetBSD: rump_namei.h,v 1.26 2015/01/16 21:41:07 dennis Exp $ */
+/* $NetBSD: rump_namei.h,v 1.27 2015/04/21 03:19:03 riastradh Exp $ */
/*
* WARNING: GENERATED FILE. DO NOT EDIT
* (edit namei.src and run make namei in src/sys/sys)
* by: NetBSD: gennameih.awk,v 1.5 2009/12/23 14:17:19 pooka Exp
- * from: NetBSD: namei.src,v 1.35 2015/01/16 21:38:26 dennis Exp
+ * from: NetBSD: namei.src,v 1.37 2015/04/21 03:18:21 riastradh Exp
*/
#ifndef _RUMP_RUMP_NAMEI_H_
@@ -34,8 +34,6 @@
#define RUMP_NAMEI_DOWHITEOUT 0x0040000
#define RUMP_NAMEI_REQUIREDIR 0x0080000
#define RUMP_NAMEI_CREATEDIR 0x0200000
-#define RUMP_NAMEI_INRENAME 0x0400000
-#define RUMP_NAMEI_INRELOOKUP 0x0800000
-#define RUMP_NAMEI_PARAMASK 0x0eee300
+#define RUMP_NAMEI_PARAMASK 0x02ee300
#endif /* _RUMP_RUMP_NAMEI_H_ */
Index: src/sys/sys/namei.h
diff -u src/sys/sys/namei.h:1.92 src/sys/sys/namei.h:1.93
--- src/sys/sys/namei.h:1.92 Fri Jan 16 21:41:07 2015
+++ src/sys/sys/namei.h Tue Apr 21 03:19:03 2015
@@ -1,11 +1,11 @@
-/* $NetBSD: namei.h,v 1.92 2015/01/16 21:41:07 dennis Exp $ */
+/* $NetBSD: namei.h,v 1.93 2015/04/21 03:19:03 riastradh Exp $ */
/*
* WARNING: GENERATED FILE. DO NOT EDIT
* (edit namei.src and run make namei in src/sys/sys)
* by: NetBSD: gennameih.awk,v 1.5 2009/12/23 14:17:19 pooka Exp
- * from: NetBSD: namei.src,v 1.35 2015/01/16 21:38:26 dennis Exp
+ * from: NetBSD: namei.src,v 1.37 2015/04/21 03:18:21 riastradh Exp
*/
/*
@@ -173,9 +173,7 @@ struct nameidata {
#define DOWHITEOUT 0x0040000 /* do whiteouts */
#define REQUIREDIR 0x0080000 /* must be a directory */
#define CREATEDIR 0x0200000 /* trailing slashes are ok */
-#define INRENAME 0x0400000 /* operation is a part of ``rename'' */
-#define INRELOOKUP 0x0800000 /* set while inside relookup() */
-#define PARAMASK 0x0eee300 /* mask of parameter descriptors */
+#define PARAMASK 0x02ee300 /* mask of parameter descriptors */
/*
* Initialization of a nameidata structure.
@@ -354,8 +352,6 @@ struct nchstats _NAMEI_CACHE_STATS(uint6
#define NAMEI_DOWHITEOUT 0x0040000
#define NAMEI_REQUIREDIR 0x0080000
#define NAMEI_CREATEDIR 0x0200000
-#define NAMEI_INRENAME 0x0400000
-#define NAMEI_INRELOOKUP 0x0800000
-#define NAMEI_PARAMASK 0x0eee300
+#define NAMEI_PARAMASK 0x02ee300
#endif /* !_SYS_NAMEI_H_ */