All of lore.kernel.org
 help / color / mirror / Atom feed
* [frederic-dynticks:timers/anna 23/29] kernel/time/timer_migration.c:1379: warning: Function parameter or struct member 'nextevt' not described in 'tmigr_quick_check'
@ 2024-02-10  5:09 kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2024-02-10  5:09 UTC (permalink / raw)
  To: Anna-Maria Behnsen; +Cc: oe-kbuild-all, Frederic Weisbecker

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/frederic/linux-dynticks.git timers/anna
head:   9a654aebd02182bcaff7e5a26b632ee20a1536ad
commit: a7631399e1db94d34d3cc5bae11e1bfcbb02db30 [23/29] do not forget quick check
config: x86_64-defconfig (https://download.01.org/0day-ci/archive/20240210/202402101310.rPIZXVPJ-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-12) 11.3.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240210/202402101310.rPIZXVPJ-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/202402101310.rPIZXVPJ-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> kernel/time/timer_migration.c:1379: warning: Function parameter or struct member 'nextevt' not described in 'tmigr_quick_check'
>> kernel/time/timer_migration.c:1379: warning: Excess function parameter 'nextexp' description in 'tmigr_quick_check'


vim +1379 kernel/time/timer_migration.c

0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1359  
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1360  /**
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1361   * tmigr_quick_check() - Quick forecast of next tmigr event when CPU wants to
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1362   *			 go idle
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1363   * @nextexp:	The next global timer expiry of the current CPU
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1364   *
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1365   * Return:
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1366   * * KTIME_MAX		- when it is probable that nothing has to be done (not
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1367   *	 		  the only one in the level 0 group; and if it is the
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1368   *	 		  only one in level 0 group, but there are more than a
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1369   *	 		  single group active on the way to top level)
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1370   * * nextevt		- when CPU is offline and has to handle timer on his own
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1371   *			  or when on the way to top in every group only a single
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1372   *			  child is active and but @nextevt is before next_expiry
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1373   *			  of top level group
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1374    * * next_expiry (top)	- value of top level group, when on the way to top in
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1375   * 			  every group only a single child is active and @nextevt
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1376   * 			  is after this value active child.
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1377   */
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1378  u64 tmigr_quick_check(u64 nextevt)
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09 @1379  {
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1380  	struct tmigr_cpu *tmc = this_cpu_ptr(&tmigr_cpu);
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1381  	struct tmigr_group *group = tmc->tmgroup;
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1382  
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1383  	if (tmigr_is_not_available(tmc))
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1384  		return nextevt;
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1385  
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1386  	if (WARN_ON_ONCE(tmc->idle))
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1387  		return nextevt;
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1388  
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1389  	if (!tmigr_check_migrator_and_lonely(tmc->tmgroup, tmc->childmask))
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1390  		return KTIME_MAX;
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1391  
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1392  	do {
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1393  		if (!tmigr_check_lonely(group)) {
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1394  			return KTIME_MAX;
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1395  		} else if (!group->parent) {
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1396  			u64 first_global = READ_ONCE(group->next_expiry);
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1397  			return min_t(u64, nextevt, first_global);
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1398  		}
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1399  		group = group->parent;
a7631399e1db94 Anna-Maria Behnsen 2024-02-09  1400  	} while (group);
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1401  
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1402  	return KTIME_MAX;
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1403  }
0a3bafadcf1c9d Anna-Maria Behnsen 2024-02-09  1404  

:::::: The code at line 1379 was first introduced by commit
:::::: 0a3bafadcf1c9d0de98b840e6e906e8f86167a12 timers: Implement the hierarchical pull model

:::::: TO: Anna-Maria Behnsen <anna-maria@linutronix.de>
:::::: CC: Frederic Weisbecker <frederic@kernel.org>

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

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2024-02-10  5:10 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-02-10  5:09 [frederic-dynticks:timers/anna 23/29] kernel/time/timer_migration.c:1379: warning: Function parameter or struct member 'nextevt' not described in 'tmigr_quick_check' kernel test robot

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.