PM: sleep: Remove obsolete comment from unlock_system_sleep()
authorKevin Hao <haokexin@gmail.com>
Wed, 20 Dec 2023 00:35:35 +0000 (08:35 +0800)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 20 Dec 2023 18:16:55 +0000 (19:16 +0100)
With the freezer changes introduced by commit f5d39b020809
("freezer,sched: Rewrite core freezer logic"), the comment in
unlock_system_sleep() has become obsolete, there is no need to
retain it.

Signed-off-by: Kevin Hao <haokexin@gmail.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
kernel/power/main.c

index f6425ae..b1ae9b6 100644 (file)
@@ -60,22 +60,6 @@ EXPORT_SYMBOL_GPL(lock_system_sleep);
 
 void unlock_system_sleep(unsigned int flags)
 {
-       /*
-        * Don't use freezer_count() because we don't want the call to
-        * try_to_freeze() here.
-        *
-        * Reason:
-        * Fundamentally, we just don't need it, because freezing condition
-        * doesn't come into effect until we release the
-        * system_transition_mutex lock, since the freezer always works with
-        * system_transition_mutex held.
-        *
-        * More importantly, in the case of hibernation,
-        * unlock_system_sleep() gets called in snapshot_read() and
-        * snapshot_write() when the freezing condition is still in effect.
-        * Which means, if we use try_to_freeze() here, it would make them
-        * enter the refrigerator, thus causing hibernation to lockup.
-        */
        if (!(flags & PF_NOFREEZE))
                current->flags &= ~PF_NOFREEZE;
        mutex_unlock(&system_transition_mutex);