WITH_OFED build option: fix
ClosedPublic
Actions

Authored by kib on Feb 2 2021, 1:36 PM.

Details

Reviewers

kevans
emaste
• hselasky

Commits

rGff975f15d860: WITH_OFED build option: fix

Summary

Userspace has OFED build enabled for quite some time, but kernel modules were not. This is useless config because any userspace IB code requires kernel support. So enable modules build by default.

Move WITH_OFED to WITHOUT_OFED since defaults are now enabled.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kib requested review of this revision.Feb 2 2021, 1:36 PM

kib created this revision.

• hselasky accepted this revision.Feb 2 2021, 2:37 PM

This revision is now accepted and ready to land.Feb 2 2021, 2:37 PM

I think we're slightly inclined towards leaving WITH_OFED in-tree even if it's the default, but not a solid objection.

LGTM, one little nit on the option description

tools/build/options/WITHOUT_OFED
3	In our conventional WITH/WITHOUT style this would be "Set to disable the build of the" IMO these descriptions are very redundant; the description of the option is clearly the action taken when the option is set, so I think just "Do not build the" is sufficient.