Page MenuHomeFreeBSD

Add KTR tracing for some MI ptrace events.
ClosedPublic

Authored by jhb on May 25 2015, 12:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 11, 1:57 AM
Unknown Object (File)
Wed, Dec 4, 10:11 PM
Unknown Object (File)
Wed, Dec 4, 10:08 PM
Unknown Object (File)
Wed, Nov 27, 2:53 AM
Unknown Object (File)
Sun, Nov 24, 7:48 AM
Unknown Object (File)
Nov 14 2024, 5:34 PM
Unknown Object (File)
Nov 14 2024, 2:52 PM
Unknown Object (File)
Nov 10 2024, 9:21 PM
Subscribers

Details

Summary

Add KTR tracing for some MI ptrace events.

Test Plan
  • I'm using this while debugging ptrace() bugs.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb retitled this revision from to Add KTR tracing for some MI ptrace events..
jhb updated this object.
jhb edited the test plan for this revision. (Show Details)
jhb added a reviewer: kib.

Should MD ptrace actions be traced as well ? I mean the ops handled in arch/arch/ptrace_machdep.c.

sys/kern/kern_exit.c
1224 ↗(On Diff #5675)

Please, provide the stock p->p_xstat value there.

sys/kern/sys_process.c
884 ↗(On Diff #5675)

Wouldn't previous follow-fork mode value also be useful there ?

jhb marked 2 inline comments as done.May 25 2015, 1:21 PM
In D2643#49325, @kib wrote:

Should MD ptrace actions be traced as well ? I mean the ops handled in arch/arch/ptrace_machdep.c.

I think so, though I would maybe add those later. I will probably add x86 in a follow up at least. For now my focus is on wait()'ing for child exit status (and the follow fork stuff) which is all MI.

sys/kern/kern_exit.c
1224 ↗(On Diff #5675)

Ok, I will provide both as I think it is useful to also see the "cooked" status returned by wait.

jhb marked an inline comment as done.
jhb edited edge metadata.
  • Two suggestions from kib@.
  • Compile.
kib edited edge metadata.
This revision is now accepted and ready to land.May 25 2015, 5:16 PM
This revision was automatically updated to reflect the committed changes.