From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 9C4CA106ACEF for ; Thu, 12 Mar 2026 21:58:00 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 0540C6B00E1; Thu, 12 Mar 2026 17:58:00 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 03EBC6B00E3; Thu, 12 Mar 2026 17:57:59 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id E58ED6B00E4; Thu, 12 Mar 2026 17:57:59 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id D5D6B6B00E1 for ; Thu, 12 Mar 2026 17:57:59 -0400 (EDT) Received: from smtpin22.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id 773E713AC68 for ; Thu, 12 Mar 2026 21:57:59 +0000 (UTC) X-FDA: 84538774278.22.B8DA40C Received: from flow-b6-smtp.messagingengine.com (flow-b6-smtp.messagingengine.com [202.12.124.141]) by imf08.hostedemail.com (Postfix) with ESMTP id 8214A160006 for ; Thu, 12 Mar 2026 21:57:57 +0000 (UTC) Authentication-Results: imf08.hostedemail.com; dkim=pass header.d=ownmail.net header.s=fm1 header.b=OmN1n9vT; dkim=pass header.d=messagingengine.com header.s=fm1 header.b=DUkArunM; spf=pass (imf08.hostedemail.com: domain of neilb@ownmail.net designates 202.12.124.141 as permitted sender) smtp.mailfrom=neilb@ownmail.net; dmarc=pass (policy=none) header.from=ownmail.net ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1773352677; h=from:from:sender:reply-to:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=tG1sAZ33EfRbToxRaM5utlCqwlH2sGHBZL6Q+fSZ/kg=; b=VU6Ghwu0nNxcxfrvX43a0yXoyQjrBhCOsLGoT4h4spDXeng3H9SX5pGIFiYPbk7dkZn8yq JrYFHgP8LzjW2CKUgLIpVl1u4gDsOapLtJTLNbbYADVMzQ84+x5EW/kbfb505NYId575R9 QHVA27zZ3NGWb6jBFjM6GctP/Mzu6B4= ARC-Authentication-Results: i=1; imf08.hostedemail.com; dkim=pass header.d=ownmail.net header.s=fm1 header.b=OmN1n9vT; dkim=pass header.d=messagingengine.com header.s=fm1 header.b=DUkArunM; spf=pass (imf08.hostedemail.com: domain of neilb@ownmail.net designates 202.12.124.141 as permitted sender) smtp.mailfrom=neilb@ownmail.net; dmarc=pass (policy=none) header.from=ownmail.net ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1773352677; a=rsa-sha256; cv=none; b=ZoUtSWGjlGJkmNJM1DLYN4mKZIitLKqQBwB5ENfFs1VpW7uf75b7KsUt4kHtUGnEBbOUVA A5+9wdNPnmjMpH+dgJ2ByZ/edA5d4uRs6qNuV5j2Ri4vup8c7NaE1mL15xm61g5Kzjvu5H erDNd77TjBAwpXsPmMYcl3BG1s/QcY4= Received: from phl-compute-12.internal (phl-compute-12.internal [10.202.2.52]) by mailflow.stl.internal (Postfix) with ESMTP id 9E9601300F6C; Thu, 12 Mar 2026 17:57:55 -0400 (EDT) Received: from phl-frontend-03 ([10.202.2.162]) by phl-compute-12.internal (MEProxy); Thu, 12 Mar 2026 17:57:56 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ownmail.net; h= cc:cc:content-transfer-encoding:content-type:date:date:from:from :in-reply-to:in-reply-to:message-id:mime-version:references :reply-to:reply-to:subject:subject:to:to; s=fm1; t=1773352675; x=1773359875; bh=tG1sAZ33EfRbToxRaM5utlCqwlH2sGHBZL6Q+fSZ/kg=; b= OmN1n9vTcFKpHoHUGImFy0CQZvz9KEtReJX8p0OX7kUzjl9V1nlaH0yIJxxNJ5o5 oET4VMjoMdSHj7IP5T4w/Rg6YRHXE0F+yv/zv6JIVNaEu04OcGt8i2Z31ixKUARE DgWSGk08+PgsvbboVZfumYegJ49ceyganU4rmEo80NNQt8dBC4CP9Wg3fGyUFm/H JqPNMalfP80ve+bbpg9QT7R10mOUfPal3L+Ic8nXjpuv3ShR8ZVXFaW54uMweLe0 Uq9Qii9ZXDmV3HkZ8LfHyMvZo42uRI7668NHYPRCfw63wFbH2Rx07hB+CGfTU/+0 5h1RkHy0wyR2U7RjIeMU+g== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:date:date:feedback-id:feedback-id:from:from :in-reply-to:in-reply-to:message-id:mime-version:references :reply-to:reply-to:subject:subject:to:to:x-me-proxy:x-me-sender :x-me-sender:x-sasl-enc; s=fm1; t=1773352675; x=1773359875; bh=t G1sAZ33EfRbToxRaM5utlCqwlH2sGHBZL6Q+fSZ/kg=; b=DUkArunM0luViPOSC jYYHJCnZBtZFu1yArXIwY0hvu6BErdZBDdHUc16HB0A4qYJKkcPzwpLxrDs9bVos bzYgAA+8UttTRhFKZ0bpoi8CaHHQoxiV1ip3XmTGLLS3OhflYcg/lSPRa3aw9Esw 9Dby830jyY6S1LECOhApUa2X7BsIoOuRyilaw6sD+gyYb3+jHWgRcHxC9iAw/Z2u IRdPeUpUN5OYAHfWgBBL/r028pjWmk4Ha2AjbAGHczLxiNwGIUOrONuduvNIGDUf DAhRMe7hKq4+w9i2cCgr2MKWCq0AYV+anj+y7k2zwww+ByYy121ZFJq9XFjo8x6e Ofj3w== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeefgedrtddtgddvkeejledvucetufdoteggodetrf dotffvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfurfetoffkrfgpnffqhgenuceu rghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmnecujf gurhephffvvefufffkofgjfhhrggfgsedtkeertdertddtnecuhfhrohhmpefpvghilheu rhhofihnuceonhgvihhlsgesohifnhhmrghilhdrnhgvtheqnecuggftrfgrthhtvghrnh epveevkeffudeuvefhieeghffgudektdelkeejiedtjedugfeukedvkeffvdefvddunecu vehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhepnhgvihhlsg esohifnhhmrghilhdrnhgvthdpnhgspghrtghpthhtohephedupdhmohguvgepshhmthhp ohhuthdprhgtphhtthhopehvihhrohesiigvnhhivhdrlhhinhhugidrohhrghdruhhkpd hrtghpthhtoheplhhinhhugidqgihfshesvhhgvghrrdhkvghrnhgvlhdrohhrghdprhgt phhtthhopehlihhnuhigqdhunhhiohhnfhhssehvghgvrhdrkhgvrhhnvghlrdhorhhgpd hrtghpthhtoheplhhinhhugidqthhrrggtvgdqkhgvrhhnvghlsehvghgvrhdrkhgvrhhn vghlrdhorhhgpdhrtghpthhtoheplhhinhhugidqnhhfshesvhhgvghrrdhkvghrnhgvlh drohhrghdprhgtphhtthhopehlihhnuhigqdhkvghrnhgvlhesvhhgvghrrdhkvghrnhgv lhdrohhrghdprhgtphhtthhopehlihhnuhigqdhfshguvghvvghlsehvghgvrhdrkhgvrh hnvghlrdhorhhgpdhrtghpthhtoheplhhinhhugidqvgigthegsehvghgvrhdrkhgvrhhn vghlrdhorhhgpdhrtghpthhtoheplhhinhhugidqvghfihesvhhgvghrrdhkvghrnhgvlh drohhrgh X-ME-Proxy: Feedback-ID: i9d664b8f:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Thu, 12 Mar 2026 17:57:42 -0400 (EDT) From: NeilBrown To: Linus Torvalds , Alexander Viro , Christian Brauner , Jan Kara , Jeff Layton , Trond Myklebust , Anna Schumaker , Carlos Maiolino , Miklos Szeredi , Amir Goldstein , Jan Harkes , Hugh Dickins , Baolin Wang , David Howells , Marc Dionne , Steve French , Namjae Jeon , Sungjong Seo , Yuezhang Mo , Andreas Hindborg , Breno Leitao , "Theodore Ts'o" , Andreas Dilger , Steven Rostedt , Masami Hiramatsu , Ilya Dryomov , Alex Markuze , Viacheslav Dubeyko , Tyler Hicks , Andreas Gruenbacher , Richard Weinberger , Anton Ivanov , Johannes Berg , Jeremy Kerr , Ard Biesheuvel Cc: linux-fsdevel@vger.kernel.org, linux-nfs@vger.kernel.org, linux-xfs@vger.kernel.org, linux-unionfs@vger.kernel.org, coda@cs.cmu.edu, linux-mm@kvack.org, linux-afs@lists.infradead.org, linux-cifs@vger.kernel.org, linux-ext4@vger.kernel.org, linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org, ceph-devel@vger.kernel.org, ecryptfs@vger.kernel.org, gfs2@lists.linux.dev, linux-um@lists.infradead.org, linux-efi@vger.kernel.org Subject: [PATCH 50/53] VFS: remove lookup_one() and lookup_noperm() Date: Fri, 13 Mar 2026 08:12:37 +1100 Message-ID: <20260312214330.3885211-51-neilb@ownmail.net> X-Mailer: git-send-email 2.50.0.107.gf914562f5916.dirty In-Reply-To: <20260312214330.3885211-1-neilb@ownmail.net> References: <20260312214330.3885211-1-neilb@ownmail.net> Reply-To: NeilBrown MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Rspamd-Queue-Id: 8214A160006 X-Stat-Signature: jb7o6keawjmrs575m3zxio1u6f6dgbe4 X-Rspam-User: X-Rspamd-Server: rspam05 X-HE-Tag: 1773352677-954462 X-HE-Meta: U2FsdGVkX1+t3KBZASy+ARBny15YZ3p8WrHUeO5NOjq99rrKD2SLKFlBbFmEH0ayQ+Hl2IcMzXWVD8EoeeuEvGHQhHyz2FkH9Md0QbQL6Qf3E6PvA8ETlI6lPf4twte5mZG40lmQ2SGu1kHp1GqsOfjoZjNX12F8yUkZajeKDeRKQ6hDcwh2eJbdfeOMTiJhtKW6rGg52h1wWaFYqc286BjXLTbvqAuAQYsv1rwKti3tjp76Ri2HzsJERmXhOcHEYOp8GzqWzhO6eJCNgjep0GT+IGU3RkR9eOOwrRran0QWkDFGGDxfbWC0aLb8SGv4XebK8ApEcuEVeqhEZB4bF2q9I/MU2UO+h840biuD0gMCtTWY1B8Yk4OmWM0wFwUNcQHwpTHXvrdOOGdoWxl4JyPmas7277LFh863Hq6+oWGfyagJDZ17dsEJYEICbbj28hSTpnVQ4k3oCmJZ789s5ulkU+VuguhwdmkNPpCLpR1EfiduWoCQkE2DmC/hb09BT9qrpbGumipU/WAx9uKlBZYuaJ+CxQOgBmFzYGmDzITHK3HVozRyz8Uh7XqWAPYI/zXk0ShDv33bD/yrIahFvg1EUpualHx8rX5SQaots2BQJqw8Xa0/c5nw0b3pp2RJ4PZ64NOZBIi8VRDHY5wDLIclKApMZu9cNXWXD+OZVMIN5vZuDkABO83WNtAtzlPMEgTxQ+T52hej77IvXv5tPnt+QBAAta8ejT59V8DsGK8EVwrpDRxZA1fyFmhWfUXynRlZtaE7qkFPgAXCvVdBv3EILdEmqIalUNHsHH5M9Y3DAIk0OHANfxgoGw/yHswAj7OTnUJN4zOeurJNBZnXy+O0R3pS0D8xrMRmfytCITVDZFb4qfZb73CJctAxQIfj7xDykkmr28al9J9TprRFews9q52R5COgSNzOZjjd7Rj6s2Sh1gQoa3dDsmBhgMbleO+xvQGs5KQlhOtRMvs UF3rJFCF f4ZKXA/7Sl6uAQqmPguJzIT3KIXTg0ioLjtV+H56hrqiyWE+tz3r0UYn9sQ4rXJUd+YPVmjJfUKioAK11mYVYv6bashBTDGGvwvwx8TRXUGQQDEkIuW+1NNaM6H0w6lQKGYcEGdt533LsyAqAJO1jMzU8NdCsda+t/BBo11l26wr7ePBi2YtmsNzYvIIP1QHYJNMsiTVjODB2kbDlkj0rMNR62ASwZ4agr2Z0BY8yewS0QSmNGL62lTy9FxmHqWQZ+Met1AfJHvhtO6k67caV8mlEsU+DOHp6PWwYOCOhCmsnDIjTooC7NVk5bPsQE5SVitsom33WWeCUHqd36twYe8av5LSIUTon6CxoHBAY8IoBwtXkH+Ackmz5RC876dtQGkV+Q32ubPTxUqHV63HC0t62zHztj1U7Zh+g7WlriLq+eIpsXbml1JE0aG6YI4B8rSRIh2p2VWztmSrYmprKoKZs10YAJGZ0VbEqWIfLBz9BtFtSMOlt+82YrH4Xm/XHzrKmOnI97tHZubfLzKet1PWnFsyOY3B6xVIapw+PtGhOw/EARaiPf59E8Y7MTjAbmPGRw7nF817+yyK5Afd9cxW2cHf3SDu/3qZd8YYq9AkZfSJgwz+yOj7bpz7FCqUUsnTrbFkVGdIRZaI= Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: From: NeilBrown These are no longer used, so remove them. Signed-off-by: NeilBrown --- Documentation/filesystems/porting.rst | 7 +++ fs/ecryptfs/inode.c | 2 +- fs/namei.c | 61 ++------------------------- include/linux/namei.h | 2 - 4 files changed, 12 insertions(+), 60 deletions(-) diff --git a/Documentation/filesystems/porting.rst b/Documentation/filesystems/porting.rst index 154a38cd7801..7e83bd3c5a12 100644 --- a/Documentation/filesystems/porting.rst +++ b/Documentation/filesystems/porting.rst @@ -1396,3 +1396,10 @@ and d_alloc_parallel() or d_alloc_noblock() when standard interfaces can be used d_rehash() is gone. It should never be needed. Only unhash a dentry if you really don't want it. +--- + +** mandatory** + +lookup_one() and lookup_noperm() are no longer available. Use +start_creating() or similar instead. + diff --git a/fs/ecryptfs/inode.c b/fs/ecryptfs/inode.c index beb9e2c8b8b3..a7a596d51d67 100644 --- a/fs/ecryptfs/inode.c +++ b/fs/ecryptfs/inode.c @@ -414,7 +414,7 @@ static struct dentry *ecryptfs_lookup(struct inode *ecryptfs_dir_inode, lower_dentry = lookup_noperm_unlocked(&qname, lower_dir_dentry); if (IS_ERR(lower_dentry)) { - ecryptfs_printk(KERN_DEBUG, "%s: lookup_noperm() returned " + ecryptfs_printk(KERN_DEBUG, "%s: lookup_noperm_unlocked() returned " "[%ld] on lower_dentry = [%s]\n", __func__, PTR_ERR(lower_dentry), qname.name); diff --git a/fs/namei.c b/fs/namei.c index eed388ee8a30..cb80490a869f 100644 --- a/fs/namei.c +++ b/fs/namei.c @@ -3148,59 +3148,6 @@ struct dentry *try_lookup_noperm(struct qstr *name, struct dentry *base) } EXPORT_SYMBOL(try_lookup_noperm); -/** - * lookup_noperm - filesystem helper to lookup single pathname component - * @name: qstr storing pathname component to lookup - * @base: base directory to lookup from - * - * Note that this routine is purely a helper for filesystem usage and should - * not be called by generic code. It does no permission checking. - * - * The caller must hold base->i_rwsem. - */ -struct dentry *lookup_noperm(struct qstr *name, struct dentry *base) -{ - struct dentry *dentry; - int err; - - WARN_ON_ONCE(!inode_is_locked(base->d_inode)); - - err = lookup_noperm_common(name, base); - if (err) - return ERR_PTR(err); - - dentry = lookup_dcache(name, base, 0); - return dentry ? dentry : __lookup_slow(name, base, 0); -} -EXPORT_SYMBOL(lookup_noperm); - -/** - * lookup_one - lookup single pathname component - * @idmap: idmap of the mount the lookup is performed from - * @name: qstr holding pathname component to lookup - * @base: base directory to lookup from - * - * This can be used for in-kernel filesystem clients such as file servers. - * - * The caller must hold base->i_rwsem. - */ -struct dentry *lookup_one(struct mnt_idmap *idmap, struct qstr *name, - struct dentry *base) -{ - struct dentry *dentry; - int err; - - WARN_ON_ONCE(!inode_is_locked(base->d_inode)); - - err = lookup_one_common(idmap, name, base); - if (err) - return ERR_PTR(err); - - dentry = lookup_dcache(name, base, 0); - return dentry ? dentry : __lookup_slow(name, base, 0); -} -EXPORT_SYMBOL(lookup_one); - /** * lookup_one_unlocked - lookup single pathname component * @idmap: idmap of the mount the lookup is performed from @@ -3209,8 +3156,8 @@ EXPORT_SYMBOL(lookup_one); * * This can be used for in-kernel filesystem clients such as file servers. * - * Unlike lookup_one, it should be called without the parent - * i_rwsem held, and will take the i_rwsem itself if necessary. + * It should be called without the parent i_rwsem held, and will take + * the i_rwsem itself if necessary. * * Returns: - A dentry, possibly negative, or * - same errors as try_lookup_noperm() or @@ -3322,8 +3269,8 @@ EXPORT_SYMBOL(lookup_one_positive_unlocked); * Note that this routine is purely a helper for filesystem usage and should * not be called by generic code. It does no permission checking. * - * Unlike lookup_noperm(), it should be called without the parent - * i_rwsem held, and will take the i_rwsem itself if necessary. + * This should be called without the parent i_rwsem held, and will take + * the i_rwsem itself if necessary. * * Unlike try_lookup_noperm() it *does* revalidate the dentry if it already * existed. diff --git a/include/linux/namei.h b/include/linux/namei.h index b3346a513d8f..cb79e84c718d 100644 --- a/include/linux/namei.h +++ b/include/linux/namei.h @@ -74,10 +74,8 @@ int vfs_path_lookup(struct dentry *, struct vfsmount *, const char *, unsigned int, struct path *); extern struct dentry *try_lookup_noperm(struct qstr *, struct dentry *); -extern struct dentry *lookup_noperm(struct qstr *, struct dentry *); extern struct dentry *lookup_noperm_unlocked(struct qstr *, struct dentry *); extern struct dentry *lookup_noperm_positive_unlocked(struct qstr *, struct dentry *); -struct dentry *lookup_one(struct mnt_idmap *, struct qstr *, struct dentry *); struct dentry *lookup_one_unlocked(struct mnt_idmap *idmap, struct qstr *name, struct dentry *base); struct dentry *lookup_one_positive_unlocked(struct mnt_idmap *idmap, -- 2.50.0.107.gf914562f5916.dirty