linux-kernel - Re: + freezer-check-oom-kill-while-being-frozen.patch added to -mm tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 17 Oct 2014 00:22:37 +0200
From:	Oleg Nesterov <oleg@...hat.com>
To:	Cong Wang <xiyou.wangcong@...il.com>
Cc:	Michal Hocko <mhocko@...e.cz>,
	David Rientjes <rientjes@...gle.com>,
	"Rafael J. Wysocki" <rjw@...ysocki.net>, Tejun Heo <tj@...nel.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: + freezer-check-oom-kill-while-being-frozen.patch added to -mm
	tree

On 10/16, Cong Wang wrote:
>
> On Thu, Oct 16, 2014 at 2:35 PM, Oleg Nesterov <oleg@...hat.com> wrote:
> >
> > If a task B is already frozen, it sleeps in D state.
> >
> > If OOM selects B as a victim after that, it won't be woken by
> > SIGKILL, thus it obviously can't call should_thaw_current() and
> > notice TIF_MEMDIE.
>
> I see your point now, it would be more clear if you can just quote
> the patch instead of changelog.
>
> So are you saying the loop in __refrigerator() is useless?

No.

> Since
> it will always stay in asleep after schedule()?

Not always. But it will stay asleep in this particular case.

> > Btw, I also do not understand the cgroup_freezing() check in
> > should_thaw_current(), but this is another story.
>
> I hate to repeat the previous discussion. Maybe you can just follow
> the link I gave to you? :)

May be, but this thread is huge. Will try tomorrow to read it tomorrow,
but you know, I hope that someone else from cc list can copy-and-paste the
relevant part of this discussion, or give me the link to some specific
email. To me the comment should be more clear in any case, but perhaps
it is just me who can't understand it immediately.

Oleg.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/