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)
	Sun, Apr 27, 11:58 AM

	Unknown Object (File)
	Mon, Apr 14, 3:24 AM

	Unknown Object (File)
	Mar 5 2025, 12:21 AM

	Unknown Object (File)
	Mar 3 2025, 7:54 AM

	Unknown Object (File)
	Feb 28 2025, 7:33 PM

	Unknown Object (File)
	Feb 12 2025, 7:01 PM

	Unknown Object (File)
	Feb 5 2025, 6:13 PM

	Unknown Object (File)
	Jan 28 2025, 8:56 PM

View All 27 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