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

[David Hildenbrand]

On 27.07.21 14:04, Pavel Machek wrote:
> Hi!
>
> > 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 :)

> > 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