Paths

Table of Contentst

Differential D58385

pwait: Test the new -r option
ClosedPublic
Actions

Authored by des on Jul 21 2026, 8:08 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Sep 17, 10:07 PM

	Unknown Object (File)
	Thu, Sep 17, 7:58 PM

	Unknown Object (File)
	Wed, Sep 9, 1:15 PM

	Unknown Object (File)
	Tue, Sep 8, 3:13 PM

	Unknown Object (File)
	Tue, Sep 8, 2:38 PM

	Unknown Object (File)
	Tue, Sep 8, 1:24 PM

	Unknown Object (File)
	Sat, Sep 5, 9:57 PM

	Unknown Object (File)
	Sat, Sep 5, 8:28 PM

Subscribers

Details

Reviewers

kib
kevans
markj

Group Reviewers

Commits

rG4e6679c9c422: pwait: Test the new -r option
rGe87afada8e8a: pwait: Test the new -r option
rGe115066370dc: pwait: Test the new -r option

Summary

Test that pwait without -r reports a process as soon as it terminates,
while pwait with -r does not report it until it has been reaped.

MFC after: 1 week
Sponsored by: Klara, Inc.
Sponsored by: NetApp, Inc.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 74989
Build 71872: arc lint + arc unit

Event Timeline

des created this revision.Jul 21 2026, 8:08 PM

Herald added a subscriber: imp. · View Herald TranscriptJul 21 2026, 8:08 PM

des requested review of this revision.Jul 21 2026, 8:08 PM

Harbormaster completed remote builds in B74989: Diff 182418.Jul 21 2026, 8:08 PM

des added a parent revision: D58314: pwait: Optionally wait until process is reaped.Jul 21 2026, 8:09 PM

des added a parent revision: D58386: pwait: Add a SIGINFO handler.Jul 21 2026, 8:19 PM

des retitled this revision from pwait: Test that pwait does not report zombies to pwait: Test the new -r option.Jul 22 2026, 8:16 AM

des edited the summary of this revision. (Show Details)

new behavior is now optional, test both variants

Harbormaster completed remote builds in B74997: Diff 182439.Jul 22 2026, 8:18 AM

rename test

Harbormaster completed remote builds in B75001: Diff 182443.Jul 22 2026, 8:33 AM

kib added inline comments.Jul 22 2026, 11:44 PM

bin/pwait/tests/pwait_reap.c
125 ↗	(On Diff #182443)	Why not using posix_spawn() there?
135 ↗	(On Diff #182443)	You are not waiting for the termination there, wait_zero(false) only checks for termination and asserts if not. As the consequence, the test is racy, since nothing guarantees that the target has exited when waitpid() is called.

des added inline comments.Jul 23 2026, 6:20 AM

bin/pwait/tests/pwait_reap.c
125 ↗	(On Diff #182443)	What would be the advantage?
135 ↗	(On Diff #182443)	I think you're confusing `WNOWAIT` with `WNOHANG`.

des added a reviewer: Klara.Jul 23 2026, 5:50 PM

kib added inline comments.Jul 23 2026, 11:32 PM

bin/pwait/tests/pwait_reap.c
125 ↗	(On Diff #182443)	That the used API is specifically designed for the task at hand.
135 ↗	(On Diff #182443)	No, I do not. Re-read what I wrote: wait_zero(false) may assert falsely.

des added inline comments.Jul 24 2026, 10:00 AM

bin/pwait/tests/pwait_reap.c
135 ↗	(On Diff #182443)	No. It will block until the target terminates. It will fail if interrupted, but I don't consider that a problem.

tolerate EINTR

Harbormaster completed remote builds in B75098: Diff 182621.Jul 24 2026, 4:28 PM

kib accepted this revision.Jul 24 2026, 10:02 PM

This revision is now accepted and ready to land.Jul 24 2026, 10:02 PM

Closed by commit rGe115066370dc: pwait: Test the new -r option (authored by des). · Explain WhyJul 27 2026, 10:17 AM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGe115066370dc: pwait: Test the new -r option.

markj added inline comments.Jul 27 2026, 1:13 PM

bin/pwait/tests/pwait_reap.c
169 ↗	(On Diff #182771)	Shouldn't this be `pwait_reap_test(false)`?

des added inline comments.Jul 27 2026, 6:49 PM

bin/pwait/tests/pwait_reap.c
169 ↗	(On Diff #182771)	You're right, will fix.

des added a commit: rGe87afada8e8a: pwait: Test the new -r option.Aug 3 2026, 11:39 AM

des added a commit: rG4e6679c9c422: pwait: Test the new -r option.

Revision Contents
Changeset List

Path

Size

bin/

pwait/

tests/

1 line

122 lines

Diff 182418

bin/pwait/tests/Makefile

Loading...

bin/pwait/tests/pwait_zombie.c

Loading...