kernel test robot reports kernel-doc warning: >> fs/btrfs/zstd.c:98: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst The comment is for zstd_reclaim_timer_fn(). Since the function is static, the comment isn't meant for kernel-doc consumption. Remove the extraneous (second) asterix at the head of function comment. Link: https://lore.kernel.org/linux-doc/202204151934.CkKcnvuJ-lkp@xxxxxxxxx/ Fixes: b672526e2ee935 ("btrfs: use non-bh spin_lock in zstd timer callback") Cc: Chris Mason <clm@xxxxxx> Cc: Josef Bacik <josef@xxxxxxxxxxxxxx> Cc: David Sterba <dsterba@xxxxxxxx> Cc: Nick Terrell <terrelln@xxxxxx> Cc: Schspa Shi <schspa@xxxxxxxxx> Cc: linux-btrfs@xxxxxxxxxxxxxxx Cc: linux-kernel@xxxxxxxxxxxxxxx Signed-off-by: Bagas Sanjaya <bagasdotme@xxxxxxxxx> --- In the previous fix attempt [1], Matthew Wilcox suggested to remove the second asterix instead since the function is static and it shouldn't have kernel-doc comments. [1]: https://lore.kernel.org/linux-doc/YluGmERvtQY9ju7Y@xxxxxxxxxxxxxxxxxxxx/ This patch is based on btrfs-devel/misc-next tree [2]. [2]: https://github.com/kdave/btrfs-devel.git fs/btrfs/zstd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/btrfs/zstd.c b/fs/btrfs/zstd.c index 0fe31a6f6e68f0..ca2102a46faefd 100644 --- a/fs/btrfs/zstd.c +++ b/fs/btrfs/zstd.c @@ -94,7 +94,7 @@ static inline struct workspace *list_to_workspace(struct list_head *list) void zstd_free_workspace(struct list_head *ws); struct list_head *zstd_alloc_workspace(unsigned int level); -/** +/* * Timer callback to free unused workspaces. * * @t: timer base-commit: 550a34e972578538fd0826916ae4fc407b62bb68 -- An old man doll... just what I always wanted! - Clara
