From: NeilBrown <neilb@ownmail.net>
To: "Theodore Ts'o" <tytso@mit.edu>,
Andreas Dilger <adilger.kernel@dilger.ca>,
Jan Kara <jack@suse.cz>
Cc: linux-ext4@vger.kernel.org, linux-fsdevel@vger.kernel.org
Subject: [PATCH 0/3] ext4: remove use of d_alloc()
Date: Wed, 18 Mar 2026 09:39:48 +1100 [thread overview]
Message-ID: <20260317224638.3809014-1-neilb@ownmail.net> (raw)
This is a revised version of the ext4 patches from my recent patchset
for revising directory locking. My particular interest in changing ext4
is to remove the use of d_alloc(). I will want to deprecate d_alloc()
as it doesn't fit the new model well.
The use of d_alloc() in ext4 is incidental to the actual task at hand.
The code really wants to pass around a parent directory and a name, and
only uses the dentry because that seems convenient. As these patches
show the code actually becomes simpler when we avoid the dentry.
The second patch here isn't needed. I thought it would be, wrote it,
then found it didn't directly help. However I think it is still a nice
simplification so I left it. If you don't want it, please drop it.
Thanks to Jan Kara for his initial review which helpped me see some
problems with my initial attempt more clearly.
NeilBrown
Note: these patches are quite independant of the rest of that big
patchset, except for the late removal of d_alloc(). They can safely
land indepenantly if you would like to take them now in the ext4 tree.
[PATCH 1/3] ext4: split __ext4_add_entry() out of ext4_add_entry()
[PATCH 2/3] ext4: add ext4_fc_eligible()
[PATCH 3/3] ext4: move dcache manipulation out of __ext4_link()
next reply other threads:[~2026-03-17 22:47 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-17 22:39 NeilBrown [this message]
2026-03-17 22:39 ` [PATCH 1/3] ext4: split __ext4_add_entry() out of ext4_add_entry() NeilBrown
2026-03-18 0:20 ` Andreas Dilger
2026-03-18 17:56 ` Jan Kara
2026-03-17 22:39 ` [PATCH 2/3] ext4: add ext4_fc_eligible() NeilBrown
2026-03-18 0:27 ` Andreas Dilger
2026-03-18 17:57 ` Jan Kara
2026-03-19 23:31 ` NeilBrown
2026-03-20 5:12 ` Andreas Dilger
2026-03-20 10:24 ` Jan Kara
2026-03-17 22:39 ` [PATCH 3/3] ext4: move dcache manipulation out of __ext4_link() NeilBrown
2026-03-18 18:03 ` Jan Kara
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=20260317224638.3809014-1-neilb@ownmail.net \
--to=neilb@ownmail.net \
--cc=adilger.kernel@dilger.ca \
--cc=jack@suse.cz \
--cc=linux-ext4@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=neil@brown.name \
--cc=tytso@mit.edu \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox