Page MenuHomeFreeBSD
Differential D21298

OFED: Unbreak SDP support in ibcore
ClosedPublic
Actions

Authored by cem on Aug 17 2019, 3:16 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 26, 10:00 AM
Unknown Object (File)
Sun, Apr 7, 11:28 PM
Unknown Object (File)
Sun, Apr 7, 9:56 PM
Unknown Object (File)
Sun, Apr 7, 1:09 PM
Unknown Object (File)
Dec 22 2023, 9:32 PM
Unknown Object (File)
Sep 11 2023, 4:56 AM
Unknown Object (File)
Aug 27 2023, 2:31 PM
Unknown Object (File)
Aug 12 2023, 11:58 PM
View All 23 Files
Subscribers
imp

Details

Reviewers
markj
• hselasky
rstone
Commits
rS351176: OFED: Unbreak SDP support in ibcore
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

cem created this revision.Aug 17 2019, 3:16 AM
Harbormaster completed remote builds in B25929: Diff 60926.Aug 17 2019, 3:16 AM
• hselasky accepted this revision.Aug 17 2019, 8:15 AM

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
cem added inline comments.Aug 17 2019, 9:31 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.

• hselasky added a comment.Aug 17 2019, 10:03 AM

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

cem added a comment.Aug 17 2019, 10:13 AM
In D21298#463284, @hselasky wrote:

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.)

Closed by commit rS351176: OFED: Unbreak SDP support in ibcore (authored by cem). · Explain WhyAug 17 2019, 6:54 PM
This revision was automatically updated to reflect the committed changes.
cem marked an inline comment as done.
cem added a commit: rS351176: OFED: Unbreak SDP support in ibcore.
Herald added a subscriber: imp. · View Herald TranscriptAug 17 2019, 6:54 PM

Revision Contents
Changeset List

PathSize
head/
sys/
ofed/
drivers/
infiniband/
core/
233 lines
ulp/
sdp/
44 lines
include/
rdma/
158 lines

Diff 60952

View Options

head/sys/ofed/drivers/infiniband/core/ib_cma.c

Loading...
View Options

head/sys/ofed/drivers/infiniband/ulp/sdp/sdp.h

Loading...
View Options

head/sys/ofed/include/rdma/rdma_sdp.h

Loading...