Fix make universe with WITH_OFED=yes.

[kib]

Summary

This patch does the following:

• Corrects include paths.
• Remove duplicated symbols from the linker version script. Bfd linker is permissive but lld issues an error.
• Correctly selects the unordered_map definition namespace.
• Defines barriers for mips/arm using strongest CPU barriers.

RISC-V was not handled but the logistics required to make it included into make universe on the machine is not worth the efforts.

Test Plan

The

make -s -j 32 SRCCONF=src.conf MAKE_JUST_WORLDS=yes tinderbox

command returns success with src.conf containing the line

WITH_OFED=yes

[• hselasky]

contrib/ofed/infiniband-diags/build/Makefile.inc

11

Style: Space before "-I".

Can you explain why you need this CFLAGS? All the relevant includes should be installed in /usr/obj/xxx in the correct location and that should be used instead of the source tree!

contrib/ofed/libibnetdisc/g_hash_table.cpp

34

Please test this change with both clang and gcc. Last time I tried this on my box, clang didn't define __cplusplus ...

[kib]

contrib/ofed/infiniband-diags/build/Makefile.inc

11

I do not know for sure why exactly I needed the change, from the PoV of internals of the build system, but without it compiler cannot find the includes.

But what I suspect happens in your case, where you did not see the build errors, is that the headers were installed into the host and your build was polluted.

Anyway, I do not know about build enough to answer this and I added Brian to might be provide some feedback.

contrib/ofed/libibnetdisc/g_hash_table.cpp

34

If you read the revision annotation, you would note that I build whole tinderbox with the change. Which includes the architectures which are (cross-)build with gcc.

Clang does not define __cplusplus IFF it builds in non-C++ mode.

[• hselasky]

contrib/ofed/libibnetdisc/g_hash_table.cpp

34

I see, so adding the -std=c++11 for clang makes __cplusplus defined.

[• hselasky]

contrib/ofed/infiniband-diags/build/Makefile.inc

11

I'll double check this.

[• hselasky]

contrib/ofed/infiniband-diags/build/Makefile.inc

11

Did you clean /usr/obj and /usr/include/infiniband before doing this build?

[kib]

contrib/ofed/infiniband-diags/build/Makefile.inc

11

/usr/include/infiniband is empty on my machine, and /usr/obj was not used due to custom MAKEOBJDIRPEFIX.

contrib/ofed/libibnetdisc/g_hash_table.cpp

34

No, absolutely NOT. __cplusplus must be defined for any compilation of c++ sources. -std=c++XX changes its value, but not presence.

[• hselasky]

Can you check inside your MAKEOBJDIRPEFIX if there is "include/infiniband" and what its contents might be?

[kib]

In D13329#278886, @hselasky wrote:

Can you check inside your MAKEOBJDIRPEFIX if there is "include/infiniband" and what its contents might be?

[root@r-freeb43 ~]# find !$ -type d -name infiniband
find obj/root/deviant2/sparc64.sparc64/ -type d -name infiniband
obj/root/deviant2/sparc64.sparc64/tmp/usr/include/infiniband
obj/root/deviant2/sparc64.sparc64/tmp/legacy/usr/include/infiniband
[root@r-freeb43 ~]# ls obj/root/deviant2/sparc64.sparc64/tmp/usr/include/infiniband
arch.h          driver.h        kern-abi.h      sa-kern-abi.h   umad_sa.h
byteorder.h     endian.h        mad.h           sa.h            umad_sm.h
byteswap.h      ib.h            mad_osd.h       types.h         umad_str.h
cm.h            iba             marshall.h      udma_barrier.h  umad_types.h
cm_abi.h        ibnetdisc.h     opcode.h        umad.h          vendor
complib         ibnetdisc_osd.h opensm          umad_cm.h       verbs.h
[root@r-freeb43 ~]# ls obj/root/deviant2/sparc64.sparc64/tmp/legacy/usr/include/infiniband
complib iba     opensm  vendor

[• hselasky]

Hi,

Yes, it looks like you're right. My test-builds picked up /usr/include/xxx instead of /usr/obj/xxx . In the Makefiles for OFED I've added these CFLAGS, but INCLUDEDIR is still pointing to /usr/include instead of /usr/obj/xxx :-(

contrib/ofed/infiniband-diags/build/Makefile.inc:CFLAGS+= -I${INCLUDEDIR}/infiniband
contrib/ofed/libibnetdisc/Makefile:CFLAGS+= -I${INCLUDEDIR}/infiniband
contrib/ofed/opensm/complib/Makefile:CFLAGS+= -I${INCLUDEDIR}/infiniband
contrib/ofed/opensm/libopensm/Makefile:CFLAGS+= -I${INCLUDEDIR}/infiniband
contrib/ofed/opensm/libvendor/Makefile:CFLAGS+= -I${INCLUDEDIR}/infiniband
contrib/ofed/opensm/opensm/Makefile:CFLAGS+= -I${INCLUDEDIR}/infiniband

If you see how to solve this let me know. I'm looking into it.

[• hselasky]

I found it. This change should be the correct one:

-CFLAGS+= -I${INCLUDEDIR}/infiniband
+CFLAGS+= -I${SYSROOT:U${DESTDIR}}/${INCLUDEDIR}/infiniband