From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Fri, 28 Feb 2003 10:46:37 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Fri, 28 Feb 2003 10:46:36 -0500 Received: from pixpat.austin.ibm.com ([192.35.232.241]:54662 "EHLO baldur.austin.ibm.com") by vger.kernel.org with ESMTP id ; Fri, 28 Feb 2003 10:46:33 -0500 Date: Fri, 28 Feb 2003 09:56:34 -0600 From: Dave McCracken To: Hugh Dickins , Andrew Morton cc: Con Kolivas , linux-kernel@vger.kernel.org, linux-mm@kvack.org Subject: Re: Rising io_load results Re: 2.5.63-mm1 Message-ID: <3050000.1046447794@baldur.austin.ibm.com> In-Reply-To: References: X-Mailer: Mulberry/2.2.1 (Linux/x86) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org --On Friday, February 28, 2003 12:48:06 +0000 Hugh Dickins wrote: > On Thu, 27 Feb 2003, Andrew Morton wrote: >> >> No, it is still wrong. Mapped cannot exceed MemTotal. > > It needs this in addition to Dave's patch from yesterday: > > --- 2.5.63-objfix-1/mm/rmap.c Thu Feb 27 23:37:28 2003 > +++ 2.5.63-objfix-2/mm/rmap.c Fri Feb 28 12:33:58 2003 > @@ -349,7 +349,8 @@ > BUG(); > if (atomic_read(&page->pte.mapcount) == 0) > BUG(); > - atomic_dec(&page->pte.mapcount); > + if (atomic_dec_and_test(&page->pte.mapcount)) > + dec_page_state(nr_mapped); > return; > } D'oh. I should have seen that one. Thanks. Dave McCracken ====================================================================== Dave McCracken IBM Linux Base Kernel Team 1-512-838-3059 dmccr@us.ibm.com T/L 678-3059