aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChristoph Hellwig <hch@infradead.org>2007-09-19 01:27:30 -0400
committerTim Shimmin <tes@chook.melbourne.sgi.com>2007-09-20 05:40:40 -0400
commit1bc5858d0d40e07697b5eda47ed8628b8a934235 (patch)
treee393b4f69b21d1d074a5ba5142f06ffa727079eb
parentbcc7b445eff295664a3a3ab14e742b3c9d88e6e3 (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.c4
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