[LU-10391] LNET: Support IPv6 Created: 14/Feb/12  Updated: 06/Feb/24

Status: Reopened
Project: Lustre
Component/s: None
Affects Version/s: None
Fix Version/s: Upstream, Lustre 2.16.0

Type: New Feature Priority: Major
Reporter: Doug Oucharek (Inactive) Assignee: Neil Brown
Resolution: Unresolved Votes: 3
Labels: IPv6, lnet, upstream

Attachments: File NetlinkDiscussion.m4a     Microsoft PowerPoint NetlinkYAML.pptx    
Issue Links:
Blocker
Related
is related to LU-8915 Don't use linux list structure as use... Reopened
is related to LU-12511 Prepare lustre for adoption into the ... Open
is related to LU-5881 Allow hostnames in NID Resolved
is related to LU-17294 sanity-lnet test_219: timeout on Ubun... Open
is related to LU-13642 lnetctl: Allow IP specification Open
is related to LU-10003 lnetctl error "cannot add network: in... Reopened
is related to LU-15478 Regression in 005bd7075c LU-10391 lne... Resolved
is related to LU-9823 LNet fails to come up when using lctl... Resolved
is related to LU-14477 lnet doesn't work with no IPV6 suppor... Resolved
is related to LU-15508 Lnet IPv6 work exposed memory leak in... Resolved
is related to LU-10360 use Imperative Recovery logs for clie... Open
is related to LU-13625 socklnd can select the wrong IP addre... Open
is related to LU-14288 Enhance nodemap ranges to work better... Open
is related to LU-17479 Update lnetctl to allow cYAML to hand... Open
is related to LU-10359 remove NIDs from config llogs Open
is related to LU-13308 update changelog_ext_nid to handle IP... Open
is related to LU-16738 Improve mount.lustre with many MGS NIDs Open
is related to LU-16822 test IPv6 NIDs in sanity-lnet.sh Open
is related to LU-17273 Update GSS context init to support IPv6 Open
is related to LU-13306 allow clients to accept mgs_nidtbl_en... Resolved
is related to LU-13307 add LCFG_NODEMAP_ADD_RANGEv6 records ... Resolved
is related to LU-13340 add LCFG_ADD_UUIDv6 and related commands Resolved
is related to LU-16823 add LNet and OBD connect flags for IP... Resolved
Rank (Obsolete): 13781

 Description   

This improvement calls for fully supporting IPv6 throughout LNet and the LND's. This will allow Lustre to run in IPV6 networks, in conjunction with related changes to any configuration and PtlRPC changes where NIDs are processed (e.g. config llog records, Imperative Recovery, etc.).



 Comments   
Comment by Andreas Dilger [ 16/Dec/15 ]

Comments from Multi-Rail HLD https://wiki.whamcloud.com/pages/viewpage.action?pageId=31073572 :

One proposal was to create an incompatible IPv6-capable network protocol, simply by increasing the field size at compile time and have separate LND types for the new protocol. Those sites that can't just switch entirely over to the IPv6 protocol could use LNet routers to translate messages between the protocols as needed. Of course, the longer this is delayed, the more sites will be affected.

In any case, increasing the field size here is trivial before implementation and the fewer places that need to be fixed in the future, the better, especially when dealing with userspace interfaces.

The main issue is NIDs, which are 64 bits, with only 32 bits reserved for the node interface id.

The MGS could just assign a 32 bit ID, if there were something like the MGS. But there isn't, so it can't. (In contrast, CXFS has a cellid concept separate from any IP address or other identifier, so when we did IPv6 support the work really was limited to the network layers and configurationhandling.)

For this work I'd really like to have some node ID separate from IP address (or whatever), as it would help resolve some edge cases. I'm just not seeing an easy way to add it and retain backward compatibility. And anything complex in that area becomes a beyond-scope issue...

For lnet_ping_info_t it seems fairly straight forward to set a new LNET_PING_FEAT_IPV6 flag in pi_features and then have a new lnet_ni_status32_t with a new lnet_nid32_t large enough to hold a 128-bit IPv6 NID.

There is a separate problem of the other uses of lnet_nid_t in the protocol, but if we can get the initial handshake to exchange support for larger NIDs then the rest of the messages can follow suit.

That said, IPv6 support is way beyond the scope of the multi-rail implementation. My main goal is to ensure that we don't put any more roadblocks in the way of implementing that, especially in initial connection establishment messages like PING and PUSH. If anything else can be done to make the LNet protocol "more accepting" of IPv6 NIDs in the future (e.g. better protocol feature exchange at connection time), that would be welcome.

For the ping_info, a feature flag could indicate a separate header following the current pi_nid[] array, specifying the number of the new-format NIDs. A downrev node would ignore that part of the data.

It would also be possible to use a different match bit for a new-style ping request. That way a downrev node would not be seeing the new form at all.

I like the idea of adding the IPv6 (or other larger-format addresses) after the regular lnet_nid_t addresses, since that avoids sending larger packets in the case where it is not needed.

For socklnd, I see it might be possible to change the ksock_hello_msg_t.kshm_version and lnet_magicversion_tactually be a bitfield of features rather than a numeric version number. In particular, version_major and version_minor are described as compatible and incompatible "versions" but they could be bitfields as well (equivalent to incrementing the version in power-of-two values for old clients that don't understand the feature flags).

My goal for projects like this is to try and lay the groundwork early on, if possible, and then when it comes time to implement the feature (maybe a few years/releases later) it can be done in a way that doesn't cause so many problems for older clients.

At socknal level you'd likely use a new ksock_hello_ipv6_msg_t. At the point where you're sending it, you know for certain whether you're talking over IPv4 or IPv6.

And (hypothetically speaking) the multi-rail work should render obsolete the reason for ksocknal to send IP addresses in its hello message.

Comment by Andreas Dilger [ 14/Dec/17 ]

This ticket isn't due to any specific requirement for this feature, but just to preserve discussion/brainstorming on this topic in case it is needed in the future. IPv6 support is definitely something to keep in mind as other changes are being made in the LNet layer.

Comment by Andreas Dilger [ 16/Jan/20 ]

Some discussion ongoing at https://wiki.whamcloud.com/display/LNet/Upstream+IPv6+Proposal

Comment by Gerrit Updater [ 22/Jan/20 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37302
Subject: LU-10391 lnet: use data_ready callback to trigger accept()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 20aad747f31c00cc72dee23e0cb15ac5bd2891c4

Comment by Gerrit Updater [ 22/Jan/20 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37303
Subject: LU-10391 lnet: discard lnet_sock_accept()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7a201c3f7f83c36ea9312b413b88a3b3714b9af4

Comment by Gerrit Updater [ 20/Feb/20 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37302/
Subject: LU-10391 lnet: use data_ready callback to trigger accept()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 40e4d957e33268354a3fd938545bf717ed415372

Comment by Gerrit Updater [ 20/Feb/20 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37303/
Subject: LU-10391 lnet: discard lnet_sock_accept()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: a1b8fd633d2eb3f92151eafcb22e7413ec33bf03

Comment by Gerrit Updater [ 24/Feb/20 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37694
Subject: LU-10391 lnet: remove 'accept' variable.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: e66e91a3caeeb9627425a13dcaffbc1243f32925

Comment by Gerrit Updater [ 24/Feb/20 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37695
Subject: LU-10391 lnet: simplify use of lnet_count_acceptor_nets()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 055fab3e3f8d6f1a0779c145151d097a87d6d671

Comment by Gerrit Updater [ 25/Feb/20 ]

Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/37699
Subject: LU-10391 lnet: document the accept_type parameter
Project: doc/manual
Branch: master
Current Patch Set: 1
Commit: 32dd22eb253f15be043b50fcd86afa060d000db0

Comment by Gerrit Updater [ 25/Feb/20 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37700
Subject: LU-10391 lnet: remove 'fatal' arg from lnet_

{sock_}

connect
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: ab0472426a7f21abe94eb8b8a454d630bc9a584b

Comment by Gerrit Updater [ 25/Feb/20 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37701
Subject: LU-10391 lnet: change various function to return pointer
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 5504dc5b5a3b84d93f5a654c8a9c4b204b7333bf

Comment by Gerrit Updater [ 25/Feb/20 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37702
Subject: LU-10391 socklnd: use interface index to track local addr
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 40c72ad9edc77cac157fcf9d5291d78754a58c82

Comment by Gerrit Updater [ 25/Feb/20 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37703
Subject: LU-10391 lnet: change lnet_sock_connect to take sockaddr
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 431032f585ccaf6325a9d52381f37710256b54ea

Comment by Gerrit Updater [ 25/Feb/20 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37704
Subject: LU-10391 socklnd: use sockaddr instead of __u32 addresses.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0195d02f9033e07c170ba5b1562d1b910f2ee8ef

Comment by Gerrit Updater [ 25/Feb/20 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37705
Subject: LU-10391 lnet: allow creation of IPv6 socket.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: ba2baedcbde4347f3417637f810c57c61683157d

Comment by Gerrit Updater [ 25/Feb/20 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37706
Subject: LU-10391 lnet: allow lnet_connect() to use IPv6 addresses.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: b18033798fe7db05f2ae3a8e73599cfc78d12ec8

Comment by Gerrit Updater [ 26/Feb/20 ]

Joseph Gmitter (jgmitter@whamcloud.com) merged in patch https://review.whamcloud.com/37699/
Subject: LU-10391 lnet: document the accept= parameter properly
Project: doc/manual
Branch: master
Current Patch Set:
Commit: eb19b25530064e7622ef51083306a669f9a99419

Comment by Gerrit Updater [ 11/Mar/20 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37694/
Subject: LU-10391 lnet: remove 'accept' variable.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 24ab342d6e7b22628b096d89cdfe1d0d54d5551a

Comment by Gerrit Updater [ 24/Mar/20 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37700/
Subject: LU-10391 lnet: remove 'fatal' arg from lnet_

{sock_}

connect
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 97084f70a6c2956d49d8dc289aeb8ede84e5f7b4

Comment by Gerrit Updater [ 24/Mar/20 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37701/
Subject: LU-10391 lnet: change various function to return pointer
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 3e42620b5b79170ec219e52d2a7e915559addba4

Comment by Gerrit Updater [ 14/Apr/20 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37695/
Subject: LU-10391 lnet: simplify use of lnet_count_acceptor_nets()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 1477027d073be1efb6ffbb368ed68b97a65c72f1

Comment by Gerrit Updater [ 14/Apr/20 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37702/
Subject: LU-10391 socklnd: use interface index to track local addr
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: d720fbaadad9343a481863ae3b04ba580541328d

Comment by Gerrit Updater [ 20/Apr/20 ]

James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/38287
Subject: LU-10391 socklnd: properly order headers in socklnd.h
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 53e10cce44376f36d0a4816cf8df814e6a5d28d2

Comment by Gerrit Updater [ 23/Apr/20 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37703/
Subject: LU-10391 lnet: change lnet_sock_connect to take sockaddr
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: a486ca181d44682187016d41d6922c68474b226d

Comment by Gerrit Updater [ 29/Apr/20 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/38408
Subject: LU-10391 socklnd: convert ksocknal_add_peer to take sockaddr
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: a368e5203ef95f23c2b133893494777088ab15f1

Comment by Gerrit Updater [ 01/May/20 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38287/
Subject: LU-10391 socklnd: properly order headers in socklnd.h
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 5ba2cc615ba9e60075412cecbce9edaaf3eea2e3

Comment by Gerrit Updater [ 03/Jun/20 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/38823
Subject: LU-10391 lnet: fix uninitialize var in choose_ipv4_src()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 9e9080880b644eb41b6a1fa51ba89138d1620671

Comment by Gerrit Updater [ 10/Jun/20 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38823/
Subject: LU-10391 lnet: fix uninitialize var in choose_ipv4_src()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 8ea895389a92dbd08fb398dbda0a8f0f2dde063b

Comment by Gerrit Updater [ 15/Jul/20 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/39373
Subject: LU-10391 lnet: extended nids.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: d9d888e8f9d14682b85e5b4a86fc21a9b68e1e49

Comment by Gerrit Updater [ 26/Feb/21 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37704/
Subject: LU-10391 socklnd: use sockaddr instead of __u32 addresses.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: c35c1babc7466ed1f49a9efc879a4aaba8d000e6

Comment by Gerrit Updater [ 26/Feb/21 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37705/
Subject: LU-10391 lnet: allow creation of IPv6 socket.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: e4fa181abf1032198ad278cf9f0560e45158c9bc

Comment by Gerrit Updater [ 26/Feb/21 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37706/
Subject: LU-10391 lnet: allow lnet_connect() to use IPv6 addresses.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: fb40f0b62d77d699ad22b635cc8275fbd5d1347f

Comment by Gerrit Updater [ 16/Mar/21 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38408/
Subject: LU-10391 socklnd: convert ksocknal_add_peer to take sockaddr
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: fdce7be68eb7ab2fc7945e81d2dc49f3595ec790

Comment by Neil Brown [ 19/Mar/21 ]

My design for this change is to introduce a new "struct lnet_nid" which has room for 16 bytes of address.

This will then be gradually spread throughout the code replacing 'lnet_nid_t' which will eventually be deprecated.

Where lnet_nid_t is often passed to functions, and returned, by value, 'struct lnet_nid' is larger and will be passed or returned by reference.  This change in particular means that it is easiest to change the type gradually, rather than trying to prepare all the code to allow "lnet_nid_t" to be atomically changed to a larger size.

Where possible, we change the arguements to a function from old to new style.  Sometimes we need to keep both old-style and new-style functions available concurrently.

When this happens I have used two approaches to naming.  If there are relatively few uses of the function, the old one is renamed to have a "_4b" suffix (because it supports addresses with a 4-byte address), and the name is then used for new-style format.  When there are too many uses for this to be practical, a new name which is spelt slightly differently is used.  e.g. libcfs_nid2str becomes libcfs_nidstr.  This will probably cause confusion during the transition, but the compiler will easily warn about any such confusion.

I plan to submit patches (I currently have 35, but there is quite a lot more work to do) in smallish batches over coming weeks - depending on review feedback.

 

 

Comment by Gerrit Updater [ 19/Mar/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/42100
Subject: LU-10391 lnet: introduce struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 19db1615bbc436e4ed7818822a5b9fefecef9995

Comment by Gerrit Updater [ 19/Mar/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/42101
Subject: LU-10391 lnet: change lpni_nid in lnet_peer_ni to lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: eba3fab8f2ab3c04ef7a234731919e0bdc1868f9

Comment by Gerrit Updater [ 19/Mar/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/42102
Subject: LU-10391 lnet: change lp_primary_nid to struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 48157bf75bcf8d54f5d66b4f65ed83d7e9cca5ae

Comment by Gerrit Updater [ 19/Mar/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/42103
Subject: LU-10391 socklnd: factor out key calculation for ksnd_peers
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4487a93e2680edc303ceb8206bfbc1854ada65fe

Comment by Gerrit Updater [ 19/Mar/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/42104
Subject: LU-10391 lnet: introduce lnet_processid for ksock_peer_ni
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7313152304b39a87880034afed52d227e7238e06

Comment by Gerrit Updater [ 19/Mar/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/42105
Subject: LU-10391 lnet: enhance connect/accept to support large addr
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 866363f9e278aaa03b347f0065abf4c17f27a39c

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43602
Subject: LU-10391 socklnd: don't deref lnet_hdr in LNDs
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: c7394bd6c7ac0395be62383ffc3cc7ff3c678111

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43601
Subject: LU-10391 socklnd: prepare for new KSOCK_MSG type
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: ae5785665fa0efd253350ca17240d1f80977993d

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43604
Subject: LU-10391 lnet: change lnet_hdr to store large nids.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7fbfd9b0fea6011d755c679c4a921f52345e5c40

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43599
Subject: LU-10391 lnet: Change lnet_send() to take large-addr nids
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0dda32010d527a97c48281be28d2cbbd079834f0

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43596
Subject: LU-10391 lnet: extend prefered nids in struct lnet_peer_ni
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: f97d65932ddc9ed8f271f851cf1ecd6ffc94be2b

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43598
Subject: LU-10391 lnet: extend nids in struct lnet_msg
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: bac00deb99859703aadc28466b4a8bb5c387000d

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43603
Subject: LU-10391 lnet: separate lnet_hdr in msg from that in lnd.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 859daa629790266b7bf3c22a6d6f50f77ec3220d

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43600
Subject: LU-10391 lnet: use large nids in struct lnet_event
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: a7550b6e387a5203c10650d7c9ed7e96d0ef95a3

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43594
Subject: LU-10391 lnet: extend rspt_next_hop_nid in lnet_rsp_tracker
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: c6dde91fbec7aad0bbf9735d1b250a341c9286b6

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43595
Subject: LU-10391 lnet: change tp_nid to tp_nidx in lnet_test_peer.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4fd69f373f972bb6d2bf3da79272fbca46b39c36

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43607
Subject: LU-10391 lnet: convert LNetGetID to return an large-addr pid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: a447f978860b2c06e4b3681ec37089a3864db62b

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43593
Subject: LU-10391 lnet: change lr_nid to struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 778199af877bc8a70ef5be6e55545bf826c509c5

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43605
Subject: LU-10391 lnet: change lnet_prep_send to take net_processid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: cb7e8e39bc1b84c6b41482a692c6798f5abdabe4

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43606
Subject: LU-10391 lnet: convert to struct lnet_process_id in lib-move
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 6ce5940878addceda0fcbbb27dc42a9162d41dee

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43597
Subject: LU-10391 lnet: switch to large lnet_processid for matching
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: b5ac8e556f6adc72fa894d4cb17dfe02f5e9d57b

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43608
Subject: LU-10391 lnet: alter lnd_notify_peer_down() to take lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 289b1a0101103f0cbf1565cff407f3f6759b8fc9

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43609
Subject: LU-10391 socklnd: move lnet_hdr unpack into ->pro_unpack
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 90dc25b7e751e33a160ea2d0678b5c8be1ddef3d

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43616
Subject: LU-10391 lnet: change LNetPrimaryNID to use struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 9ac20dc8f426fc958bebb0b11975f6181745f537

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43612
Subject: LU-10391 lnet: Convert ping to support 16-bytes address
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 8dceda3d5e051a6716c1f0241b6d79346da847f2

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43610
Subject: LU-10391 socklnd: Change ksock_hello_msg to struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: dd9daf8c0b63f4feed05903204a91c0efc52c518

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43611
Subject: LU-10391 socklnd: add hello message version 4
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4b1b09a1b265b25df2442b5d2f25efeefc246735

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43613
Subject: LU-10391 lnet: convert nids in lnet_parse to lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 991a00743fad9f571bd536a2ffc54d51c8d38a1a

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43614
Subject: LU-10391 lnet: change src_nid arg to lnet_parse() to 16byte
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 39853c4a7a32729e1e227c18d5a4d66603c3e200

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43617
Subject: LU-10391 lnet: alter lnet_drop_rule_match() to take lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: dbea7682cf6c4ba3f8d2cb87c6eb8b421ecfd6a7

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43618
Subject: LU-10391 lnet: Change LNetDist to work with struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 10879bad04e4deb70f57cf6e7e82c777aa9b62c6

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43615
Subject: LU-10391 lnet: change lnet_del_route() to take lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 5fbe19203a56856b8e7035be6b7c4dd68ed28d67

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43621
Subject: LU-10391 lnet: switch LNetIsPeerLocal() to take 16-byte addr
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: ad5788250c2d66f0881772a5fad61cdb244ac77a

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43619
Subject: LU-10391 lnet: convert LNetPut to take 16byte nid and pid.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: ddd4fe7f8ada0c96bfad1bd81cf655eee1d57eec

Comment by Gerrit Updater [ 10/May/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43620
Subject: LU-10391 lnet: change LNetGet to take 16byte nid and pid.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 54ea0a4448f2fc8dcad62022f903f0b318ded46f

Comment by Gerrit Updater [ 08/Jun/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43942
Subject: LU-10391 lnet: add string formating/parsing for IPv6 nids
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 5258926381c4fb8a54d8ffe6a3d641445280ba5f

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44620
Subject: LU-10391 lnet: change lp_disc_*_nid to struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: c4952d2a83901ad1d7395531a9d89e6734a20de1

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44621
Subject: LU-10391 socklnd: pass large processid to ksocknal_add_peer
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 32041048a2317a81b3b14305f92d404166ebeb68

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44622
Subject: LU-10391 socklnd: large processid for ksocknal_get_peer_info
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 09e0aecf97e8d31036e99815c3655e1f944f7f1d

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44623
Subject: LU-10391 socklnd: switch ksocknal_del_peer to lnet_processid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 9e13ee1b530124c279b5c902fe050a379006675f

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44624
Subject: LU-10391 lnet: discard some peer_ni lookup functions
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4c07cecc9fd9fe473e5a32aae973d447fbfc11b5

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44625
Subject: LU-10391 lnet: change lnet_*_peer_ni to take struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 67fb98b61af337770bcbdc83a04863f0c3615c23

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44626
Subject: LU-10391 lnet: change ni_status in lnet_ni to __u32*
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 1f4f6b2f23f41d9eaf1bb48658ba7996bd116a58

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44627
Subject: LU-10391 lnet: track pinginfo size in bytes, not nis.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: abf0640a9e45c04fc04cbed942f0aeea48a58f5a

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44628
Subject: LU-10391 lnet: allow ping packet to contain large nids
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: c7ac157f919458313373d1a58c1ad695440c90b9

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44629
Subject: LU-10391 lnet: extend lnet_is_nid_in_ping_info()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: b8b175d8cb939e86f9157b35696d02bd126f09ba

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44630
Subject: LU-10391 lnet: lnet_peer_merge_data to understand large addr
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: d51a36f6cab7dd93a99f4ecfa488fd46b5ceee2e

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44631
Subject: LU-10391 lnet: router_discover - handle large addrs in ping
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 6bc2bceab7f0d24d5a3816d0579c054af97cb221

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44632
Subject: LU-10391 lnet: find correct primary for peer
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 1c4adbe11b16051c8767b3daf3c585032a3be128

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44633
Subject: LU-10391 lnet: change lnet_notify() to take struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 460b98629d91ea5405c318d420bdf2b773cd4bf6

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44634
Subject: LU-10391 lnet: discard lnet_nid2ni_*()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 267eb4571541b01aa163bab26dac138e521824ee

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44635
Subject: LU-10391 lnet: change lnet_debug_peer() to struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 583c900d253b9a99e3a44cec34edbe61389daad8

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44636
Subject: LU-10391 ptlrpc: change rq_self to struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 669f9afefa9b44120f02b60b0fd81dc2e4efc9b7

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44637
Subject: LU-10391 ptlrpc: pass net num to ptlrpc_uuid_to_connection
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: c435255f13b0e08538b45ec4e3e83e2dfbd7b258

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44638
Subject: LU-10391 ptlrpc: change rq_peer to struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: bec60629e3dfac689ffc1d51283575dce9a06f72

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44639
Subject: LU-10391 ptlrpc: change rq_source to struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 3490718fbb01b4a3124c13a34e7b63960722973a

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44640
Subject: LU-10391 ptlrpc: change bd_sender in ptlrpc_bulk_frag_ops
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7e86d6a36273dbb3b2f14a3a46c2fd6eb0bb6521

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44641
Subject: LU-10391 ptlrpc: pass lnet_nid for self to ptl_send_buf()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 5a6f85755294486200a6fe939a7be10f0c532fe2

Comment by Gerrit Updater [ 12/Aug/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44642
Subject: LU-10391 ptlrpc: lprocfs_exp_setup() to take struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0203e4cfbfc13ba10f6db201c3c418b6d21d84fc

Comment by Gerrit Updater [ 31/Aug/21 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/42100/
Subject: LU-10391 lnet: introduce struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 82a17076f880770a267746eafc9d956998189531

Comment by Gerrit Updater [ 31/Aug/21 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43942/
Subject: LU-10391 lnet: add string formating/parsing for IPv6 nids
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 7224b21156639a639d9755e1c2bf05affde50785

Comment by Gerrit Updater [ 31/Aug/21 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/42101/
Subject: LU-10391 lnet: change lpni_nid in lnet_peer_ni to lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 6c5561a1e1eeab18e8226d410f4aa9922fbab0d8

Comment by Gerrit Updater [ 31/Aug/21 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/42102/
Subject: LU-10391 lnet: change lp_primary_nid to struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 36dd83ee8e143a4726317565e1bdc8834edcbe70

Comment by Gerrit Updater [ 31/Aug/21 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44620/
Subject: LU-10391 lnet: change lp_disc_*_nid to struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: f38529cd3a1722119e48abe0931c3e232d3cc1f7

Comment by Gerrit Updater [ 31/Aug/21 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/42103/
Subject: LU-10391 socklnd: factor out key calculation for ksnd_peers
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 96a0c378c2e0a0c8f7e21404252e66944e163100

Comment by Gerrit Updater [ 31/Aug/21 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/42104/
Subject: LU-10391 lnet: introduce lnet_processid for ksock_peer_ni
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: e1dbfdd53e2ce95433f17ea610a5987c128e9c51

Comment by Gerrit Updater [ 31/Aug/21 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/42105/
Subject: LU-10391 lnet: enhance connect/accept to support large addr
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 9b0738c53c962f4269178e420d45565f9ec5bd09

Comment by Gerrit Updater [ 31/Aug/21 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43593/
Subject: LU-10391 lnet: change lr_nid to struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: d1e2f6fc688762222c63095173cfb5d19a93d377

Comment by Gerrit Updater [ 31/Aug/21 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43594/
Subject: LU-10391 lnet: extend rspt_next_hop_nid in lnet_rsp_tracker
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: a34afe7f20ec7d618d3c6a129f0eda3a18972928

Comment by Gerrit Updater [ 03/Sep/21 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44838
Subject: LU-10391 lnet: Fix NULL-deref in lnet_nidstr_r()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0a1ae426338e3e6866536fc57cb02ba3e8df45a3

Comment by Gerrit Updater [ 03/Sep/21 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/44844
Subject: LU-10391 uapi: move out kernel only code.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: a0bd88c47d458e3d20d3e0866fec5ef5cd33e688

Comment by Gerrit Updater [ 20/Nov/21 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43595/
Subject: LU-10391 lnet: change tp_nid to 16byte in lnet_test_peer.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 7e89b556ea7dc4b4cf5e6bce3b1da6bed0190e8b

Comment by Gerrit Updater [ 20/Nov/21 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43596/
Subject: LU-10391 lnet: extend prefered nids in struct lnet_peer_ni
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 47cc77462343533b4d706836e7e087f7a1844318

Comment by Gerrit Updater [ 20/Nov/21 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43597/
Subject: LU-10391 lnet: switch to large lnet_processid for matching
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: db49fbf00d24edc8347006241f314fb0f82f1b63

Comment by James A Simmons [ 21/Nov/21 ]

In the interop testing 2.12.7 is being used which always fails racer for RHEL7. A fix has landed to 2.12.8. Can someone bump the version the interop testing is being done against. Thanks.

Comment by Andreas Dilger [ 21/Nov/21 ]

That will happen when 2.12.8 is released.

Comment by Gerrit Updater [ 30/Nov/21 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44844/
Subject: LU-10391 uapi: move out kernel only code.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 0631caae7d804720de275ac557dda52d7f6e5084

Comment by Gerrit Updater [ 18/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43598/
Subject: LU-10391 lnet: extend nids in struct lnet_msg
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 57c03f3070753146a1d374939197e8c838a0bcc1

Comment by Gerrit Updater [ 18/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43599/
Subject: LU-10391 lnet: Change lnet_send() to take large-addr nids
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 005bd7075c8710fb8efaede1148870d2c0601aad

Comment by Gerrit Updater [ 18/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43600/
Subject: LU-10391 lnet: use large nids in struct lnet_event
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: e1d0224fb4045571a2d71e3a802095d82086e5e9

Comment by Gerrit Updater [ 18/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43601/
Subject: LU-10391 socklnd: prepare for new KSOCK_MSG type
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 6940303aad5375bb2c83160f87c0cacf9129b5aa

Comment by Gerrit Updater [ 18/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43602/
Subject: LU-10391 socklnd: don't deref lnet_hdr in LNDs
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 321fe9fa12d244bb70a6662d3d4d9a6c1709fe3e

Comment by Gerrit Updater [ 26/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43603/
Subject: LU-10391 lnet: separate lnet_hdr in msg from that in lnd.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 4ea5cf154a4663a73621dad687873b05290cd420

Comment by Gerrit Updater [ 26/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43604/
Subject: LU-10391 lnet: change lnet_hdr to store large nids.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 5bb421cdfd4ce6a29202ce9d680c5d95d04a5254

Comment by Gerrit Updater [ 26/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43605/
Subject: LU-10391 lnet: change lnet_prep_send to take net_processid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: c0ccfaf1e359352731b50d07e33a79c69c21a791

Comment by Gerrit Updater [ 26/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43606/
Subject: LU-10391 lnet: convert to struct lnet_process_id in lib-move
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 9feddf7e5d01be437e1a4bd2bd4f417a1206edf3

Comment by Gerrit Updater [ 26/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43607/
Subject: LU-10391 lnet: convert LNetGetID to return an large-addr pid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: a42e0f6471bf5aad37f15b929be5bf3e76f892ef

Comment by Gerrit Updater [ 26/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43608/
Subject: LU-10391 lnet: alter lnd_notify_peer_down() to take lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 6a9bdf59e6306d49c4d2d44e95773a257114c639

Comment by Gerrit Updater [ 26/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43609/
Subject: LU-10391 socklnd: move lnet_hdr unpack into ->pro_unpack
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 15365f3de34ed7d25166d347d0df9b6741deffe7

Comment by Gerrit Updater [ 26/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43610/
Subject: LU-10391 socklnd: Change ksock_hello_msg to struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: d1fb459cca931f84f087aae5519bd86eca92009f

Comment by Gerrit Updater [ 26/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43611/
Subject: LU-10391 socklnd: add hello message version 4
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 7b31ef0bbac99bfd0f681fb9256f82db8bc9b20d

Comment by Gerrit Updater [ 26/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43612/
Subject: LU-10391 lnet: Convert ping to support 16-bytes address
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 3e37ac8bb7e068a3070f5770570766ad87270186

Comment by Gerrit Updater [ 26/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43613/
Subject: LU-10391 lnet: convert nids in lnet_parse to lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: ac439ce87b0acbff35deaf52c283e3fa93e52510

Comment by Gerrit Updater [ 26/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43614/
Subject: LU-10391 lnet: change src_nid arg to lnet_parse() to 16byte
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: eb0eedbb1a68297b879cbec691ce85e1f208b89e

Comment by Gerrit Updater [ 26/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44838/
Subject: LU-10391 lnet: Fix NULL-deref in lnet_nidstr_r()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 8ed370864c4747281dab22f3fafceb5310c1cd53

Comment by Gerrit Updater [ 26/Jan/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43615/
Subject: LU-10391 lnet: change lnet_del_route() to take lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 58993799f29d12c1a60c24d9e096e38ed998ff87

Comment by Gerrit Updater [ 30/May/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43616/
Subject: LU-10391 lnet: change LNetPrimaryNID to use struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: ac881498fa19e6b04cbd51e206376657d24a8eb9

Comment by Gerrit Updater [ 30/May/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43617/
Subject: LU-10391 lnet: alter lnet_drop_rule_match() to take lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 57b7b3d36f5fa15274cc8da939556f4d814cbbb7

Comment by Gerrit Updater [ 30/May/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43618/
Subject: LU-10391 lnet: Change LNetDist to work with struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: c87f70acd86c59425c9fd89682cd46498dcaeef5

Comment by Gerrit Updater [ 30/May/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43619/
Subject: LU-10391 lnet: convert LNetPut to take 16byte nid and pid.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 50f6bb62987c54ea9a5d622df4871ad9a416c018

Comment by Gerrit Updater [ 30/May/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43620/
Subject: LU-10391 lnet: change LNetGet to take 16byte nid and pid.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: d727ec56b26cbd1b8444e83e05fe408aa97cbeb1

Comment by Gerrit Updater [ 30/May/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43621/
Subject: LU-10391 lnet: switch LNetIsPeerLocal() to take 16-byte addr
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 3b8f253c10759027d37d45cf6e961f77fc9c0f81

Comment by Gerrit Updater [ 30/May/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44621/
Subject: LU-10391 socklnd: pass large processid to ksocknal_add_peer
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 6deddc3d46704643dbc6c91b63ee52b1d91ce212

Comment by Gerrit Updater [ 30/May/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44622/
Subject: LU-10391 socklnd: large processid for ksocknal_get_peer_info
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: e4a49294530a5d5f78b970f562165f598813c3fc

Comment by Gerrit Updater [ 30/May/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44623/
Subject: LU-10391 socklnd: switch ksocknal_del_peer to lnet_processid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 782e37e54f5c54886972320e526845da422d9a03

Comment by Gerrit Updater [ 11/Jul/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44624/
Subject: LU-10391 lnet: discard some peer_ni lookup functions
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 9768d8929a305588f07caaffe9c9d16087133bf7

Comment by Gerrit Updater [ 11/Jul/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44625/
Subject: LU-10391 lnet: change lnet_*_peer_ni to take struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: d9af9b5a7ee7066607d14d61f92aec5350a36008

Comment by James A Simmons [ 18/Jul/22 ]

Currently I'm working on the user land interface to support IPv6 which will unify both the per MR and DLC apis we currently support. This will enable us to test the IPv6.

Comment by Gerrit Updater [ 01/Sep/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44626/
Subject: LU-10391 lnet: change ni_status in lnet_ni to u32*
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 9e2df7e5cc5fca3c60e048f3e4004fb8d10f03c1

Comment by James A Simmons [ 13/Sep/22 ]

Neil using my Netlink patch I attempted to bring up an IPv6 NID but it failed. Looking at ksocklnd it's still hardcoded to ipv4. Could you update the LND driver while I work on the Netlink ping patch.

Comment by Gerrit Updater [ 16/Sep/22 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/48569
Subject: LU-10391 lnet: add iface index to struct lnet_inetdev
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 8a3a766d7f0f9c159aef6619b2a4a77099ca4d82

Comment by Gerrit Updater [ 16/Sep/22 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/48570
Subject: LU-10391 socklnd: support IPv6 in ksocknal_ip2index()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 854e687fc52b400c7d54a06159a938bc1d22d90c

Comment by Gerrit Updater [ 16/Sep/22 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/48571
Subject: LU-10391 lnet: only use PUBLIC IP6 addresses for connections
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: b6ea6fc7e3b20d1006de2015e3792e6f0e887483

Comment by Gerrit Updater [ 16/Sep/22 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/48572
Subject: LU-10391 lnet: support IPv6 in lnet_inet_enumerate()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: bceef444a747c9667e6eab97462791094150668e

Comment by James A Simmons [ 24/Sep/22 ]

With the ksocklnd patches and my Netlink work I can bring up IPv6 interfaces.

lctl list_nids
fe80::f68:45bd:7b60:e933@tcp

I also created a Netlink ping patch and testing pinging with IPv6. I'm seeing

89440.313309] LNetError: 336027:0:(lib-socket.c:307:lnet_sock_create()) Error trying to bind to port 1023: -22
[89440.323903] LNetError: 11e-e: Unexpected error -22 connecting to fe80::f68:45bd:7b60:e933@tcp at host [fe80:0000:0000:0000:0f68:45bd:7b60:e933]:988
[89501.674740] LNetError: 336028:0:(lib-socket.c:307:lnet_sock_create()) Error trying to bind to port 1023: -22
[89501.685288] LNetError: 11e-e: Unexpected error -22 connecting to fe80::f68:45bd:7b60:e933@tcp at host [fe80:0000:0000:0000:0f68:45bd:7b60:e933]:988
[89677.568616] LNetError: 336029:0:(lib-socket.c:307:lnet_sock_create()) Error trying to bind to port 1023: -22
[89677.579170] LNetError: 11e-e: Unexpected error -22 connecting to fe80::a242:3fff:fe39:d974@tcp at host [fe80:0000:0000:0000:a242:3fff:fe39:d974]:988
[89752.123634] LNetError: 336030:0:(lib-socket.c:307:lnet_sock_create()) Error trying to bind to port 1023: -22
[89752.134238] LNetError: 11e-e: Unexpected error -22 connecting to fe80::a242:3fff:fe39:d974@tcp at host [fe80:0000:0000:0000:a242:3fff:fe39:d974]:988

This is ping the local NID and a remote NID.

Comment by Andreas Dilger [ 24/Sep/22 ]
host [fe80:0000:0000:0000:0f68:45bd:7b60:e933]:988

Can't help with the LNet part, but please update theIPv6 NID printing to use the standard shortcut to skip blocks that are all-zero:

[fe80::::0f68:45bd:7b60:e933]
Comment by Neil Brown [ 25/Sep/22 ]

> but please update theIPv6 NID printing to use the standard shortcut to skip blocks that are all-zero:

What are there two struct netstrfns libcfs_netstrfns[], one in libcfs and one in lnet?

The one in lnet will get this right, the one in libcfs doesn't - because I didn't realise there were two and that I needed to extend them both.

 

Comment by Andreas Dilger [ 26/Sep/22 ]

It looks  like  one for  the kernel and one for the userspace utils.  These functions used to be shared between the kernel and userspace, but they were split into two separate copies in commit v2_7_51_0-30-g05d13ca333:

LU-6245 libcfs: create userland and kernel string operations
    
Additonal string handling and NID string parsing code are both
used by kernel space and user land. This prevents us from
moving forward for cleaning up the userland and kernel space
headers. With the code duplicated for both environments we
can then clean up the headers independently. Since NID string
handling is only done for LNET we move it there.

At a minimum, it would make sense to add a comment to both copies of this struct reporting on its peer's existence, and possibly have some sanity checking for consistency.

Comment by Neil Brown [ 28/Sep/22 ]

Sorry - I didn't realise that the libcfs was in the user-space libcfs, not the kernel-space one.  That makes more sense.

So I've looked at the problem again and now see that I misunderstood it.  The IPv6 address stored in a nid IS being formatted correctly by libcfs_strnid().  The problem is that when an address (not in a NID) is being reported in the error message, it has the less desirable format.

This can be fixed by using %pIScp rather than %pISp for reporting network addresses.  The 'c' means compressed, and is ignored for IPv4.

Comment by Gerrit Updater [ 28/Sep/22 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/48685
Subject: LU-10391 lnet: use %pISc for formatting IP addresses
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 892269b27a859bf07ee91a29bfe8b85bb2ad80b2

Comment by Gerrit Updater [ 04/Oct/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44627/
Subject: LU-10391 lnet: track pinginfo size in bytes, not nis.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 941218e09e1d6bb9b5b3ed4780e1c60ed75dbce7

Comment by Gerrit Updater [ 04/Oct/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48569/
Subject: LU-10391 lnet: add iface index to struct lnet_inetdev
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 860182ee6e84d391aaa8b7e3cf810b6e030b1672

Comment by Gerrit Updater [ 04/Oct/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48570/
Subject: LU-10391 socklnd: support IPv6 in ksocknal_ip2index()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: c0fdf9efbf927db405fa662292bd7f5d4001e1bb

Comment by Gerrit Updater [ 04/Oct/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48571/
Subject: LU-10391 lnet: only use PUBLIC IP6 addresses for connections
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: cd3b89be221b4c5b67e14aa32001cd97ac3d54a8

Comment by Gerrit Updater [ 10/Oct/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48685/
Subject: LU-10391 lnet: use %pISc for formatting IP addresses
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: ed6b125bab5f7e383d8fa00acf75836922cfcb2a

Comment by Gerrit Updater [ 15/Oct/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48572/
Subject: LU-10391 lnet: support IPv6 in lnet_inet_enumerate()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 781499eee645a635d065f40a5400a3b0de3aa051

Comment by James A Simmons [ 17/Oct/22 ]

I added attachments from LAD 2022 developers day when I went over the Netlink YAML API I developed. It's an 1.5 hour discussion which does with the presentation.

Comment by James A Simmons [ 18/Oct/22 ]

Fixed the IPv6 ping issues. It works in my testing. Please review the patch set.

Comment by Gerrit Updater [ 31/Oct/22 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/48990
Subject: LU-10391 lnet: fix build issue when IPv6 is disabled.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 35dd40a9cfd82028eaebfbba702a9144fbdf004d

Comment by Gerrit Updater [ 08/Nov/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44628/
Subject: LU-10391 lnet: allow ping packet to contain large nids
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: db0fb8f2b649c0c389a6915f1bbd49b36bfec4b9

Comment by Gerrit Updater [ 08/Nov/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48990/
Subject: LU-10391 lnet: fix build issue when IPv6 is disabled.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 896cd5b7bcf94d4fd2ec20c808d66e11ca87939f

Comment by James A Simmons [ 08/Nov/22 ]

Ping patches are ready for review.

Comment by Gerrit Updater [ 17/Nov/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44629/
Subject: LU-10391 lnet: extend lnet_is_nid_in_ping_info()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 56bcfbf22d91b96c3367533145e5099e63f26d14

Comment by Gerrit Updater [ 17/Nov/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44632/
Subject: LU-10391 lnet: find correct primary for peer
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 022b46d887603f703ecc897c0aaa40e609f2d926

Comment by Gerrit Updater [ 17/Nov/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44633/
Subject: LU-10391 lnet: change lnet_notify() to take struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 4a88236f40a47c05d0cc6a253dcfc105427987f4

Comment by Gerrit Updater [ 17/Nov/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44634/
Subject: LU-10391 lnet: discard lnet_nid2ni_*()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: cbfbe6d132c6d0fe5c53118c792d9924b1bfb58e

Comment by Gerrit Updater [ 17/Nov/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44635/
Subject: LU-10391 lnet: change lnet_debug_peer() to struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: e834ad5992adef5989332abce1d8eb2102f86cd2

Comment by Gerrit Updater [ 17/Nov/22 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/49181
Subject: LU-10391 lnet: change lnet_find_best_lpni to handle large NIDs
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 5eac75b6cba0c0f01c01ae06c969f53f54016c8e

Comment by Gerrit Updater [ 29/Nov/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44630/
Subject: LU-10391 lnet: lnet_peer_merge_data to understand large addr
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 57bcd6aa7f5f347de81665307e76ce47c7491052

Comment by Gerrit Updater [ 29/Nov/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44631/
Subject: LU-10391 lnet: router_discover - handle large addrs in ping
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 2d916eec68e8a7d353ae4ae2a4923d7f6cfcec0d

Comment by James A Simmons [ 05/Dec/22 ]

Status update:

One LNet core patch is outstanding. We have code to handle sending new LNet headers for bulk I/O  but its not available to send over the wire when using a large NID. Once we have that in place we can send data between two IPv6 based NIDs. I'm working on more Netlink code to allow us to use the new large NIDs for user land APIs. I just pushed LNet selftest group querying handling to Netlink. The last bits for lnetctl / lctl ping for large NIDs is almost done. After that LNet peer support for large NIDs will be added.

Comment by Andreas Dilger [ 05/Dec/22 ]

James, has any effort been spent on the Lustre side of this equation yet (LU-13307, LU-13308, LU-13340, LU-14288)?

Getting some of these issues addressed (in particular the basic config llog and Imperative Recovery records) to allow IPv6 addresses in the logs (even if the clients don't know how to connect to them yet) would avoid interoperability issues with older clients seeing these larger records. It would great if something could be included into 2.12.10 (which will be the end of the line for 2.12 releases) to maximize the backward compatibility.

Comment by James A Simmons [ 05/Dec/22 ]

I like to get traffic flowing between LNet nodes first. I really want to make sure we get Lustre support right.

If we can get LU-10360 working then Lustre can bring up LNet itself without any user configuration.  This setup could be a way to test Lustre with IPv6 without the need for LNet utilities being totally updated.

Comment by Gerrit Updater [ 06/Dec/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/49181/
Subject: LU-10391 lnet: change lnet_find_best_lpni to handle large NIDs
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 4c0c01e29c891d100b134d0988947cfb3aeee6c1

Comment by Gerrit Updater [ 06/Jan/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/49574
Subject: LU-10391 lnet: migrate peer NI control to Netlink
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 887112ef909ac191ee103e44b84ed503650c7aed

Comment by Bjoern Olausson [ 10/Feb/23 ]

Hej,

what is the staus of the overall Lustre/LNet IPv6 support? Is it ready for prime time?

I am working on a tender (100 PB, 1.25 GB/s, HDR or NDR InfiniBand) that requires IPv6.

Greetings,
Bjoern

Comment by Neil Brown [ 10/Feb/23 ]

IPv6 support is getting close to the point where we might be able to start testing some bits of the new code.
The LNET side might be mostly functional in the near future.

We are still quite a way before we could think about trying to mount a Lustre filesystem using IPv6.

So no - not ready for prime time yet.

 

Comment by Bjoern Olausson [ 10/Feb/23 ]

Thanks Neil,

is there a rough ETA?

Comment by James A Simmons [ 10/Feb/23 ]

I plan to push all the LNet user interface changes needed to support IPv6 to the repo by the end of the month. It will take time to review and merge the patches. Currently I'm working on regression found with multi interface nodes and then I have a few patches out standing to collect peer and router information. I also pushed a patch for managing peers but it needs an update. What is missing is full collecting of the LNet NI (lnetctl net show) and managing the routes. That should cover most of tools for LNet with IPv6. LNet selftest is still in the works.

How much do you need done to start your testing? 

P.S    I do want to aim at getting AF_IB support going as well before the close window of this release.

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50085
Subject: LU-10391 lnet: change LNetAddPeer() to take struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 39d2920231348a7bc14a29a8d615c958af04eade

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50086
Subject: LU-10391 obdclass: change class_add/check_uuid to large nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0e1fbb0a8f6f844e4d095358143d458f1cfea012

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50087
Subject: LU-10391 obdclass: support large nids in mgs_nidtbl_entry
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: ac136b0128958026ea260d01a441da43195f970f

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50088
Subject: LU-10391 lustre: rename class_parse_nid to class_parse_nid4
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: afebbbd2b887852b45858a3051c8abbd63d3845a

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50089
Subject: LU-10391 lustre: introduce class_parse_nid()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 00d99d2fa85a0890bf413b988090b1491ace9dd0

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50090
Subject: LU-10391 obdclass: change class_match_nid to take lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: dc37eff5987c1803f07f993a807c4cb063000652

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50091
Subject: LU-10391 mgs: fix lots of white-space irregularities
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 15c2bdd883541f6ff3857bb786434619ad3e4589

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50092
Subject: LU-10391 libcfs: add large-nid string conversion functions.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7cd75e3f836b19c247dd02548dfd2b2dfaaf1a6d

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50093
Subject: LU-10391 mgs: change record_add_uuid() to take large nids
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 25664092879a652c79c877e176c3668d586aaeff

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50094
Subject: LU-10391 mgs: convert class_parse_nid4 to class_parse_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 839d002c831371e0840cdef1131bc1e5e6f7b231

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50095
Subject: LU-10391 obdclass: remove class_parse_nid4()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 02f663527f487fdf238c824bdf98a6f17760e10c

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50096
Subject: LU-10391 lustre: Support large nids in LCFG_ADD_UUID
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 3f80a0e0a4a88b3ce873553491129a322bdc4a51

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50097
Subject: LU-10391 lustre: obd_connect and reconnect now use large nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 8b88d0a0c36253bbe27b9276c3133ac311c06f4e

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50098
Subject: LU-10391 lustre: change cfs_match_nid to take large nid.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 8b41dd0101e98ae55b7c186607ab6ec47fa03ada

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50099
Subject: LU-10391 mdd: add LLOG_F_EXT_X_NID_BE
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 2e8e4abef2cbeecf23da56729f6707d60dcf928d

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50100
Subject: LU-10391 ptlrpc: change cc_nid in nrs to be struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: e01d8bffacba81c9d052e32f30678deeed83d8ba

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50101
Subject: LU-10391 ptlrpc: change tc_nid in nrs to be struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 05c37fe41328b43f9d1ec5cb215547ce7e770f9a

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50102
Subject: LU-10391 ptlrpc: switch sptlrpc_rule_set_choose to large nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 6b8098c5ce3a5e9471caf89a43ef5083989d6fee

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50103
Subject: LU-10391 mdt: change md_perm.mp_nid to large nid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 6c5242c07c495f02b146dcc6d099f12351448549

Comment by Gerrit Updater [ 21/Feb/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50104
Subject: LU-10391 mdt: extend downcall to handle large nids
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: f6dedc86a2e142336cb22900e6fb0d6b8398bf08

Comment by Gerrit Updater [ 10/Mar/23 ]

"jsimmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50253
Subject: LU-10391 lnet: handle discovery with Netlink
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7cd4b3b712a87410266a7d30936a76751f1fa1d2

Comment by Gerrit Updater [ 10/Mar/23 ]

"jsimmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50254
Subject: LU-10391 lnet: migrate router management to Netlink
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4cf54ec382d73647d4ff71b7ae165b887e1888e4

Comment by Gerrit Updater [ 10/Mar/23 ]

"jsimmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50255
Subject: LU-10391 lnet: migrate full LNet NI information collection
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: bb7bcc11d4608ebba33d09b7fe1758019e871a13

Comment by James A Simmons [ 10/Mar/23 ]

Since several people are looking to give this a go I pushed what I have so far. Some is ready for ready and some needs more work.

These are ready.

https://review.whamcloud.com/#/c/fs/lustre-release/+50026
https://review.whamcloud.com/#/c/fs/lustre-release/+49516
https://review.whamcloud.com/#/c/fs/lustre-release/+49839

 

More work is need for the following but you use them for general testing.

https://review.whamcloud.com/#/c/fs/lustre-release/+49361
https://review.whamcloud.com/#/c/fs/lustre-release/+50253
https://review.whamcloud.com/#/c/fs/lustre-release/+49574
https://review.whamcloud.com/#/c/fs/lustre-release/+50254
https://review.whamcloud.com/#/c/fs/lustre-release/+50255

Comment by Gerrit Updater [ 22/Mar/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50362
Subject: LU-10391 utils: enable large NID support for convert_hostnames
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: eaf9a0e171190d75f04e614e8af21ac957f13497

Comment by James A Simmons [ 30/Mar/23 ]

In patch https://review.whamcloud.com/#/c/fs/lustre-release/+/49516 Oleg has reported an __might_sleep. I believe this to be a bug in the RHEL7 kernel where it is lacking a fix (git commit 707693c8a498697aa8db240b93eb76ec62e30892). I have asked Oleg to try the IPv6 patch out with a RHEL8 debug  kernel or perhaps patch his debug kernel with the mentioned patch to see if it resolves it. If does come to this issue I can open a ticket with RedHat to have the patch applied to its RHEL7 kernels.

Comment by Gerrit Updater [ 11/Apr/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50085/
Subject: LU-10391 lnet: change LNetAddPeer() to take struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 42b49afdc8a4d2ec6597d3db6553ed79bad7fed8

Comment by Gerrit Updater [ 11/Apr/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50086/
Subject: LU-10391 obdclass: change class_add/check_uuid to large nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 16d84b030520c431daafcca4f1df00a1de5d20bb

Comment by Gerrit Updater [ 11/Apr/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50088/
Subject: LU-10391 lustre: rename class_parse_nid to class_parse_nid4
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: b4a28a3269fadb105912325054c4ba82984a981c

Comment by Gerrit Updater [ 11/Apr/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50089/
Subject: LU-10391 lustre: introduce class_parse_nid()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 163331cb81c9f7d7aaaed2056691abb5401580f0

Comment by Gerrit Updater [ 11/Apr/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50090/
Subject: LU-10391 obdclass: change class_match_nid to take lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: a6915dd9d82e2f2b47b9a103baae4391eedf94e2

Comment by Gerrit Updater [ 11/Apr/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50091/
Subject: LU-10391 mgs: fix lots of white-space irregularities
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 60e6e35f4cad3f79b2e96ddf41a8d8a02d6047ac

Comment by Gerrit Updater [ 11/Apr/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50100/
Subject: LU-10391 ptlrpc: change cc_nid in nrs to be struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 36a199db2b174ba95b683b5e641443357ad24036

Comment by Andreas Dilger [ 14/Apr/23 ]

I've filed LU-16738, which relates to how the mount.lustre command can be improved for multiple NIDs on the command line. While it is useful for systems today, I think it will be increasingly important for IPv6 due to the longer NID length.

Comment by Gerrit Updater [ 22/Apr/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50092/
Subject: LU-10391 libcfs: add large-nid string conversion functions.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: b6c702df5d4de8e5a806169404ba1c069ea8fff2

Comment by Gerrit Updater [ 22/Apr/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50095/
Subject: LU-10391 obdclass: remove class_parse_nid4()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 89aac3d3b39f68982530287e11fb29fef0edf456

Comment by Gerrit Updater [ 22/Apr/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50098/
Subject: LU-10391 lustre: change cfs_match_nid to take large nid.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 3d0b1b0200542f845ceb00c7d2be9a85fcf19388

Comment by Gerrit Updater [ 22/Apr/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50101/
Subject: LU-10391 ptlrpc: change tc_nid in nrs to be struct lnet_nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: e2602bd49c1d670b8bf0ad75cdd115941b885575

Comment by Gerrit Updater [ 23/Apr/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50716
Subject: LU-10391 lnet: set msg field for lnet message header
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 818c7fb748b5d7e13208bffd283d67cea67aa9d0

Comment by Gerrit Updater [ 26/Apr/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50097/
Subject: LU-10391 lustre: obd_connect and reconnect now use large nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 3b2926161d27906642a747f3449006267fecba18

Comment by Gerrit Updater [ 01/May/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50813
Subject: LU-10391 mgc: gracefully ignore unrecognised recovery logs
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 2c5d73fc8a9606b120db71b33e8aab2adae9faba

Comment by Gerrit Updater [ 09/May/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50716/
Subject: LU-10391 lnet: set msg field for lnet message header
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 562865630e67afb06d0fbf679ebee243691bec66

Comment by James A Simmons [ 11/May/23 ]

The following patches are ready for review:

https://review.whamcloud.com/#/c/fs/lustre-release/+/49839/

https://review.whamcloud.com/#/c/fs/lustre-release/+/50254/

https://review.whamcloud.com/#/c/fs/lustre-release/+/50440/

https://review.whamcloud.com/#/c/fs/lustre-release/+/49574/

Comment by Gerrit Updater [ 31/May/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50102/
Subject: LU-10391 ptlrpc: switch sptlrpc_rule_set_choose to large nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 80a4a2ebe0a269d564e13b0931c9852379da86d8

Comment by James A Simmons [ 08/Jun/23 ]

Status update:

LU-13306 : patch for MGS to handle large NIDs is ready for review. Its passing all test.  https://review.whamcloud.com/#/c/fs/lustre-release/+/50896/

LU-10391:  patch https://review.whamcloud.com/#/c/fs/lustre-release/+/50103 for internal MDS identity upcall is ready to land and is in master-next

                      patch https://review.whamcloud.com/#/c/fs/lustre-release/+/50362 is ready to handle large NID strings for mounting

Several patches for IPv6 support for LNet are also ready for review.

https://review.whamcloud.com/#/c/fs/lustre-release/+/50254

https://review.whamcloud.com/#/c/fs/lustre-release/+/50440/

https://review.whamcloud.com/#/c/fs/lustre-release/+/49574/

https://review.whamcloud.com/#/c/fs/lustre-release/+/49516/

https://review.whamcloud.com/#/c/fs/lustre-release/+/50255/

 

Comment by Gerrit Updater [ 09/Jun/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50103/
Subject: LU-10391 mdt: change md_perm.mp_nid to large nid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 8683b953347fc16a7fa55a838d30b0754df95cfa

Comment by James A Simmons [ 20/Jun/23 ]

My latest batch of LNet IPv6 work went without a review for 3 weeks outside of Chris Horn. Has this work been slated for the 2.17 release?

Comment by Andreas Dilger [ 22/Jun/23 ]

I was away for ISC and two weeks vacation, so didn't do (m)any patch reviews during that time, but I'd hope that Neil or Serguei or Frank would have done reviews on at least the first patches in the series to keep it moving along in my absence.

Comment by Gerrit Updater [ 26/Jun/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51442
Subject: LU-10391 gnilnd: typo fixes for lnet_notify() changes.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 9dc7f816ef1a650f7df91d62c54b73fcbadc9a51

Comment by Gerrit Updater [ 24/Aug/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50362/
Subject: LU-10391 obdclass: handle large NIDs for mount strings
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 101f6e84889a9b48238ca320557101058d935fb0

Comment by Mikhail Pershin [ 02/Sep/23 ]

the patch "LU-10391 obdclass: handle large NIDs for mount strings" caused my local system stops to mount servers with error like this:

[11210.421883] LNetError: 120-3: Refusing connection from 192.168.56.101 for 0.0.0.0@tcp: No matching NI
[11210.423194] LNetError: 122557:0:(socklnd_cb.c:1783:ksocknal_recv_hello()) Error -104 reading HELLO from 192.168.56.101
[11210.424063] LNetError: 11b-b: Connection to 0.0.0.0@tcp at host 0.0.0.0:988 was reset: is it running a compatible version of Lustre and is 0.0.0.0@tcp one of its NIDs?

the reason was that my hostname listed in '/etc/hostname' was not listed in '/etc/hosts' with related address. I am not sure that is correct solution for the problem because without that patch it worked, so problem occurred due to patch and I suppose it should be  fixed in patch as well. Anyway, if there will be a report about 0.0.0.0@ address that can be fixed by listing hostname in /etc/hosts

Comment by James A Simmons [ 13/Sep/23 ]

The change was the move from gethostname() which only supports IPv4 to getaddrinfo() which supports both IPvv4 and IPv6. What did your /etc/hostname look like so I can reproduce this. I suspect your /etc/hostname lacked an actually IP.

Comment by James A Simmons [ 13/Sep/23 ]

Ah to support what you want you need to install the package containing nss-myhostname. Try that and let me know if it works. 

Comment by Mikhail Pershin [ 13/Sep/23 ]

James, I had the following in configs:

[root@nodez ~]# cat /etc/hostname 
nodez
[root@nodez ~]# cat /etc/hosts
127.0.0.1   localhost localhost.localdomain localhost4 localhost4.localdomain4
::1         localhost localhost.localdomain localhost6 localhost6.localdomain6
192.168.56.101   nodez <--- this one was added later
[root@nodez ~]# 

The last line in /etc/hosts was added to fix the issue. Without it the address of nodez was reported as 0.0.0.0

I have no nss-myhostname tool in my Centos 8 and didn't figure out yet what package contains it. Any ideas?

Comment by Gerrit Updater [ 25/Oct/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50255/
Subject: LU-10391 lnet: migrate full LNet NI information collection
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: d15bfca07858c175c67ac6715fca2a876834b1ce

Comment by Gerrit Updater [ 25/Oct/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/49574/
Subject: LU-10391 lnet: migrate peer NI control to Netlink
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 8a0fdfa0b281b20b6acdd25ceb4c3fb5a9efacb3

Comment by Gerrit Updater [ 31/Oct/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52911
Subject: LU-10391 ksocklnd: use ksocknal_protocol v4 for IPv6
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 373d879460210248513679495ec7a59a7550106c

Comment by Gerrit Updater [ 03/Nov/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50254/
Subject: LU-10391 lnet: migrate router management to Netlink
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 6557cd4b8c8cf8f8a7d33dadf5a86b4fe0d6eaa2

Comment by Gerrit Updater [ 06/Nov/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53003
Subject: LU-10391 lnet: update Netlink commands functionality
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: c27ff2701f5d42963c642cb2aaa801b2f4b3d2fd

Comment by Gerrit Updater [ 07/Nov/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53013
Subject: LU-10391 lnet: support setting LND timeouts
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 6f9a02948f58f02fec6b2d1af04078706f2869c4

Comment by Gerrit Updater [ 07/Nov/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53016
Subject: LU-10391 lnet: use lnet_ni_get_status_locked for lnet_net_show_dump
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: e6ea13f6d9d896aea87e93685512c1008246bef7

Comment by Gerrit Updater [ 07/Nov/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53018
Subject: LU-10391 lnet: missing some peer functionality
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: f5bbe42a780d58f20c3a114eb85eae69fbcfb683

Comment by Gerrit Updater [ 07/Nov/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53019
Subject: LU-10391 lnet: hops -1 is valid
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 79f27873db318a064d966dbfe005dd1269f386a5

Comment by Gerrit Updater [ 07/Nov/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53020
Subject: LU-10391 lnet: filter out white spaces
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 6a821b38391841447a8b2e7d896900ad1f6f31ba

Comment by Gerrit Updater [ 08/Nov/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50253/
Subject: LU-10391 lnet: handle discovery with Netlink
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 68254c484abfb5a9e212f72f937c0af9710dfb73

Comment by Gerrit Updater [ 08/Nov/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/52911/
Subject: LU-10391 ksocklnd: use ksocknal_protocol v4 for IPv6
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: cb5f92c0e38eaca4b76f17356ba6d7817c220f56

Comment by Gerrit Updater [ 13/Nov/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53115
Subject: LU-10391 mgs: copy fill nid string
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7949932586ae660929e61b16508eb3f92d7349df

Comment by Gerrit Updater [ 13/Nov/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53117
Subject: LU-10391 ptlrpc: update import debugfs to support IPv6 formats
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: b77af6a674c1714ebe35e078a9f3da312aa259d7

Comment by James A Simmons [ 14/Nov/23 ]

Pushed the 3 needed patches required that we need to allow mounting Lustre with IPv6. Something kind of works but its unknown what is lacking with testing. I have shorten IPv6 address so I haven't seen the UUID overflow just yet. That needs to be fix but it works in general.

Comment by Gerrit Updater [ 15/Nov/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53150
Subject: LU-10391 socklnd: handle IPv6 for zero copy messages
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 2e0a2b218ecdbf21922707dd43f1b5152bfb4843

Comment by Gerrit Updater [ 18/Nov/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53013/
Subject: LU-10391 lnet: support setting LND timeouts
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: aa2fa25325edfb6ee03ee343a46cfc2acfd68b42

Comment by Gerrit Updater [ 18/Nov/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53016/
Subject: LU-10391 lnet: use lnet_ni_get_status_locked for lnet_net_show_dump
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: f3237a16e367e13eb8d29bab66903757ba8fdd62

Comment by Gerrit Updater [ 18/Nov/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53019/
Subject: LU-10391 lnet: hops -1 is valid
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: c75b6fe3f147206ee3fd10fb0ee55a06245c0a4f

Comment by Gerrit Updater [ 18/Nov/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53020/
Subject: LU-10391 lnet: filter out white spaces
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 0f61c9ff72f34270b31e1b0a5d0a67691b815994

Comment by Gerrit Updater [ 29/Nov/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53018/
Subject: LU-10391 lnet: missing some peer functionality
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 774c3d2883c86117e9e2da992843756a5b721064

Comment by Gerrit Updater [ 29/Nov/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53115/
Subject: LU-10391 mgs: copy full nid string
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 8276ade19d573c6eb3b561dbcbc30de607a703fd

Comment by Gerrit Updater [ 29/Nov/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53150/
Subject: LU-10391 socklnd: handle IPv6 for zero copy messages
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 17c1cdf40e54f7be235f6b896dee575ef897d7a1

Comment by Gerrit Updater [ 04/Dec/23 ]

"Chris Horn <chris.horn@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53320
Subject: LU-10391 lnet: Ping target corrupted
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: fc2fc1176be1c71b50583d40b152cf652977994e

Comment by James A Simmons [ 06/Dec/23 ]

Status update:

Patch https://review.whamcloud.com/c/fs/lustre-release/+/53117 is in master-next and ready to land. The other patch needed for mounting lustre with IPv6 is https://review.whamcloud.com/c/fs/lustre-release/+/53135 which is currently being actively worked on. With this work we can be considered feature complete.

Addition work is needed and for more complete LNet setup support we need 3 patches:

https://review.whamcloud.com/c/fs/lustre-release/+/49361

https://review.whamcloud.com/c/fs/lustre-release/+/53003

https://review.whamcloud.com/c/fs/lustre-release/+/53004

Those patch are WIP but they mostly work for me. Bugs are being worked out.

Comment by Gerrit Updater [ 10/Dec/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53399
Subject: LU-10391 nrs: use struct lnet_nid for nrs_crrn_hashfn()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 923e6442c8871b5002f40b431b83eeb47b4a8ae3

Comment by Gerrit Updater [ 13/Dec/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53117/
Subject: LU-10391 ptlrpc: update import debugfs to support IPv6 formats
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 03f4ef7f32692cfdbe9fc20c218fea75ec3aee08

Comment by James A Simmons [ 13/Dec/23 ]

Status report:

For basic support for mount and using lustre with IPv6 we are done to one patch needed to mount. That is patch https://review.whamcloud.com/c/fs/lustre-release/+/53135.

As you can see we have a bunch of patches to make testing pass.

Comment by Gerrit Updater [ 20/Dec/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53320/
Subject: LU-10391 lnet: Ping target corrupted
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 1b099c404ae4780179bbcd0622fac255edf40352

Comment by Gerrit Updater [ 27/Dec/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53560
Subject: LU-10391 lnet: support updating LNet local NI settings
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: f3906ada20ea243d933c1a35d452ba5f20857be4

Comment by Gerrit Updater [ 02/Jan/24 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53571
Subject: LU-10391 mgs: handle strscpy return value properly
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0a1217e26a34e0317014b9c9eaf3baf8dac379bd

Comment by Gerrit Updater [ 03/Jan/24 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/49361/
Subject: LU-10391 lnet: update ping to handle multiple NIDs
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: c384a39d2889e93229813a204f7de5ca5f6bd142

Comment by Gerrit Updater [ 04/Jan/24 ]

"Andreas Dilger <adilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53592
Subject: LU-10391 revert: "lnet: update ping to handle multiple NIDs"
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0b4eac4e80b7556f4521a3d9f4d80445ad0e29f0

Comment by Andreas Dilger [ 04/Jan/24 ]

I've pushed a revert for the most recent patch in this ticket because it is causing almost constant failures since 2024-01-03 when it landed (build version 2.15.60.1). Almost all of the PASS results since that date are for patches that have not yet rebased:
https://testing.whamcloud.com/search?client_branch_type_id=24a6947e-04a9-11e1-bb5f-52540025f9af&server_branch_type_id=24a6947e-04a9-11e1-bb5f-52540025f9af&test_set_script_id=a2b1c4b2-b449-11e9-b88c-52540065bddc&sub_test_script_id=5a631bd3-f3e2-48aa-937f-fa3662bd2d47&start_date=2024-01-03&end_date=2024-01-04&source=sub_tests#redirect

/usr/sbin/lnetctl set transaction_timeout 10
Added delay rule 10.240.39.244@tcp->10.240.39.245@tcp (1/1)
Issued 8 pings to 10.240.39.245@tcp from 10.240.39.244@tcp
Removed 1 delay rules
ping:
- primary nid: 10.240.39.245@tcp
  Multi-Rail: true
  peer_ni:
  - nid: 10.240.39.245@tcp
 sanity-lnet test_253: @@@@@@ FAIL: Expect 1 dropped GET but found 0 
  Trace dump:
  = /usr/lib64/lustre/tests/test-framework.sh:6947:error()
  = /usr/lib64/lustre/tests/sanity-lnet.sh:3330:do_expired_message_drop_test()
  = /usr/lib64/lustre/tests/sanity-lnet.sh:3340:test_253()
  = /usr/lib64/lustre/tests/test-framework.sh:7287:run_one()
  = /usr/lib64/lustre/tests/test-framework.sh:7343:run_one_logged()
  = /usr/lib64/lustre/tests/test-framework.sh:7173:run_test()
Comment by James A Simmons [ 04/Jan/24 ]

It might be good to expect the test for now. It doesn't fail all the time so I suspect its a timing issue with the test.

Comment by Gerrit Updater [ 04/Jan/24 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53593
Subject: LU-10391 tests: recent landing for ping support has timing issue
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: a70b9a6d31de4547ca2a9588b7061c1f9a7676a1

Comment by Gerrit Updater [ 04/Jan/24 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53594
Subject: LU-10391 tests: debug ping failures
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: b749bc8be9af0c0133dd069c568f15e731718c54

Comment by Gerrit Updater [ 04/Jan/24 ]

"Andreas Dilger <adilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53597
Subject: LU-10391 lnet: use GFP_ATOMIC for alloc under spinlock
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: f2aa2ca04ccf0980b4897ef7823847ce52eecf5d

Comment by Andreas Dilger [ 04/Jan/24 ]

It doesn't fail all the time so I suspect its a timing issue with the test.

It fails about 90% of the runs since the patch landed, so it should be possible to debug fairly easily.

The recent patch about GFP_ATOMIC doesn't have anything to do with this sanity-lnet failure, but just fixing a longstanding problem in lnet_discover() that shows up in Gerrit Janitor testing.

Comment by Gerrit Updater [ 05/Jan/24 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53593/
Subject: LU-10391 tests: except sanity-lnet/253+254 due to failures
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 3bfc335b04500041adf4d6ea19a6d6c52520605e

Comment by Gerrit Updater [ 10/Jan/24 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53571/
Subject: LU-10391 mgs: handle strscpy return value properly
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 015f645651173fbc65ee21eadb14470881d4d512

Comment by Gerrit Updater [ 18/Jan/24 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53597/
Subject: LU-10391 lnet: use GFP_ATOMIC for alloc under spinlock
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 7d9643b5b9373b1f89caa64c92c624f4b68a08b3

Comment by Gerrit Updater [ 18/Jan/24 ]

"Chris Horn <chris.horn@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53727
Subject: LU-10391 tests: Skip tests lacking large NID support
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 39e9095a21401d0e1956bab00a69aedbdce29e2d

Comment by Gerrit Updater [ 18/Jan/24 ]

"Chris Horn <chris.horn@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53728
Subject: LU-10391 tests: Update sanity-lnet router tests for IPv6
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: a8d631f71a7c3d574799023972a6342c776c6f5c

Comment by Gerrit Updater [ 18/Jan/24 ]

"Chris Horn <chris.horn@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53729
Subject: LU-10391 lnet: Add yaml event helpers
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 8005abd8f73d5a8fd323c5fd07740cba03a91fe8

Comment by Gerrit Updater [ 18/Jan/24 ]

"Chris Horn <chris.horn@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53730
Subject: LU-10391 lnet: Add helper for netlink initialization
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 745a348f0fa344b637a28073a4fb9e5553b71ff5

Comment by Gerrit Updater [ 22/Jan/24 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53761
Subject: LU-10391 ksocklnd: support IPv6 for link state
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: ee1694cb0cdaa9c688bf0482d6f21bc8cade6931

Comment by Gerrit Updater [ 23/Jan/24 ]

"Chris Horn <chris.horn@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53791
Subject: LU-10391 lnet: Fix lnetctl peer set --all
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: c95dfcb87363789c96884ddbe7a3fd9f8b1bd14f

Comment by Gerrit Updater [ 01/Feb/24 ]

"Chris Horn <chris.horn@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53882
Subject: LU-10391 obd: Update lmd_parse to handle IPv6 NIDs
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: ed6d9689aceed0218b7cf04f94cc94d296b4721d

Comment by Gerrit Updater [ 06/Feb/24 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53946
Subject: LU-10391 tests: collect gdb backtrace
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0016c918d201debc08457a7a11815eff43f80da0

Generated at Sat Feb 10 02:34:38 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.