Page MenuHomeFreeBSD

Enable OFED build by default.
ClosedPublic

Authored by kib on Jul 11 2018, 9:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, May 13, 2:49 AM
Unknown Object (File)
Thu, May 2, 7:18 AM
Unknown Object (File)
Thu, May 2, 7:17 AM
Unknown Object (File)
Thu, May 2, 7:14 AM
Unknown Object (File)
Thu, May 2, 5:07 AM
Unknown Object (File)
Thu, May 2, 1:08 AM
Unknown Object (File)
Thu, May 2, 12:02 AM
Unknown Object (File)
Sun, Apr 21, 4:18 AM
Subscribers

Details

Summary

This change enables both userspace and modules build for OFED by default. Also, it fixes some issues, namely:

  • correct location for libmlx5 and libibverbs, required by libpcap when OFED is used;
  • move mostly useless examples, as well as subnet manager, builds under the new option WITH_OFED_EXTRA, disabled by default.
Test Plan

Tested by tinderbox with default options and with WITH_OFED_EXTRA enabled.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Remove libmlx5.so.1 and libibverbs.so.1 from /usr/lib.

You also need to create a WITHOUT_OFED_EXTRA file.

You also need to create a WITHOUT_OFED_EXTRA file.

Why ? Each option has only one WITH/WITHOUT description file.

It is common to define both?? See for example:

./tools/build/options/WITHOUT_MLX5TOOL
./tools/build/options/WITH_MLX5TOOL

Among other.

It is common to define both?? See for example:

./tools/build/options/WITHOUT_MLX5TOOL
./tools/build/options/WITH_MLX5TOOL

Among other.

I think it is excessive for MLX5TOOL, was it added by you ?

I see that both WITH_XXX and WITHOUT_XXX files are present when operation modes descriptions are different enough to warrant different text. When an option turns some feature on and off, without side effects worth describing, the text for one mode is enough.

Yes, I added those files and I see there are a couple of more pairs.

This revision is now accepted and ready to land.Jul 12 2018, 8:07 PM
bdrewery added inline comments.
usr.bin/ofed/Makefile
11 ↗(On Diff #45187)

SIBDIR should be spelled SUBDIR.
But you can also just make these 3 lines: SUBDIR.${MK_OFED_EXTRA}+= opensm

This revision now requires changes to proceed.Jul 19 2018, 6:40 PM
kib marked an inline comment as done.

Fix typo.

This revision is now accepted and ready to land.Jul 19 2018, 7:21 PM
This revision was automatically updated to reflect the committed changes.
This revision was not accepted when it landed; it landed in state Needs Review.Jul 20 2018, 11:52 PM
This revision was automatically updated to reflect the committed changes.