2007-10-05 15:31:26

by Cordenner jean noel

[permalink] [raw]
Subject: [PATCH 2/2] i_version update - ext4 part

This patch update the i_version field of the inode and add a mount
option to enable this feature. The other condition to enable this
feature is that the inode size should be 256-bytes.

Signed-off-by: Jean Noel Cordenner <[email protected]>
---
fs/ext4/inode.c | 4 +++-
fs/ext4/super.c | 7 ++++++-
include/linux/ext4_fs.h | 1 +
3 files changed, 10 insertions(+), 2 deletions(-)

Index: linux-2.6.23-rc8-ext4-i_version/fs/ext4/inode.c
===================================================================
--- linux-2.6.23-rc8-ext4-i_version.orig/fs/ext4/inode.c 2007-10-03
18:11:17.000000000 +0200
+++ linux-2.6.23-rc8-ext4-i_version/fs/ext4/inode.c 2007-10-05
10:26:42.000000000 +0200
@@ -3173,7 +3173,9 @@
{
int err = 0;

- inode->i_version++;
+ if (test_opt(inode->i_sb, I_VERSION))
+ inode_inc_iversion(inode);
+
/* the do_update_inode consumes one bh->b_count */
get_bh(iloc->bh);

Index: linux-2.6.23-rc8-ext4-i_version/fs/ext4/super.c
===================================================================
--- linux-2.6.23-rc8-ext4-i_version.orig/fs/ext4/super.c 2007-10-03
18:11:17.000000000 +0200
+++ linux-2.6.23-rc8-ext4-i_version/fs/ext4/super.c 2007-10-03
18:17:44.000000000 +0200
@@ -742,7 +742,7 @@
Opt_jqfmt_vfsold, Opt_jqfmt_vfsv0, Opt_quota, Opt_noquota,
Opt_ignore, Opt_barrier, Opt_err, Opt_resize, Opt_usrquota,
Opt_grpquota, Opt_extents, Opt_noextents, Opt_delalloc,
- Opt_mballoc, Opt_nomballoc, Opt_stripe,
+ Opt_mballoc, Opt_nomballoc, Opt_stripe, Opt_i_version,
};

static match_table_t tokens = {
@@ -800,6 +800,7 @@
{Opt_mballoc, "mballoc"},
{Opt_nomballoc, "nomballoc"},
{Opt_stripe, "stripe=%u"},
+ {Opt_i_version, "i_version"},
{Opt_err, NULL},
{Opt_resize, "resize"},
};
@@ -1161,6 +1162,10 @@
return 0;
sbi->s_stripe = option;
break;
+ case Opt_i_version:
+ set_opt (sbi->s_mount_opt, I_VERSION);
+ sb->s_flags |= MS_I_VERSION;
+ break;
default:
printk (KERN_ERR
"EXT4-fs: Unrecognized mount option \"%s\" "
Index: linux-2.6.23-rc8-ext4-i_version/include/linux/ext4_fs.h
===================================================================
--- linux-2.6.23-rc8-ext4-i_version.orig/include/linux/ext4_fs.h
2007-10-03 18:11:17.000000000 +0200
+++ linux-2.6.23-rc8-ext4-i_version/include/linux/ext4_fs.h 2007-10-03
18:11:54.000000000 +0200
@@ -500,6 +500,7 @@
#define EXT4_MOUNT_JOURNAL_ASYNC_COMMIT 0x1000000 /* Journal Async
Commit */
#define EXT4_MOUNT_DELALLOC 0x2000000 /* Delalloc support */
#define EXT4_MOUNT_MBALLOC 0x4000000 /* Buddy allocation support */
+#define EXT4_MOUNT_I_VERSION 0x8000000 /* i_version support */
/* Compatibility, for having both ext2_fs.h and ext4_fs.h included at
once */
#ifndef _LINUX_EXT2_FS_H
#define clear_opt(o, opt) o &= ~EXT4_MOUNT_##opt


2007-10-06 00:58:52

by Mingming Cao

[permalink] [raw]
Subject: Re: [PATCH 2/2] i_version update - ext4 part

On Fri, 2007-10-05 at 17:28 +0200, Cordenner jean noel wrote:
> This patch update the i_version field of the inode and add a mount
> option to enable this feature. The other condition to enable this
> feature is that the inode size should be 256-bytes.
>
> Signed-off-by: Jean Noel Cordenner <[email protected]>
> ---
> fs/ext4/inode.c | 4 +++-
> fs/ext4/super.c | 7 ++++++-
> include/linux/ext4_fs.h | 1 +
> 3 files changed, 10 insertions(+), 2 deletions(-)
>
> Index: linux-2.6.23-rc8-ext4-i_version/fs/ext4/inode.c
> ===================================================================
> --- linux-2.6.23-rc8-ext4-i_version.orig/fs/ext4/inode.c 2007-10-03
> 18:11:17.000000000 +0200
> +++ linux-2.6.23-rc8-ext4-i_version/fs/ext4/inode.c 2007-10-05
> 10:26:42.000000000 +0200
> @@ -3173,7 +3173,9 @@
> {
> int err = 0;
>
> - inode->i_version++;
> + if (test_opt(inode->i_sb, I_VERSION))
> + inode_inc_iversion(inode);
> +
> /* the do_update_inode consumes one bh->b_count */
> get_bh(iloc->bh);
>
> Index: linux-2.6.23-rc8-ext4-i_version/fs/ext4/super.c
> ===================================================================
> --- linux-2.6.23-rc8-ext4-i_version.orig/fs/ext4/super.c 2007-10-03
> 18:11:17.000000000 +0200
> +++ linux-2.6.23-rc8-ext4-i_version/fs/ext4/super.c 2007-10-03
> 18:17:44.000000000 +0200
> @@ -742,7 +742,7 @@
> Opt_jqfmt_vfsold, Opt_jqfmt_vfsv0, Opt_quota, Opt_noquota,
> Opt_ignore, Opt_barrier, Opt_err, Opt_resize, Opt_usrquota,
> Opt_grpquota, Opt_extents, Opt_noextents, Opt_delalloc,
> - Opt_mballoc, Opt_nomballoc, Opt_stripe,
> + Opt_mballoc, Opt_nomballoc, Opt_stripe, Opt_i_version,
> };
>
> static match_table_t tokens = {
> @@ -800,6 +800,7 @@
> {Opt_mballoc, "mballoc"},
> {Opt_nomballoc, "nomballoc"},
> {Opt_stripe, "stripe=%u"},
> + {Opt_i_version, "i_version"},
> {Opt_err, NULL},
> {Opt_resize, "resize"},
> };
> @@ -1161,6 +1162,10 @@
> return 0;
> sbi->s_stripe = option;
> break;
> + case Opt_i_version:
> + set_opt (sbi->s_mount_opt, I_VERSION);
> + sb->s_flags |= MS_I_VERSION;
> + break;

Need to make sure this flag is cleared if remounted fs without I_VERSION

> default:
> printk (KERN_ERR
> "EXT4-fs: Unrecognized mount option \"%s\" "
> Index: linux-2.6.23-rc8-ext4-i_version/include/linux/ext4_fs.h
> ===================================================================
> --- linux-2.6.23-rc8-ext4-i_version.orig/include/linux/ext4_fs.h
> 2007-10-03 18:11:17.000000000 +0200
> +++ linux-2.6.23-rc8-ext4-i_version/include/linux/ext4_fs.h 2007-10-03
> 18:11:54.000000000 +0200
> @@ -500,6 +500,7 @@
> #define EXT4_MOUNT_JOURNAL_ASYNC_COMMIT 0x1000000 /* Journal Async
> Commit */
> #define EXT4_MOUNT_DELALLOC 0x2000000 /* Delalloc support */
> #define EXT4_MOUNT_MBALLOC 0x4000000 /* Buddy allocation support */
> +#define EXT4_MOUNT_I_VERSION 0x8000000 /* i_version support */
> /* Compatibility, for having both ext2_fs.h and ext4_fs.h included at
> once */
> #ifndef _LINUX_EXT2_FS_H
> #define clear_opt(o, opt) o &= ~EXT4_MOUNT_##opt
>


I don't see places where this counter is being stored/load to/from disk,
so I assume this is the not the full patch series?


Mingming

2007-10-25 17:09:15

by Cordenner jean noel

[permalink] [raw]
Subject: Re: [PATCH 2/2] i_version update - ext4 part

This patch adds 64-bit inode version support to ext4. The lower 32 bits
are stored in the osd1.linux1.l_i_version field while the high 32 bits
are stored in the i_version_hi field newly created in the ext4_inode.
This field is incremented in case the ext4_inode is large enough. A
i_version mount option has been added to enable the feature.

Signed-off-by: Mingming Cao <[email protected]>
Signed-off-by: Andreas Dilger <[email protected]>
Signed-off-by: Kalpak Shah <[email protected]>
Signed-off-by: Aneesh Kumar K.V <[email protected]>
Signed-off-by: Jean Noel Cordenner <[email protected]>
---
fs/ext4/inode.c | 18 +++++++++++++++++-
fs/ext4/super.c | 8 ++++++--
include/linux/ext4_fs.h | 6 +++++-
3 files changed, 28 insertions(+), 4 deletions(-)

Index: linux-2.6.23-ext4-1/fs/ext4/inode.c
===================================================================
--- linux-2.6.23-ext4-1.orig/fs/ext4/inode.c 2007-10-25
18:49:24.000000000 +0200
+++ linux-2.6.23-ext4-1/fs/ext4/inode.c 2007-10-25 18:57:35.000000000
+0200
@@ -2806,6 +2806,13 @@
EXT4_INODE_GET_XTIME(i_atime, inode, raw_inode);
EXT4_EINODE_GET_XTIME(i_crtime, ei, raw_inode);

+ inode->i_version = le32_to_cpu(raw_inode->i_disk_version);
+ if (EXT4_INODE_SIZE(inode->i_sb) > EXT4_GOOD_OLD_INODE_SIZE) {
+ if (EXT4_FITS_IN_INODE(raw_inode, ei, i_version_hi))
+ inode->i_version |=
+ (__u64)(le32_to_cpu(raw_inode->i_version_hi)) << 32;
+ }
+
if (S_ISREG(inode->i_mode)) {
inode->i_op = &ext4_file_inode_operations;
inode->i_fop = &ext4_file_operations;
@@ -2944,8 +2951,14 @@
} else for (block = 0; block < EXT4_N_BLOCKS; block++)
raw_inode->i_block[block] = ei->i_data[block];

- if (ei->i_extra_isize)
+ raw_inode->i_disk_version = cpu_to_le32(inode->i_version);
+ if (ei->i_extra_isize) {
+ if (EXT4_FITS_IN_INODE(raw_inode, ei, i_version_hi))
+ raw_inode->i_version_hi =
+ cpu_to_le32(inode->i_version >> 32);
raw_inode->i_extra_isize = cpu_to_le16(ei->i_extra_isize);
+ }
+

BUFFER_TRACE(bh, "call ext4_journal_dirty_metadata");
rc = ext4_journal_dirty_metadata(handle, bh);
@@ -3161,6 +3174,9 @@
{
int err = 0;

+ if (test_opt(inode->i_sb, I_VERSION))
+ inode_inc_iversion(inode);
+
/* the do_update_inode consumes one bh->b_count */
get_bh(iloc->bh);

Index: linux-2.6.23-ext4-1/fs/ext4/super.c
===================================================================
--- linux-2.6.23-ext4-1.orig/fs/ext4/super.c 2007-10-25
18:49:24.000000000 +0200
+++ linux-2.6.23-ext4-1/fs/ext4/super.c 2007-10-25 18:57:35.000000000
+0200
@@ -742,7 +742,7 @@
Opt_jqfmt_vfsold, Opt_jqfmt_vfsv0, Opt_quota, Opt_noquota,
Opt_ignore, Opt_barrier, Opt_err, Opt_resize, Opt_usrquota,
Opt_grpquota, Opt_extents, Opt_noextents, Opt_delalloc,
- Opt_mballoc, Opt_nomballoc, Opt_stripe,
+ Opt_mballoc, Opt_nomballoc, Opt_stripe, Opt_i_version,
};

static match_table_t tokens = {
@@ -800,6 +800,7 @@
{Opt_mballoc, "mballoc"},
{Opt_nomballoc, "nomballoc"},
{Opt_stripe, "stripe=%u"},
+ {Opt_i_version, "i_version"},
{Opt_err, NULL},
{Opt_resize, "resize"},
};
@@ -1161,6 +1162,10 @@
return 0;
sbi->s_stripe = option;
break;
+ case Opt_i_version:
+ set_opt (sbi->s_mount_opt, I_VERSION);
+ sb->s_flags |= MS_I_VERSION;
+ break;
default:
printk (KERN_ERR
"EXT4-fs: Unrecognized mount option \"%s\" "
@@ -2968,7 +2973,6 @@
i_size_write(inode, off+len-towrite);
EXT4_I(inode)->i_disksize = inode->i_size;
}
- inode->i_version++;
inode->i_mtime = inode->i_ctime = CURRENT_TIME;
ext4_mark_inode_dirty(handle, inode);
mutex_unlock(&inode->i_mutex);
Index: linux-2.6.23-ext4-1/include/linux/ext4_fs.h
===================================================================
--- linux-2.6.23-ext4-1.orig/include/linux/ext4_fs.h 2007-10-25
18:49:24.000000000 +0200
+++ linux-2.6.23-ext4-1/include/linux/ext4_fs.h 2007-10-25
18:57:55.000000000 +0200
@@ -314,7 +314,7 @@
__le32 i_flags; /* File flags */
union {
struct {
- __u32 l_i_reserved1;
+ __le32 l_i_version;
} linux1;
struct {
__u32 h_i_translator;
@@ -356,6 +356,7 @@
__le32 i_atime_extra; /* extra Access time (nsec << 2 | epoch)
*/
__le32 i_crtime; /* File Creation time */
__le32 i_crtime_extra; /* extra FileCreationtime (nsec << 2 | epoch)
*/
+ __le32 i_version_hi; /* high 32 bits for 64-bit version */
};

#define i_size_high i_dir_acl
@@ -430,6 +431,8 @@
raw_inode->xtime ## _extra); \
} while (0)

+#define i_disk_version osd1.linux1.l_i_version
+
#if defined(__KERNEL__) || defined(__linux__)
#define i_reserved1 osd1.linux1.l_i_reserved1
#define i_file_acl_high osd2.linux2.l_i_file_acl_high
@@ -493,6 +496,7 @@
#define EXT4_MOUNT_JOURNAL_ASYNC_COMMIT 0x1000000 /* Journal Async
Commit */
#define EXT4_MOUNT_DELALLOC 0x2000000 /* Delalloc support */
#define EXT4_MOUNT_MBALLOC 0x4000000 /* Buddy allocation support */
+#define EXT4_MOUNT_I_VERSION 0x8000000 /* i_version support */
/* Compatibility, for having both ext2_fs.h and ext4_fs.h included at
once */
#ifndef _LINUX_EXT2_FS_H
#define clear_opt(o, opt) o &= ~EXT4_MOUNT_##opt