Re: [Patch v4 1/2] freezer: check OOM kill while being frozen

From: Cong Wang
Date: Wed Sep 10 2014 - 19:21:01 EST


On Wed, Sep 10, 2014 at 4:38 PM, Rafael J. Wysocki <rjw@xxxxxxxxxxxxx> wrote:
> On Wednesday, September 10, 2014 01:30:25 PM Cong Wang wrote:
>> I am wondering if the folllowing check makes any sense with regarding
>> to rule out PM freeze:
>>
>> if ((!pm_nosig_freezing && !pm_freezing) &&
>> cgroup_freezing(current) && test_thread_flag(TIF_MEMDIE))
>> return true;
>>
>> PM will not freeze the process again if it is already frozen by cgroup,
>> so here we just make sure that it will not thaw if PM freeze is in progress.
>>
>> Am I missing anything?
>
> And where exactly would you like to put this check? Into __refrigerator()?

Yes, still in should_thaw_current() which is called in __refrigerator().
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/