Hello, Sorry about the delay. On Tue, Oct 21, 2014 at 04:29:39PM +0200, Michal Hocko wrote: > Reduce the race window by checking all tasks after OOM killer has been Ugh... this is never a good direction to take. It often just ends up making bugs harder to reproduce and track down. > disabled. This is still not race free completely unfortunately because > oom_killer_disable cannot stop an already ongoing OOM killer so a task > might still wake up from the fridge and get killed without > freeze_processes noticing. Full synchronization of OOM and freezer is, > however, too heavy weight for this highly unlikely case. Both oom killing and PM freezing are exremely rare events and I have difficult time why their exclusion would be heavy weight. Care to elaborate? Overall, this is a lot of complexity for something which doesn't really fix the problem and the comments while referring to the race don't mention that the implemented "fix" is broken, which is pretty bad as it gives readers of the code a false sense of security and another hurdle to overcome in actually tracking down what went wrong if this thing ever shows up as an actual breakage. I'd strongly recommend implementing something which is actually correct. Thanks. -- tejun -- 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>