diff options
author | Christoph Hellwig <hch@infradead.org> | 2007-09-19 01:27:30 -0400 |
---|---|---|
committer | Tim Shimmin <tes@chook.melbourne.sgi.com> | 2007-09-20 05:40:40 -0400 |
commit | 1bc5858d0d40e07697b5eda47ed8628b8a934235 (patch) | |
tree | e393b4f69b21d1d074a5ba5142f06ffa727079eb | |
parent | bcc7b445eff295664a3a3ab14e742b3c9d88e6e3 (diff) |
[XFS] fix valid but harmless sparse warning
The new xlog_recover_do_reg_buffer checks call be16_to_cpu on di_gen which
is a 32bit value so sparse rightly complains. Fortunately the warning is
harmless because we don't care for the value, but only whether it's
non-NULL. Due to that fact we can simply kill the endian swaps on this and
the previous di_mode check entirely.
SGI-PV: 969656
SGI-Modid: xfs-linux-melb:xfs-kern:29709a
Signed-off-by: Christoph Hellwig <hch@infradead.org>
Signed-off-by: Lachlan McIlroy <lachlan@sgi.com>
Signed-off-by: Tim Shimmin <tes@sgi.com>
-rw-r--r-- | fs/xfs/xfs_log_recover.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/xfs/xfs_log_recover.c b/fs/xfs/xfs_log_recover.c index dacb19739cc2..7174991f4bef 100644 --- a/fs/xfs/xfs_log_recover.c +++ b/fs/xfs/xfs_log_recover.c | |||
@@ -1920,9 +1920,9 @@ xlog_recover_do_reg_buffer( | |||
1920 | stale_buf = 1; | 1920 | stale_buf = 1; |
1921 | break; | 1921 | break; |
1922 | } | 1922 | } |
1923 | if (be16_to_cpu(dip->di_core.di_mode)) | 1923 | if (dip->di_core.di_mode) |
1924 | mode_count++; | 1924 | mode_count++; |
1925 | if (be16_to_cpu(dip->di_core.di_gen)) | 1925 | if (dip->di_core.di_gen) |
1926 | gen_count++; | 1926 | gen_count++; |
1927 | } | 1927 | } |
1928 | 1928 | ||