Page MenuHomeFreeBSD
Differential D3857

Switch pl_child_pid from int to pid_t.
ClosedPublic
Actions

Authored by jhb on Oct 9 2015, 4:22 PM.
Tags
None
Referenced Files
F148522906: D3857.id9546.diff
Wed, Mar 18, 11:08 AM
F148520402: D3857.id9281.diff
Wed, Mar 18, 10:39 AM
Unknown Object (File)
Sun, Mar 15, 10:45 AM
Unknown Object (File)
Tue, Mar 10, 5:29 PM
Unknown Object (File)
Tue, Mar 10, 5:28 PM
Unknown Object (File)
Jan 24 2026, 10:57 PM
Unknown Object (File)
Dec 20 2025, 2:25 PM
Unknown Object (File)
Nov 24 2025, 9:28 AM
View All Files
Subscribers
emaste
imp

Details

Reviewers
kib
emaste
Group Reviewers
manpages
Commits
rS289636: Switch pl_child_pid from int to pid_t.
Summary

Switch pl_child_pid from int to pid_t.

Test Plan
  • make universe passed
  • verified with gdb that size of ptrace_lwpinfo and offset of pl_syscall_code were identical for old and new binaries

Diff Detail

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

Event Timeline

jhb updated this revision to Diff 9281.Oct 9 2015, 4:22 PM
jhb retitled this revision from to Switch pl_child_pid from int to pid_t..
jhb updated this object.
jhb edited the test plan for this revision. (Show Details)
jhb added a reviewer: kib.
Herald added a reviewer: manpages. · View Herald TranscriptOct 9 2015, 4:22 PM
Herald added subscribers: emaste, imp. · View Herald Transcript
emaste accepted this revision.Oct 9 2015, 5:06 PM
emaste added a reviewer: emaste.
This revision is now accepted and ready to land.Oct 9 2015, 5:06 PM
kib accepted this revision.Oct 10 2015, 5:56 AM
kib edited edge metadata.

I think that I used int because it is more explicit for ABI. But since the struct ptrace_lwpinfo is not 32/64 bit invariant, this is moot.

Closed by commit rS289636: Switch pl_child_pid from int to pid_t. (authored by jhb). · Explain WhyOct 20 2015, 5:58 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
lib/
libc/
sys/
4 lines
sys/
kern/
2 lines
sys/
2 lines

Diff 9546

View Options

head/lib/libc/sys/ptrace.2

Loading...
View Options

head/sys/kern/sys_process.c

Loading...
View Options

head/sys/sys/ptrace.h

Loading...