Page MenuHomeFreeBSD

Fix fallout from the emancipation of dump(8) workers
ClosedPublic

Authored by cperciva on Jun 22 2020, 8:37 PM.
Tags
None
Referenced Files
F103064754: D25403.diff
Wed, Nov 20, 11:29 AM
Unknown Object (File)
Thu, Oct 24, 2:35 AM
Unknown Object (File)
Thu, Oct 24, 2:35 AM
Unknown Object (File)
Thu, Oct 24, 2:23 AM
Unknown Object (File)
Oct 4 2024, 8:57 AM
Unknown Object (File)
Oct 4 2024, 4:45 AM
Unknown Object (File)
Oct 2 2024, 12:22 AM
Unknown Object (File)
Oct 1 2024, 9:07 PM
Subscribers

Details

Summary

The change from "slave" processes to "minion" processes to "worker" processes left some less-than-coherent names:

  1. "enslave" turned into the ungrammatical "enworker".
  2. "slp" (SLave Pointer) turned into "mlp" (Minion [L] Pointer?).

Convert "enworker" to "create_workers" (the function in question forks off 3 worker processes), and replace "mlp" with "wp" (Worker Pointer) and "tmlp" with "twp" (Temporary Worker Pointer).

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 31880

Event Timeline

cperciva created this revision.

Thanks for taking care of this Colin. The diff looks fine at a quick glance.

Diff looks good to me, thanks. I would suggest not using "emancipation" in the commit subject.

This revision is now accepted and ready to land.Jun 22 2020, 8:50 PM
This revision was automatically updated to reflect the committed changes.

Thanks for this cleanup. I found the previous transformation grating for the reasons that you listed.