From: Al Viro <viro@zeniv.linux.org.uk>
Date: Tue, 10 Mar 2020 14:09:26 +0000 (-0400)
Subject: open_last_lookups(): don't abuse complete_walk() when all we want is unlazy
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=72287417abd16c42f1d1ea8d93ee60ba08022fc1;p=linux.git

open_last_lookups(): don't abuse complete_walk() when all we want is unlazy

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
---

diff --git a/fs/namei.c b/fs/namei.c
index 6cb41608c47c9..9f2d95aa2502d 100644
--- a/fs/namei.c
+++ b/fs/namei.c
@@ -3147,15 +3147,11 @@ static const char *open_last_lookups(struct nameidata *nd,
 		BUG_ON(nd->flags & LOOKUP_RCU);
 	} else {
 		/* create side of things */
-		/*
-		 * This will *only* deal with leaving RCU mode - LOOKUP_JUMPED
-		 * has been cleared when we got to the last component we are
-		 * about to look up
-		 */
-		error = complete_walk(nd);
-		if (unlikely(error))
-			return ERR_PTR(error);
-
+		if (nd->flags & LOOKUP_RCU) {
+			error = unlazy_walk(nd);
+			if (unlikely(error))
+				return ERR_PTR(error);
+		}
 		audit_inode(nd->name, dir, AUDIT_INODE_PARENT);
 		/* trailing slashes? */
 		if (unlikely(nd->last.name[nd->last.len]))