From nobody Mon May 25 05:55:43 2026 Received: from n169-114.mail.139.com (n169-114.mail.139.com [120.232.169.114]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1ADEA349B16; Mon, 18 May 2026 04:21:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=120.232.169.114 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779078106; cv=none; b=AZSrwzxM3aipduW0GZSA7UvlMdfqAoDWe6LZdOg/wkVDr8FG5WCgQvcwuQZgA14DE5cn+Bm2D3sC9v5zIlDK5Np3cp8StjzCUcEkpFCtS4cduljtNhkmhVRg4fYmelYhiJgToI9o70RJ8SRoERdOsXE0lq5sFqHbF5YA71m8N24= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779078106; c=relaxed/simple; bh=8aXYRl//JGNCU2h37QhnEWAcwTENSu3Si3q7xsFzPhw=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version; b=NRrBKuV/mdJSVjuQPNfLsWh7Y+GPlU9g/ryfWCHPhYdsTVvl1aNnX0BnpeHGA7Sk6w/36hWoJ9iuExpCvx2An3SrP4Ncek2+yU9EY3fRani4N42Ns/71s2WoLoaxhg0BjQ9AeQGMJrgy/Y9k2m/HUkRVKJuFExjRtDLJRwQ14ww= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=139.com; spf=pass smtp.mailfrom=139.com; dkim=pass (1024-bit key) header.d=139.com header.i=@139.com header.b=LFU+w16i; arc=none smtp.client-ip=120.232.169.114 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=139.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=139.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=139.com header.i=@139.com header.b="LFU+w16i" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=139.com; s=dkim; l=0; h=from:subject:message-id:to:cc:mime-version; bh=47DEQpj8HBSa+/TImW+5JCeuQeRkm5NMpJWZG3hSuFU=; b=LFU+w16iKWArwZ/IR/xbuS6WW2XIRnICRpluGJKHbY2F7tZ1fYp6A8aC9V4g89UGVTo5VUi6P/6LB cLdMFVXGQmB8F0UdY4aCxJh2q1iJKJaTHcyvy3tFTch+Y6afOT6PFjtusEu/hq6kBIEmEcRUfQe8iO dN3iA9to0JB9OoVQ= X-RM-TagInfo: emlType=0 X-RM-SPAM: X-RM-SPAM-FLAG: 00000000 Received: from NTT-kernel-dev (unknown[60.247.85.88]) by rmsmtp-lg-appmail-40-12054 (RichMail) with SMTP id 2f166a0a93c86e1-006ea; Mon, 18 May 2026 12:21:32 +0800 (CST) X-RM-TRANSID: 2f166a0a93c86e1-006ea From: Li hongliang <1468888505@139.com> To: gregkh@linuxfoundation.org, stable@vger.kernel.org, viro@zeniv.linux.org.uk Cc: patches@lists.linux.dev, linux-kernel@vger.kernel.org, almaz.alexandrovich@paragon-software.com, ntfs3@lists.linux.dev Subject: [PATCH 6.12.y] ntfs: ->d_compare() must not block Date: Mon, 18 May 2026 12:21:30 +0800 Message-Id: <20260518042130.489507-1-1468888505@139.com> X-Mailer: git-send-email 2.34.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Al Viro [ Upstream commit ca2a04e84af79596e5cd9cfe697d5122ec39c8ce ] ... so don't use __getname() there. Switch it (and ntfs_d_hash(), while we are at it) to kmalloc(PATH_MAX, GFP_NOWAIT). Yes, ntfs_d_hash() almost certainly can do with smaller allocations, but let ntfs folks deal with that - keep the allocation size as-is for now. Stop abusing names_cachep in ntfs, period - various uses of that thing in there have nothing to do with pathnames; just use k[mz]alloc() and be done with that. For now let's keep sizes as-in, but AFAICS none of the users actually want PATH_MAX. Signed-off-by: Al Viro Signed-off-by: Li hongliang <1468888505@139.com> --- fs/ntfs3/dir.c | 5 ++--- fs/ntfs3/fsntfs.c | 4 ++-- fs/ntfs3/inode.c | 13 ++++++------- fs/ntfs3/namei.c | 17 ++++++++--------- fs/ntfs3/xattr.c | 5 ++--- 5 files changed, 20 insertions(+), 24 deletions(-) diff --git a/fs/ntfs3/dir.c b/fs/ntfs3/dir.c index 600e66035c1b..522ebc14b1fb 100644 --- a/fs/ntfs3/dir.c +++ b/fs/ntfs3/dir.c @@ -425,8 +425,7 @@ static int ntfs_readdir(struct file *file, struct dir_c= ontext *ctx) if (!dir_emit_dots(file, ctx)) return 0; =20 - /* Allocate PATH_MAX bytes. */ - name =3D __getname(); + name =3D kmalloc(PATH_MAX, GFP_KERNEL); if (!name) return -ENOMEM; =20 @@ -504,7 +503,7 @@ static int ntfs_readdir(struct file *file, struct dir_c= ontext *ctx) =20 out: =20 - __putname(name); + kfree(name); put_indx_node(node); =20 if (err =3D=3D 1) { diff --git a/fs/ntfs3/fsntfs.c b/fs/ntfs3/fsntfs.c index 37c5d9a1f77b..5972f160e566 100644 --- a/fs/ntfs3/fsntfs.c +++ b/fs/ntfs3/fsntfs.c @@ -2670,7 +2670,7 @@ int ntfs_set_label(struct ntfs_sb_info *sbi, u8 *labe= l, int len) u32 uni_bytes; struct ntfs_inode *ni =3D sbi->volume.ni; /* Allocate PATH_MAX bytes. */ - struct cpu_str *uni =3D __getname(); + struct cpu_str *uni =3D kmalloc(PATH_MAX, GFP_KERNEL); =20 if (!uni) return -ENOMEM; @@ -2714,6 +2714,6 @@ int ntfs_set_label(struct ntfs_sb_info *sbi, u8 *labe= l, int len) err =3D _ni_write_inode(&ni->vfs_inode, 0); =20 out: - __putname(uni); + kfree(uni); return err; } \ No newline at end of file diff --git a/fs/ntfs3/inode.c b/fs/ntfs3/inode.c index 8113d47b0ceb..b50c9dff327d 100644 --- a/fs/ntfs3/inode.c +++ b/fs/ntfs3/inode.c @@ -1301,7 +1301,7 @@ int ntfs_create_inode(struct mnt_idmap *idmap, struct= inode *dir, fa |=3D FILE_ATTRIBUTE_READONLY; =20 /* Allocate PATH_MAX bytes. */ - new_de =3D kmem_cache_zalloc(names_cachep, GFP_KERNEL); + new_de =3D kzalloc(PATH_MAX, GFP_KERNEL); if (!new_de) { err =3D -ENOMEM; goto out1; @@ -1713,7 +1713,7 @@ int ntfs_create_inode(struct mnt_idmap *idmap, struct= inode *dir, ntfs_mark_rec_free(sbi, ino, false); =20 out2: - __putname(new_de); + kfree(new_de); kfree(rp); =20 out1: @@ -1734,7 +1734,7 @@ int ntfs_link_inode(struct inode *inode, struct dentr= y *dentry) struct NTFS_DE *de; =20 /* Allocate PATH_MAX bytes. */ - de =3D kmem_cache_zalloc(names_cachep, GFP_KERNEL); + de =3D kzalloc(PATH_MAX, GFP_KERNEL); if (!de) return -ENOMEM; =20 @@ -1748,7 +1748,7 @@ int ntfs_link_inode(struct inode *inode, struct dentr= y *dentry) =20 err =3D ni_add_name(ntfs_i(d_inode(dentry->d_parent)), ni, de); out: - __putname(de); + kfree(de); return err; } =20 @@ -1771,8 +1771,7 @@ int ntfs_unlink_inode(struct inode *dir, const struct= dentry *dentry) if (ntfs_is_meta_file(sbi, ni->mi.rno)) return -EINVAL; =20 - /* Allocate PATH_MAX bytes. */ - de =3D kmem_cache_zalloc(names_cachep, GFP_KERNEL); + de =3D kzalloc(PATH_MAX, GFP_KERNEL); if (!de) return -ENOMEM; =20 @@ -1808,7 +1807,7 @@ int ntfs_unlink_inode(struct inode *dir, const struct= dentry *dentry) =20 out: ni_unlock(ni); - __putname(de); + kfree(de); return err; } =20 diff --git a/fs/ntfs3/namei.c b/fs/ntfs3/namei.c index 71a5a959a48c..fa4f7d9f3845 100644 --- a/fs/ntfs3/namei.c +++ b/fs/ntfs3/namei.c @@ -68,7 +68,7 @@ static struct dentry *ntfs_lookup(struct inode *dir, stru= ct dentry *dentry, u32 flags) { struct ntfs_inode *ni =3D ntfs_i(dir); - struct cpu_str *uni =3D __getname(); + struct cpu_str *uni =3D kmalloc(PATH_MAX, GFP_KERNEL); struct inode *inode; int err; =20 @@ -85,7 +85,7 @@ static struct dentry *ntfs_lookup(struct inode *dir, stru= ct dentry *dentry, inode =3D dir_search_u(dir, uni, NULL); ni_unlock(ni); } - __putname(uni); + kfree(uni); } =20 /* @@ -287,8 +287,7 @@ static int ntfs_rename(struct mnt_idmap *idmap, struct = inode *dir, return err; } =20 - /* Allocate PATH_MAX bytes. */ - de =3D __getname(); + de =3D kmalloc(PATH_MAX, GFP_KERNEL); if (!de) return -ENOMEM; =20 @@ -333,7 +332,7 @@ static int ntfs_rename(struct mnt_idmap *idmap, struct = inode *dir, ni_unlock(ni); ni_unlock(dir_ni); out: - __putname(de); + kfree(de); return err; } =20 @@ -391,7 +390,7 @@ static int ntfs_d_hash(const struct dentry *dentry, str= uct qstr *name) /* * Try slow way with current upcase table */ - uni =3D kmem_cache_alloc(names_cachep, GFP_NOWAIT); + uni =3D kmalloc(PATH_MAX, GFP_NOWAIT); if (!uni) return -ENOMEM; =20 @@ -413,7 +412,7 @@ static int ntfs_d_hash(const struct dentry *dentry, str= uct qstr *name) err =3D 0; =20 out: - kmem_cache_free(names_cachep, uni); + kfree(uni); return err; } =20 @@ -452,7 +451,7 @@ static int ntfs_d_compare(const struct dentry *dentry, = unsigned int len1, * Try slow way with current upcase table */ sbi =3D dentry->d_sb->s_fs_info; - uni1 =3D __getname(); + uni1 =3D kmalloc(PATH_MAX, GFP_NOWAIT); if (!uni1) return -ENOMEM; =20 @@ -482,7 +481,7 @@ static int ntfs_d_compare(const struct dentry *dentry, = unsigned int len1, ret =3D !ntfs_cmp_names_cpu(uni1, uni2, sbi->upcase, false) ? 0 : 1; =20 out: - __putname(uni1); + kfree(uni1); return ret; } =20 diff --git a/fs/ntfs3/xattr.c b/fs/ntfs3/xattr.c index e0055dcf8fe3..6861c09d66d7 100644 --- a/fs/ntfs3/xattr.c +++ b/fs/ntfs3/xattr.c @@ -552,8 +552,7 @@ struct posix_acl *ntfs_get_acl(struct mnt_idmap *idmap,= struct dentry *dentry, int err; void *buf; =20 - /* Allocate PATH_MAX bytes. */ - buf =3D __getname(); + buf =3D kmalloc(PATH_MAX, GFP_KERNEL); if (!buf) return ERR_PTR(-ENOMEM); =20 @@ -584,7 +583,7 @@ struct posix_acl *ntfs_get_acl(struct mnt_idmap *idmap,= struct dentry *dentry, if (!IS_ERR(acl)) set_cached_acl(inode, type, acl); =20 - __putname(buf); + kfree(buf); =20 return acl; } --=20 2.34.1