diff options
author | Dave Chinner <david@fromorbit.com> | 2008-11-27 22:23:37 -0500 |
---|---|---|
committer | Niv Sardi <xaiki@sgi.com> | 2008-11-30 19:37:10 -0500 |
commit | 26c5295135d10fc90cbf160adfda392d91f58279 (patch) | |
tree | 783ec22542fc70bd2bcab6c111205dc62b214e71 /fs/xfs/xfs_rename.c | |
parent | 207fcfad58482c7c9f92939a1f6df9f7e8873a34 (diff) |
[XFS] remove i_gen from incore inode
i_gen is incremented in directory operations when the
directory is changed. It is never read or otherwise used
so it should be removed to help reduce the size of the
struct xfs_inode.
The patch also removes a duplicate logging of the directory
inode core. We only need to do this once per transaction
so kill the one associated with the i_gen increment.
Signed-off-by: Dave Chinner <david@fromorbit.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Niv Sardi <xaiki@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_rename.c')
-rw-r--r-- | fs/xfs/xfs_rename.c | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/fs/xfs/xfs_rename.c b/fs/xfs/xfs_rename.c index d700dacdb10e..02f0e8f53a94 100644 --- a/fs/xfs/xfs_rename.c +++ b/fs/xfs/xfs_rename.c | |||
@@ -367,19 +367,11 @@ xfs_rename( | |||
367 | &first_block, &free_list, spaceres); | 367 | &first_block, &free_list, spaceres); |
368 | if (error) | 368 | if (error) |
369 | goto abort_return; | 369 | goto abort_return; |
370 | xfs_ichgtime(src_dp, XFS_ICHGTIME_MOD | XFS_ICHGTIME_CHG); | ||
371 | 370 | ||
372 | /* | 371 | xfs_ichgtime(src_dp, XFS_ICHGTIME_MOD | XFS_ICHGTIME_CHG); |
373 | * Update the generation counts on all the directory inodes | ||
374 | * that we're modifying. | ||
375 | */ | ||
376 | src_dp->i_gen++; | ||
377 | xfs_trans_log_inode(tp, src_dp, XFS_ILOG_CORE); | 372 | xfs_trans_log_inode(tp, src_dp, XFS_ILOG_CORE); |
378 | 373 | if (new_parent) | |
379 | if (new_parent) { | ||
380 | target_dp->i_gen++; | ||
381 | xfs_trans_log_inode(tp, target_dp, XFS_ILOG_CORE); | 374 | xfs_trans_log_inode(tp, target_dp, XFS_ILOG_CORE); |
382 | } | ||
383 | 375 | ||
384 | /* | 376 | /* |
385 | * If this is a synchronous mount, make sure that the | 377 | * If this is a synchronous mount, make sure that the |