Re: ext2_discard_prealloc() called on each iput?

2007-05-30 Thread Jan Kara
On Tue 29-05-07 14:10:52, Mingming Cao wrote: On Mon, 2007-05-28 at 18:04 +0200, Jan Kara wrote: On Wed 23-05-07 08:37:43, Theodore Tso wrote: On Tue, May 22, 2007 at 06:11:27PM +0200, Jan Kara wrote: while fixing some problems with preallocation in UDF, I had a look how ext2

[Patch 0/2] Journal Checksums

2007-05-30 Thread Girish Shilamkar
Hi, The following patch set adds journal checksumming support for linux kernel and e2fsprogs. Two new flags have been added i.e JBD2_FEATURE_INCOMPAT_ASYNC_COMMIT and JBD2_FEATURE_COMPAT_CHECKSUM. _CHECKSUM flag indicates that the commit block contains the checksum for the blocks

[Patch][1/2] Journal Checksums - ext4 patch

2007-05-30 Thread Girish Shilamkar
Hi, Patch for adding journal checksums support ext4. Signed-off-by: Andreas Dilger [EMAIL PROTECTED] Signed-off-by: Girish Shilamkar [EMAIL PROTECTED] diffstat patches/ext4-journal-chksum-2.6.20.patch fs/ext4/super.c | 25 ++ fs/jbd2/commit.c| 181

[Patch][2/2] Journal Checksums - e2fsprogs

2007-05-30 Thread Girish Shilamkar
Hi, This patch has been created on top of the e2fsprogs-1.39-WIP and number of other patches sent to list. But as it contains changes to recovery part, should be able to apply safely on top of e2fsprogs-1.39-WIP. Signed-off-by: Andreas Dilger [EMAIL PROTECTED] Signed-off-by: Girish

Re: [Patch 0/2] Journal Checksums

2007-05-30 Thread Girish Shilamkar
Hi, I forgot mention, this patch is spinoff of patch originally written by Vijayan Prabhakaran. Thanks Regards. Girish On Wed, 2007-05-30 at 16:01 +0530, Girish Shilamkar wrote: Hi, The following patch set adds journal checksumming support for linux kernel and e2fsprogs.

Online defragmentation

2007-05-30 Thread Aneesh Kumar K.V
Hi Takashi, I was looking at online defrag code and found that the tmp_inode is created with tmp_inode-i_nlink equal to zero. Now i am not sure whether i understand the code correctly, but AFAIU we allocate contiguous block using this tmp_inode. That means tmp_inode have extent details

Re: Online defragmentation

2007-05-30 Thread Andreas Dilger
On May 30, 2007 12:34 +0530, Aneesh Kumar K.V wrote: I haven't tested the defrag code. This came up when i was doing the online migration. With large file having large number of fragmented blocks I was getting the below error during the iput of the temporary inode. ext4_free_blocks

Re: [RFC][PATCH] Multiple mount protection

2007-05-30 Thread Kalpak Shah
On Sat, 2007-05-26 at 03:06 +0530, Kalpak Shah wrote: Hi Ted, On Fri, 2007-05-25 at 10:39 -0400, Theodore Tso wrote: Hi Kalpak, On Tue, May 22, 2007 at 01:22:32AM +0530, Kalpak Shah wrote: It will also protect against running e2fsck on a mounted filesystem by adding similar logic

Re: [patch 2/2] i_version update - ext4 part

2007-05-30 Thread Mingming Cao
On Tue, 2007-05-29 at 16:58 -0600, Andreas Dilger wrote: On May 29, 2007 12:44 -0700, Mingming Cao wrote: I am a little bit confused about the two patches. It appears in the ext4_expand_inode_extra_isize patch by Kalpak, there a new 64 bit i_fs_version field is added to ext4 inode

Re: [patch 0/2] i_version update

2007-05-30 Thread Neil Brown
On Wednesday May 30, [EMAIL PROTECTED] wrote: On Fri, May 25, 2007 at 06:25:31PM +0200, Jean noel Cordenner wrote: The aim is to fulfill a NFSv4 requirement for rfc3530: 5.5. Mandatory Attributes - Definitions Name# DataType Access Description

Re: [patch 0/2] i_version update

2007-05-30 Thread David Chinner
On Wed, May 30, 2007 at 04:32:57PM -0700, Mingming Cao wrote: On Wed, 2007-05-30 at 10:21 +1000, David Chinner wrote: On Fri, May 25, 2007 at 06:25:31PM +0200, Jean noel Cordenner wrote: Hi, This is an update of the i_version patch. The i_version field is a 64bit counter that is