Re: [PATCH v3] zone_reclaim is always 0 by default

[Robin Holt <holt@sgi.com>]

On Wed, May 27, 2009 at 05:06:18PM +0900, KOSAKI Motohiro wrote:
> your last patch is one of considerable thing. but it has one weakness.
> in general "ifdef x86" is wrong idea. almost minor architecture don't
> have sufficient tester. the difference against x86 often makes bug.
> Then, unnecessary difference is hated by much people.

Let me start by saying I can barely understand this entire email.
I appreciate that english is a second language for you and you are
doing a service to the linux community with your contributions despite
the language barrier.  I commend you for your efforts.  I do ask that if
there was more information contained in your email than I am replying too,
please reword it so I may understand.

IIRC, my last patch made it an arch header option to set zone_reclaim_mode
to any value it desired while leaving the default as 1.  The only arch
that changed the default was x86 (both 32 and 64 bit).  That seems the
least disruptive to existing users.

> So, I think we have two selectable choice.
> 
> 1. remove zone_reclaim default setting completely (this patch)
> 2. Only PowerPC and IA64 have default zone_reclaim_mode settings,
>    other architecture always use zone_reclaim_mode=0.

Looks like 2 is the inverse of my patch.  That is fine as well.  The only
reason I formed the patch with the default of 1 and override on x86 is
it was one less line of change and one less file.

Thanks,
Robin

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>