From: Brian Foster <bfoster@redhat.com>
To: Dave Chinner <david@fromorbit.com>
Cc: xfs@oss.sgi.com
Subject: Re: [PATCH 7/9] xfs: move inode generation count to VFS inode
Date: Mon, 8 Feb 2016 10:29:34 -0500 [thread overview]
Message-ID: <20160208152933.GE19597@bfoster.bfoster> (raw)
In-Reply-To: <1454905461-2773-8-git-send-email-david@fromorbit.com>
On Mon, Feb 08, 2016 at 03:24:19PM +1100, Dave Chinner wrote:
> From: Dave Chinner <dchinner@redhat.com>
>
> Pull another 4 bytes out of the xfs_icdinode.
>
> Signed-off-by: Dave Chinner <dchinner@redhat.com>
> ---
Reviewed-by: Brian Foster <bfoster@redhat.com>
> fs/xfs/libxfs/xfs_inode_buf.c | 8 ++++----
> fs/xfs/libxfs/xfs_inode_buf.h | 1 -
> fs/xfs/xfs_export.c | 2 +-
> fs/xfs/xfs_icache.c | 2 ++
> fs/xfs/xfs_inode.c | 5 +----
> fs/xfs/xfs_inode_item.c | 2 +-
> fs/xfs/xfs_ioctl.c | 2 +-
> fs/xfs/xfs_iops.c | 1 -
> fs/xfs/xfs_itable.c | 2 +-
> 9 files changed, 11 insertions(+), 14 deletions(-)
>
> diff --git a/fs/xfs/libxfs/xfs_inode_buf.c b/fs/xfs/libxfs/xfs_inode_buf.c
> index 0dcaa9a..0a3f10b 100644
> --- a/fs/xfs/libxfs/xfs_inode_buf.c
> +++ b/fs/xfs/libxfs/xfs_inode_buf.c
> @@ -237,6 +237,7 @@ xfs_inode_from_disk(
> inode->i_mtime.tv_nsec = (int)be32_to_cpu(from->di_mtime.t_nsec);
> inode->i_ctime.tv_sec = (int)be32_to_cpu(from->di_ctime.t_sec);
> inode->i_ctime.tv_nsec = (int)be32_to_cpu(from->di_ctime.t_nsec);
> + inode->i_generation = be32_to_cpu(from->di_gen);
>
> to->di_size = be64_to_cpu(from->di_size);
> to->di_nblocks = be64_to_cpu(from->di_nblocks);
> @@ -248,7 +249,6 @@ xfs_inode_from_disk(
> to->di_dmevmask = be32_to_cpu(from->di_dmevmask);
> to->di_dmstate = be16_to_cpu(from->di_dmstate);
> to->di_flags = be16_to_cpu(from->di_flags);
> - to->di_gen = be32_to_cpu(from->di_gen);
>
> if (to->di_version == 3) {
> to->di_changecount = be64_to_cpu(from->di_changecount);
> @@ -286,6 +286,7 @@ xfs_inode_to_disk(
> to->di_ctime.t_sec = cpu_to_be32(inode->i_ctime.tv_sec);
> to->di_ctime.t_nsec = cpu_to_be32(inode->i_ctime.tv_nsec);
> to->di_nlink = cpu_to_be32(inode->i_nlink);
> + to->di_gen = cpu_to_be32(inode->i_generation);
>
> to->di_size = cpu_to_be64(from->di_size);
> to->di_nblocks = cpu_to_be64(from->di_nblocks);
> @@ -297,7 +298,6 @@ xfs_inode_to_disk(
> to->di_dmevmask = cpu_to_be32(from->di_dmevmask);
> to->di_dmstate = cpu_to_be16(from->di_dmstate);
> to->di_flags = cpu_to_be16(from->di_flags);
> - to->di_gen = cpu_to_be32(from->di_gen);
>
> if (from->di_version == 3) {
> to->di_changecount = cpu_to_be64(from->di_changecount);
> @@ -443,7 +443,7 @@ xfs_iread(
> !(mp->m_flags & XFS_MOUNT_IKEEP)) {
> /* initialise the on-disk inode core */
> memset(&ip->i_d, 0, sizeof(ip->i_d));
> - ip->i_d.di_gen = prandom_u32();
> + VFS_I(ip)->i_generation = prandom_u32();
> if (xfs_sb_version_hascrc(&mp->m_sb))
> ip->i_d.di_version = 3;
> else
> @@ -491,7 +491,7 @@ xfs_iread(
> * that xfs_ialloc won't overwrite or relies on being correct.
> */
> ip->i_d.di_version = dip->di_version;
> - ip->i_d.di_gen = be32_to_cpu(dip->di_gen);
> + VFS_I(ip)->i_generation = be32_to_cpu(dip->di_gen);
> ip->i_d.di_flushiter = be16_to_cpu(dip->di_flushiter);
>
> /*
> diff --git a/fs/xfs/libxfs/xfs_inode_buf.h b/fs/xfs/libxfs/xfs_inode_buf.h
> index 320b723..29fd9f1 100644
> --- a/fs/xfs/libxfs/xfs_inode_buf.h
> +++ b/fs/xfs/libxfs/xfs_inode_buf.h
> @@ -46,7 +46,6 @@ struct xfs_icdinode {
> __uint32_t di_dmevmask; /* DMIG event mask */
> __uint16_t di_dmstate; /* DMIG state info */
> __uint16_t di_flags; /* random flags, XFS_DIFLAG_... */
> - __uint32_t di_gen; /* generation number */
>
> __uint64_t di_changecount; /* number of attribute changes */
> __uint64_t di_flags2; /* more random flags */
> diff --git a/fs/xfs/xfs_export.c b/fs/xfs/xfs_export.c
> index 652cd3c..2816d42 100644
> --- a/fs/xfs/xfs_export.c
> +++ b/fs/xfs/xfs_export.c
> @@ -152,7 +152,7 @@ xfs_nfs_get_inode(
> return ERR_PTR(error);
> }
>
> - if (ip->i_d.di_gen != generation) {
> + if (VFS_I(ip)->i_generation != generation) {
> IRELE(ip);
> return ERR_PTR(-ESTALE);
> }
> diff --git a/fs/xfs/xfs_icache.c b/fs/xfs/xfs_icache.c
> index 4c184f7..6401e3c 100644
> --- a/fs/xfs/xfs_icache.c
> +++ b/fs/xfs/xfs_icache.c
> @@ -149,10 +149,12 @@ xfs_reinit_inode(
> {
> int error;
> uint32_t nlink = inode->i_nlink;
> + uint32_t generation = inode->i_generation;
>
> error = inode_init_always(mp->m_super, inode);
>
> set_nlink(inode, nlink);
> + inode->i_generation = generation;
> return error;
> }
>
> diff --git a/fs/xfs/xfs_inode.c b/fs/xfs/xfs_inode.c
> index ed8e3d2..e8f87c7 100644
> --- a/fs/xfs/xfs_inode.c
> +++ b/fs/xfs/xfs_inode.c
> @@ -835,9 +835,6 @@ xfs_ialloc(
> inode->i_atime = tv;
> inode->i_ctime = tv;
>
> - /*
> - * di_gen will have been taken care of in xfs_iread.
> - */
> ip->i_d.di_extsize = 0;
> ip->i_d.di_dmevmask = 0;
> ip->i_d.di_dmstate = 0;
> @@ -2428,7 +2425,7 @@ xfs_ifree(
> * Bump the generation count so no one will be confused
> * by reincarnations of this inode.
> */
> - ip->i_d.di_gen++;
> + VFS_I(ip)->i_generation++;
> xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE);
>
> if (xic.deleted)
> diff --git a/fs/xfs/xfs_inode_item.c b/fs/xfs/xfs_inode_item.c
> index 193e0bd..6367780 100644
> --- a/fs/xfs/xfs_inode_item.c
> +++ b/fs/xfs/xfs_inode_item.c
> @@ -350,6 +350,7 @@ xfs_inode_to_log_dinode(
> to->di_ctime.t_sec = inode->i_ctime.tv_sec;
> to->di_ctime.t_nsec = inode->i_ctime.tv_nsec;
> to->di_nlink = inode->i_nlink;
> + to->di_gen = inode->i_generation;
>
> to->di_size = from->di_size;
> to->di_nblocks = from->di_nblocks;
> @@ -361,7 +362,6 @@ xfs_inode_to_log_dinode(
> to->di_dmevmask = from->di_dmevmask;
> to->di_dmstate = from->di_dmstate;
> to->di_flags = from->di_flags;
> - to->di_gen = from->di_gen;
>
> if (from->di_version == 3) {
> to->di_changecount = from->di_changecount;
> diff --git a/fs/xfs/xfs_ioctl.c b/fs/xfs/xfs_ioctl.c
> index 478d04e..cdd6c31 100644
> --- a/fs/xfs/xfs_ioctl.c
> +++ b/fs/xfs/xfs_ioctl.c
> @@ -114,7 +114,7 @@ xfs_find_handle(
> handle.ha_fid.fid_len = sizeof(xfs_fid_t) -
> sizeof(handle.ha_fid.fid_len);
> handle.ha_fid.fid_pad = 0;
> - handle.ha_fid.fid_gen = ip->i_d.di_gen;
> + handle.ha_fid.fid_gen = inode->i_generation;
> handle.ha_fid.fid_ino = ip->i_ino;
>
> hsize = XFS_HSIZE(handle);
> diff --git a/fs/xfs/xfs_iops.c b/fs/xfs/xfs_iops.c
> index 8982e56..a4daa3f 100644
> --- a/fs/xfs/xfs_iops.c
> +++ b/fs/xfs/xfs_iops.c
> @@ -1231,7 +1231,6 @@ xfs_setup_inode(
> break;
> }
>
> - inode->i_generation = ip->i_d.di_gen;
> i_size_write(inode, ip->i_d.di_size);
> xfs_diflags_to_iflags(inode, ip);
>
> diff --git a/fs/xfs/xfs_itable.c b/fs/xfs/xfs_itable.c
> index cfb6527..6162e65 100644
> --- a/fs/xfs/xfs_itable.c
> +++ b/fs/xfs/xfs_itable.c
> @@ -100,11 +100,11 @@ xfs_bulkstat_one_int(
> buf->bs_mtime.tv_nsec = inode->i_mtime.tv_nsec;
> buf->bs_ctime.tv_sec = inode->i_ctime.tv_sec;
> buf->bs_ctime.tv_nsec = inode->i_ctime.tv_nsec;
> + buf->bs_gen = inode->i_generation;
>
> buf->bs_xflags = xfs_ip2xflags(ip);
> buf->bs_extsize = dic->di_extsize << mp->m_sb.sb_blocklog;
> buf->bs_extents = dic->di_nextents;
> - buf->bs_gen = dic->di_gen;
> memset(buf->bs_pad, 0, sizeof(buf->bs_pad));
> buf->bs_dmevmask = dic->di_dmevmask;
> buf->bs_dmstate = dic->di_dmstate;
> --
> 2.5.0
>
> _______________________________________________
> xfs mailing list
> xfs@oss.sgi.com
> http://oss.sgi.com/mailman/listinfo/xfs
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
next prev parent reply other threads:[~2016-02-08 15:29 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-08 4:24 [PATCH v3 0/9] xfs: gut the struct xfs_icdinode Dave Chinner
2016-02-08 4:24 ` [PATCH 1/9] xfs: introduce inode log format object Dave Chinner
2016-02-08 9:23 ` Christoph Hellwig
2016-02-08 19:28 ` Dave Chinner
2016-02-08 4:24 ` [PATCH 2/9] xfs: remove timestamps from incore inode Dave Chinner
2016-02-08 9:28 ` Christoph Hellwig
2016-02-08 4:24 ` [PATCH 3/9] xfs: cull unnecessary icdinode fields Dave Chinner
2016-02-08 9:29 ` Christoph Hellwig
2016-02-08 15:29 ` Brian Foster
2016-02-08 4:24 ` [PATCH 4/9] xfs: move v1 inode conversion to xfs_inode_from_disk Dave Chinner
2016-02-08 9:31 ` Christoph Hellwig
2016-02-08 19:44 ` Dave Chinner
2016-02-08 4:24 ` [PATCH 5/9] xfs: reinitialise recycled VFS inode correctly Dave Chinner
2016-02-08 9:33 ` Christoph Hellwig
2016-02-08 15:29 ` Brian Foster
2016-02-08 4:24 ` [PATCH 6/9] xfs: use vfs inode nlink field everywhere Dave Chinner
2016-02-08 9:40 ` Christoph Hellwig
2016-02-08 19:47 ` Dave Chinner
2016-02-08 15:29 ` Brian Foster
2016-02-08 4:24 ` [PATCH 7/9] xfs: move inode generation count to VFS inode Dave Chinner
2016-02-08 9:40 ` Christoph Hellwig
2016-02-08 15:29 ` Brian Foster [this message]
2016-02-08 4:24 ` [PATCH 8/9] xfs: move di_changecount " Dave Chinner
2016-02-08 9:41 ` Christoph Hellwig
2016-02-08 4:24 ` [PATCH 9/9] xfs: mode di_mode to vfs inode Dave Chinner
2016-02-08 9:42 ` Christoph Hellwig
2016-02-08 19:50 ` Dave Chinner
2016-02-08 15:29 ` Brian Foster
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160208152933.GE19597@bfoster.bfoster \
--to=bfoster@redhat.com \
--cc=david@fromorbit.com \
--cc=xfs@oss.sgi.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox