[Ocfs2-devel] [PATCH 2/5] ocfs2: Remove ENTRY from masklog.
Tao Ma
tao.ma at oracle.com
Wed Nov 10 06:19:40 PST 2010
Joel Becker wrote:
> On Wed, Nov 10, 2010 at 10:52:20AM +0800, Tao Ma wrote:
>
>> ENTRY is used to record the entry 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.
>>
>> So for mlog_entry_void, we just remove it.
>> for mlog_entry(...), we replace it with mlog(0,...).
>>
>
> I'm not sure I like this. One of the benefits of mlog_entry()
> was that you could turn on ML_xxx and not see these entry lines. By
> replacing it with mlog(0, ...), all of those entry lines will appear if
> the file is enabled.
> I realize that many of them will be replaced in later patches.
> I just hope they all are ;-)
>
Yeah, I will replace them with trace events later.
But you are right that the commit log should explain it. I will add some
more comments in the next round.
Regards,
Tao
More information about the Ocfs2-devel
mailing list