Page MenuHomeFreeBSD
Differential D58292

EVFILT_PROCDESC: support NOTE_FORK
ClosedPublic
Actions

Authored by kib on Fri, Jul 17, 5:41 AM.
Tags
None
Referenced Files
F164384430: D58292.id182124.diff
Fri, Jul 31, 9:22 AM
F164370113: D58292.id182159.diff
Fri, Jul 31, 6:14 AM
F164368798: D58292.id182159.diff
Fri, Jul 31, 6:00 AM
F164362444: D58292.id182243.diff
Fri, Jul 31, 4:45 AM
F164334796: D58292.id182258.diff
Thu, Jul 30, 10:10 PM
F164323212: D58292.id.diff
Thu, Jul 30, 7:14 PM
F164314560: D58292.id182243.diff
Thu, Jul 30, 4:54 PM
Unknown Object (File)
Thu, Jul 30, 3:31 AM
View All 59 Files
Subscribers
imp
olce
ziaee

Details

Reviewers
kevans
markj
Commits
rG3f88f6b89942: kqueue.2: document EVFILT_PROCDESC support for NOTE_FORK
rGe8d4d754e1c7: EVFILT_PROCDESC: support NOTE_FORK

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

kib created this revision.Fri, Jul 17, 5:41 AM
Herald added subscribers: ziaee, olce, imp. · View Herald TranscriptFri, Jul 17, 5:41 AM
kib requested review of this revision.Fri, Jul 17, 5:41 AM
markj added inline comments.Fri, Jul 17, 1:32 PM
lib/libsys/kqueue.2
597
601
602

Also, if the process raises NOTE_EXIT, the last PID will be overwritten (but I think we will return NOTE_FORK | NOTE_EXIT).

605

Perhaps mention ptrace(2) explicitly?

sys/kern/sys_procdesc.c
521

Probably you want to kn->kn_flags &= ~NOTE_FORK here?

kib marked 5 inline comments as done.Fri, Jul 17, 1:57 PM
kib added inline comments.
sys/kern/sys_procdesc.c
521

Hm, no. I will explicitly mention that exit code is returned.

kib updated this revision to Diff 182124.Fri, Jul 17, 1:57 PM
kib marked an inline comment as done.

Update the man page with the review comments.

markj added inline comments.Fri, Jul 17, 1:58 PM
sys/kern/sys_procdesc.c
521

(I meant kn->kn_fflags above of course.)

kib marked an inline comment as done.Fri, Jul 17, 2:11 PM
kib added inline comments.
sys/kern/sys_procdesc.c
521

I do want (need) to see NOTE_FORK in this situation.

kib updated this revision to Diff 182152.Fri, Jul 17, 8:02 PM
kib marked an inline comment as done.

Report NOTE_FORK for procdesc after the child is attached to the debugger.

markj added a comment.Fri, Jul 17, 8:19 PM
In D58292#1337327, @kib wrote:

Report NOTE_FORK for procdesc after the child is attached to the debugger.

I suggest adding a test for this, or at least a comment explaining why it is important.

lib/libsys/kqueue.2
597

since you use it below. Or write, The process identifier (PID) of the ...

605
606
kib marked 3 inline comments as done.Fri, Jul 17, 8:48 PM
In D58292#1337369, @markj wrote:
In D58292#1337327, @kib wrote:

Report NOTE_FORK for procdesc after the child is attached to the debugger.

I suggest adding a test for this, or at least a comment explaining why it is important.

I added the comment. truss(1) in the cap mode is the practical test.

kib updated this revision to Diff 182159.Fri, Jul 17, 8:50 PM

Add comment for the placement of NOTE_FORK activation.
Edit the man page.

markj accepted this revision.Sun, Jul 19, 6:38 PM
markj added inline comments.
lib/libsys/kqueue.2
629
sys/kern/kern_fork.c
830
832
833
This revision is now accepted and ready to land.Sun, Jul 19, 6:38 PM
kib updated this revision to Diff 182243.Sun, Jul 19, 6:50 PM
kib marked 4 inline comments as done.

Edit comments and the man page update.

This revision now requires review to proceed.Sun, Jul 19, 6:50 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sun, Jul 19, 8:41 PM
Closed by commit rGe8d4d754e1c7: EVFILT_PROCDESC: support NOTE_FORK (authored by kib). · Explain Why
This revision was automatically updated to reflect the committed changes.
kib added a commit: rGe8d4d754e1c7: EVFILT_PROCDESC: support NOTE_FORK.
kib added a commit: rG3f88f6b89942: kqueue.2: document EVFILT_PROCDESC support for NOTE_FORK.

Revision Contents
Changeset List

PathSize
lib/
libsys/
35 lines
sys/
kern/
9 lines
19 lines
sys/
3 lines
2 lines

Diff 182159

View Options

lib/libsys/kqueue.2

Loading...
View Options

sys/kern/kern_fork.c

Loading...
View Options

sys/kern/sys_procdesc.c

Loading...
View Options

sys/sys/event.h

Loading...
View Options

sys/sys/procdesc.h

Loading...