linux-ext4 - Re: [RFC] [patch 1/2] i_version update

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <1179179817.3772.16.camel@dyn9047017103.beaverton.ibm.com>
Date:	Mon, 14 May 2007 14:56:56 -0700
From:	Mingming Cao <cmm@...ibm.com>
To:	jean-noel.cordenner@...l.net
Cc:	linux-ext4@...r.kernel.org
Subject: Re: [RFC] [patch 1/2] i_version update - vfs part

On Mon, 2007-05-14 at 13:04 +0200, Cordenner jean noel wrote:
> Concerning the first part of the set, the i_version field of the inode
> structure has been reused. The field  has been redefined from unsigned
> long to unsigned long long as the counter has to be 64bit.
> 
Since this chnged the vfs layer inode structure, it would be better to
send this patch to the linux-fsdev list for a wider review.

The unsigned long long could still be 32 bit on 32 bit arch though.


> Signed-off-by: Jean Noel Cordenner <jean-noel.cordenner@...l.net>
> 
> fs/binfmt_misc.c   |    1 +
> fs/libfs.c         |    9 +++++++++
> fs/pipe.c          |    1 +
> include/linux/fs.h |    2 +-
> 
> Index: linux-2.6.21-rc4-i_version/fs/binfmt_misc.c
> ===================================================================
> --- linux-2.6.21-rc4-i_version.orig/fs/binfmt_misc.c	2007-05-10 14:14:48.000000000 +0200
> +++ linux-2.6.21-rc4-i_version/fs/binfmt_misc.c	2007-05-10 14:18:45.000000000 +0200
> @@ -508,6 +508,7 @@
>  		inode->i_blocks = 0;
>  		inode->i_atime = inode->i_mtime = inode->i_ctime =
>  			current_fs_time(inode->i_sb);
> +		inode->i_version = 1;
>  	}
>  	return inode;
>  }
> Index: linux-2.6.21-rc4-i_version/fs/libfs.c
> ===================================================================
> --- linux-2.6.21-rc4-i_version.orig/fs/libfs.c	2007-05-10 14:14:48.000000000 +0200
> +++ linux-2.6.21-rc4-i_version/fs/libfs.c	2007-05-10 17:26:13.000000000 +0200
> @@ -223,6 +223,7 @@
>  	root->i_mode = S_IFDIR | S_IRUSR | S_IWUSR;
>  	root->i_uid = root->i_gid = 0;
>  	root->i_atime = root->i_mtime = root->i_ctime = CURRENT_TIME;
> +	root->i_version = 1;
>  	dentry = d_alloc(NULL, &d_name);
>  	if (!dentry) {
>  		iput(root);
> @@ -246,6 +247,8 @@
>  	struct inode *inode = old_dentry->d_inode;
> 
>  	inode->i_ctime = dir->i_ctime = dir->i_mtime = CURRENT_TIME;
> +	inode->i_version++;
> +	dir->i_version++;
>  	inc_nlink(inode);
>  	atomic_inc(&inode->i_count);
>  	dget(dentry);
> @@ -278,6 +281,8 @@
>  	struct inode *inode = dentry->d_inode;
> 
>  	inode->i_ctime = dir->i_ctime = dir->i_mtime = CURRENT_TIME;
> +	inode->i_version++;
> +	dir->i_version++;
>  	drop_nlink(inode);
>  	dput(dentry);
>  	return 0;
> @@ -314,6 +319,8 @@
> 
>  	old_dir->i_ctime = old_dir->i_mtime = new_dir->i_ctime =
>  		new_dir->i_mtime = inode->i_ctime = CURRENT_TIME;
> +	old_dir->i_version++;
> +	new_dir->i_version++;
> 
>  	return 0;
>  }
> @@ -380,6 +387,7 @@
>  	inode->i_uid = inode->i_gid = 0;
>  	inode->i_blocks = 0;
>  	inode->i_atime = inode->i_mtime = inode->i_ctime = CURRENT_TIME;
> +	inode->i_version = 1;
>  	inode->i_op = &simple_dir_inode_operations;
>  	inode->i_fop = &simple_dir_operations;
>  	inode->i_nlink = 2;
> @@ -401,6 +409,7 @@
>  		inode->i_uid = inode->i_gid = 0;
>  		inode->i_blocks = 0;
>  		inode->i_atime = inode->i_mtime = inode->i_ctime = CURRENT_TIME;
> +		inode->i_version = 1;
>  		inode->i_fop = files->ops;
>  		inode->i_ino = i;
>  		d_add(dentry, inode);
> Index: linux-2.6.21-rc4-i_version/fs/pipe.c
> ===================================================================
> --- linux-2.6.21-rc4-i_version.orig/fs/pipe.c	2007-05-10 14:14:48.000000000 +0200
> +++ linux-2.6.21-rc4-i_version/fs/pipe.c	2007-05-10 14:18:45.000000000 +0200
> @@ -872,6 +872,7 @@
>  	inode->i_uid = current->fsuid;
>  	inode->i_gid = current->fsgid;
>  	inode->i_atime = inode->i_mtime = inode->i_ctime = CURRENT_TIME;
> +	inode->i_version = 1;
> 
>  	return inode;
> 
> Index: linux-2.6.21-rc4-i_version/include/linux/fs.h
> ===================================================================
> --- linux-2.6.21-rc4-i_version.orig/include/linux/fs.h	2007-04-26 16:23:59.000000000 +0200
> +++ linux-2.6.21-rc4-i_version/include/linux/fs.h	2007-05-10 17:23:38.000000000 +0200
> @@ -536,7 +536,7 @@
>  	uid_t			i_uid;
>  	gid_t			i_gid;
>  	dev_t			i_rdev;
> -	unsigned long		i_version;
> +	unsigned long long	i_version;
>  	loff_t			i_size;
>  #ifdef __NEED_I_SIZE_ORDERED
>  	seqcount_t		i_size_seqcount;
> 
> 
> -
> To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

-
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html