Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761488AbXJPJXV (ORCPT ); Tue, 16 Oct 2007 05:23:21 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1757124AbXJPJXN (ORCPT ); Tue, 16 Oct 2007 05:23:13 -0400 Received: from ebiederm.dsl.xmission.com ([166.70.28.69]:54483 "EHLO ebiederm.dsl.xmission.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756608AbXJPJXM convert rfc822-to-8bit (ORCPT ); Tue, 16 Oct 2007 05:23:12 -0400 From: ebiederm@xmission.com (Eric W. Biederman) To: Christian Borntraeger Cc: Andrew Morton , Nick Piggin , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Martin Schwidefsky , "Theodore Ts'o" , stable@kernel.org Subject: Re: [PATCH] rd: Mark ramdisk buffers heads dirty References: <200710151028.34407.borntraeger@de.ibm.com> <200710160956.58061.borntraeger@de.ibm.com> Date: Tue, 16 Oct 2007 03:22:30 -0600 In-Reply-To: <200710160956.58061.borntraeger@de.ibm.com> (Christian Borntraeger's message of "Tue, 16 Oct 2007 09:56:57 +0200") Message-ID: User-Agent: Gnus/5.110006 (No Gnus v0.6) Emacs/21.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1539 Lines: 38 Christian Borntraeger writes: > Am Dienstag, 16. Oktober 2007 schrieb Eric W. Biederman: > >>?fs/buffer.c | ? ?3 +++ >> 1 files changed, 3 insertions(+), 0 deletions(-) >> drivers/block/rd.c | 13 +------------ >> 1 files changed, 1 insertions(+), 12 deletions(-) > > Your patches look sane so far. I have applied both patches, and the problem > seems gone. I will try to get these patches to our testers. > > As long as they dont find new problems: Sounds good. Please make certain to test reiserfs as well as ext2+ as it seems to strain the ramdisk code more aggressively. > Acked-by: Christian Borntraeger > > Nick, Eric. What is the best patch for the stable series? Both patches from > Eric or mine? I CCed stable, so they know that something is coming. My gut feel says my patches assuming everything tests out ok, as they actually fix the problem instead of papering over it, and there isn't really a size difference. In addition the change to init_page_buffers is interesting all by itself. With that patch we now have the option of running block devices in mode where we don't bother to cache the buffer heads which should reduce memory pressure a bit. Not that an enhancement like that will show up in stable, but... Eric - 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/