https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 https://jira.whamcloud.com/browse/LU-7593 Lustre <p> tgt_client_del() and<br/> ofd_commitrw_write-&gt;tgt_last_rcvd_update<br/> take transaction and ted-&gt;ted_lcd_lock<br/> in different order:</p> <p> thread1:<br/> osd_trans_start<br/> tgt_client_data_update<br/> tgt_client_del &lt;&lt;&lt; mutex_lock(&amp;ted-&gt;ted_lcd_lock);<br/> ofd_obd_disconnect<br/> class_disconnect_export_list<br/> class_disconnect_exports<br/> class_cleanup<br/> ...<br/> sys_umount</p> <p> thread2:<br/> __mutex_lock_slowpath<br/> mutex_lock &lt;&lt;&lt; mutex_lock(&amp;ted-&gt;ted_lcd_lock);<br/> tgt_last_rcvd_update<br/> tgt_txn_stop_cb<br/> dt_txn_hook_stop<br/> osd_trans_stop<br/> ofd_trans_stop<br/> ofd_commitrw_write<br/> ...<br/> tgt_brw_write</p> LU-7593

umount vs tgt_last_rcvd_update deadlock

Bug Major Resolved Fixed nasf Andriy Skulysh bgti patch Tue, 22 Dec 2015 11:38:39 +0000 Mon, 27 May 2019 16:23:08 +0000 Tue, 25 Oct 2016 03:13:38 +0000 Lustre 2.9.0 0 5 <p>Andriy Skulysh (andriy.skulysh@seagate.com) uploaded a new patch: <a href="http://review.whamcloud.com/17703" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/17703</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-7593" title="umount vs tgt_last_rcvd_update deadlock" class="issue-link" data-issue-key="LU-7593"><del>LU-7593</del></a> target: umount vs tgt_last_rcvd_update deadlock<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: f3e97908af29d76962bd20542f7c20ff9e4c3fe8</p> <p>Andriy Skulysh (andriy.skulysh@seagate.com) uploaded a new patch: <a href="http://review.whamcloud.com/17704" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/17704</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-7593" title="umount vs tgt_last_rcvd_update deadlock" class="issue-link" data-issue-key="LU-7593"><del>LU-7593</del></a> target: umount vs tgt_last_rcvd_update deadlock<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 2c3f38babef3a657e5a6e0dc022ec273ae0447a1</p> <p>Andriy</p> <p>Could you please elaborate as to why there are two patches here? Was this issue hit in testing on master?</p> <p>Peter</p> <p>I forgot to add a test for the deadlock. Second patch contains both test and fix</p> <p>Ah I see. So only the second patch is intended to land and the first one will be abandoned?</p> <p>Yes, only second patch is needed.</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/17704/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/17704/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-7593" title="umount vs tgt_last_rcvd_update deadlock" class="issue-link" data-issue-key="LU-7593"><del>LU-7593</del></a> target: umount vs tgt_last_rcvd_update deadlock<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 9197849bc761f7fbbdee65916ad1346eec79d412</p> <p>Landed for 2.9</p> <blockquote> <p>I forgot to add a test for the deadlock. Second patch contains both test and fix</p></blockquote> <p>The 2nd patch <a href="http://review.whamcloud.com/#/c/17704/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/#/c/17704/</a> does NOT contain test. So I think you refer to the 1st patch <a href="http://review.whamcloud.com/#/c/17703/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/#/c/17703/</a> ?<br/> And the two patches looks quite different.</p> <p>Fan Yong (fan.yong@intel.com) uploaded a new patch: <a href="http://review.whamcloud.com/23129" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/23129</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-7593" title="umount vs tgt_last_rcvd_update deadlock" class="issue-link" data-issue-key="LU-7593"><del>LU-7593</del></a> target: take ted_lcd_lock after transaction started<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 57ab9916062248a102bbe7938509c41390c67e0c</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/23129/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/23129/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-7593" title="umount vs tgt_last_rcvd_update deadlock" class="issue-link" data-issue-key="LU-7593"><del>LU-7593</del></a> target: take ted_lcd_lock after transaction started<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: cbca8055b75d084bc6554d1a06f3cd8ccf2f8c09</p> <p>The patch <a href="http://review.whamcloud.com/23129/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/23129/</a> has been landed to Lustre-2.9</p> Duplicate Related Development Rank 1|hzxwkf: Rank (Obsolete) 9223372036854775807 Severity