Commit 5e8be76a authored by Zhongqiu Han's avatar Zhongqiu Han Committed by Rafael J. Wysocki
Browse files

PM: sleep: Drop superfluous might_sleep() calls 



Drop superfluous might_sleep() calls from dpm_resume(), dpm_complete(),
and dpm_prepare(). These functions already invoke primitives that
implicitly check for sleep in atomic context:

 - dpm_resume() and dpm_complete() invoke mutex_lock(), which internally
   triggers might_sleep().

 - dpm_prepare() calls wait_for_device_probe(), which internally uses
   flush_work(), and thus might_sleep().

These annotations are unnecessary and can be dropped to reduce clutter.

Signed-off-by: default avatarZhongqiu Han <quic_zhonhan@quicinc.com>
Link: https://patch.msgid.link/20250617084650.341262-1-quic_zhonhan@quicinc.com


Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent 4266e8fa

drivers/base/power/main.c

+0 −3
Original line number Diff line number Diff line
@@ -1095,7 +1095,6 @@ void dpm_resume(pm_message_t state) 
	ktime_t starttime = ktime_get();



	trace_suspend_resume(TPS("dpm_resume"), state.event, true);

	might_sleep();



	pm_transition = state;

	async_error = 0;
@@ -1198,7 +1197,6 @@ void dpm_complete(pm_message_t state) 
	struct list_head list;



	trace_suspend_resume(TPS("dpm_complete"), state.event, true);

	might_sleep();



	INIT_LIST_HEAD(&list);

	mutex_lock(&dpm_list_mtx);
@@ -2110,7 +2108,6 @@ int dpm_prepare(pm_message_t state) 
	int error = 0;



	trace_suspend_resume(TPS("dpm_prepare"), state.event, true);

	might_sleep();



	/*

	 * Give a chance for the known devices to complete their probes, before