xfs
[Top] [All Lists]

Re: Review: Fix inverted quiet mount logic

To: Timothy Shimmin <tes@xxxxxxx>
Subject: Re: Review: Fix inverted quiet mount logic
From: David Chatterton <chatz@xxxxxxxxxxxxxxxxx>
Date: Wed, 22 Nov 2006 16:02:04 +1100
Cc: David Chinner <dgc@xxxxxxx>, xfs-dev@xxxxxxx, xfs@xxxxxxxxxxx
In-reply-to: <7013720DE121BA0C2CA1A09D@timothy-shimmins-power-mac-g5.local>
Organization: SGI
References: <20061122044656.GS37654165@melbourne.sgi.com> <7013720DE121BA0C2CA1A09D@timothy-shimmins-power-mac-g5.local>
Reply-to: chatz@xxxxxxxxxxxxxxxxx
Sender: xfs-bounce@xxxxxxxxxxx
User-agent: Thunderbird 1.5.0.8 (Windows/20061025)
One of the bugs we found on the training course that I had not raised 
yet...thanks!

Timothy Shimmin wrote:
> Add me to the reviewer list :)
> --Tim
> 
> --On 22 November 2006 3:46:56 PM +1100 David Chinner <dgc@xxxxxxx> wrote:
> 
>>
>> 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__ */
> 
> 
> 
> 

-- 
David Chatterton
XFS Engineering Manager
SGI Australia


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