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
| ||
|
Date: Wed, 26 Jul 2017 16:24:34 +0200 From: Michal Hocko <mhocko@...nel.org> To: Roman Gushchin <guro@...com> Cc: linux-mm@...ck.org, Vladimir Davydov <vdavydov.dev@...il.com>, Johannes Weiner <hannes@...xchg.org>, Tetsuo Handa <penguin-kernel@...ove.SAKURA.ne.jp>, David Rientjes <rientjes@...gle.com>, Tejun Heo <tj@...nel.org>, kernel-team@...com, cgroups@...r.kernel.org, linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [v4 1/4] mm, oom: refactor the TIF_MEMDIE usage On Wed 26-07-17 15:06:07, Roman Gushchin wrote: > On Wed, Jul 26, 2017 at 03:56:22PM +0200, Michal Hocko wrote: > > On Wed 26-07-17 14:27:15, Roman Gushchin wrote: > > [...] > > > @@ -656,13 +658,24 @@ static void mark_oom_victim(struct task_struct *tsk) > > > struct mm_struct *mm = tsk->mm; > > > > > > WARN_ON(oom_killer_disabled); > > > - /* OOM killer might race with memcg OOM */ > > > - if (test_and_set_tsk_thread_flag(tsk, TIF_MEMDIE)) > > > + > > > + if (!cmpxchg(&tif_memdie_owner, NULL, current)) { > > > + struct task_struct *t; > > > + > > > + rcu_read_lock(); > > > + for_each_thread(current, t) > > > + set_tsk_thread_flag(t, TIF_MEMDIE); > > > + rcu_read_unlock(); > > > + } > > > > I would realy much rather see we limit the amount of memory reserves oom > > victims can consume rather than build on top of the current hackish > > approach of limiting the number of tasks because the fundamental problem > > is still there (a heavy multithreaded process can still deplete the > > reserves completely). > > > > Is there really any reason to not go with the existing patch I've > > pointed to the last time around? You didn't seem to have any objects > > back then. > > Hi Michal! > > I had this patch in mind and mentioned in the commit log, that TIF_MEMDIE > as an memory reserves access indicator will probably be eliminated later. > > But that patch is not upstream yet, and it's directly related to the theme. > The proposed refactoring of TIF_MEMDIE usage is not against your approach, > and will not make harder to go this way further. So what is the reason to invent another tricky way to limit access to memory reserves? The patch is not upstream but nothin prevents you from picking it up and post along with your series. Or if you prefer I can post it separately? > I'm slightly concerned about an idea to give TIF_MEMDIE to all tasks > in case we're killing a really large cgroup. Why? There shouldn't be any issue if the access is limited. -- Michal Hocko SUSE Labs
Powered by blists - more mailing lists