diff options
author | Christoph Hellwig <hch@lst.de> | 2011-07-08 08:35:19 -0400 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2011-07-08 08:35:19 -0400 |
commit | 78f70cd7b7e8fd44dc89af4f72fb110d865338a1 (patch) | |
tree | a0faa6e46c6d6504de239a1b34e90938a5763bca /fs/xfs/xfs_dir2_sf.c | |
parent | ac8ba50f6be4a3ab49a2ad2cb676422af4ebaf8a (diff) |
xfs: cleanup the definition of struct xfs_dir2_sf_entry
Remove the inumber member which is at a variable offset after the actual
name, and make name a real variable sized C99 array instead of the incorrect
one-sized array which confuses (not only) gcc. Based on this clean up
the helpers to calculate the entry size.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Alex Elder <aelder@sgi.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Diffstat (limited to 'fs/xfs/xfs_dir2_sf.c')
-rw-r--r-- | fs/xfs/xfs_dir2_sf.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/xfs/xfs_dir2_sf.c b/fs/xfs/xfs_dir2_sf.c index 806b348f496f..3a1e3f4e3fa3 100644 --- a/fs/xfs/xfs_dir2_sf.c +++ b/fs/xfs/xfs_dir2_sf.c | |||
@@ -371,7 +371,7 @@ xfs_dir2_sf_addname( | |||
371 | /* | 371 | /* |
372 | * Compute entry (and change in) size. | 372 | * Compute entry (and change in) size. |
373 | */ | 373 | */ |
374 | add_entsize = xfs_dir2_sf_entsize_byname(sfp, args->namelen); | 374 | add_entsize = xfs_dir2_sf_entsize(sfp, args->namelen); |
375 | incr_isize = add_entsize; | 375 | incr_isize = add_entsize; |
376 | objchange = 0; | 376 | objchange = 0; |
377 | #if XFS_BIG_INUMS | 377 | #if XFS_BIG_INUMS |
@@ -465,7 +465,7 @@ xfs_dir2_sf_addname_easy( | |||
465 | /* | 465 | /* |
466 | * Grow the in-inode space. | 466 | * Grow the in-inode space. |
467 | */ | 467 | */ |
468 | xfs_idata_realloc(dp, xfs_dir2_sf_entsize_byname(sfp, args->namelen), | 468 | xfs_idata_realloc(dp, xfs_dir2_sf_entsize(sfp, args->namelen), |
469 | XFS_DATA_FORK); | 469 | XFS_DATA_FORK); |
470 | /* | 470 | /* |
471 | * Need to set up again due to realloc of the inode data. | 471 | * Need to set up again due to realloc of the inode data. |
@@ -1001,7 +1001,7 @@ xfs_dir2_sf_removename( | |||
1001 | * Calculate sizes. | 1001 | * Calculate sizes. |
1002 | */ | 1002 | */ |
1003 | byteoff = (int)((char *)sfep - (char *)sfp); | 1003 | byteoff = (int)((char *)sfep - (char *)sfp); |
1004 | entsize = xfs_dir2_sf_entsize_byname(sfp, args->namelen); | 1004 | entsize = xfs_dir2_sf_entsize(sfp, args->namelen); |
1005 | newsize = oldsize - entsize; | 1005 | newsize = oldsize - entsize; |
1006 | /* | 1006 | /* |
1007 | * Copy the part if any after the removed entry, sliding it down. | 1007 | * Copy the part if any after the removed entry, sliding it down. |