* [PATCH] inode: move @isnew kdoc from inode_insert5 to ilookup5_nowait
@ 2025-12-28 16:04 Ryota Sakamoto
2025-12-29 1:22 ` Bagas Sanjaya
0 siblings, 1 reply; 2+ messages in thread
From: Ryota Sakamoto @ 2025-12-28 16:04 UTC (permalink / raw)
To: Alexander Viro, Christian Brauner, Jan Kara, Mateusz Guzik
Cc: linux-fsdevel, linux-kernel, Ryota Sakamoto
The commit reworking I_NEW handling accidentally documented the @isnew
parameter in the kernel-doc for inode_insert5(), which does not take the
parameter. Meanwhile, ilookup5_nowait() gained the @isnew parameter but
lacked the corresponding kernel-doc.
Move the description to the correct function to ensure the kernel-doc
accuracy.
Fixes: a27628f43634 ("fs: rework I_NEW handling to operate without fences")
Signed-off-by: Ryota Sakamoto <sakamo.ryota@gmail.com>
---
fs/inode.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/inode.c b/fs/inode.c
index 521383223d8a455a2d09caff70615032213e3dfc..68056473d65ed4beba5241f4b1bac79b29cd84e9 100644
--- a/fs/inode.c
+++ b/fs/inode.c
@@ -1278,7 +1278,6 @@ EXPORT_SYMBOL(unlock_two_nondirectories);
* @test: callback used for comparisons between inodes
* @set: callback used to initialize a new struct inode
* @data: opaque data pointer to pass to @test and @set
- * @isnew: pointer to a bool which will indicate whether I_NEW is set
*
* Search for the inode specified by @hashval and @data in the inode cache,
* and if present return it with an increased reference count. This is a
@@ -1593,6 +1592,7 @@ EXPORT_SYMBOL(igrab);
* @hashval: hash value (usually inode number) to search for
* @test: callback used for comparisons between inodes
* @data: opaque data pointer to pass to @test
+ * @isnew: pointer to a bool which will indicate whether I_NEW is set
*
* Search for the inode specified by @hashval and @data in the inode cache.
* If the inode is in the cache, the inode is returned with an incremented
---
base-commit: d26143bb38e2546fe6f8c9860c13a88146ce5dd6
change-id: 20251228-fix-kdoc-ilookup5_nowait-b17dc447bf2c
Best regards,
--
Ryota Sakamoto <sakamo.ryota@gmail.com>
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] inode: move @isnew kdoc from inode_insert5 to ilookup5_nowait
2025-12-28 16:04 [PATCH] inode: move @isnew kdoc from inode_insert5 to ilookup5_nowait Ryota Sakamoto
@ 2025-12-29 1:22 ` Bagas Sanjaya
0 siblings, 0 replies; 2+ messages in thread
From: Bagas Sanjaya @ 2025-12-29 1:22 UTC (permalink / raw)
To: Ryota Sakamoto, Alexander Viro, Christian Brauner, Jan Kara,
Mateusz Guzik
Cc: linux-fsdevel, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 559 bytes --]
On Mon, Dec 29, 2025 at 01:04:07AM +0900, Ryota Sakamoto wrote:
> The commit reworking I_NEW handling accidentally documented the @isnew
> parameter in the kernel-doc for inode_insert5(), which does not take the
> parameter. Meanwhile, ilookup5_nowait() gained the @isnew parameter but
> lacked the corresponding kernel-doc.
>
> Move the description to the correct function to ensure the kernel-doc
> accuracy.
Much better, thanks!
Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>
--
An old man doll... just what I always wanted! - Clara
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-12-29 1:22 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-12-28 16:04 [PATCH] inode: move @isnew kdoc from inode_insert5 to ilookup5_nowait Ryota Sakamoto
2025-12-29 1:22 ` Bagas Sanjaya
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox