Whamcloud Community JIRA

Whamcloud Community JIRA https://jira.whamcloud.com This file is an XML representation of an issue en-us

9.4.14

940014

05-12-2023

[LU-9416] lmv_hsm_ct_register() should add kkuc before sending registration RPCs https://jira.whamcloud.com/browse/LU-9416 Lustre LU-9416

lmv_hsm_ct_register() should add kkuc before sending registration RPCs

Bug Minor Resolved Fixed Henri Doreau John Hammond hsm Fri, 28 Apr 2017 15:30:18 +0000 Tue, 24 Oct 2017 21:48:33 +0000 Thu, 19 Oct 2017 11:57:23 +0000 Lustre 2.11.0 Lustre 2.10.2 0 4 Henri Doreau (henri.doreau@cea.fr) uploaded a new patch: <a href="https://review.whamcloud.com/28751" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/28751</a> Subject: <a href="https://jira.whamcloud.com/browse/LU-9416" title="lmv_hsm_ct_register() should add kkuc before sending registration RPCs" class="issue-link" data-issue-key="LU-9416"><del>LU-9416</del></a> hsm: add kkuc before sending registration RPCs Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: f76ade97e83d909f0247c93ae4e560ff2b442b0b John, Can you at least comment/detail here about the main reason/need for this change ??!! Could this be to avoid that some KUC msgs/requests from Server to Client could be lost, or to cause other inconvenient, during some racy cases ??... Or simply to do local operations first and thus prevent from doing useless remote/MDTs registrations in case of failure ? > Could this be to avoid that some KUC msgs/requests from Server to Client could be lost, or to cause other inconvenient, during some racy cases ?? Correct. I have seen in sanity-hsm runs where the resistration completed and the CDTs sent HSM actions just before the kkuc registration happens. In this case the client drops the actions because there are no CT pipes in the KKUC list. Then the CDT must wait and resent the actions causing a 200+ second delay. So this should be kind of a follow-on to my previous patch for <a href="https://jira.whamcloud.com/browse/LU-9038" title="sanity-hsm: kernel panic in lbcfs_kkuc_group_put()" class="issue-link" data-issue-key="LU-9038"><del>LU-9038</del></a>, that fixed the crashes that could occur before and during the same race but introduced this retry mechanism from CDT, right... Correct. Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/28751/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/28751/</a> Subject: <a href="https://jira.whamcloud.com/browse/LU-9416" title="lmv_hsm_ct_register() should add kkuc before sending registration RPCs" class="issue-link" data-issue-key="LU-9416"><del>LU-9416</del></a> hsm: add kkuc before sending registration RPCs Project: fs/lustre-release Branch: master Current Patch Set: Commit: bdb04079570dd2de3b9310221e0206d64efb9ecc Landed for 2.11 Minh Diep (minh.diep@intel.com) uploaded a new patch: <a href="https://review.whamcloud.com/29687" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/29687</a> Subject: <a href="https://jira.whamcloud.com/browse/LU-9416" title="lmv_hsm_ct_register() should add kkuc before sending registration RPCs" class="issue-link" data-issue-key="LU-9416"><del>LU-9416</del></a> hsm: add kkuc before sending registration RPCs Project: fs/lustre-release Branch: b2_10 Current Patch Set: 1 Commit: 45b51a053c014a11b49768757ab6d08103f8547e John L. Hammond (john.hammond@intel.com) merged in patch <a href="https://review.whamcloud.com/29687/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/29687/</a> Subject: <a href="https://jira.whamcloud.com/browse/LU-9416" title="lmv_hsm_ct_register() should add kkuc before sending registration RPCs" class="issue-link" data-issue-key="LU-9416"><del>LU-9416</del></a> hsm: add kkuc before sending registration RPCs Project: fs/lustre-release Branch: b2_10 Current Patch Set: Commit: 46aee8eba9000e458d231c975716feb275b8eb92 Development Rank 1|hzzbgn: Rank (Obsolete) 9223372036854775807 Severity