Page MenuHomeFreeBSD

OFED: Unbreak SDP support in ibcore
ClosedPublic

Authored by cem on Aug 17 2019, 3:16 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 19, 12:12 AM
Unknown Object (File)
Tue, Sep 30, 1:35 PM
Unknown Object (File)
Jul 8 2025, 10:58 AM
Unknown Object (File)
Jul 7 2025, 2:33 PM
Unknown Object (File)
Jul 5 2025, 11:05 PM
Unknown Object (File)
Jul 5 2025, 10:54 AM
Unknown Object (File)
Jul 3 2025, 6:48 AM
Unknown Object (File)
Jun 22 2025, 2:37 AM
Subscribers

Details

Summary

This regression was introduced in the r326169 Linux v4.9 Infiniband upgrade.
Restore the functionality.

Test Plan

Basic functionality established with mthca(4) devices.

Diff Detail

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

Event Timeline

Looks good. See my one comment.

sys/ofed/include/rdma/rdma_sdp.h
28 ↗(On Diff #60926)

Can you change this into a normal include guard?

This revision is now accepted and ready to land.Aug 17 2019, 8:15 AM
sys/ofed/include/rdma/rdma_sdp.h
28 ↗(On Diff #60926)

I prefer once, but if you have a strong preference for the boilerplate, I'm happy to change it.

If all compilers support \#pragma once I'm fine with that, but else we should follow the style in FreeBSD.

If all compilers support \#pragma once I'm fine with that, but else we should follow the style in FreeBSD.

All compilers do support #pragma once: https://en.wikipedia.org/wiki/Pragma_once#Portability

(The list we care about is much shorter: Clang and GCC.)

This revision was automatically updated to reflect the committed changes.
cem marked an inline comment as done.