Page MenuHomeFreeBSD
Differential D30513

ktrace: Fix some problems with process subtree traversal
Needs ReviewPublic
Actions

Authored by markj on May 27 2021, 7:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jun 21, 2:35 PM
Unknown Object (File)
Apr 29 2026, 9:21 AM
Unknown Object (File)
Nov 28 2025, 1:33 AM
Unknown Object (File)
Nov 26 2025, 12:47 AM
Unknown Object (File)
Nov 21 2025, 6:45 AM
Unknown Object (File)
Nov 15 2025, 7:53 AM
Unknown Object (File)
Nov 5 2025, 7:21 PM
Unknown Object (File)
Oct 25 2025, 4:23 AM
View All 44 Files
Subscribers
imp

Details

Reviewers
kib
Summary


- Avoid visiting processes that were reparented, since they represent
debuggees possibly belonging to a different subtree.
- Visit orphan processes, to ensure that reparented debuggees that do
belong to the subtree are treated consistently.

Reported by: kib

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 39520
Build 36409: arc lint + arc unit

Event Timeline

markj created this revision.May 27 2021, 7:29 PM
Herald added a subscriber: imp. · View Herald TranscriptMay 27 2021, 7:29 PM
markj requested review of this revision.May 27 2021, 7:29 PM
Harbormaster completed remote builds in B39520: Diff 90025.May 27 2021, 7:29 PM
markj added a reviewer: kib.May 27 2021, 7:29 PM
kib added inline comments.May 27 2021, 9:52 PM
sys/kern/kern_ktrace.c
1244

I understand why this assert should be true for 'p' equal to the function argument. But after we started iterating over siblings, I think this is no longer true, generally.

1268

For orphans, I believe the iteration should be also done in depth, same as for siblings. Children of an orphan are also descendants of the process.

Revision Contents
Changeset List

PathSize
sys/
kern/
63 lines

Diff 90025

View Options

sys/kern/kern_ktrace.c

Loading...