[Ocfs2-devel] [PATCH 0/2] ocfs2: add trace event support for ocfs2
TaoMa
tao.ma at oracle.com
Thu Oct 15 15:29:00 PDT 2009
Joel Becker wrote:
> On Thu, Oct 15, 2009 at 11:05:45AM +0800, Tao Ma wrote:
>
>> Hi Joel/Wengang,
>>
>> Joel Becker wrote:
>>
>>> On Thu, Sep 17, 2009 at 06:19:44AM +0800, Wengang Wang wrote:
>>>
>>>> following patches add trace event support for ocfs2.
>>>>
>>> Wengang,
>>> This looks like a nice start. Thank you for getting this going.
>>> I'm sorry I haven't gotten back until now, I've been busy.
>>> What I'd like to do next is get a small starting set of
>>> tracepoints. You have one tracepoint in this patch as a good example,
>>> but we should get a few more. I don't want every mlog changed into a
>>> tracepoint. Some of them are old and useless :-) Instead, I'd like to
>>> find information we often want while debugging.
>>> Tao, Tiger, and Sunil, what are parts of the code you often come
>>> across when debugging? Certain mlog() prints you always find really
>>> important to see. If you point them out, Wengang can cook up
>>> tracepoints for them.
>>>
>> aops.c maybe? And recovery/dlm(The most awesome part)?
>>
>
> But what things? We're not just going to replace every mlog,
> and we're not going to do every function. We really want to figure out
> what information is actually useful and add tracepoints for it. So if
> you have particular functions you need data on, or particular mlog()
> calls you always like to see, we'd love to hear about it.
>
No idea of recovery/dlm since I don't have much experience in it.
As for aops.c, I think the most important part is ocfs2_write_begin_no_lock.
It is very complicated, sometimes it will write inline_data, sometimes
CoW a refcount record, sometimes makes a extent written. It is helpful
for us to know some detail information about it.
Regards,
Tao
> Wengang, dlmglue.c is definitely a place to look at. I'd love
> to see events for the inc/dec of l_ro_holders and l_ex_holders. I'd
> also love to see the mlogs for 'BAST fired' and 'UNLOCK AST fired'
> converted, and a similar event for the regular AST.
>
> Joel
>
>
More information about the Ocfs2-devel
mailing list