On Saturday, September 06, 2014 07:45:54 AM Tejun Heo wrote: > Hello, > > On Fri, Sep 05, 2014 at 11:12:24AM -0700, Cong Wang wrote: > > > Rafael, can you please help? > > > > Rafael is known not responsive at least for this topic. :) > > :(
Well, am I? I haven't commented patches in this thread so far, mostly because other people have. How can I help actually? > > > Shouldn't the primary goal of the comment be explaining why we need > > > TIF_MEMDIE check there at all anyway? The deadlock possiblity is not > > > very obvious. > > > > The changelog is not long enough?? ;-) I hate to copy+paste changelog > > into comments, changelog is essentially necessary for people to understand > > kernel code (at least networking) , so I don't think we have to move it > > into comments in this case. > > It doesn't have to be the same text but the current comment is > basically content-less. e.g. it can just say "OOM killer may get > stuck trying to kill a cgroup frozen task" and actualy provide > information on what condition the conditional tries to address. Or something like "We need to check X to prevent Y from happening". Rafael -- 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/