All of lore.kernel.org
 help / color / mirror / Atom feed
* [tip:sched/core 24/27] kernel/power/suspend.c:431:10: error: implicit declaration of function 'suspend_disable_secondary_cpus'
@ 2019-05-03 12:34 kbuild test robot
  2019-05-03 16:04 ` Peter Zijlstra
  0 siblings, 1 reply; 4+ messages in thread
From: kbuild test robot @ 2019-05-03 12:34 UTC (permalink / raw)
  To: Nicholas Piggin
  Cc: kbuild-all, linux-kernel, tipbuild, Ingo Molnar, Peter Zijlstra (Intel)

[-- Attachment #1: Type: text/plain, Size: 3989 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git sched/core
head:   65874bd36e6ae3028539e989bfb5c28ad457368e
commit: c2cb30bfceceba8a2a0d5713230a250dd6140e22 [24/27] power/suspend: Add function to disable secondaries for suspend
config: x86_64-randconfig-l3-05031806 (attached as .config)
compiler: gcc-5 (Debian 5.5.0-3) 5.4.1 20171010
reproduce:
        git checkout c2cb30bfceceba8a2a0d5713230a250dd6140e22
        # save the attached .config to linux build tree
        make ARCH=x86_64 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   kernel/power/suspend.c: In function 'suspend_enter':
>> kernel/power/suspend.c:431:10: error: implicit declaration of function 'suspend_disable_secondary_cpus' [-Werror=implicit-function-declaration]
     error = suspend_disable_secondary_cpus();
             ^
>> kernel/power/suspend.c:461:2: error: implicit declaration of function 'suspend_enable_secondary_cpus' [-Werror=implicit-function-declaration]
     suspend_enable_secondary_cpus();
     ^
   cc1: some warnings being treated as errors

vim +/suspend_disable_secondary_cpus +431 kernel/power/suspend.c

   389	
   390	/**
   391	 * suspend_enter - Make the system enter the given sleep state.
   392	 * @state: System sleep state to enter.
   393	 * @wakeup: Returns information that the sleep state should not be re-entered.
   394	 *
   395	 * This function should be called after devices have been suspended.
   396	 */
   397	static int suspend_enter(suspend_state_t state, bool *wakeup)
   398	{
   399		int error;
   400	
   401		error = platform_suspend_prepare(state);
   402		if (error)
   403			goto Platform_finish;
   404	
   405		error = dpm_suspend_late(PMSG_SUSPEND);
   406		if (error) {
   407			pr_err("late suspend of devices failed\n");
   408			goto Platform_finish;
   409		}
   410		error = platform_suspend_prepare_late(state);
   411		if (error)
   412			goto Devices_early_resume;
   413	
   414		if (state == PM_SUSPEND_TO_IDLE && pm_test_level != TEST_PLATFORM) {
   415			s2idle_loop();
   416			goto Platform_early_resume;
   417		}
   418	
   419		error = dpm_suspend_noirq(PMSG_SUSPEND);
   420		if (error) {
   421			pr_err("noirq suspend of devices failed\n");
   422			goto Platform_early_resume;
   423		}
   424		error = platform_suspend_prepare_noirq(state);
   425		if (error)
   426			goto Platform_wake;
   427	
   428		if (suspend_test(TEST_PLATFORM))
   429			goto Platform_wake;
   430	
 > 431		error = suspend_disable_secondary_cpus();
   432		if (error || suspend_test(TEST_CPUS))
   433			goto Enable_cpus;
   434	
   435		arch_suspend_disable_irqs();
   436		BUG_ON(!irqs_disabled());
   437	
   438		system_state = SYSTEM_SUSPEND;
   439	
   440		error = syscore_suspend();
   441		if (!error) {
   442			*wakeup = pm_wakeup_pending();
   443			if (!(suspend_test(TEST_CORE) || *wakeup)) {
   444				trace_suspend_resume(TPS("machine_suspend"),
   445					state, true);
   446				error = suspend_ops->enter(state);
   447				trace_suspend_resume(TPS("machine_suspend"),
   448					state, false);
   449			} else if (*wakeup) {
   450				error = -EBUSY;
   451			}
   452			syscore_resume();
   453		}
   454	
   455		system_state = SYSTEM_RUNNING;
   456	
   457		arch_suspend_enable_irqs();
   458		BUG_ON(irqs_disabled());
   459	
   460	 Enable_cpus:
 > 461		suspend_enable_secondary_cpus();
   462	
   463	 Platform_wake:
   464		platform_resume_noirq(state);
   465		dpm_resume_noirq(PMSG_RESUME);
   466	
   467	 Platform_early_resume:
   468		platform_resume_early(state);
   469	
   470	 Devices_early_resume:
   471		dpm_resume_early(PMSG_RESUME);
   472	
   473	 Platform_finish:
   474		platform_resume_finish(state);
   475		return error;
   476	}
   477	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 32942 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [tip:sched/core 24/27] kernel/power/suspend.c:431:10: error: implicit declaration of function 'suspend_disable_secondary_cpus'
  2019-05-03 12:34 [tip:sched/core 24/27] kernel/power/suspend.c:431:10: error: implicit declaration of function 'suspend_disable_secondary_cpus' kbuild test robot
@ 2019-05-03 16:04 ` Peter Zijlstra
  2019-05-03 23:51   ` Nicholas Piggin
  0 siblings, 1 reply; 4+ messages in thread
From: Peter Zijlstra @ 2019-05-03 16:04 UTC (permalink / raw)
  To: kbuild test robot
  Cc: Nicholas Piggin, kbuild-all, linux-kernel, tipbuild, Ingo Molnar

On Fri, May 03, 2019 at 08:34:57PM +0800, kbuild test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git sched/core
> head:   65874bd36e6ae3028539e989bfb5c28ad457368e
> commit: c2cb30bfceceba8a2a0d5713230a250dd6140e22 [24/27] power/suspend: Add function to disable secondaries for suspend
> config: x86_64-randconfig-l3-05031806 (attached as .config)
> compiler: gcc-5 (Debian 5.5.0-3) 5.4.1 20171010
> reproduce:
>         git checkout c2cb30bfceceba8a2a0d5713230a250dd6140e22
>         # save the attached .config to linux build tree
>         make ARCH=x86_64 
> 

The below appears to fix.


--- a/include/linux/cpu.h
+++ b/include/linux/cpu.h
@@ -150,6 +150,8 @@ static inline void suspend_enable_second
 #else /* !CONFIG_PM_SLEEP_SMP */
 static inline int disable_nonboot_cpus(void) { return 0; }
 static inline void enable_nonboot_cpus(void) {}
+static inline int suspend_disable_secondary_cpus(void) { return 0; }
+static inline void suspend_enable_secondary_cpus(void) { }
 #endif /* !CONFIG_PM_SLEEP_SMP */
 
 void cpu_startup_entry(enum cpuhp_state state);

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [tip:sched/core 24/27] kernel/power/suspend.c:431:10: error: implicit declaration of function 'suspend_disable_secondary_cpus'
  2019-05-03 16:04 ` Peter Zijlstra
@ 2019-05-03 23:51   ` Nicholas Piggin
  2019-05-04  0:50     ` Ingo Molnar
  0 siblings, 1 reply; 4+ messages in thread
From: Nicholas Piggin @ 2019-05-03 23:51 UTC (permalink / raw)
  To: kbuild test robot, Peter Zijlstra
  Cc: kbuild-all, linux-kernel, Ingo Molnar, tipbuild

Peter Zijlstra's on May 4, 2019 2:04 am:
> On Fri, May 03, 2019 at 08:34:57PM +0800, kbuild test robot wrote:
>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git sched/core
>> head:   65874bd36e6ae3028539e989bfb5c28ad457368e
>> commit: c2cb30bfceceba8a2a0d5713230a250dd6140e22 [24/27] power/suspend: Add function to disable secondaries for suspend
>> config: x86_64-randconfig-l3-05031806 (attached as .config)
>> compiler: gcc-5 (Debian 5.5.0-3) 5.4.1 20171010
>> reproduce:
>>         git checkout c2cb30bfceceba8a2a0d5713230a250dd6140e22
>>         # save the attached .config to linux build tree
>>         make ARCH=x86_64 
>> 
> 
> The below appears to fix.
> 
> 
> --- a/include/linux/cpu.h
> +++ b/include/linux/cpu.h
> @@ -150,6 +150,8 @@ static inline void suspend_enable_second
>  #else /* !CONFIG_PM_SLEEP_SMP */
>  static inline int disable_nonboot_cpus(void) { return 0; }
>  static inline void enable_nonboot_cpus(void) {}
> +static inline int suspend_disable_secondary_cpus(void) { return 0; }
> +static inline void suspend_enable_secondary_cpus(void) { }
>  #endif /* !CONFIG_PM_SLEEP_SMP */
>  
>  void cpu_startup_entry(enum cpuhp_state state);
> 

Oops, thanks for that, it looks okay.

Thanks,
Nick

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [tip:sched/core 24/27] kernel/power/suspend.c:431:10: error: implicit declaration of function 'suspend_disable_secondary_cpus'
  2019-05-03 23:51   ` Nicholas Piggin
@ 2019-05-04  0:50     ` Ingo Molnar
  0 siblings, 0 replies; 4+ messages in thread
From: Ingo Molnar @ 2019-05-04  0:50 UTC (permalink / raw)
  To: Nicholas Piggin
  Cc: kbuild test robot, Peter Zijlstra, kbuild-all, linux-kernel,
	tipbuild, Frédéric Weisbecker


* Nicholas Piggin <npiggin@gmail.com> wrote:

> Peter Zijlstra's on May 4, 2019 2:04 am:
> > On Fri, May 03, 2019 at 08:34:57PM +0800, kbuild test robot wrote:
> >> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git sched/core
> >> head:   65874bd36e6ae3028539e989bfb5c28ad457368e
> >> commit: c2cb30bfceceba8a2a0d5713230a250dd6140e22 [24/27] power/suspend: Add function to disable secondaries for suspend
> >> config: x86_64-randconfig-l3-05031806 (attached as .config)
> >> compiler: gcc-5 (Debian 5.5.0-3) 5.4.1 20171010
> >> reproduce:
> >>         git checkout c2cb30bfceceba8a2a0d5713230a250dd6140e22
> >>         # save the attached .config to linux build tree
> >>         make ARCH=x86_64 
> >> 
> > 
> > The below appears to fix.
> > 
> > 
> > --- a/include/linux/cpu.h
> > +++ b/include/linux/cpu.h
> > @@ -150,6 +150,8 @@ static inline void suspend_enable_second
> >  #else /* !CONFIG_PM_SLEEP_SMP */
> >  static inline int disable_nonboot_cpus(void) { return 0; }
> >  static inline void enable_nonboot_cpus(void) {}
> > +static inline int suspend_disable_secondary_cpus(void) { return 0; }
> > +static inline void suspend_enable_secondary_cpus(void) { }
> >  #endif /* !CONFIG_PM_SLEEP_SMP */
> >  
> >  void cpu_startup_entry(enum cpuhp_state state);
> > 
> 
> Oops, thanks for that, it looks okay.

I back-merged the fix into tip:sched/core.

For these bits to make it upstream in the merge window which starts in 
two days, Frederic's questions about this patch need to be addressed:

   Re: [tip:sched/core] sched/isolation: Require a present CPU in housekeeping mask

Thanks,

	Ingo

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-05-04  0:50 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-03 12:34 [tip:sched/core 24/27] kernel/power/suspend.c:431:10: error: implicit declaration of function 'suspend_disable_secondary_cpus' kbuild test robot
2019-05-03 16:04 ` Peter Zijlstra
2019-05-03 23:51   ` Nicholas Piggin
2019-05-04  0:50     ` Ingo Molnar

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.