Page MenuHomeFreeBSD

ofed/infiniband: fix ifdefs for new INET changes, fixing LINT-NOIP

Authored by karels on Jul 16 2022, 9:14 PM.
Referenced Files
Unknown Object (File)
Mon, Jun 3, 5:03 PM
Unknown Object (File)
Mon, Jun 3, 3:59 AM
Unknown Object (File)
Sun, May 26, 11:01 PM
Unknown Object (File)
May 9 2024, 7:27 PM
Unknown Object (File)
Apr 26 2024, 11:01 PM
Unknown Object (File)
Apr 22 2024, 8:04 AM
Unknown Object (File)
Apr 19 2024, 4:33 PM
Unknown Object (File)
Apr 10 2024, 10:58 PM



Some of the ofed/infiniband code has INET and INET6 address handling
code without using ifdefs. This failed with a recent change to INET,
in which IN_LOOPBACK() started using a VNET variable, and which is not
present if INET is not configured. Add #ifdef INET, and INET6 for good
measure, in cma_loopback_addr(), along with inclusion of the options
headers in ib_cma.c.

Test Plan

verified that correct symbols are referenced with and without

Diff Detail

rG FreeBSD src repository
Lint Not Applicable
Tests Not Applicable

Event Timeline

rgrimes added a subscriber: rgrimes.

Adding some more eyes to this, IIRC this is ofed code and actually maintained and imported from some place else. Though I agree with Glebius that this is the right way to fix it, it may need or be desired to do this upstream. Hselesky is the infiniband expert.

This revision is now accepted and ready to land.Jul 17 2022, 5:15 PM
bz added a subscriber: bz.
bz added inline comments.

Do we need this comment?


I thought it was worth an explanation of why this particular code has ifdefs.