On Tue 03-07-18 23:25:01, Tetsuo Handa wrote: > This series provides > > (1) Mitigation and a fix for CVE-2016-10723. > > (2) A mitigation for needlessly selecting next OOM victim reported > by David Rientjes and rejected by Michal Hocko. > > (3) A preparation for handling many concurrent OOM victims which > could become real by introducing memcg-aware OOM killer. It would have been great to describe the overal design in the cover letter. So let me summarize just to be sure I understand the proposal. You are removing the oom_reaper and moving the oom victim tear down to the oom path. To handle cases where we cannot get mmap_sem to do that work you simply decay oom_badness over time if there are no changes in the victims oom score. In order to not block in the oom context for too long because the address space might be quite large, you allow to direct oom reap from multiple contexts. You fail to explain why is this approach more appropriate and how you have settled with your current tuning with 3s timeout etc... Considering how subtle this whole area is I am not overly happy about another rewrite without a really strong reasoning behind. There is none here, unfortunately. Well, except for statements how I reject something without telling the whole story etc... > Tetsuo Handa (7): > mm,oom: Don't call schedule_timeout_killable() with oom_lock held. > mm,oom: Check pending victims earlier in out_of_memory(). > mm,oom: Fix unnecessary killing of additional processes. > mm,page_alloc: Make oom_reserves_allowed() even. > mm,oom: Bring OOM notifier to outside of oom_lock. > mm,oom: Make oom_lock static variable. > mm,oom: Do not sleep with oom_lock held. > Michal Hocko (1): > mm,page_alloc: Move the short sleep to should_reclaim_retry(). > > drivers/tty/sysrq.c | 2 - > include/linux/memcontrol.h | 9 +- > include/linux/oom.h | 6 +- > include/linux/sched.h | 7 +- > include/trace/events/oom.h | 64 ------- > kernel/fork.c | 2 + > mm/memcontrol.c | 24 +-- > mm/mmap.c | 17 +- > mm/oom_kill.c | 439 +++++++++++++++++++++------------------------ > mm/page_alloc.c | 134 ++++++-------- > 10 files changed, 287 insertions(+), 417 deletions(-) > > -- > 1.8.3.1 -- Michal Hocko SUSE Labs
