Re: [patch -mm 4/9 v2] oom: remove compulsory panic_on_oom mode

From: KOSAKI Motohiro
Date: Tue Feb 16 2010 - 21:19:57 EST


> +/*
> + * The pagefault handler calls here because it is out of memory, so kill a
> + * memory-hogging task. If a populated zone has ZONE_OOM_LOCKED set, a parallel
> + * oom killing is already in progress so do nothing. If a task is found with
> + * TIF_MEMDIE set, it has been killed so do nothing and allow it to exit.
> + */
> +void pagefault_out_of_memory(void)
> +{
> + if (!try_set_system_oom())
> + return;
> + out_of_memory(NULL, 0, 0, NULL);
> + clear_system_oom();
> +}

At least, I agree pagefault oom part. it need ZONE_OOM_LOCKED too.
if you make separated patch, I'll ack it. I don't know memcg part is
correct or not.



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/