Page MenuHomeFreeBSD
Differential D58391

rc.subr: Fix premature return from wait_for_pids
ClosedPublic
Actions

Authored by des on Wed, Jul 22, 8:18 AM.
Tags
None
Referenced Files
F164288732: D58391.id182668.diff
Thu, Jul 30, 10:37 AM
F164286324: D58391.id.diff
Thu, Jul 30, 9:59 AM
F164284605: D58391.id182546.diff
Thu, Jul 30, 9:33 AM
F164277024: D58391.id182668.diff
Thu, Jul 30, 7:38 AM
F164276828: D58391.id182668.diff
Thu, Jul 30, 7:35 AM
F164276337: D58391.id182440.diff
Thu, Jul 30, 7:26 AM
F164275949: D58391.id182440.diff
Thu, Jul 30, 7:20 AM
F164258961: D58391.id182546.diff
Thu, Jul 30, 4:29 AM
View All 41 Files
Subscribers
imp
markj

Details

Reviewers
dch
feld
markj
Group Reviewers
Klara
Commits
rG356d0b79cf6f: rc.subr: Fix premature return from wait_for_pids
Summary

Use pwait's new -r option to wait until the target processes have not
only terminated, but also been reaped.

PR: 293183
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 75000
Build 71883: arc lint + arc unit

Event Timeline

des created this revision.Wed, Jul 22, 8:18 AM
Herald added a subscriber: imp. · View Herald TranscriptWed, Jul 22, 8:18 AM
des requested review of this revision.Wed, Jul 22, 8:18 AM
Harbormaster completed remote builds in B74998: Diff 182440.Wed, Jul 22, 8:18 AM
des added a parent revision: D58314: pwait: Optionally wait until process is reaped.Wed, Jul 22, 8:19 AM
des updated this revision to Diff 182442.Wed, Jul 22, 8:28 AM

add a note in UPDATING

Harbormaster completed remote builds in B75000: Diff 182442.Wed, Jul 22, 8:28 AM
markj accepted this revision as: markj.Thu, Jul 23, 4:03 PM
markj added a subscriber: markj.

Should this have a Fixes: tag?

Maybe we should provide a backwards compat shim which checks whether the installed pwait supports -r and falls back to something else if not. I'm not sure if it's worth it.

UPDATING
34

I don't know what "fastboot" means, maybe write "use fastboot(8)" to be a bit more explicit.

This revision is now accepted and ready to land.Thu, Jul 23, 4:03 PM
des marked an inline comment as done.Thu, Jul 23, 4:41 PM
des updated this revision to Diff 182546.Thu, Jul 23, 5:48 PM

rf

This revision now requires review to proceed.Thu, Jul 23, 5:48 PM
Harbormaster completed remote builds in B75051: Diff 182546.Thu, Jul 23, 5:48 PM
des updated this revision to Diff 182668.Sat, Jul 25, 12:36 PM

simplify test

Harbormaster completed remote builds in B75121: Diff 182668.Sat, Jul 25, 12:36 PM
des mentioned this in D58314: pwait: Optionally wait until process is reaped.Sun, Jul 26, 9:38 AM
This revision was not accepted when it landed; it landed in state Needs Review.Mon, Jul 27, 10:17 AM
Closed by commit rG356d0b79cf6f: rc.subr: Fix premature return from wait_for_pids (authored by des). · Explain Why
This revision was automatically updated to reflect the committed changes.
des added a commit: rG356d0b79cf6f: rc.subr: Fix premature return from wait_for_pids.

Revision Contents
Changeset List

PathSize
11 lines
libexec/
rc/
2 lines

Diff 182442

View Options

UPDATING

Loading...
View Options

libexec/rc/rc.subr

Loading...