Re: [PATCH v2] mm: Enable suspend-only swap spaces

On 27.07.21 14:04, Pavel Machek wrote:

Hi!

quoted

Let's take a look at hibernate() callers:

drivers/mfd/tps65010.c: calls hibernate() from IRQ contex, based on HW
                         event

No it does not. Look again.

Oh, dead code :)

quoted

kernel/power/autosleep.c: calls hibernate() when it thinks it might be
  			  a good time to go to sleep

Ok, you are right, it is there. But I don't believe anyone uses this
configuration.

If it's dead code, we might want to look into deprecating and removing it. It was introduced around 2012:

https://lwn.net/Articles/479841/


-- 
Thanks,

David / dhildenb