Re: [PATCH printk v3 2/7] printk: nbcon: Add acquire/release logic

From: kernel test robot
Date: Sun Sep 03 2023 - 12:48:54 EST


Hi John,

kernel test robot noticed the following build warnings:

[auto build test WARNING on cb65d08d735e00cc55ad7700a82a453bb88c93a3]

url: https://github.com/intel-lab-lkp/linux/commits/John-Ogness/printk-Add-non-BKL-nbcon-console-basic-infrastructure/20230903-230708
base: cb65d08d735e00cc55ad7700a82a453bb88c93a3
patch link: https://lore.kernel.org/r/20230903150539.245076-3-john.ogness%40linutronix.de
patch subject: [PATCH printk v3 2/7] printk: nbcon: Add acquire/release logic
config: riscv-allnoconfig (https://download.01.org/0day-ci/archive/20230904/202309040026.yyn4RCk1-lkp@xxxxxxxxx/config)
compiler: riscv64-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20230904/202309040026.yyn4RCk1-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/202309040026.yyn4RCk1-lkp@xxxxxxxxx/

All warnings (new ones prefixed by >>):

>> kernel/printk/nbcon.c:392: warning: expecting prototype for nbcon_context_try_acquire_hostile(). Prototype was for nbcon_context_acquire_hostile() instead


vim +392 kernel/printk/nbcon.c

379
380 /**
381 * nbcon_context_try_acquire_hostile - Acquire via unsafe hostile takeover
382 * @ctxt: The context of the caller
383 * @cur: The current console state
384 *
385 * @cur is updated to the new console state.
386 *
387 * The general procedure is to set @prio (forcing ownership). This method
388 * must only be used as a final attempt during panic.
389 */
390 static void nbcon_context_acquire_hostile(struct nbcon_context *ctxt,
391 struct nbcon_state *cur)
> 392 {
393 unsigned int cpu = smp_processor_id();
394 struct console *con = ctxt->console;
395 struct nbcon_state new;
396
397 do {
398 new.atom = cur->atom;
399 new.cpu = cpu;
400 new.prio = ctxt->prio;
401 new.unsafe |= cur->unsafe_takeover;
402 new.unsafe_takeover |= cur->unsafe;
403
404 } while (!nbcon_state_try_cmpxchg(con, cur, &new));
405
406 cur->atom = new.atom;
407 }
408

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