On 10/16, Cong Wang wrote: > > On Thu, Oct 16, 2014 at 2:11 PM, Oleg Nesterov <o...@redhat.com> wrote: > > > > But also I can't understand why this patch helps. The changelog says: > > > > do_send_sig_info will wake up the thread > > > > why? > > > > This is a question for Michal who rewrites my changelog: > > http://marc.info/?l=linux-kernel&m=140986986423092&w=2 > > :)
OK, I hope Michal can answer my question if you do not want to do this ;) So far I think this patch is not right. 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. Btw, I also do not understand the cgroup_freezing() check in should_thaw_current(), but this is another story. Oleg. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
