xfs
[Top] [All Lists]

Review: Fix inverted quiet mount logic

To: xfs-dev@xxxxxxx
Subject: Review: Fix inverted quiet mount logic
From: David Chinner <dgc@xxxxxxx>
Date: Wed, 22 Nov 2006 15:46:56 +1100
Cc: xfs@xxxxxxxxxxx
Sender: xfs-bounce@xxxxxxxxxxx
User-agent: Mutt/1.4.2.1i
Simple problem Noticed by Eric Sandeen at about the same time I did
- we are not getting error messages in dmesg when certain checks
fail during mount.  Turns out the XFS_MFSI_QUIET flag usage is
inverted, so we suppress messages when we should be noisy and
shout when we should be silent....

Cheers,

Dave.
-- 
Dave Chinner
Principal Engineer
SGI Australian Software Group

---
 fs/xfs/xfs_error.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: 2.6.x-xfs-new/fs/xfs/xfs_error.h
===================================================================
--- 2.6.x-xfs-new.orig/fs/xfs/xfs_error.h       2006-10-17 12:17:25.000000000 
+1000
+++ 2.6.x-xfs-new/fs/xfs/xfs_error.h    2006-11-16 09:45:03.444451972 +1100
@@ -180,6 +180,6 @@ extern void xfs_fs_cmn_err(int level, st
        xfs_fs_cmn_err(level, mp, fmt "  Unmount and run xfs_repair.", ## args)
 
 #define xfs_fs_mount_cmn_err(f, fmt, args...) \
-       ((f & XFS_MFSI_QUIET)? cmn_err(CE_WARN, "XFS: " fmt, ## args) : (void)0)
+       ((f & XFS_MFSI_QUIET)? (void)0 : cmn_err(CE_WARN, "XFS: " fmt, ## args))
 
 #endif /* __XFS_ERROR_H__ */


<Prev in Thread] Current Thread [Next in Thread>