Differential D20192

random: x86 driver: Prefer RDSEED over RDRAND when available
ClosedPublic
Actions

Authored by cem on May 8 2019, 12:15 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Apr 19, 1:24 AM

	Unknown Object (File)
	Mon, Apr 8, 3:25 PM

	Unknown Object (File)
	Jan 4 2024, 2:43 PM

	Unknown Object (File)
	Dec 31 2023, 7:19 AM

	Unknown Object (File)
	Dec 19 2023, 9:05 PM

	Unknown Object (File)
	Nov 11 2023, 6:27 PM

	Unknown Object (File)
	Nov 10 2023, 10:46 PM

	Unknown Object (File)
	Oct 13 2023, 3:15 PM

View All 26 Files

Subscribers

None

Details

Reviewers

delphij
markm
imp
jhb
kib
jmg

Group Reviewers

O3: Kernel Random Numbers Generator

(Owns No Changed Paths)

Commits

rS347239: random: x86 driver: Prefer RDSEED over RDRAND when available

Summary

Per
https://software.intel.com/en-us/blogs/2012/11/17/the-difference-between-rdrand-and-rdseed
, RDRAND is a PRNG seeded from the same source as RDSEED. The source is
more suitable as PRNG seed material, so prefer it when the RDSEED intrinsic
is available (indicated in CPU feature bits).

Test Plan

Bhyve blocks RDSEED feature bit, unfortunately. So I have not tested RDSEED
functionality of this patch yet.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 24148
Build 23000: arc lint + arc unit

Event Timeline

cem created this revision.May 8 2019, 12:15 AM

Harbormaster completed remote builds in B24145: Diff 57153.May 8 2019, 12:15 AM

seems sane to me...

This revision is now accepted and ready to land.May 8 2019, 12:17 AM

jhb added inline comments.May 8 2019, 12:22 AM

sys/dev/random/ivy.c
84	Maybe "unsupported compiler" or something. I'd honestly just remove the #ifdef and force compile failure though.
98	I don't think you need the '!!' since these variables are 'bool'.

jhb added a reviewer: kib.May 8 2019, 12:22 AM

cem added inline comments.May 8 2019, 12:33 AM

sys/dev/random/ivy.c
84	Will fix. :)
98	Ah, you're totally right. I'll drop that bit.

Drop ifndef wat
Drop bangbang "operator" -- bool implicitly coerces on assignment already

This revision now requires review to proceed.May 8 2019, 12:35 AM

Harbormaster completed remote builds in B24148: Diff 57156.May 8 2019, 12:35 AM

LGTM in principle

This revision is now accepted and ready to land.May 8 2019, 12:35 AM

cem added a parent revision: D20194: vmm(4): Pass through RDSEED feature bit to guests.May 8 2019, 12:38 AM

jhb accepted this revision.May 8 2019, 12:41 AM

Closed by commit rS347239: random: x86 driver: Prefer RDSEED over RDRAND when available (authored by cem). · Explain WhyMay 8 2019, 12:45 AM

This revision was automatically updated to reflect the committed changes.

Owners added a reviewer: O3: Kernel Random Numbers Generator.May 8 2019, 12:45 AM

Herald added a reviewer: jmg. · View Herald TranscriptMay 8 2019, 12:45 AM

Revision Contents
Changeset List

Path

Size

sys/

dev/

random/

48 lines

Diff 57156

sys/dev/random/ivy.c

Loading...