[linux-next:master 5773/9539] Warning: fs/iomap/buffered-io.c:624 function parameter 'private' not described in 'iomap_readahead'

[linux-next:master 5773/9539] Warning: fs/iomap/buffered-io.c:624 function parameter 'private' not described in 'iomap_readahead'

[kernel test robot]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   ca3a02fda4da8e2c1cb6baee5d72352e9e2cfaea
commit: 8806f279244bf300dca2c99735d5a51cd24b86df [5773/9539] iomap: stash iomap read ctx in the private field of iomap_iter
config: x86_64-rhel-9.4 (https://download.01.org/0day-ci/archive/20260126/202601261111.vIL9rhgD-lkp@intel.com/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260126/202601261111.vIL9rhgD-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202601261111.vIL9rhgD-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> Warning: fs/iomap/buffered-io.c:624 function parameter 'private' not described in 'iomap_readahead'
>> Warning: fs/iomap/buffered-io.c:624 function parameter 'private' not described in 'iomap_readahead'

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

Re: [linux-next:master 5773/9539] Warning: fs/iomap/buffered-io.c:624 function parameter 'private' not described in 'iomap_readahead'

[Gao Xiang]

Hi Hongbo,

On 2026/1/26 11:56, kernel test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> head:   ca3a02fda4da8e2c1cb6baee5d72352e9e2cfaea
> commit: 8806f279244bf300dca2c99735d5a51cd24b86df [5773/9539] iomap: stash iomap read ctx in the private field of iomap_iter
> config: x86_64-rhel-9.4 (https://download.01.org/0day-ci/archive/20260126/202601261111.vIL9rhgD-lkp@intel.com/config)
> compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260126/202601261111.vIL9rhgD-lkp@intel.com/reproduce)
> 
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <lkp@intel.com>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202601261111.vIL9rhgD-lkp@intel.com/
> 
> All warnings (new ones prefixed by >>):
> 
>>> Warning: fs/iomap/buffered-io.c:624 function parameter 'private' not described in 'iomap_readahead'
>>> Warning: fs/iomap/buffered-io.c:624 function parameter 'private' not described in 'iomap_readahead'

It seems the kernel-doc isn't updated, could you
submit another iomap patch to fix that?

  608 /**
  609  * iomap_readahead - Attempt to read pages from a file.
  610  * @ops: The operations vector for the filesystem.
  611  * @ctx: The ctx used for issuing readahead.

^ here.

  612  *
  613  * This function is for filesystems to call to implement their readahead
  614  * address_space operation.
  615  *
  616  * Context: The @ops callbacks may submit I/O (eg to read the addresses of
  617  * blocks from disc), and may wait for it.  The caller may be trying to
  618  * access a different page, and so sleeping excessively should be avoided.
  619  * It may allocate memory, but should avoid costly allocations.  This
  620  * function is called with memalloc_nofs set, so allocations will not cause
  621  * the filesystem to be reentered.
  622  */
  623 void iomap_readahead(const struct iomap_ops *ops,
  624                 struct iomap_read_folio_ctx *ctx, void *private)

Thanks,
Gao Xiang



> 

Re: [linux-next:master 5773/9539] Warning: fs/iomap/buffered-io.c:624 function parameter 'private' not described in 'iomap_readahead'

[Hongbo Li]

On 2026/1/26 12:01, Gao Xiang wrote:
> Hi Hongbo,
> 
> On 2026/1/26 11:56, kernel test robot wrote:
>> tree:   
>> https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git 
>> master
>> head:   ca3a02fda4da8e2c1cb6baee5d72352e9e2cfaea
>> commit: 8806f279244bf300dca2c99735d5a51cd24b86df [5773/9539] iomap: 
>> stash iomap read ctx in the private field of iomap_iter
>> config: x86_64-rhel-9.4 
>> (https://download.01.org/0day-ci/archive/20260126/202601261111.vIL9rhgD-lkp@intel.com/config)
>> compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
>> reproduce (this is a W=1 build): 
>> (https://download.01.org/0day-ci/archive/20260126/202601261111.vIL9rhgD-lkp@intel.com/reproduce)
>>
>> If you fix the issue in a separate patch/commit (i.e. not just a new 
>> version of
>> the same patch/commit), kindly add following tags
>> | Reported-by: kernel test robot <lkp@intel.com>
>> | Closes: 
>> https://lore.kernel.org/oe-kbuild-all/202601261111.vIL9rhgD-lkp@intel.com/
>>
>> All warnings (new ones prefixed by >>):
>>
>>>> Warning: fs/iomap/buffered-io.c:624 function parameter 'private' not 
>>>> described in 'iomap_readahead'
>>>> Warning: fs/iomap/buffered-io.c:624 function parameter 'private' not 
>>>> described in 'iomap_readahead'
> 
> It seems the kernel-doc isn't updated, could you
> submit another iomap patch to fix that?

Ok, thank you to point it out.

Thanks,
Hongbo
> 
>   608 /**
>   609  * iomap_readahead - Attempt to read pages from a file.
>   610  * @ops: The operations vector for the filesystem.
>   611  * @ctx: The ctx used for issuing readahead.
> 
> ^ here.
> 
>   612  *
>   613  * This function is for filesystems to call to implement their 
> readahead
>   614  * address_space operation.
>   615  *
>   616  * Context: The @ops callbacks may submit I/O (eg to read the 
> addresses of
>   617  * blocks from disc), and may wait for it.  The caller may be 
> trying to
>   618  * access a different page, and so sleeping excessively should be 
> avoided.
>   619  * It may allocate memory, but should avoid costly allocations.  This
>   620  * function is called with memalloc_nofs set, so allocations will 
> not cause
>   621  * the filesystem to be reentered.
>   622  */
>   623 void iomap_readahead(const struct iomap_ops *ops,
>   624                 struct iomap_read_folio_ctx *ctx, void *private)
> 
> Thanks,
> Gao Xiang
> 
> 
> 
>>
>