On Tue, Oct 30, 2012 at 1:30 PM, Luigi Semenzato <semenzato@xxxxxxxxxx> wrote:
> On Tue, Oct 30, 2012 at 12:12 PM, Luigi Semenzato <semenzato@xxxxxxxxxx> wrote:
>
>> OK, now someone is going to fix this, right? :-)
>
> Actually, there is a very simple fix:
>
> @@ -355,14 +364,6 @@ static struct task_struct
> *select_bad_process(unsigned int *ppoints,
> if (p == current) {
> chosen = p;
> *ppoints = 1000;
> - } else if (!force_kill) {
> - /*
> - * If this task is not being ptraced on exit,
> - * then wait for it to finish before killing
> - * some other task unnecessarily.
> - */
> - if (!(p->group_leader->ptrace & PT_TRACE_EXIT))
> - return ERR_PTR(-1UL);
> }
> }
>
> I'd rather kill some other task unnecessarily than hang! My load
> works fine with this change.
For completeness, I would like to report that the page fault in
mm_release looks legitimate. The fault happens near here:
if (unlikely(tsk->robust_list)) {
exit_robust_list(tsk);
tsk->robust_list = NULL;
}
and robust_list is a userspace structure.
--
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>