Re: [PATCH] VFS: d_duplicate: repair comments to avoid kernel-doc warnings

[Randy Dunlap]

I need to send a v2 of this patch...
Please do not apply this one.

On 9/28/26 2:10 PM, Randy Dunlap wrote:
> Use a typical list style for comments to prevent kernel-doc warnings:
> 
> Documentation/filesystems/api-summary:20: ../fs/dcache.c:2029: ERROR: Unexpected indentation. [docutils]
> Documentation/filesystems/api-summary:20: ../fs/dcache.c:2030: WARNING: Block quote ends without a blank line; unexpected unindent. [docutils]
> 
> Also, modify the spelling of "whiteout" to match all other uses in
> the kernel source tree and to be consistent on the same line.
> 
> Fixes: 33d76519f0cc ("VFS: add d_duplicate()")
> Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> ---
> Cc: NeilBrown <neil@xxxxxxxxxx>
> Cc: Alexander Viro <viro@xxxxxxxxxxxxxxxxxx>
> Cc: Christian Brauner <brauner@xxxxxxxxxx>
> Cc: Jan Kara <jack@xxxxxxx>
> Cc: linux-fsdevel@xxxxxxxxxxxxxxx
> 
>  fs/dcache.c |    7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> --- linux-next-20260928.orig/fs/dcache.c
> +++ linux-next-20260928/fs/dcache.c
> @@ -2025,10 +2025,11 @@ EXPORT_SYMBOL(d_alloc);
>   * @dentry: the dentry to duplicate
>   *
>   * Some rename operations need to be combined with another operation
> - * inside the filesystem.
> - * 1/ A cluster filesystem when renaming to an in-use file might need to
> + * inside the filesystem::
> + *
> + * 1. A cluster filesystem when renaming to an in-use file might need to
>   *   first "silly-rename" that target out of the way before the main rename
> - * 2/ A filesystem that supports white-out might want to create a whiteout
> + * 2. A filesystem that supports whiteout might want to create a whiteout
>   *   in place of the file being moved.
>   *
>   * For this they need two dentries which temporarily have the same name,

-- 
~Randy