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
F170747123: D3857.id.diff
Sun, Sep 6, 9:36 AM
F170734215: D3857.id9281.diff
Sun, Sep 6, 7:22 AM
F170710850: D3857.diff
Sun, Sep 6, 3:50 AM
Unknown Object (File)
Sat, Sep 5, 11:53 AM
Unknown Object (File)
Thu, Sep 3, 7:24 AM
Unknown Object (File)
Thu, Sep 3, 3:35 AM
Unknown Object (File)
Wed, Sep 2, 9:04 AM
Unknown Object (File)
Tue, Sep 1, 10:55 PM
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...