On Thu 09-07-15 14:07:37, David Rientjes wrote: > On Thu, 9 Jul 2015, Michal Hocko wrote: [...] > > So I am not > > sure it belongs outside of the oom killer proper. > > > > Umm it has nothing to do with oom killing, it quite obviously doesn't > belong in the oom killer. The naming of the API would disagree. To me register_oom_notifier sounds like a mechanism to be notified when we are oom. > It belongs prior to invoking the oom killer if memory could be freed. Shrinkers are there to reclaim and prevent from OOM. This API is a gray zone. It looks generic method for the notification yet it allows to prevent from oom killer. I can imagine somebody might abuse this interface to implement OOM killer policies. Anyway, I think it would be preferable to kill it altogether rather than play with its placing. It will always be a questionable API. [...] -- Michal Hocko SUSE Labs -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>