tests/procdesc: Fix race in pdopenpid_pdwait_only_one
ClosedPublic
Actions

Authored by olivier on Jul 29 2026, 8:22 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Sep 11, 8:35 AM

	Unknown Object (File)
	Thu, Sep 10, 7:46 PM

	Unknown Object (File)
	Thu, Sep 10, 1:00 AM

	Unknown Object (File)
	Thu, Sep 10, 1:00 AM

	Unknown Object (File)
	Wed, Sep 9, 2:49 PM

	Unknown Object (File)
	Sun, Sep 6, 1:18 PM

	Unknown Object (File)
	Sun, Sep 6, 2:24 AM

	Unknown Object (File)
	Thu, Sep 3, 8:33 AM

View All 39 Files

Subscribers

imp

Details

Reviewers

markj
asomers

Commits

rG727a83e90098: tests/procdesc: Fix race in pdopenpid_pdwait_only_one

Summary

The child exited immediately after pdfork(), so the parent's pdopenpid() could catch it mid-exit (P_WEXIT) and fail with EBUSY.
Block the child on a pipe until the parent has opened the second descriptor, then release it

Test Plan

Without this patch, I sometimes meet this error:

tests/sys/kern/procdesc.c:534: pdopenpid: Device busy

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

olivier requested review of this revision.Jul 29 2026, 8:22 PM

olivier created this revision.

markj accepted this revision.Jul 30 2026, 2:15 PM

This revision is now accepted and ready to land.Jul 30 2026, 2:15 PM

Closed by commit rG727a83e90098: tests/procdesc: Fix race in pdopenpid_pdwait_only_one (authored by olivier). · Explain WhyJul 30 2026, 2:33 PM

This revision was automatically updated to reflect the committed changes.

olivier added a commit: rG727a83e90098: tests/procdesc: Fix race in pdopenpid_pdwait_only_one.

Herald added a subscriber: imp. · View Herald TranscriptJul 30 2026, 2:33 PM

Revision Contents
Changeset List

Path

Size

tests/

sys/

kern/

procdesc.c

17 lines

Diff 183109

View Options

tests/procdesc: Fix race in pdopenpid_pdwait_only_oneClosedPublicActions