[Ocfs2-devel] [PATCH 3/5] ocfs2: Remove EXIT from masklog.

[Tao Ma]

Joel Becker wrote:
> On Wed, Nov 10, 2010 at 10:52:21AM +0800, Tao Ma wrote:
>   
>> mlog_exit is used to record the exit status of a function.
>> But because it is added in so many functions, if we enable it,
>> the system logs get filled up quickly and cause too much I/O.
>> So actually no one can open it for a production system or even
>> for a test.
>>
>> This patch just try to remove it. So in case of report error, a
>> mlog_errno is used while if all the errors in the function are
>> properly reported, mlog_exit is just removed.
>> for mlog_exit:
>> 1. if all the error paths already use mlog_errno, it is just removed.
>>    Otherwise, it will be replaced by mlog_errno.
>> 2. if it is used to print some return value, it is replaced with
>>    mlog(0,...).
>> mlog_exit_ptr is changed to mlog(0.
>>     
>
> 	This one is great, except I would only select certain exit
> return codes to log.  Most of them aren't very interesting.
>   
Actually I have checked all of them to make sure that only the error 
code is prompted.
But maybe I may miss something, so let us wait and see.

Regards,
Tao