Messages in this thread | | | Date | Wed, 27 May 2009 04:50:06 -0500 | From | Robin Holt <> | Subject | Re: [PATCH v3] zone_reclaim is always 0 by default |
| |
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
| |