Page MenuHomeFreeBSD

sys: Remove DEV_RANDOM device option
ClosedPublic

Authored by cem on Apr 15 2019, 8:46 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 15 2024, 7:46 AM
Unknown Object (File)
Feb 23 2024, 11:14 PM
Unknown Object (File)
Dec 17 2023, 2:22 PM
Unknown Object (File)
Nov 29 2023, 10:06 PM
Unknown Object (File)
Nov 18 2023, 6:53 PM
Unknown Object (File)
Nov 18 2023, 6:49 PM
Unknown Object (File)
Nov 18 2023, 6:13 PM
Unknown Object (File)
Nov 4 2023, 4:19 PM

Details

Summary

Remove 'device random' from kernel configurations that reference it (most).
Replace perhaps mistaken 'nodevice random' in two MIPS configs with 'options
RANDOM_LOADABLE' instead. Document removal in UPDATING; update NOTES and
random.4.

Test Plan

tinderbox is clean

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 23697
Build 22666: arc lint + arc unit

Event Timeline

I'm in favor of this change; please consider this as an explicit "accepted" if nobody objects in a week.

UPDATING
35

Maybe make the second sentence first (Entropy collection.. are no longer an optional component, and the "device random" option has been removed) to emphasize that it's now enabled by default?

I'm in favor of this change; please consider this as an explicit "accepted" if nobody objects in a week.

Thanks! I will hold off on this change a bit (probably more than 1 week) until I've put out the fires from the last change.

UPDATING
35

Sure, that sounds reasonable to me. Perhaps s/an optional component/optional components/ as well to match the plurality of the object to the subject.

Rebase on WIP changes; no functional change.

Hi @delphij , @markm . Is this still something that seems reasonable or should I drop it?

No objection from me.

This revision is now accepted and ready to land.Jun 20 2019, 7:37 AM

I think I have accepted back in April but let's make it an explicit one. Be sure to update the dates though.

This revision now requires review to proceed.Jun 20 2019, 8:52 PM
UPDATING
35

will fix.

This revision was not accepted when it landed; it landed in state Needs Review.Jun 21 2019, 12:16 AM
This revision was automatically updated to reflect the committed changes.