From mboxrd@z Thu Jan 1 00:00:00 1970 Message-ID: <447D8725.4060506@shadowen.org> Date: Wed, 31 May 2006 13:08:05 +0100 From: Andy Whitcroft MIME-Version: 1.0 Subject: Re: [stable] [PATCH 0/2] Zone boundary alignment fixes, default configuration References: <447173EF.9090000@shadowen.org> <20060531001322.GJ18769@moss.sous-sol.org> <447D80ED.7070403@yahoo.com.au> In-Reply-To: <447D80ED.7070403@yahoo.com.au> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org Return-Path: To: Nick Piggin Cc: Chris Wright , Andrew Morton , Mel Gorman , stable@kernel.org, Linux Memory Management List-ID: Nick Piggin wrote: > Chris Wright wrote: > >> * Andy Whitcroft (apw@shadowen.org) wrote: >> >>> I think a concensus is forming that the checks for merging across >>> zones were removed from the buddy allocator without anyone noticing. >>> So I propose that the configuration option UNALIGNED_ZONE_BOUNDARIES >>> default to on, and those architectures which have been auditied >>> for alignment may turn it off. >> >> >> >> So what's the final outcome here for -stable? The only >> relevant patch upstream appears to be Bob Picco's patch > > > I think you need zone checks? [ ie. page_zone(page) == page_zone(buddy) ] > I had assumed Andy was going to do a patch for that. The stack for the full optional check in -mm seems like a lot for a stable patch. I think for stable we should just add the check for unconditionally, its very light weight and safe that way. Am just putting together a patch for that now. Will respond to this email shortly with that patch once its been through a few tests. -apw -- 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: email@kvack.org