roundup_pow_of_two: don't take the log of it
ClosedPublic
Actions

Authored by dougm on Sep 28 2024, 12:02 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Oct 24, 12:40 AM

	Unknown Object (File)
	Mon, Oct 21, 7:29 AM

	Unknown Object (File)
	Mon, Oct 21, 6:29 AM

	Unknown Object (File)
	Sun, Oct 20, 8:28 AM

	Unknown Object (File)
	Sat, Oct 19, 5:52 PM

	Unknown Object (File)
	Oct 3 2024, 9:39 PM

	Unknown Object (File)
	Oct 2 2024, 12:06 PM

	Unknown Object (File)
	Oct 1 2024, 6:22 PM

View All 11 Files

Subscribers

imp

kib

Details

Reviewers

bz
kib

Commits

rGc44fbfdb5686: roundup_pow_of_two: don't take the log of it

Summary

Based on the definitions, ilog2(roundup_pow_of_two(x)) == order_base_2(x). Replace the former with the latter in a few places to save a few calculations.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dougm requested review of this revision.Sep 28 2024, 12:02 AM

dougm created this revision.

Add @kib in the hope that he's maintaining the mlx* bits these days.

To me looks ok

kib accepted this revision.Sep 28 2024, 11:40 AM

This revision is now accepted and ready to land.Sep 28 2024, 11:40 AM

Closed by commit rGc44fbfdb5686: roundup_pow_of_two: don't take the log of it (authored by dougm). · Explain WhySep 28 2024, 5:01 PM

This revision was automatically updated to reflect the committed changes.

dougm added a commit: rGc44fbfdb5686: roundup_pow_of_two: don't take the log of it.

Herald added a subscriber: imp. · View Herald TranscriptSep 28 2024, 5:01 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

mlx4/

mlx4_en/

en.h

2 lines

mlx4_ib/

mlx4_ib_qp.c

4 lines

Diff 143871

View Options

sys/dev/mlx4/mlx4_en/en.h