Supporting blktrace in DRBD

View: New views
2 Messages — Rating Filter:   Alert me  

Supporting blktrace in DRBD

by Graham, Simon :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Some parts of this message have been removed. Learn more about Nabble's security policy.

I’ve been using blktrace to investigate performance of DRBD and have found a few places where DRBD needs to issue blktrace API calls that it does not – usually, the library routines in ll_rw_blk.c make these calls on behalf of block devices but those like DRBD that write their own routines need to also make some trace calls. Specifically, DRBD needs to trace request completion and calls to lower level unplug routines (it’s hard to be sure but it looks to me like the caller of the unplug routine is supposed to make the trace call).

 

Patch against 8.2 (and tested with kernel 2.6.18) attached for your review

Simon

 



_______________________________________________
drbd-dev mailing list
drbd-dev@...
http://lists.linbit.com/mailman/listinfo/drbd-dev

blktrace.patch (4K) Download Attachment

Re: Supporting blktrace in DRBD

by Lars Ellenberg :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

On Sun, Nov 01, 2009 at 05:10:24PM -0500, Graham, Simon wrote:

> I've been using blktrace to investigate performance of DRBD and have
> found a few places where DRBD needs to issue blktrace API calls that it
> does not - usually, the library routines in ll_rw_blk.c make these calls
> on behalf of block devices but those like DRBD that write their own
> routines need to also make some trace calls. Specifically, DRBD needs to
> trace request completion and calls to lower level unplug routines (it's
> hard to be sure but it looks to me like the caller of the unplug routine
> is supposed to make the trace call).
>
>  
>
> Patch against 8.2 (and tested with kernel 2.6.18) attached for your
> review

Will do...
though only for current git and the next kernel submission iteration.
BTW, our "homegrown" tracing framework has been ripped out upstream,
so we will probably replace it by trace_events "soonish".
Unless you beat us to it ;)

Thanks,


--
: Lars Ellenberg
: LINBIT | Your Way to High Availability
: DRBD/HA support and consulting http://www.linbit.com

DRBD® and LINBIT® are registered trademarks of LINBIT, Austria.
_______________________________________________
drbd-dev mailing list
drbd-dev@...
http://lists.linbit.com/mailman/listinfo/drbd-dev