[Bug dyninst/15743] New: dyninst "utrace" probes (with a path) seem to be broken

[Bug dyninst/15743] New: dyninst "utrace" probes (with a path) seem to be broken

[dsmith at redhat dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=15743

            Bug ID: 15743
           Summary: dyninst "utrace" probes (with a path) seem to be
                    broken
           Product: systemtap
           Version: unspecified
            Status: NEW
          Severity: normal
          Priority: P2
         Component: dyninst
          Assignee: systemtap at sourceware dot org
          Reporter: dsmith at redhat dot com

On rawhide (3.11.0-0.rc0.git7.1.fc20.x86_64), the following works:

=====
# stap --dyninst -e 'probe process.begin { printf("begin\n") }' -c "echo hi"
begin
hi
====

However, this fails:

====
# stap --dyninst -e 'probe process("echo").begin { printf("begin\n") }' -c
"echo hi"--SERIOUS-- #100: ../../dyninstAPI/src/BPatch_module.C[513]:  Module
stap_1a8b5b33112cf2e384608705faa351a8_951.so: unable to find function
enter_dyninst_uprobe_regs
--SERIOUS-- #100: ../../dyninstAPI/src/BPatch_module.C[513]:  Module
stap_1a8b5b33112cf2e384608705faa351a8_951.so: unable to find function
enter_dyninst_uprobe
hi
====

(these failures were pointed out by systemtap.base/utrace_p5.exp)

The '--SERIOUS--' error message are coming from dyninst itself I believe.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug dyninst/15743] dyninst "utrace" probes (with a path) seem to be broken

[dsmith at redhat dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=15743

--- Comment #1 from David Smith <dsmith at redhat dot com> ---
"git bisect" tells me the following commit is the guilty one:

====
commit 3debb935ae062df047be2563b7b7dcc0bdfd2d83
Author: Josh Stone <jistone@redhat.com>
Date:   Wed May 22 16:14:18 2013 -0700

    stapdyn: separate enter_dyninst_uprobe_regs

    That function calls enter_dyninst_uprobe, which is only defined when
    uprobes are active, so we have a weak binding in place.  By instead
    splitting this into a new file, which is only included when uprobes are
    known to be active, it doesn't need the weak reference.
====

Unfortunately, that weak reference was needed in the case where there are no
uprobe probes.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug dyninst/15743] dyninst "utrace" probes (with a path) seem to be broken

[dsmith at redhat dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=15743

David Smith <dsmith at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from David Smith <dsmith at redhat dot com> ---
Fixed in commit 126305a by only looking up the uprobe functions if needed.

-- 
You are receiving this mail because:
You are the assignee for the bug.