From: Marc-Christian Petersen <m.c.p@wolk-project.de>
Organization: Working Overloaded Linux Kernel
To: linux-kernel@vger.kernel.org
Subject: Re: 2.6.5-rc1-mm1
Date: Tue, 23 Mar 2004 14:05:56 +0100
User-Agent: KMail/1.6.1
Cc: Andrew Morton <akpm@osdl.org>, piotr@larroy.com
References: <20040316015338.39e2c48e.akpm@osdl.org> <20040322195450.GB2306@larroy.com> <20040322193502.41704cda.akpm@osdl.org>
In-Reply-To: <20040322193502.41704cda.akpm@osdl.org>
MIME-Version: 1.0
Content-Disposition: inline
Content-Type: text/plain;
  charset="iso-8859-15"
Content-Transfer-Encoding: 7bit
Message-Id: <200403231405.56777@WOLK>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1344
Lines: 43

On Tuesday 23 March 2004 04:35, Andrew Morton wrote:

Hi Andrew,

thanks alot. All is back to normal and running smooth and fine! :)

> >  > /proc/meminfo, /proc/vmstat and /proc/slabinfo.  Also sysrq-M.
> A memory leak was introduced in the ext3 changes.
>  25-akpm/fs/jbd/commit.c |    8 ++++++++
>  1 files changed, 8 insertions(+)
>
> diff -puN fs/jbd/commit.c~jbd-move-locked-buffers-leak-fixes
> fs/jbd/commit.c ---
> 25/fs/jbd/commit.c~jbd-move-locked-buffers-leak-fixes	2004-03-22
> 19:34:37.492865816 -0800 +++ 25-akpm/fs/jbd/commit.c	2004-03-22
> 19:34:41.987182576 -0800
> @@ -296,6 +296,13 @@ write_out_data:
>  		}
>  	}
>
> +	if (bufs) {
> +		spin_unlock(&journal->j_list_lock);
> +		ll_rw_block(WRITE, bufs, wbuf);
> +		journal_brelse_array(wbuf, bufs);
> +		spin_lock(&journal->j_list_lock);
> +	}
> +
>  	/*
>  	 * Wait for all previously submitted IO to complete.
>  	 */
> @@ -322,6 +329,7 @@ write_out_data:
>  			jh->b_transaction = NULL;
>  			jbd_unlock_bh_state(bh);
>  			journal_remove_journal_head(bh);
> +			put_bh(bh);
>  		} else {
>  			jbd_unlock_bh_state(bh);
>  		}
-
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/