All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: "Darrick J. Wong" <darrick.wong@oracle.com>
Cc: oe-kbuild-all@lists.linux.dev
Subject: [djwong-xfs:fsverity-cleanups-6.9 26/44] fs/verity/verify.c:419: warning: Function parameter or struct member 'level' not described in 'fsverity_read_merkle_tree_block'
Date: Wed, 13 Mar 2024 16:27:47 +0800	[thread overview]
Message-ID: <202403131626.aADdoCAL-lkp@intel.com> (raw)

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/djwong/xfs-linux.git fsverity-cleanups-6.9
head:   f0f02c9c8136c2900f1a5e3fd479cb42a5453ab0
commit: 9f776a322d851cf08ce43dfb0129d1d49bc628af [26/44] fsverity: reorganize read_merkle_tree_block signature
config: sh-allmodconfig (https://download.01.org/0day-ci/archive/20240313/202403131626.aADdoCAL-lkp@intel.com/config)
compiler: sh4-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240313/202403131626.aADdoCAL-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/202403131626.aADdoCAL-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> fs/verity/verify.c:419: warning: Function parameter or struct member 'level' not described in 'fsverity_read_merkle_tree_block'


vim +419 fs/verity/verify.c

057a4e47b75182 Andrey Albershteyn 2024-02-28  404  
7e233393e1188a Darrick J. Wong    2024-03-12  405  /**
7e233393e1188a Darrick J. Wong    2024-03-12  406   * fsverity_read_merkle_tree_block() - read Merkle tree block
7e233393e1188a Darrick J. Wong    2024-03-12  407   * @inode: inode to which this Merkle tree blocks belong
7e233393e1188a Darrick J. Wong    2024-03-12  408   * @params: merkle tree parameters
7e233393e1188a Darrick J. Wong    2024-03-12  409   * @pos: byte position within merkle tree
7e233393e1188a Darrick J. Wong    2024-03-12  410   * @ra_bytes: try to read ahead this many btes
7e233393e1188a Darrick J. Wong    2024-03-12  411   * @block: block to be loaded
7e233393e1188a Darrick J. Wong    2024-03-12  412   *
7e233393e1188a Darrick J. Wong    2024-03-12  413   * This function loads data from a merkle tree.
7e233393e1188a Darrick J. Wong    2024-03-12  414   */
7e233393e1188a Darrick J. Wong    2024-03-12  415  int fsverity_read_merkle_tree_block(struct inode *inode,
7e233393e1188a Darrick J. Wong    2024-03-12  416  				    const struct merkle_tree_params *params,
9f776a322d851c Darrick J. Wong    2024-03-08  417  				    int level, u64 pos, unsigned long ra_bytes,
057a4e47b75182 Andrey Albershteyn 2024-02-28  418  				    struct fsverity_blockbuf *block)
057a4e47b75182 Andrey Albershteyn 2024-02-28 @419  {
7e233393e1188a Darrick J. Wong    2024-03-12  420  	const struct fsverity_operations *vops = inode->i_sb->s_vop;
7e233393e1188a Darrick J. Wong    2024-03-12  421  	unsigned long page_idx;
7e233393e1188a Darrick J. Wong    2024-03-12  422  	struct page *page;
7e233393e1188a Darrick J. Wong    2024-03-12  423  	unsigned long index;
7e233393e1188a Darrick J. Wong    2024-03-12  424  	unsigned int offset_in_page;
057a4e47b75182 Andrey Albershteyn 2024-02-28  425  
9f776a322d851c Darrick J. Wong    2024-03-08  426  	block->offset = pos;
9f776a322d851c Darrick J. Wong    2024-03-08  427  	block->size = params->block_size;
9f776a322d851c Darrick J. Wong    2024-03-08  428  
9f776a322d851c Darrick J. Wong    2024-03-08  429  	if (fsverity_caches_blocks(inode)) {
9f776a322d851c Darrick J. Wong    2024-03-08  430  		struct fsverity_readmerkle req = {
9f776a322d851c Darrick J. Wong    2024-03-08  431  			.inode = inode,
9f776a322d851c Darrick J. Wong    2024-03-08  432  			.level = level,
9f776a322d851c Darrick J. Wong    2024-03-08  433  			.num_levels = params->num_levels,
9f776a322d851c Darrick J. Wong    2024-03-08  434  			.log_blocksize = params->log_blocksize,
9f776a322d851c Darrick J. Wong    2024-03-08  435  			.ra_bytes = ra_bytes,
9f776a322d851c Darrick J. Wong    2024-03-08  436  		};
9f776a322d851c Darrick J. Wong    2024-03-08  437  
9f776a322d851c Darrick J. Wong    2024-03-08  438  		return vops->read_merkle_tree_block(&req, block);
9f776a322d851c Darrick J. Wong    2024-03-08  439  	}
7e233393e1188a Darrick J. Wong    2024-03-12  440  
7e233393e1188a Darrick J. Wong    2024-03-12  441  	index = pos >> params->log_blocksize;
7e233393e1188a Darrick J. Wong    2024-03-12  442  	page_idx = round_down(index, params->blocks_per_page);
7e233393e1188a Darrick J. Wong    2024-03-12  443  	offset_in_page = pos & ~PAGE_MASK;
7e233393e1188a Darrick J. Wong    2024-03-12  444  
7e233393e1188a Darrick J. Wong    2024-03-12  445  	page = vops->read_merkle_tree_page(inode, page_idx,
7e233393e1188a Darrick J. Wong    2024-03-12  446  			ra_bytes >> PAGE_SHIFT);
7e233393e1188a Darrick J. Wong    2024-03-12  447  	if (IS_ERR(page))
7e233393e1188a Darrick J. Wong    2024-03-12  448  		return PTR_ERR(page);
7e233393e1188a Darrick J. Wong    2024-03-12  449  
7e233393e1188a Darrick J. Wong    2024-03-12  450  	block->kaddr = kmap_local_page(page) + offset_in_page;
7e233393e1188a Darrick J. Wong    2024-03-12  451  	block->context = page;
7e233393e1188a Darrick J. Wong    2024-03-12  452  	return 0;
7e233393e1188a Darrick J. Wong    2024-03-12  453  }
7e233393e1188a Darrick J. Wong    2024-03-12  454  

:::::: The code at line 419 was first introduced by commit
:::::: 057a4e47b751820ed58aca21323c229c7b9322ec fsverity: support block-based Merkle tree caching

:::::: TO: Andrey Albershteyn <aalbersh@redhat.com>
:::::: CC: Darrick J. Wong <djwong@kernel.org>

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

                 reply	other threads:[~2024-03-13  8:28 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202403131626.aADdoCAL-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=darrick.wong@oracle.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.