From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ted Ts'o Subject: Re: [PATCH] ext4: nblocks arg to ext4_truncate_restart_trans() is ignored Date: Sun, 27 Feb 2011 23:41:39 -0500 Message-ID: <20110228044139.GP2924@thunk.org> References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Jan Kara , Ext4 Developers List To: Amir Goldstein Return-path: Received: from li9-11.members.linode.com ([67.18.176.11]:49782 "EHLO test.thunk.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754436Ab1B1QFc (ORCPT ); Mon, 28 Feb 2011 11:05:32 -0500 Content-Disposition: inline In-Reply-To: Sender: linux-ext4-owner@vger.kernel.org List-ID: On Sun, Feb 27, 2011 at 11:54:28AM +0200, Amir Goldstein wrote: > nblocks is passed into ext4_truncate_restart_trans() from > ext4_ext_truncate_extend_restart() with a value different from the default > blocks_for_truncate(), but is being ignored. > > The two other calls to ext4_truncate_restart_trans() already pass the > default value, which is then being recalculated inside the function. > > Fix the problem by using the passed argument. > > Signed-off-by: Amir Goldstein Thanks, added to the ext4 patch queue. - Ted