[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: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| 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 :
|
| 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 |
| Comment by Gerrit Updater [ 22/Jan/20 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37303 |
| Comment by Gerrit Updater [ 20/Feb/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37302/ |
| Comment by Gerrit Updater [ 20/Feb/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37303/ |
| Comment by Gerrit Updater [ 24/Feb/20 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37694 |
| Comment by Gerrit Updater [ 24/Feb/20 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37695 |
| Comment by Gerrit Updater [ 25/Feb/20 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/37699 |
| Comment by Gerrit Updater [ 25/Feb/20 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37700 connect |
| Comment by Gerrit Updater [ 25/Feb/20 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37701 |
| Comment by Gerrit Updater [ 25/Feb/20 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37702 |
| Comment by Gerrit Updater [ 25/Feb/20 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37703 |
| Comment by Gerrit Updater [ 25/Feb/20 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37704 |
| Comment by Gerrit Updater [ 25/Feb/20 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37705 |
| Comment by Gerrit Updater [ 25/Feb/20 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/37706 |
| Comment by Gerrit Updater [ 26/Feb/20 ] |
|
Joseph Gmitter (jgmitter@whamcloud.com) merged in patch https://review.whamcloud.com/37699/ |
| Comment by Gerrit Updater [ 11/Mar/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37694/ |
| Comment by Gerrit Updater [ 24/Mar/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37700/ connect |
| Comment by Gerrit Updater [ 24/Mar/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37701/ |
| Comment by Gerrit Updater [ 14/Apr/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37695/ |
| Comment by Gerrit Updater [ 14/Apr/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37702/ |
| Comment by Gerrit Updater [ 20/Apr/20 ] |
|
James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/38287 |
| Comment by Gerrit Updater [ 23/Apr/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37703/ |
| Comment by Gerrit Updater [ 29/Apr/20 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/38408 |
| Comment by Gerrit Updater [ 01/May/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38287/ |
| Comment by Gerrit Updater [ 03/Jun/20 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/38823 |
| Comment by Gerrit Updater [ 10/Jun/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38823/ |
| Comment by Gerrit Updater [ 15/Jul/20 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/39373 |
| Comment by Gerrit Updater [ 26/Feb/21 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37704/ |
| Comment by Gerrit Updater [ 26/Feb/21 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37705/ |
| Comment by Gerrit Updater [ 26/Feb/21 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37706/ |
| Comment by Gerrit Updater [ 16/Mar/21 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38408/ |
| 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 |
| Comment by Gerrit Updater [ 19/Mar/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/42101 |
| Comment by Gerrit Updater [ 19/Mar/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/42102 |
| Comment by Gerrit Updater [ 19/Mar/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/42103 |
| Comment by Gerrit Updater [ 19/Mar/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/42104 |
| Comment by Gerrit Updater [ 19/Mar/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/42105 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43602 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43601 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43604 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43599 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43596 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43598 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43603 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43600 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43594 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43595 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43607 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43593 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43605 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43606 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43597 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43608 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43609 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43616 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43612 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43610 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43611 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43613 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43614 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43617 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43618 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43615 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43621 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43619 |
| Comment by Gerrit Updater [ 10/May/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43620 |
| Comment by Gerrit Updater [ 08/Jun/21 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/43942 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44620 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44621 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44622 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44623 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44624 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44625 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44626 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44627 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44628 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44629 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44630 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44631 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44632 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44633 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44634 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44635 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44636 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44637 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44638 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44639 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44640 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44641 |
| Comment by Gerrit Updater [ 12/Aug/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44642 |
| Comment by Gerrit Updater [ 31/Aug/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/42100/ |
| Comment by Gerrit Updater [ 31/Aug/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43942/ |
| Comment by Gerrit Updater [ 31/Aug/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/42101/ |
| Comment by Gerrit Updater [ 31/Aug/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/42102/ |
| Comment by Gerrit Updater [ 31/Aug/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44620/ |
| Comment by Gerrit Updater [ 31/Aug/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/42103/ |
| Comment by Gerrit Updater [ 31/Aug/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/42104/ |
| Comment by Gerrit Updater [ 31/Aug/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/42105/ |
| Comment by Gerrit Updater [ 31/Aug/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43593/ |
| Comment by Gerrit Updater [ 31/Aug/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43594/ |
| Comment by Gerrit Updater [ 03/Sep/21 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/44838 |
| Comment by Gerrit Updater [ 03/Sep/21 ] |
|
"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/44844 |
| Comment by Gerrit Updater [ 20/Nov/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43595/ |
| Comment by Gerrit Updater [ 20/Nov/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43596/ |
| Comment by Gerrit Updater [ 20/Nov/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43597/ |
| 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/ |
| Comment by Gerrit Updater [ 18/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43598/ |
| Comment by Gerrit Updater [ 18/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43599/ |
| Comment by Gerrit Updater [ 18/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43600/ |
| Comment by Gerrit Updater [ 18/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43601/ |
| Comment by Gerrit Updater [ 18/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43602/ |
| Comment by Gerrit Updater [ 26/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43603/ |
| Comment by Gerrit Updater [ 26/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43604/ |
| Comment by Gerrit Updater [ 26/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43605/ |
| Comment by Gerrit Updater [ 26/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43606/ |
| Comment by Gerrit Updater [ 26/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43607/ |
| Comment by Gerrit Updater [ 26/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43608/ |
| Comment by Gerrit Updater [ 26/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43609/ |
| Comment by Gerrit Updater [ 26/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43610/ |
| Comment by Gerrit Updater [ 26/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43611/ |
| Comment by Gerrit Updater [ 26/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43612/ |
| Comment by Gerrit Updater [ 26/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43613/ |
| Comment by Gerrit Updater [ 26/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43614/ |
| Comment by Gerrit Updater [ 26/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44838/ |
| Comment by Gerrit Updater [ 26/Jan/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43615/ |
| Comment by Gerrit Updater [ 30/May/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43616/ |
| Comment by Gerrit Updater [ 30/May/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43617/ |
| Comment by Gerrit Updater [ 30/May/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43618/ |
| Comment by Gerrit Updater [ 30/May/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43619/ |
| Comment by Gerrit Updater [ 30/May/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43620/ |
| Comment by Gerrit Updater [ 30/May/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/43621/ |
| Comment by Gerrit Updater [ 30/May/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44621/ |
| Comment by Gerrit Updater [ 30/May/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44622/ |
| Comment by Gerrit Updater [ 30/May/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44623/ |
| Comment by Gerrit Updater [ 11/Jul/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44624/ |
| Comment by Gerrit Updater [ 11/Jul/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44625/ |
| 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/ |
| 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 |
| Comment by Gerrit Updater [ 16/Sep/22 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/48570 |
| Comment by Gerrit Updater [ 16/Sep/22 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/48571 |
| Comment by Gerrit Updater [ 16/Sep/22 ] |
|
"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/48572 |
| 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 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 This is ping the local NID and a remote NID. |
| Comment by Andreas Dilger [ 24/Sep/22 ] |
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 |
| Comment by Gerrit Updater [ 04/Oct/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44627/ |
| Comment by Gerrit Updater [ 04/Oct/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48569/ |
| Comment by Gerrit Updater [ 04/Oct/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48570/ |
| Comment by Gerrit Updater [ 04/Oct/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48571/ |
| Comment by Gerrit Updater [ 10/Oct/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48685/ |
| Comment by Gerrit Updater [ 15/Oct/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48572/ |
| 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 |
| Comment by Gerrit Updater [ 08/Nov/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44628/ |
| Comment by Gerrit Updater [ 08/Nov/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48990/ |
| 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/ |
| Comment by Gerrit Updater [ 17/Nov/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44632/ |
| Comment by Gerrit Updater [ 17/Nov/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44633/ |
| Comment by Gerrit Updater [ 17/Nov/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44634/ |
| Comment by Gerrit Updater [ 17/Nov/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44635/ |
| 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 |
| Comment by Gerrit Updater [ 29/Nov/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44630/ |
| Comment by Gerrit Updater [ 29/Nov/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/44631/ |
| 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 ( 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/ |
| 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 |
| 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, |
| 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. 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| Comment by Gerrit Updater [ 10/Mar/23 ] |
|
"jsimmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50253 |
| Comment by Gerrit Updater [ 10/Mar/23 ] |
|
"jsimmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50254 |
| Comment by Gerrit Updater [ 10/Mar/23 ] |
|
"jsimmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50255 |
| 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
More work is need for the following but you use them for general testing. https://review.whamcloud.com/#/c/fs/lustre-release/+49361 |
| 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 |
| 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/ |
| Comment by Gerrit Updater [ 11/Apr/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50086/ |
| Comment by Gerrit Updater [ 11/Apr/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50088/ |
| Comment by Gerrit Updater [ 11/Apr/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50089/ |
| Comment by Gerrit Updater [ 11/Apr/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50090/ |
| Comment by Gerrit Updater [ 11/Apr/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50091/ |
| Comment by Gerrit Updater [ 11/Apr/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50100/ |
| 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/ |
| Comment by Gerrit Updater [ 22/Apr/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50095/ |
| Comment by Gerrit Updater [ 22/Apr/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50098/ |
| Comment by Gerrit Updater [ 22/Apr/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50101/ |
| 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 |
| Comment by Gerrit Updater [ 26/Apr/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50097/ |
| 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 |
| Comment by Gerrit Updater [ 09/May/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50716/ |
| 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/ |
| Comment by Gerrit Updater [ 31/May/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50102/ |
| Comment by James A Simmons [ 08/Jun/23 ] |
|
Status update:
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/ |
| 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 |
| Comment by Gerrit Updater [ 24/Aug/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50362/ |
| 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/ |
| Comment by Gerrit Updater [ 25/Oct/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/49574/ |
| 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 |
| Comment by Gerrit Updater [ 03/Nov/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50254/ |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| Comment by Gerrit Updater [ 08/Nov/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50253/ |
| Comment by Gerrit Updater [ 08/Nov/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/52911/ |
| 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 |
| 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 |
| 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 |
| Comment by Gerrit Updater [ 18/Nov/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53013/ |
| Comment by Gerrit Updater [ 18/Nov/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53016/ |
| Comment by Gerrit Updater [ 18/Nov/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53019/ |
| Comment by Gerrit Updater [ 18/Nov/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53020/ |
| Comment by Gerrit Updater [ 29/Nov/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53018/ |
| Comment by Gerrit Updater [ 29/Nov/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53115/ |
| Comment by Gerrit Updater [ 29/Nov/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53150/ |
| 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 |
| 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 |
| Comment by Gerrit Updater [ 13/Dec/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53117/ |
| 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/ |
| 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 |
| 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 |
| Comment by Gerrit Updater [ 03/Jan/24 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/49361/ |
| Comment by Gerrit Updater [ 04/Jan/24 ] |
|
|
| 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: /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 |
| 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 |
| 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 |
| Comment by Andreas Dilger [ 04/Jan/24 ] |
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/ |
| Comment by Gerrit Updater [ 10/Jan/24 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53571/ |
| Comment by Gerrit Updater [ 18/Jan/24 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53597/ |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |