Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755560AbYFVRHY (ORCPT ); Sun, 22 Jun 2008 13:07:24 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751671AbYFVRHL (ORCPT ); Sun, 22 Jun 2008 13:07:11 -0400 Received: from gir.skynet.ie ([193.1.99.77]:41302 "EHLO gir.skynet.ie" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751975AbYFVRHK (ORCPT ); Sun, 22 Jun 2008 13:07:10 -0400 Date: Sun, 22 Jun 2008 18:07:05 +0100 From: Mel Gorman To: Christoph Lameter Cc: Alexander Beregalov , kernel-testers@vger.kernel.org, kernel list , linux-mm@kvack.org, Lee Schermerhorn , KAMEZAWA Hiroyuki , Hugh Dickins , Nick Piggin , Andrew Morton , Linus Torvalds , bfields@fieldses.org, neilb@suse.de, linux-nfs@vger.kernel.org Subject: Re: 2.6.26-rc: nfsd hangs for a few sec Message-ID: <20080622170704.GB625@csn.ul.ie> References: <20080621224135.GD4692@csn.ul.ie> <20080622013801.GE4692@csn.ul.ie> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.13 (2006-08-11) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1612 Lines: 41 On (21/06/08 21:13), Christoph Lameter didst pronounce: > On Sun, 22 Jun 2008, Mel Gorman wrote: > > > > Before the change we walk all zones of the zonelist. > > > > > > > Yeah, but the zonelist is for GFP_KERNEL so it should not include the HIGHMEM > > zones, right? The key change is that after the patch there are fewer zonelists > > than get filtered. > > But the HIGHMEM zones etc were included before. There was no check for > HIGHMEM etc there. The gfpmask was ignored. > Well, the mask is not totally ignored, it's part of the scan_control and used later when deciding what can and can't be done as part of reclaim. However, you are right in that it is apparently ignored for zone selection. However, try_to_free_pages() received a struct zone **zones which was a zonelist which is a zonelist->zones selected based on the gfp_mask in __alloc_pages. By the time shrink_zones() is called, it can ignore the mask because only relevant zones are in there. For GFP_KERNEL, that would exclude HIGHMEM. > > I think the effect of that patch is that zones get shrunk that have > > nothing to do with the requestors requirements. Right? > > Right. AFAICT That was the behavior before the change. > -- Mel Gorman Part-time Phd Student Linux Technology Center University of Limerick IBM Dublin Software Lab -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/