The Linux Kernel Mailing List
 help / color / mirror / Atom feed
* [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