Re: [PATCH -next] fs: fix __sb_write_started() kerneldoc formatting

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu 28-12-23 11:06:08, Vegard Nossum wrote:
> When running 'make htmldocs', I see the following warning:
> 
>   Documentation/filesystems/api-summary:14: ./include/linux/fs.h:1659: WARNING: Definition list ends without a blank line; unexpected unindent.
> 
> The official guidance [1] seems to be to use lists, which will prevent
> both the "unexpected unindent" warning as well as ensure that each line
> is formatted on a separate line in the HTML output instead of being
> all considered a single paragraph.
> 
> [1]: https://docs.kernel.org/doc-guide/kernel-doc.html#return-values
> 
> Fixes: 8802e580ee64 ("fs: create __sb_write_started() helper")
> Cc: Amir Goldstein <amir73il@xxxxxxxxx>
> Cc: Josef Bacik <josef@xxxxxxxxxxxxxx>
> Cc: Jan Kara <jack@xxxxxxx>
> Signed-off-by: Vegard Nossum <vegard.nossum@xxxxxxxxxx>

Thanks! Feel free to add:

Reviewed-by: Jan Kara <jack@xxxxxxx>

								Honza

> ---
> Applies to git://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs.git vfs.rw
> 
>  include/linux/fs.h | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/include/linux/fs.h b/include/linux/fs.h
> index db5d07e6e02e..473063f385e5 100644
> --- a/include/linux/fs.h
> +++ b/include/linux/fs.h
> @@ -1650,9 +1650,9 @@ static inline bool __sb_start_write_trylock(struct super_block *sb, int level)
>   * @sb: the super we write to
>   * @level: the freeze level
>   *
> - * > 0 sb freeze level is held
> - *   0 sb freeze level is not held
> - * < 0 !CONFIG_LOCKDEP/LOCK_STATE_UNKNOWN
> + * * > 0 - sb freeze level is held
> + * *   0 - sb freeze level is not held
> + * * < 0 - !CONFIG_LOCKDEP/LOCK_STATE_UNKNOWN
>   */
>  static inline int __sb_write_started(const struct super_block *sb, int level)
>  {
> -- 
> 2.34.1
> 
-- 
Jan Kara <jack@xxxxxxxx>
SUSE Labs, CR




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux