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)
May 29 2025, 2:27 AM
Unknown Object (File)
May 24 2025, 8:13 AM
Unknown Object (File)
May 22 2025, 10:22 PM
Unknown Object (File)
Apr 4 2025, 9:22 PM
Unknown Object (File)
Feb 2 2025, 12:36 AM
Unknown Object (File)
Jan 31 2025, 11:16 PM
Unknown Object (File)
Dec 4 2024, 1:53 AM
Unknown Object (File)
Oct 2 2024, 8:36 PM
View All 22 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...