Commit:     1bc5858d0d40e07697b5eda47ed8628b8a934235
Parent:     bcc7b445eff295664a3a3ab14e742b3c9d88e6e3
Author:     Christoph Hellwig <[EMAIL PROTECTED]>
AuthorDate: Wed Sep 19 15:27:30 2007 +1000
Committer:  Tim Shimmin <[EMAIL PROTECTED]>
CommitDate: Thu Sep 20 19:40:40 2007 +1000

    [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 <[EMAIL PROTECTED]>
    Signed-off-by: Lachlan McIlroy <[EMAIL PROTECTED]>
    Signed-off-by: Tim Shimmin <[EMAIL PROTECTED]>
 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 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;
-                       if (be16_to_cpu(dip->di_core.di_mode))
+                       if (dip->di_core.di_mode)
-                       if (be16_to_cpu(dip->di_core.di_gen))
+                       if (dip->di_core.di_gen)
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at

Reply via email to