Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757863AbZJaPZz (ORCPT ); Sat, 31 Oct 2009 11:25:55 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1757846AbZJaPZz (ORCPT ); Sat, 31 Oct 2009 11:25:55 -0400 Received: from e23smtp07.au.ibm.com ([202.81.31.140]:58139 "EHLO e23smtp07.au.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757839AbZJaPZy (ORCPT ); Sat, 31 Oct 2009 11:25:54 -0400 Date: Sat, 31 Oct 2009 20:54:28 +0530 From: "Aneesh Kumar K.V" To: Theodore Tso , Andreas Dilger , Eric Sandeen , Parag Warudkar , LKML , linux-ext4@vger.kernel.org, bugzilla-daemon@bugzilla.kernel.org Subject: Re: [Bug 14354] Re: ext4 increased intolerance to unclean shutdown? Message-ID: <20091031152428.GA25548@skywalker.linux.vnet.ibm.com> References: <20091016091558.GA10184@mit.edu> <20091027101534.GA27584@skywalker.linux.vnet.ibm.com> <4AEA0B71.5050805@redhat.com> <3D6E88E0-7271-4CEC-85DC-9E4416F030F5@sun.com> <20091031091528.GO18464@mit.edu> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20091031091528.GO18464@mit.edu> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1354 Lines: 27 On Sat, Oct 31, 2009 at 05:15:28AM -0400, Theodore Tso wrote: > On Fri, Oct 30, 2009 at 01:56:27PM -0600, Andreas Dilger wrote: > > I wonder if there are multiple problems involved here? Eric, it seems > > possible that your reproducer is exercising a similar, though unrelated > > codepath. > > Note that Aneesh has pubished two patches which insert a call to > ext4_discard_preallocations(). One is a patch which inserts it into > fs/inode.c's truncate path (for direct/indirect-mapped inodes) and one > which is patch which inserts it into fs/extents.c truncate path (for > extent-mapped inodes). As near as I can tell both patches are > necessary, and it looks to me like they should be combined into a > single patch, since commit 487caeef9 affects both truncate paths. > Aneesh, do you concur? > We need only the patch that drop prealloc space in ext4_truncate_restart_trans ext4_ext_truncate_extend_restart calls ext4_truncate_restart_trans. So adding the prealloc space dropping in ext4_truncate_restart_trans should handle both direct/indirect-mapped inode and extent-mapped inodes. -aneesh -- 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/