[Bug dyninst/14573] New: Pass pt_regs to dyninst probe handlers

classic Classic list List threaded Threaded
7 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug dyninst/14573] New: Pass pt_regs to dyninst probe handlers

glaubitz at physik dot fu-berlin.de
http://sourceware.org/bugzilla/show_bug.cgi?id=14573

             Bug #: 14573
           Summary: Pass pt_regs to dyninst probe handlers
           Product: systemtap
           Version: unspecified
            Status: NEW
          Severity: normal
          Priority: P2
         Component: dyninst
        AssignedTo: [hidden email]
        ReportedBy: [hidden email]
    Classification: Unclassified


In order to examine much process state with stapdyn, we're going to need
registers.  We should pass a pt_regs* (/usr/include/asm/ptrace.h) as a
parameter to the probe handler.

I believe this can be allocated and prepared via BPatch_register code snippets.
 We will probably need to hardcode architecture-specific support for the
register layouts, but I think that's ok as these are generally unchanging.

Not all registers will be available.  Right now dyninst only exposes the
general purpose registers, and none of the special registers.  We do need
access to the frame and stack pointers, but that should come soon - see:
https://lists.cs.wisc.edu/archive/dyninst-api/2012/msg00093.shtml

--
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug dyninst/14573] Pass pt_regs to dyninst probe handlers

glaubitz at physik dot fu-berlin.de


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



--- Comment #1 from Josh Stone <jistone at redhat dot com> 2012-10-11 17:20:22 UTC ---

Commit dd94003 started this by passing individual registers to a separate entry

point, which then packs those up into a pt_regs.  It would still be nicer to

find a way to build pt_regs more directly though.



--

Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email

------- You are receiving this mail because: -------

You are the assignee for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug dyninst/14573] Pass pt_regs to dyninst probe handlers

glaubitz at physik dot fu-berlin.de
In reply to this post by glaubitz at physik dot fu-berlin.de
http://sourceware.org/bugzilla/show_bug.cgi?id=14573

Josh Stone <jistone at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Depends on|                            |15136

--
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug dyninst/14573] Pass pt_regs to dyninst probe handlers

glaubitz at physik dot fu-berlin.de
In reply to this post by glaubitz at physik dot fu-berlin.de
http://sourceware.org/bugzilla/show_bug.cgi?id=14573

--- Comment #2 from Josh Stone <jistone at redhat dot com> ---
Commit 09bf82fd adds basic powerpc support, but it's limited to only 6
registers (including IP) due to a limitation noted in Dyninst's
EmitterPOWER::emitCall().

--
You are receiving this mail because:
You are the assignee for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug dyninst/14573] Pass pt_regs to dyninst probe handlers

glaubitz at physik dot fu-berlin.de
In reply to this post by glaubitz at physik dot fu-berlin.de
http://sourceware.org/bugzilla/show_bug.cgi?id=14573

Josh Stone <jistone at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Depends on|                            |15614

--
You are receiving this mail because:
You are the assignee for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug dyninst/14573] Pass pt_regs to dyninst probe handlers

Sourceware - systemtap mailing list
In reply to this post by glaubitz at physik dot fu-berlin.de
https://sourceware.org/bugzilla/show_bug.cgi?id=14573
Bug 14573 depends on bug 15614, which changed state.

Bug 15614 Summary: stapdyn only partially fills registers on ppc
https://sourceware.org/bugzilla/show_bug.cgi?id=15614

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

--
You are receiving this mail because:
You are the assignee for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug dyninst/14573] Pass pt_regs to dyninst probe handlers

Sourceware - systemtap mailing list
In reply to this post by glaubitz at physik dot fu-berlin.de
https://sourceware.org/bugzilla/show_bug.cgi?id=14573

Stan Cox <scox at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |DUPLICATE
                 CC|                            |scox at redhat dot com

--- Comment #3 from Stan Cox <scox at redhat dot com> ---
Fixed by setting up all the regs using multiple synthesized calls.

*** This bug has been marked as a duplicate of bug 24954 ***

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