Subject: Re: [patch 4/4] mm, oom: remove statically defined arch functions of same name

From: Kamezawa Hiroyuki <[email-masked]>
Date: Thu, 15 Nov 2012 17:48:39 +0900

(2012/11/14 18:15), David Rientjes wrote:
> out_of_memory() is a globally defined function to call the oom killer.
> x86, sh, and powerpc all use a function of the same name within file
> scope in their respective fault.c unnecessarily.  Inline the functions
> into the pagefault handlers to clean the code up.
>
> Cc: Ingo Molnar <[email-masked]>
> Cc: "H. Peter Anvin" <[email-masked]>
> Cc: Thomas Gleixner <[email-masked]>
> Cc: Benjamin Herrenschmidt <[email-masked]>
> Cc: Paul Mackerras <[email-masked]>
> Cc: Paul Mundt <[email-masked]>
> Signed-off-by: David Rientjes <[email-masked]>

I think this is good.

Reviewed-by: KAMEZAWA Hiroyuki <[email-masked]>