[LU-8769] Remove obsolete LND types

• Type: Bug
• Resolution: Fixed
• Priority: Trivial
• Fix Version/s: Lustre 2.10.0
• Labels: easy
• Severity: 3
• Assignee: Sonia Sharma (Inactive)
• Reporter: Steve Guminski (Inactive)

Description

There are several LND types that are no longer in use, and should be removed. The LND types are initialized in this array in libcfs/libcfs/util/nidstrings.c:

static struct netstrfns libcfs_netstrfns[]

The following LNDs should be removed:

• CIBLND
• OPENIBLND
• IIBLND
• VIBLND
• RALND
• QSWLND
• GMLND
• MXLND
• PTLLND

Comments

[Gerrit Updater]

Sonia Sharma ([e-mail]) uploaded a new patch: http://review.whamcloud.com/23628
Subject: LU-8769 lnds: <remove obsolete LNDs>
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 06c83c6a51c5b7ee2fb68358b22b534fe5934cf8

[Gerrit Updater]

Sonia Sharma ([e-mail]) uploaded a new patch: http://review.whamcloud.com/23627
Subject: LU-8769 lnds: <Remove Obsolete LND types>
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: bfbac77aa4dab97b4ffe203fb4e8036a14b01375

[Gerrit Updater]

Sonia Sharma ([e-mail]) uploaded a new patch: http://review.whamcloud.com/23623
Subject: LU-8769 lnds: <removal of obsolete LNDs>
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4c678e349a2dd5c1d8835c6dd8efc4ff87284fb9

[Gerrit Updater]

Sonia Sharma ([e-mail]) uploaded a new patch: http://review.whamcloud.com/23622
Subject: LU-8769 lnds: <removal of obsolete LNDs>
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: f46c57b6ef24a7809bf7f52bf022ef9061812607

[Gerrit Updater]

Sonia Sharma ([e-mail]) uploaded a new patch: http://review.whamcloud.com/23621
Subject: LU-8769 lnds: <removal of obsolete LNDs>
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 30f89305b86bf7bb1ebee80cb0cb45e560797b9b

[Andreas Dilger]

Just to clarify, the LNDs were removed long ago. All that this ticket is about is removing the name<->network number mapping for these obsolete LNDs, and (preferably, as stated above) adding clear comments for the removed LND network numbers.

[Joseph Gmitter (Inactive)]

Hi Sonia,

Would you be able to assess these LND's for possible removal?

Thanks.
Joe

[Patrick Farrell (Inactive)]

FWIW, speaking for Cray, the PTLLND line can be completely commented out. We are confident about never running 2.9 or 2.10 on anything that has a connection with a PTLLND system.

[Andreas Dilger]

Note that some of the LNDs were only recently removed (after 2.5) and at least the name->network mapping should stick around in order for a client/server to route to that network type (e.g. on a Cray cluster running an older version of Lustre that connects to newer servers), even if it doesn't have support for the network locally. The other IBLND and GMLND types are have not been used for ages so it isn't a concern and they can be removed completely. Please keep commented-out definitions in lnet/include/lnet/nidstr.h (the unnamed enum, that should really get a name like enum lnet_types so that this can properly be used elsewhere in the code) to avoid re-using these values:

enum lnet_types {    
        /* Only add to these values (i.e. don't ever change or redefine them):
         * network addresses depend on them... */ 
        QSWLND          = 1, /* removed v2_7_50_0-34-g8be9e41 */
        SOCKLND         = 2,    
        /* GMLND        = 3,    removed v2_0_0-rc1a-16-gc660aac */
        PTLLND          = 4, /* removed v2_5_53_0-21-g61bcfb2 */
        O2IBLND         = 5,    
        /* CIBLND       = 6,    removed v2_0_0-rc1a-175-gd2b8a0e */
        /* OPENIBLND    = 7,    removed v2_0_0-rc1a-175-gd2b8a0e */
        /* IIBLND       = 8,    removed v2_0_0-rc1a-175-gd2b8a0e */
        LOLND           = 9,    
        RALND           = 10, /* removed v2_7_50_0-34-g8be9e41 */
        /* VIBLND       = 11,   removed v2_0_0-rc1a-175-gd2b8a0e */
        MXLND           = 12, /* removed v2_7_50_0-34-g8be9e41 */
        GNILND          = 13,
        GNIIPLND        = 14,
};

If you are removing the remaining LNDs, please also look at removing the rest of the places where these constants are used (headers, Makefile.*, autoconf, etc). Run e.g. git grep VIBLND to see all of the places this is used.