This code has a compile problem. Missing argument to spin_lock_irqsave(). regards, dan carpenter
On Wed, Apr 09, 2014 at 01:34:15AM +0800, kbuild test robot wrote: > TO: Jet Chen <jet.c...@intel.com> > > tree: git://bee.sh.intel.com/git/tchen37/linux.git timer_debug3 > head: c7c05c6a3c9c585a0d74c052a2908bf159d09f7f > commit: c7c05c6a3c9c585a0d74c052a2908bf159d09f7f [2/2] Should use > SOFTIRQ-safe lock in this scenario > :::::: branch date: 66 minutes ago > :::::: commit date: 66 minutes ago > > kernel/time/clocksource.c:367 clocksource_cpu_notify() warn: statement has no > effect 3 > > git remote add tchen git://bee.sh.intel.com/git/tchen37/linux.git > git remote update tchen > git checkout c7c05c6a3c9c585a0d74c052a2908bf159d09f7f > vim +367 kernel/time/clocksource.c > > 9fb60336 Thomas Gleixner 2011-09-12 351 * We only clear the > watchdog_reset_pending, when we did a > 9fb60336 Thomas Gleixner 2011-09-12 352 * full cycle through all > clocksources. > 9fb60336 Thomas Gleixner 2011-09-12 353 */ > 9fb60336 Thomas Gleixner 2011-09-12 354 if (reset_pending) > 9fb60336 Thomas Gleixner 2011-09-12 355 > atomic_dec(&watchdog_reset_pending); > 9fb60336 Thomas Gleixner 2011-09-12 356 > 90585a71 Viresh Kumar 2014-04-05 357 queue_timer_on_next_cpu(); > fb63a0eb Martin Schwidefsky 2009-08-14 358 out: > 5d8b34fd Thomas Gleixner 2007-02-16 359 spin_unlock(&watchdog_lock); > 5d8b34fd Thomas Gleixner 2007-02-16 360 } > 0f8e8ef7 Martin Schwidefsky 2009-08-14 361 > 90585a71 Viresh Kumar 2014-04-05 362 static int > clocksource_cpu_notify(struct notifier_block *self, > 90585a71 Viresh Kumar 2014-04-05 363 > unsigned long action, void *hcpu) > 90585a71 Viresh Kumar 2014-04-05 364 { > 90585a71 Viresh Kumar 2014-04-05 365 long cpu = (long)hcpu; > 90585a71 Viresh Kumar 2014-04-05 366 > c7c05c6a Jet Chen 2014-04-09 @367 > spin_lock_irqsave(&watchdog_lock); > 90585a71 Viresh Kumar 2014-04-05 368 if (!watchdog_running) > 90585a71 Viresh Kumar 2014-04-05 369 goto notify_out; > 90585a71 Viresh Kumar 2014-04-05 370 > 90585a71 Viresh Kumar 2014-04-05 371 switch (action) { > 90585a71 Viresh Kumar 2014-04-05 372 case CPU_DEAD: > 90585a71 Viresh Kumar 2014-04-05 373 case CPU_DEAD_FROZEN: > 90585a71 Viresh Kumar 2014-04-05 374 if (cpu != timer_cpu) > 90585a71 Viresh Kumar 2014-04-05 375 break; > > --- > 0-DAY kernel build testing backend Open Source Technology Center > http://lists.01.org/mailman/listinfo/kbuild Intel Corporation _______________________________________________ kbuild mailing list kbuild@lists.01.org https://lists.01.org/mailman/listinfo/kbuild