Paths

Table of Contentst

LinuxKPI: implement get_random_u32_inclusive()
ClosedPublic
Actions

Authored by bz on May 16 2023, 9:08 PM.

Details

Reviewers

cperciva
dwmalone

Group Reviewers

linuxkpi

Commits

rG1dcd1a539275: LinuxKPI: implement get_random_u32_inclusive()

Summary

Attempt of an implementation of get_random_u32_inclusive().
This is needed by an update for wireless drivers.

Sponsored by: The FreeBSD Foundation
MFC after: 10 days

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bz created this revision.May 16 2023, 9:08 PM

Herald added subscribers: emaste, imp. · View Herald TranscriptMay 16 2023, 9:08 PM

bz requested review of this revision.May 16 2023, 9:08 PM

Harbormaster completed remote builds in B51513: Diff 122013.May 16 2023, 9:09 PM

emaste added inline comments.May 16 2023, 11:09 PM

sys/compat/linuxkpi/common/include/linux/random.h
62	`low` and `top` seem like an odd pair - I'd think `low` and `high`, `bottom` and `top`, or `floor` and `ceil`.

Change function argument names to floor/ceil as suggested by @emaste.

Harbormaster completed remote builds in B51517: Diff 122018.May 16 2023, 11:49 PM

bz marked an inline comment as done.Jun 8 2023, 6:09 PM

Anyone?

• hselasky added a subscriber: • hselasky.Jun 9 2023, 12:31 PM

• hselasky added inline comments.

sys/compat/linuxkpi/common/include/linux/random.h
70	Is there a reason for the while loop? Is zero not valid for "x >> 32" ?

Maybe add:
MPASS(ceil >= floor);

Maybe add a comment showing the reference to the .PDF document explaining the formula ...

What do you think?

sys/compat/linuxkpi/common/include/linux/random.h
70	@emaste : Thanks for the link!

Address review comments.

@emaste, do you think this is okay now?

Harbormaster completed remote builds in B52368: Diff 124042.Jun 30 2023, 8:18 PM

This revision was not accepted when it landed; it landed in state Needs Review.Aug 7 2023, 1:00 AM

Closed by commit rG1dcd1a539275: LinuxKPI: implement get_random_u32_inclusive() (authored by bz). · Explain Why

This revision was automatically updated to reflect the committed changes.

bz added a commit: rG1dcd1a539275: LinuxKPI: implement get_random_u32_inclusive().

Revision Contents
Changeset List

Path

Size

sys/

compat/

linuxkpi/

common/

include/

linux/

random.h

30 lines

Diff 125641

View Options

sys/compat/linuxkpi/common/include/linux/random.h

LinuxKPI: implement get_random_u32_inclusive()ClosedPublicActions