From: Christoph Hellwig Date: Wed, 19 Sep 2007 05:27:30 +0000 (+1000) Subject: [XFS] fix valid but harmless sparse warning X-Git-Tag: upstream/snapshot3+hdmi~31135^2 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=1bc5858d0d40e07697b5eda47ed8628b8a934235;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git [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 Signed-off-by: Lachlan McIlroy Signed-off-by: Tim Shimmin --- diff --git a/fs/xfs/xfs_log_recover.c b/fs/xfs/xfs_log_recover.c index dacb197..7174991 100644 --- a/fs/xfs/xfs_log_recover.c +++ b/fs/xfs/xfs_log_recover.c @@ -1920,9 +1920,9 @@ xlog_recover_do_reg_buffer( stale_buf = 1; break; } - if (be16_to_cpu(dip->di_core.di_mode)) + if (dip->di_core.di_mode) mode_count++; - if (be16_to_cpu(dip->di_core.di_gen)) + if (dip->di_core.di_gen) gen_count++; }