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)
	Sat, Oct 3, 12:44 PM

	Unknown Object (File)
	Fri, Oct 2, 10:10 AM

	Unknown Object (File)
	Sat, Sep 26, 6:07 AM

	Unknown Object (File)
	Sat, Sep 26, 3:20 AM

	Unknown Object (File)
	Fri, Sep 25, 9:16 AM

	Unknown Object (File)
	Fri, Sep 25, 9:11 AM

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

	Unknown Object (File)
	Thu, Sep 17, 7:58 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 75098
Build 71981: 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
126	Why not using posix_spawn() there?
136	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
126	What would be the advantage?
136	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
126	That the used API is specifically designed for the task at hand.
136	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
136	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
170	Shouldn't this be `pwait_reap_test(false)`?

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

bin/pwait/tests/pwait_reap.c
170	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

188 lines

Diff 182621

bin/pwait/tests/Makefile

Loading...

bin/pwait/tests/pwait_reap.c

Loading...