From: Al Viro <viro@zeniv.linux.org.uk>
To: linux-fsdevel@vger.kernel.org
Subject: [PATCHES] assorted fs cleanups
Date: Wed, 20 Dec 2023 05:13:48 +0000 [thread overview]
Message-ID: <20231220051348.GY1674809@ZenIV> (raw)
Assorted cleanups in various filesystems. Currently
that pile sits in vfs.git #work.misc; if anyone wants to pick
these into relevant filesystem tree, I'll be glad to drop those
from the queue...
Diffstat:
fs/affs/namei.c | 3 ---
fs/befs/linuxvfs.c | 3 ---
fs/bfs/dir.c | 5 -----
fs/ceph/dir.c | 21 +++++++++++++--------
fs/ceph/export.c | 2 --
fs/ext4/namei.c | 2 --
fs/gfs2/export.c | 2 --
fs/gfs2/super.c | 12 +-----------
fs/hostfs/hostfs_kern.c | 8 ++------
fs/kernfs/mount.c | 3 ---
fs/nfsd/nfsctl.c | 4 ----
fs/nilfs2/namei.c | 7 +------
fs/ocfs2/dcache.c | 7 -------
fs/ocfs2/dir.c | 9 ---------
fs/orangefs/dir.c | 32 ++++++++++++--------------------
fs/proc/proc_sysctl.c | 14 ++------------
fs/reiserfs/namei.c | 7 -------
fs/udf/namei.c | 11 +----------
fs/zonefs/super.c | 2 --
security/apparmor/apparmorfs.c | 7 +------
20 files changed, 33 insertions(+), 128 deletions(-)
Shortlog:
Al Viro (22):
hostfs: use d_splice_alias() calling conventions to simplify failure exits
/proc/sys: use d_splice_alias() calling conventions to simplify failure exits
zonefs: d_splice_alias() will do the right thing on ERR_PTR() inode
udf: d_splice_alias() will do the right thing on ERR_PTR() inode
affs: d_obtain_alias(ERR_PTR(...)) will do the right thing
befs: d_obtain_alias(ERR_PTR(...)) will do the right thing
ceph: d_obtain_{alias,root}(ERR_PTR(...)) will do the right thing
gfs2: d_obtain_alias(ERR_PTR(...)) will do the right thing
kernfs: d_obtain_alias(NULL) will do the right thing...
nilfs2: d_obtain_alias(ERR_PTR(...)) will do the right thing...
udf: d_obtain_alias(ERR_PTR(...)) will do the right thing...
ext4_add_entry(): ->d_name.len is never 0
bfs_add_entry(): get rid of pointless ->d_name.len checks
__ocfs2_add_entry(), ocfs2_prepare_dir_for_insert()
reiserfs_add_entry(): get rid of pointless namelen checks
udf_fiiter_add_entry(): check for zero ->d_name.len is bogus...
get rid of passing callbacks to ceph __dentry_leases_walk()
nfsd: kill stale comment about simple_fill_super() requirements
ocfs2_find_match(): there's no such thing as NULL or negative ->d_parent
gfs2: use is_subdir()
orangefs: saner arguments passing in readdir guts
apparmorfs: don't duplicate kfree_link()
First group (1--11) is about making use of d_splice_alias() and d_obtain_alias()
calling conventions; a bunch of failure exits are pointless, since the primitive
will do the right thing. Then (12--16) killing impossible checks for dentry
name length (never 0, never exceeds what ->lookup() would reject with ENAMETOOLONG).
The rest is really without any common theme...
As far as I can tell, nothing in there has conflicts in linux-next.
Individual patches in followups. Please, review; again, if anything of
that gets picked by relevant filesystem tree, I'll gladly drop it from
this branch.
next reply other threads:[~2023-12-20 5:13 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-12-20 5:13 Al Viro [this message]
2023-12-20 5:15 ` PATCH 01/22] hostfs: use d_splice_alias() calling conventions to simplify failure exits Al Viro
2023-12-20 5:16 ` [PATCH 02/22] /proc/sys: " Al Viro
2023-12-20 5:17 ` [PATCH 03/22] zonefs: d_splice_alias() will do the right thing on ERR_PTR() inode Al Viro
2023-12-20 7:09 ` Damien Le Moal
2023-12-20 5:18 ` [PATCH 04/22] udf: " Al Viro
2023-12-21 12:19 ` Jan Kara
2023-12-20 5:19 ` [PATCH 05/22] affs: d_obtain_alias(ERR_PTR(...)) will do the right thing Al Viro
2023-12-20 5:19 ` [PATCH 06/22] befs: " Al Viro
2023-12-20 5:20 ` [PATCH 07/22] ceph: d_obtain_{alias,root}(ERR_PTR(...)) " Al Viro
2023-12-21 0:35 ` Xiubo Li
2023-12-20 5:21 ` [PATCH 08/22] gfs2: d_obtain_alias(ERR_PTR(...)) " Al Viro
2023-12-20 5:22 ` [PATCH 09/22] kernfs: d_obtain_alias(NULL) " Al Viro
2023-12-20 5:23 ` nilfs2: d_obtain_alias(ERR_PTR(...)) " Al Viro
2023-12-20 14:46 ` Ryusuke Konishi
2023-12-20 5:23 ` [PATCH 11/22] udf: " Al Viro
2023-12-21 12:20 ` Jan Kara
2023-12-20 5:24 ` [PATCH 12/22] ext4_add_entry(): ->d_name.len is never 0 Al Viro
2023-12-20 5:25 ` [PATCH 13/22] bfs_add_entry(): get rid of pointless ->d_name.len checks Al Viro
2023-12-20 11:03 ` Tigran Aivazian
2023-12-20 5:26 ` [PATCH 14/22] __ocfs2_add_entry(), ocfs2_prepare_dir_for_insert(): namelen checks Al Viro
2023-12-20 5:27 ` [PATCH 15/22] reiserfs_add_entry(): get rid of pointless " Al Viro
2023-12-20 5:28 ` [PATCH 16/22] udf_fiiter_add_entry(): check for zero ->d_name.len is bogus Al Viro
2023-12-21 12:22 ` Jan Kara
2023-12-20 5:29 ` [PATCH 17/22] get rid of passing callbacks to ceph __dentry_leases_walk() Al Viro
2023-12-21 0:45 ` Xiubo Li
2023-12-21 1:12 ` Al Viro
2023-12-21 1:16 ` Xiubo Li
2023-12-20 5:30 ` [PATCH 18/22] nfsd: kill stale comment about simple_fill_super() requirements Al Viro
2023-12-20 14:22 ` Chuck Lever
2023-12-20 5:31 ` [PATCH 19/22] ocfs2_find_match(): there's no such thing as NULL or negative ->d_parent Al Viro
2023-12-20 5:31 ` [PATCH 20/22] gfs2: use is_subdir() Al Viro
2023-12-20 5:32 ` [PATCH 21/22] orangefs: saner arguments passing in readdir guts Al Viro
2023-12-27 12:05 ` Mike Marshall
2024-01-02 16:25 ` Mike Marshall
2023-12-20 5:33 ` [PATCH 22/22] apparmorfs: don't duplicate kfree_link() Al Viro
2023-12-20 12:16 ` [PATCHES] assorted fs cleanups Andreas Grünbacher
2023-12-20 17:15 ` Al Viro
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20231220051348.GY1674809@ZenIV \
--to=viro@zeniv.linux.org.uk \
--cc=linux-fsdevel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.