All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: oe-kbuild@lists.linux.dev
Cc: lkp@intel.com
Subject: Re: [PATCH v4] ACPI: button: Report wakeup key only for power button wakeups
Date: Tue, 29 Sep 2026 04:52:14 +0200	[thread overview]
Message-ID: <202609290454.LJrhMCb3-lkp@intel.com> (raw)

:::::: 
:::::: Manual check reason: "linux-review patch is more than 7 days old, verify it wasn't already superseded"
:::::: 

BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
In-Reply-To: <20260922024232.816-1-baorliu@amd.com>
References: <20260922024232.816-1-baorliu@amd.com>
TO: "Baorui.Liu" <baorliu@amd.com>
TO: rafael@kernel.org
CC: lenb@kernel.org
CC: linux-acpi@vger.kernel.org
CC: linux-kernel@vger.kernel.org
CC: acpica-devel@lists.linuxfoundation.org
CC: "Baorui.Liu" <baorliu@amd.com>

Hi Baorui.Liu,

kernel test robot noticed the following build errors:

[auto build test ERROR on rafael-pm/linux-next]
[also build test ERROR on linus/master v7.3-rc5 next-20260928]
[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/Baorui-Liu/ACPI-button-Report-wakeup-key-only-for-power-button-wakeups/20260922-104232
base:   https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git linux-next
patch link:    https://lore.kernel.org/r/20260922024232.816-1-baorliu%40amd.com
patch subject: [PATCH v4] ACPI: button: Report wakeup key only for power button wakeups
:::::: branch date: 7 days ago
:::::: commit date: 7 days ago
config: x86_64-randconfig-2100-20260928 (https://download.01.org/0day-ci/archive/20260929/202609290454.LJrhMCb3-lkp@intel.com/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260929/202609290454.LJrhMCb3-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/r/202609290454.LJrhMCb3-lkp@intel.com/

All errors (new ones prefixed by >>):

   ld: vmlinux.o: in function `acpi_suspend_enter':
>> drivers/acpi/sleep.c:643:(.text+0x147f324): undefined reference to `acpi_button_power_wakeup_pending'


vim +643 drivers/acpi/sleep.c

e9b3aba887f47f drivers/acpi/sleep/main.c Rafael J. Wysocki     2007-07-17  589  
^1da177e4c3f41 drivers/acpi/sleep/main.c Linus Torvalds        2005-04-16  590  /**
2c6e33c366bff2 drivers/acpi/sleep/main.c Len Brown             2008-04-23  591   *	acpi_suspend_enter - Actually enter a sleep state.
e9b3aba887f47f drivers/acpi/sleep/main.c Rafael J. Wysocki     2007-07-17  592   *	@pm_state: ignored
^1da177e4c3f41 drivers/acpi/sleep/main.c Linus Torvalds        2005-04-16  593   *
50ad147aa09c82 drivers/acpi/sleep/main.c Rafael J. Wysocki     2007-07-24  594   *	Flush caches and go to sleep. For STR we have to call arch-specific
50ad147aa09c82 drivers/acpi/sleep/main.c Rafael J. Wysocki     2007-07-24  595   *	assembly, which in turn call acpi_enter_sleep_state().
^1da177e4c3f41 drivers/acpi/sleep/main.c Linus Torvalds        2005-04-16  596   *	It's unfortunate, but it works. Please fix if you're feeling frisky.
^1da177e4c3f41 drivers/acpi/sleep/main.c Linus Torvalds        2005-04-16  597   */
2c6e33c366bff2 drivers/acpi/sleep/main.c Len Brown             2008-04-23  598  static int acpi_suspend_enter(suspend_state_t pm_state)
^1da177e4c3f41 drivers/acpi/sleep/main.c Linus Torvalds        2005-04-16  599  {
^1da177e4c3f41 drivers/acpi/sleep/main.c Linus Torvalds        2005-04-16  600  	acpi_status status = AE_OK;
e9b3aba887f47f drivers/acpi/sleep/main.c Rafael J. Wysocki     2007-07-17  601  	u32 acpi_state = acpi_target_sleep_state;
979f11b060c0b3 drivers/acpi/sleep.c      Rafael J. Wysocki     2011-02-08  602  	int error;
^1da177e4c3f41 drivers/acpi/sleep/main.c Linus Torvalds        2005-04-16  603  
bb3632c6101b2f drivers/acpi/sleep.c      Todd E Brandt         2014-06-06  604  	trace_suspend_resume(TPS("acpi_suspend"), acpi_state, true);
e9b3aba887f47f drivers/acpi/sleep/main.c Rafael J. Wysocki     2007-07-17  605  	switch (acpi_state) {
e9b3aba887f47f drivers/acpi/sleep/main.c Rafael J. Wysocki     2007-07-17  606  	case ACPI_STATE_S1:
^1da177e4c3f41 drivers/acpi/sleep/main.c Linus Torvalds        2005-04-16  607  		barrier();
3f6f49c7854c92 drivers/acpi/sleep.c      Len Brown             2012-07-26  608  		status = acpi_enter_sleep_state(acpi_state);
^1da177e4c3f41 drivers/acpi/sleep/main.c Linus Torvalds        2005-04-16  609  		break;
^1da177e4c3f41 drivers/acpi/sleep/main.c Linus Torvalds        2005-04-16  610  
e9b3aba887f47f drivers/acpi/sleep/main.c Rafael J. Wysocki     2007-07-17  611  	case ACPI_STATE_S3:
d6a77ead21b69c drivers/acpi/sleep.c      Konrad Rzeszutek Wilk 2013-05-14  612  		if (!acpi_suspend_lowlevel)
d6a77ead21b69c drivers/acpi/sleep.c      Konrad Rzeszutek Wilk 2013-05-14  613  			return -ENOSYS;
f1a2003e22f6b5 drivers/acpi/sleep.c      Rafael J. Wysocki     2011-02-08  614  		error = acpi_suspend_lowlevel();
979f11b060c0b3 drivers/acpi/sleep.c      Rafael J. Wysocki     2011-02-08  615  		if (error)
979f11b060c0b3 drivers/acpi/sleep.c      Rafael J. Wysocki     2011-02-08  616  			return error;
f5ee87df7a4dab drivers/acpi/sleep.c      Hanjun Guo            2021-06-02  617  		pr_info("Low-level resume complete\n");
ef25ba04760159 drivers/acpi/sleep.c      Rafael J. Wysocki     2015-10-07  618  		pm_set_resume_via_firmware();
^1da177e4c3f41 drivers/acpi/sleep/main.c Linus Torvalds        2005-04-16  619  		break;
^1da177e4c3f41 drivers/acpi/sleep/main.c Linus Torvalds        2005-04-16  620  	}
bb3632c6101b2f drivers/acpi/sleep.c      Todd E Brandt         2014-06-06  621  	trace_suspend_resume(TPS("acpi_suspend"), acpi_state, false);
872d83d00f6702 drivers/acpi/sleep/main.c Arnaud Patard         2006-04-27  622  
b6dacf63e9fb2e drivers/acpi/sleep.c      Matthew Garrett       2010-05-11  623  	/* This violates the spec but is required for bug compatibility. */
50ffba1bd3120b drivers/acpi/sleep.c      Bob Moore             2009-02-23  624  	acpi_write_bit_register(ACPI_BITREG_SCI_ENABLE, 1);
65df78473ffbf3 drivers/acpi/sleep/main.c Rafael J. Wysocki     2008-11-26  625  
3f6f49c7854c92 drivers/acpi/sleep.c      Len Brown             2012-07-26  626  	/* Reprogram control registers */
3f6f49c7854c92 drivers/acpi/sleep.c      Len Brown             2012-07-26  627  	acpi_leave_sleep_state_prep(acpi_state);
c95d47a868f35c drivers/acpi/sleep/main.c Rafael J. Wysocki     2008-01-08  628  
81320b20cf47de drivers/acpi/sleep.c      Baorui.Liu            2026-09-22  629  	/*
81320b20cf47de drivers/acpi/sleep.c      Baorui.Liu            2026-09-22  630  	 * ACPI 3.0 (P62): OSPM should clear the fixed power-button status bit
81320b20cf47de drivers/acpi/sleep.c      Baorui.Liu            2026-09-22  631  	 * so the event does not reach userspace as KEY_POWER. Remember that a
81320b20cf47de drivers/acpi/sleep.c      Baorui.Liu            2026-09-22  632  	 * power-button wake occurred so button resume can report KEY_WAKEUP
81320b20cf47de drivers/acpi/sleep.c      Baorui.Liu            2026-09-22  633  	 * and acpi_pm_finish() can emit a PM wakeup event.
872d83d00f6702 drivers/acpi/sleep/main.c Arnaud Patard         2006-04-27  634  	 */
c10d7a13846bff drivers/acpi/sleep.c      Daniel Drake          2012-05-10  635  	if (ACPI_SUCCESS(status) && (acpi_state == ACPI_STATE_S3)) {
51468a9d3a8024 drivers/acpi/sleep.c      Al Stone              2013-11-20  636  		acpi_event_status pwr_btn_status = ACPI_EVENT_FLAG_DISABLED;
c10d7a13846bff drivers/acpi/sleep.c      Daniel Drake          2012-05-10  637  
c10d7a13846bff drivers/acpi/sleep.c      Daniel Drake          2012-05-10  638  		acpi_get_event_status(ACPI_EVENT_POWER_BUTTON, &pwr_btn_status);
c10d7a13846bff drivers/acpi/sleep.c      Daniel Drake          2012-05-10  639  
18864cc4892d20 drivers/acpi/sleep.c      Lv Zheng              2016-08-04  640  		if (pwr_btn_status & ACPI_EVENT_FLAG_STATUS_SET) {
872d83d00f6702 drivers/acpi/sleep/main.c Arnaud Patard         2006-04-27  641  			acpi_clear_event(ACPI_EVENT_POWER_BUTTON);
c10d7a13846bff drivers/acpi/sleep.c      Daniel Drake          2012-05-10  642  			pwr_btn_event_pending = true;
81320b20cf47de drivers/acpi/sleep.c      Baorui.Liu            2026-09-22 @643  			acpi_button_power_wakeup_pending();
c10d7a13846bff drivers/acpi/sleep.c      Daniel Drake          2012-05-10  644  		}
c10d7a13846bff drivers/acpi/sleep.c      Daniel Drake          2012-05-10  645  	}
872d83d00f6702 drivers/acpi/sleep/main.c Arnaud Patard         2006-04-27  646  
a3627f67b1d5dd drivers/acpi/sleep/main.c Shaohua Li            2007-06-20  647  	/*
22db06337f590d drivers/acpi/sleep.c      Rafael J. Wysocki     2023-06-14  648  	 * Disable all GPE and clear their status bits before interrupts are
22db06337f590d drivers/acpi/sleep.c      Rafael J. Wysocki     2023-06-14  649  	 * enabled. Some GPEs (like wakeup GPEs) have no handlers and this can
a508939e15356d drivers/acpi/sleep.c      Chu Guangqing         2025-11-25  650  	 * prevent them from producing spurious interrupts.
22db06337f590d drivers/acpi/sleep.c      Rafael J. Wysocki     2023-06-14  651  	 *
22db06337f590d drivers/acpi/sleep.c      Rafael J. Wysocki     2023-06-14  652  	 * acpi_leave_sleep_state() will reenable specific GPEs later.
22db06337f590d drivers/acpi/sleep.c      Rafael J. Wysocki     2023-06-14  653  	 *
22db06337f590d drivers/acpi/sleep.c      Rafael J. Wysocki     2023-06-14  654  	 * Because this code runs on one CPU with disabled interrupts (all of
22db06337f590d drivers/acpi/sleep.c      Rafael J. Wysocki     2023-06-14  655  	 * the other CPUs are offline at this time), it need not acquire any
22db06337f590d drivers/acpi/sleep.c      Rafael J. Wysocki     2023-06-14  656  	 * sleeping locks which may trigger an implicit preemption point even
22db06337f590d drivers/acpi/sleep.c      Rafael J. Wysocki     2023-06-14  657  	 * if there is no contention, so avoid doing that by using a low-level
22db06337f590d drivers/acpi/sleep.c      Rafael J. Wysocki     2023-06-14  658  	 * library routine here.
a3627f67b1d5dd drivers/acpi/sleep/main.c Shaohua Li            2007-06-20  659  	 */
22db06337f590d drivers/acpi/sleep.c      Rafael J. Wysocki     2023-06-14  660  	acpi_hw_disable_all_gpes();
d5a64513c6a171 drivers/acpi/sleep.c      Rafael J. Wysocki     2010-04-09  661  	/* Allow EC transactions to happen. */
c2b46d679b30c5 drivers/acpi/sleep.c      Lv Zheng              2016-08-03  662  	acpi_ec_unblock_transactions();
a3627f67b1d5dd drivers/acpi/sleep/main.c Shaohua Li            2007-06-20  663  
2a6b69765ad794 drivers/acpi/sleep.c      Matthew Garrett       2010-05-28  664  	suspend_nvs_restore();
2a6b69765ad794 drivers/acpi/sleep.c      Matthew Garrett       2010-05-28  665  
^1da177e4c3f41 drivers/acpi/sleep/main.c Linus Torvalds        2005-04-16  666  	return ACPI_SUCCESS(status) ? 0 : -EFAULT;
^1da177e4c3f41 drivers/acpi/sleep/main.c Linus Torvalds        2005-04-16  667  }
^1da177e4c3f41 drivers/acpi/sleep/main.c Linus Torvalds        2005-04-16  668  

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

             reply	other threads:[~2026-09-29  2:52 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-29  2:52 kernel test robot [this message]
  -- strict thread matches above, loose matches on Subject: below --
2026-09-17 14:31 [PATCH v3] ACPI: button: Report wakeup key only for power button wakeups Baorui.Liu
2026-09-22  2:42 ` [PATCH v4] " Baorui.Liu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202609290454.LJrhMCb3-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=oe-kbuild@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.