[LU-16659] mofed detection would detect multiple OFED installed by mistake for multiple kernels installed

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 2.16.0
• Fix Version/s: Lustre 2.16.0, Lustre 2.15.3
• Type: Bug
• Priority: Minor
• Reporter: Gaurang Tapase
• Assignee: Gaurang Tapase
• Resolution: Fixed
• Severity: 3

Description

If there are multiple kernels installed, specifying --with-o2ib=yes during configure fails saying that there are multiple OFED installed which is not the case as the mofed detection logic finds multiple directories with different kernels.

Need to just use the mofed path of the current running kernel while detection.

Comments

[Gerrit Updater]

"Gaurang Tapase <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50517
Subject: LU-16659 build: Detect the mofed path based on running kernel
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 56b49a8aa44ef2c07e821326341dfb9f8db75bbc

[Gerrit Updater]

"Gaurang Tapase <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50518
Subject: LU-16659 build: Detect the mofed path based on running kernel
Project: fs/lustre-release
Branch: b2_15
Current Patch Set: 1
Commit: 39423d030f2dce6e623a0f23d80e77461950dd61

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50517/
Subject: LU-16659 build: Detect the mofed path based on running kernel
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 99172e355fbb37d2ba671d08ce0370ac0e8ae971

[Peter Jones]

Landed for 2.16

[Åke Sandgren]

The change in 50517 is wrong in my opinion.

The correct way would be to do

 

— a/lnet/autoconf/lustre-lnet.m4
+++ b/lnet/autoconf/lustre-lnet.m4
@@ -147,6 +147,9 @@
                                       egrep "${O2IBDIR}$" | head -n1)
 
                        if test -n "$O2IBDIR_PATH"; then
+                                if test -d $O2IBDIR_PATH/${LINUXRELEASE}; then
+                                    O2IBDIR_PATH=$O2IBDIR_PATH/${LINUXRELEASE}
+                                fi
                                O2IBPATHS=$(find $O2IBDIR_PATH -name rdma_cm.h |
                                        sed -e 's/\/include\/rdma\/rdma_cm.h//')
                        fi

 

Using uname -r makes it impossible to build for anything else than the currently running kernel

[Minh Diep]

ake_s, could you please open an LU ticket and push the fix?

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50518/
Subject: LU-16659 build: Detect the mofed path based on running kernel
Project: fs/lustre-release
Branch: b2_15
Current Patch Set:
Commit: 42ff616c73ad759e41fed8d99759c0c42a4d907d