This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Adding systemtap probe points in pthread library(slightly revised again)


On Wed, Feb 9, 2011 at 18:09, Rayson Ho <rho@redhat.com> wrote:
> Thanks again, Roland! I've applied my changes on top of your latest
> branch refresh, and it works with & without those probes enabled.
>
> So here's my pthread probing patch again:
>
> diff --git a/nptl/DESIGN-systemtap-probes.txt b/nptl/DESIGN-systemtap-probes.txt
> index e69de29..5485ce4 100644
> --- a/nptl/DESIGN-systemtap-probes.txt
> +++ b/nptl/DESIGN-systemtap-probes.txt
> @@ -0,0 +1,88 @@

> +
> +wrlock_entry - probe for entry to the pthread_rwlock_wrlock function
> + Â Â Â Â Â Â arg1 = address of rw lock
> +rdlock_entry - probe for entry to the pthread_rwlock_rdlock function
> + Â Â Â Â Â Â arg1 = address of rw lock
> +
> +rwlock_destroy - probe for pthread_rwlock_destroy
> + Â Â Â Â Â Â Â arg1 = address of rw lock
> +rwlock_acquire_write - probe for pthread_rwlock_wrlock (after getting the lock)
> + Â Â Â Â Â Â Â Â Â Â arg1 = address of rw lock

IMHO, this is a typo. Because for 'read' you have
'rdlock_acquire_read' and 'rdlock_entry'. So judging from this
pattern, I would expect, that this probe would be
'wrlock_acquire_write', like 'wrlock_entry'. Also note, that in the
'acquire' probe names are redundancies for 'read' and 'write' (after
fixing the typo), while in the 'entry' names not.

Bert

> +rdlock_acquire_read - probe for pthread_rwlock_rdlock after successfully getting
> + Â Â Â Â Â Â Â Â Â Â Âthe lock
> + Â Â Â Â Â Â Â Â Â Âarg1 = address of rw lock
> +rwlock_unlock - probe for pthread_rwlock_unlock
> + Â Â Â Â Â Â Âarg1 = address of rw lock
> +


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]