Re: [PATCH] mm/slab: fix kernel-doc func param names

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

 



On 1/11/25 07:32, Randy Dunlap wrote:
> Use corrected function parameter names to eliminate kernel-doc
> warnings:
> 
> slab.h:142: warning: Function parameter or struct member 's' not described in 'slab_folio'
> slab.h:142: warning: Excess function parameter 'slab' description in 'slab_folio'
> slab.h:168: warning: Function parameter or struct member 's' not described in 'slab_page'
> slab.h:168: warning: Excess function parameter 'slab' description in 'slab_page'

I'm unable to get those warnings with "make htmldocs", what exactly do I
need to run?

> Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Cc: Christoph Lameter <cl@xxxxxxxxx>
> Cc: Pekka Enberg <penberg@xxxxxxxxxx>
> Cc: David Rientjes <rientjes@xxxxxxxxxx>
> Cc: Joonsoo Kim <iamjoonsoo.kim@xxxxxxx>
> Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> Cc: Vlastimil Babka <vbabka@xxxxxxx>

Anyway the problem and fix seem obvious, so applied. Thanks!

> ---
>  mm/slab.h |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> --- linux-next-20250108.orig/mm/slab.h
> +++ linux-next-20250108/mm/slab.h
> @@ -128,7 +128,7 @@ static_assert(IS_ALIGNED(offsetof(struct
>  
>  /**
>   * slab_folio - The folio allocated for a slab
> - * @slab: The slab.
> + * @s: The slab.
>   *
>   * Slabs are allocated as folios that contain the individual objects and are
>   * using some fields in the first struct page of the folio - those fields are
> @@ -159,7 +159,7 @@ static_assert(IS_ALIGNED(offsetof(struct
>  
>  /**
>   * slab_page - The first struct page allocated for a slab
> - * @slab: The slab.
> + * @s: The slab.
>   *
>   * A convenience wrapper for converting slab to the first struct page of the
>   * underlying folio, to communicate with code not yet converted to folio or





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux