Re: [PATCH 04/18] maple_tree: introduce mas_wr_store_type()

From: kernel test robot
Date: Tue Jun 04 2024 - 17:10:25 EST


Hi Sidhartha,

kernel test robot noticed the following build warnings:

[auto build test WARNING on akpm-mm/mm-nonmm-unstable]
[also build test WARNING on linus/master v6.10-rc2 next-20240604]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url: https://github.com/intel-lab-lkp/linux/commits/Sidhartha-Kumar/maple_tree-introduce-store_type-enum/20240605-014633
base: https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git mm-nonmm-unstable
patch link: https://lore.kernel.org/r/20240604174145.563900-5-sidhartha.kumar%40oracle.com
patch subject: [PATCH 04/18] maple_tree: introduce mas_wr_store_type()
config: openrisc-allnoconfig (https://download.01.org/0day-ci/archive/20240605/202406050440.xjLxhyu5-lkp@xxxxxxxxx/config)
compiler: or1k-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240605/202406050440.xjLxhyu5-lkp@xxxxxxxxx/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@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202406050440.xjLxhyu5-lkp@xxxxxxxxx/

All warnings (new ones prefixed by >>):

>> lib/maple_tree.c:4289: warning: Function parameter or struct member 'entry' not described in 'mas_prealloc_calc'


vim +4289 lib/maple_tree.c

4280
4281 /**
4282 * mas_prealloc_calc() - Calculate number of nodes needed for a
4283 * given store oepration
4284 * @mas: The maple state.
4285 *
4286 * Return: Number of nodes required for preallocation.
4287 */
4288 static inline int mas_prealloc_calc(struct ma_state *mas, void *entry)
> 4289 {
4290 int ret = mas_mt_height(mas) * 3 + 1;
4291
4292 switch (mas->store_type) {
4293 case wr_invalid:
4294 WARN_ON_ONCE(1);
4295 break;
4296 case wr_new_root:
4297 ret = 1;
4298 break;
4299 case wr_store_root:
4300 if (likely((mas->last != 0) || (mas->index != 0)))
4301 ret = 1;
4302 else if (((unsigned long) (entry) & 3) == 2)
4303 ret = 1;
4304 else
4305 ret = 0;
4306 break;
4307 case wr_spanning_store:
4308 ret = mas_mt_height(mas) * 3 + 1;
4309 break;
4310 case wr_split_store:
4311 ret = mas_mt_height(mas) * 2 + 1;
4312 break;
4313 case wr_rebalance:
4314 ret = mas_mt_height(mas) * 2 - 1;
4315 break;
4316 case wr_node_store:
4317 case wr_bnode:
4318 ret = mt_in_rcu(mas->tree) ? 1 : 0;
4319 break;
4320 case wr_append:
4321 case wr_exact_fit:
4322 case wr_slot_store:
4323 ret = 0;
4324 }
4325
4326 return ret;
4327 }
4328

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