<!-- 
RSS generated by JIRA (9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c) at Sat Feb 10 02:13:39 UTC 2024

It is possible to restrict the fields that are returned in this document by specifying the 'field' parameter in your request.
For example, to request only the issue key and summary append 'field=key&field=summary' to the URL of your request.
-->
<rss version="0.92" >
<channel>
    <title>Whamcloud Community JIRA</title>
    <link>https://jira.whamcloud.com</link>
    <description>This file is an XML representation of an issue</description>
    <language>en-us</language>    <build-info>
        <version>9.4.14</version>
        <build-number>940014</build-number>
        <build-date>05-12-2023</build-date>
    </build-info>


<item>
            <title>[LU-7988] HSM: high lock contention for cdt_llog_lock</title>
                <link>https://jira.whamcloud.com/browse/LU-7988</link>
                <project id="10000" key="LU">Lustre</project>
                    <description>&lt;p&gt;There is an important locking issue around cdt_llog_lock when adding new HSM requests.&lt;/p&gt;

&lt;div class=&quot;preformatted panel&quot; style=&quot;border-width: 1px;&quot;&gt;&lt;div class=&quot;preformattedContent panelContent&quot;&gt;
&lt;pre&gt;# time wc -l /proc/fs/lustre/mdt/snx11133-MDT0000/hsm/actions
219759 /proc/fs/lustre/mdt/snx11133-MDT0000/hsm/actions

real    11m45.068s
user    0m0.020s
sys     0m21.372s
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;

&lt;p&gt;11 minutes to cat the list is too high. Such operation should take a couple seconds at most.&lt;/p&gt;

&lt;p&gt;The contention appears to come from the coordinator. Every time a new request is posted, the whole list of request is browsed, under that lock. That&apos;s not a problem when there is only a handful of request, but it doesn&apos;t scale when there is hundreds of thousands of them.&lt;/p&gt;

&lt;p&gt;I recompiled a centos 7 kernel with CONFIG_LOCK_STAT on a VM. I ran test creating 10000 files and archiving them without a copytool present. Total time was 146 seconds. Lock contention result:&lt;/p&gt;

&lt;div class=&quot;preformatted panel&quot; style=&quot;border-width: 1px;&quot;&gt;&lt;div class=&quot;preformattedContent panelContent&quot;&gt;
&lt;pre&gt;lock_stat version 0.3
-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
                              class name    con-bounces    contentions   waittime-min   waittime-max waittime-total    acq-bounces   acquisitions   holdtime-min   holdtime-max holdtime-total
-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

[...]

                     &amp;amp;cdt-&amp;gt;cdt_llog_lock:          6296           6296          15.45       23074.17    43436574.06          17791          27134          25.09       37745.03   138558199.24
                     -------------------
                     &amp;amp;cdt-&amp;gt;cdt_llog_lock           6296          [&amp;lt;ffffffffa0fb096d&amp;gt;] cdt_llog_process+0x9d/0x3a0 [mdt]
                     -------------------
                     &amp;amp;cdt-&amp;gt;cdt_llog_lock           6296          [&amp;lt;ffffffffa0fb096d&amp;gt;] cdt_llog_process+0x9d/0x3a0 [mdt]
[...]
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;

&lt;p&gt;(time units are micro-seconds).&lt;/p&gt;

&lt;p&gt;With waittime-total=43 seconds and holdtime-total=138s, this is a very contentious lock, way above the other locks in Lustre or the whole system.&lt;/p&gt;

&lt;p&gt;AFAICS, contention is between these mechanisms:&lt;/p&gt;
&lt;ul class=&quot;alternate&quot; type=&quot;square&quot;&gt;
	&lt;li&gt;adding a new request (lfs hsm_archive, ...)&lt;/li&gt;
	&lt;li&gt;changing a request status (WAITING-&amp;gt;STARTED-&amp;gt;SUCCEED)&lt;/li&gt;
	&lt;li&gt;removing a request (archive completed)&lt;/li&gt;
	&lt;li&gt;housekeeping (coordinator loop every 10 seconds)&lt;/li&gt;
	&lt;li&gt;dumping the list of actions from /proc&lt;/li&gt;
&lt;/ul&gt;


&lt;p&gt;The net result is that when there is a lot of requests, they trickle down to the copytool, exacerbating the problem by increasing the number in the list.&lt;/p&gt;</description>
                <environment></environment>
        <key id="35819">LU-7988</key>
            <summary>HSM: high lock contention for cdt_llog_lock</summary>
                <type id="1" iconUrl="https://jira.whamcloud.com/secure/viewavatar?size=xsmall&amp;avatarId=11303&amp;avatarType=issuetype">Bug</type>
                                            <priority id="2" iconUrl="https://jira.whamcloud.com/images/icons/priorities/critical.svg">Critical</priority>
                        <status id="5" iconUrl="https://jira.whamcloud.com/images/icons/statuses/resolved.png" description="A resolution has been taken, and it is awaiting verification by reporter. From here issues are either reopened, or are closed.">Resolved</status>
                    <statusCategory id="3" key="done" colorName="success"/>
                                    <resolution id="1">Fixed</resolution>
                                        <assignee username="fzago">Frank Zago</assignee>
                                    <reporter username="fzago">Frank Zago</reporter>
                        <labels>
                            <label>cea</label>
                    </labels>
                <created>Tue, 5 Apr 2016 18:20:07 +0000</created>
                <updated>Mon, 21 Jan 2019 15:32:28 +0000</updated>
                            <resolved>Mon, 6 Nov 2017 13:28:24 +0000</resolved>
                                                    <fixVersion>Lustre 2.11.0</fixVersion>
                    <fixVersion>Lustre 2.10.4</fixVersion>
                    <fixVersion>Lustre 2.10.7</fixVersion>
                                        <due></due>
                            <votes>2</votes>
                                    <watches>30</watches>
                                                                            <comments>
                            <comment id="147875" author="gerrit" created="Tue, 5 Apr 2016 18:35:29 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/19341&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19341&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: execute HSM coordinator once per second at most&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 6aa634f1d28e16300dbf428dcec0e38e01b5a2dc&lt;/p&gt;</comment>
                            <comment id="147877" author="fzago" created="Tue, 5 Apr 2016 18:37:36 +0000"  >&lt;p&gt;Above patch is only a partial fix. Maybe executing the loop once per second is too much, and should be configurable too.&lt;/p&gt;</comment>
                            <comment id="148040" author="fzago" created="Wed, 6 Apr 2016 20:20:37 +0000"  >&lt;p&gt;250000 empty files are created on a filesystem, and hsm_archive is done on them. A copytool is then started to perform these archive requests. The MDT has the patch above. &lt;/p&gt;

&lt;p&gt;mdt-cdt-250000.svg shows, the number of records completed per second over the duration of the archival. The max number was 389 at the end. The total duration was 1h:29min:39s. It shows that the processed records is getting faster over time as the backlog queue diminishes.&lt;/p&gt;</comment>
                            <comment id="148041" author="fzago" created="Wed, 6 Apr 2016 20:22:03 +0000"  >&lt;p&gt;As a side note, the creation of these 250000 files + hsm_archive by batch of 10 took 3160 seconds with this patch. Without the patch, it takes 11822 seconds.&lt;/p&gt;</comment>
                            <comment id="148042" author="fzago" created="Wed, 6 Apr 2016 20:30:56 +0000"  >&lt;p&gt;I think the graph shows that things could be improved. For instance, the MDT shouldn&apos;t need to parse the whole backlog of HSM request, as only the first ones (up to /proc/.../hsm/max_requests?) have been seen by a client. The other ones need to be checked regularly possibly for timeout or cancelation, but this is not an operation that needs to be done as frequently.&lt;/p&gt;

&lt;p&gt;In my test setup, we should be able to perform constantly at least 200 requests per seconds, as seen during the last minute, for the duration of the archival (1250 seconds total), instead of the average of 46 per seconds (5379 total).&lt;/p&gt;</comment>
                            <comment id="148986" author="gerrit" created="Thu, 14 Apr 2016 19:45:03 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/19585&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19585&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: defer all new hals&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 756c24f770e569a240d02d9e8b1e94f9ea2ba0a2&lt;/p&gt;</comment>
                            <comment id="148987" author="gerrit" created="Thu, 14 Apr 2016 19:45:05 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/19584&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19584&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: update many cookie status at once&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 1cec5fa2b8eb264736aa7143418e3df9d5878451&lt;/p&gt;</comment>
                            <comment id="148990" author="gerrit" created="Thu, 14 Apr 2016 19:45:10 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/19578&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19578&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: remove hsd.request_sz&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 810bfd58bdf238a7f0f83dc5938f51ad218df921&lt;/p&gt;</comment>
                            <comment id="148991" author="gerrit" created="Thu, 14 Apr 2016 19:45:11 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/19579&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19579&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: Fix possible out of bounds reference in message&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 7bf47d3dc16c7554845aef211bb998d79515c84b&lt;/p&gt;</comment>
                            <comment id="148992" author="gerrit" created="Thu, 14 Apr 2016 19:45:13 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/19580&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19580&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: improve hsd.request reset&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 10119ceb49ed2724fb529760d4127ac3ff7cc1d6&lt;/p&gt;</comment>
                            <comment id="148993" author="gerrit" created="Thu, 14 Apr 2016 19:45:15 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/19581&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19581&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: only browse entries of hsd.request when needed&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: d0d0e913f927c9843d0354fbad89ad2bcdf2586e&lt;/p&gt;</comment>
                            <comment id="148994" author="gerrit" created="Thu, 14 Apr 2016 19:45:17 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/19582&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19582&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: added coordinator housekeeping flag&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 6305a604bc97f563e8a084a70f2e14e62121cef4&lt;/p&gt;</comment>
                            <comment id="148995" author="gerrit" created="Thu, 14 Apr 2016 19:45:19 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/19583&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19583&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: remove compound_id from mdt_hsm_add_actions proto&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: feec067b250099a1f051b6762bab37db3495a1e5&lt;/p&gt;</comment>
                            <comment id="148998" author="fzago" created="Thu, 14 Apr 2016 19:56:55 +0000"  >&lt;p&gt;The above series is trying to fix the issue by deferring everything that can be, so instead of calling cdt_llog_process for everything every time some new work has to be done, requests are queued and the coordinator will process them once per second. This vastly improves performance.&lt;/p&gt;

&lt;p&gt;With the patches:&lt;/p&gt;
&lt;ul class=&quot;alternate&quot; type=&quot;square&quot;&gt;
	&lt;li&gt;creating 250,000 files and hsm_archive them, without a copytool: 1400s (when hsm_archive one by one), or 844s (when hsm_archive by batches of 40).&lt;/li&gt;
	&lt;li&gt;start copytool to archive the files: 876s&lt;/li&gt;
	&lt;li&gt;stop the copytool and hsm_release the files: 948s by batches of 40&lt;/li&gt;
	&lt;li&gt;hsm_restore the files, (without a copytool): 1539s&lt;/li&gt;
	&lt;li&gt;start copytool to restore the files: 1004s&lt;/li&gt;
&lt;/ul&gt;


&lt;p&gt;Tested on a VM, with Cray&apos;s copytool.&lt;/p&gt;

&lt;p&gt;The last patch (&quot;hsm: defer all new hals&quot;) is changing the error path and may introduce some different behaviour when error happen during restoration.&lt;/p&gt;</comment>
                            <comment id="149019" author="adilger" created="Thu, 14 Apr 2016 23:26:35 +0000"  >&lt;p&gt;Frank, I was wondering what you were looking for when submitting this patch series?  All of the patches are submitted with &lt;tt&gt;Test&amp;#45;Parameters: forbuildonly&lt;/tt&gt; which means that they will only be built but never tested.  I&apos;d have thought possibly the last patch wouldn&apos;t have this keyword so that you could get some testing on the patch series.  Alternately, you could use &lt;tt&gt;Test-Paramters: fortestonly testlist=sanity-hsm&lt;/tt&gt; to run only the sanity-hsm test on each of the patches?&lt;/p&gt;

&lt;p&gt;Also, the patch series was submitted twice, which clogged up the build queue a bit.  I killed the remaining builds for the first version once I noticed, and all but the last ones for the second series, since they will never be tested.  At least you can verify that the full patch series builds on all the architectures.&lt;/p&gt;</comment>
                            <comment id="149083" author="fzago" created="Fri, 15 Apr 2016 14:41:49 +0000"  >&lt;p&gt;Thanks Andreas. I wanted some reviews before starting the testing on these patches, which may clog your test machines. I&apos;ve added testlist=sanity-hsm to the last one as you suggested.&lt;/p&gt;</comment>
                            <comment id="149339" author="fzago" created="Mon, 18 Apr 2016 22:28:55 +0000"  >&lt;p&gt;Dropping &lt;a href=&quot;http://review.whamcloud.com/19585&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19585&lt;/a&gt; as it cannot work efficiently within the current constraints. It&apos;s mostly related to restoring files as a lot of checking is done when the request is submitted and the layout locked. If we delay, then  a program accessing a release file with see a ENODATA failure since the layout is not locked. This was the reason for sanity-hsm tests 10c and 11b to fail.&lt;/p&gt;

&lt;p&gt;Maybe a tinier optimization is to only queue the archive operations, which in the real world, an HSM system would mostly see. Every other operations would act as a barrier.&lt;/p&gt;</comment>
                            <comment id="150591" author="gerrit" created="Fri, 29 Apr 2016 16:40:41 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/19886&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19886&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: defer all new hals&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 96b64c150463c52314689af06222b557f7241631&lt;/p&gt;</comment>
                            <comment id="150592" author="fzago" created="Fri, 29 Apr 2016 16:49:59 +0000"  >&lt;p&gt;The more I look at the coordinator code and the more I feel that using HAls internally is the wrong way to go. They should only be used for communication between nodes.&lt;/p&gt;

&lt;p&gt;When the MDT receives a message from a client, it makes a HAL out of it, which includes one or more HAI. Eventually most HAI are allocated again in mdt_agent_record_add() as a struct llog_agent_req_rec with the same info, so we might as well start there.&lt;/p&gt;</comment>
                            <comment id="150908" author="fzago" created="Tue, 3 May 2016 21:54:34 +0000"  >&lt;p&gt;Current series can perform 237 archive operation per seconds. The setup is a single VM with Lustre+Cray copytool; 0-sized files are created, hsm_archive&apos;d and archived immediately by Cray&apos;s copytool.&lt;/p&gt;

&lt;p&gt;The bottleneck is still the processing of the requests in the coordinator, and very likely the root issue is the catalog.&lt;/p&gt;</comment>
                            <comment id="151444" author="vinayakh" created="Mon, 9 May 2016 06:00:38 +0000"  >&lt;p&gt;Hello Frank, Andreas,&lt;/p&gt;

&lt;p&gt;&lt;a href=&quot;http://review.whamcloud.com/#/c/19578/:&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/#/c/19578/:&lt;/a&gt; Tagged build only&lt;br/&gt;
&lt;a href=&quot;http://review.whamcloud.com/#/c/19579/:&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/#/c/19579/:&lt;/a&gt; Tagged build only&lt;br/&gt;
&lt;a href=&quot;http://review.whamcloud.com/#/c/19580/:&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/#/c/19580/:&lt;/a&gt; Tagged build only&lt;br/&gt;
&lt;a href=&quot;http://review.whamcloud.com/#/c/19581/:&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/#/c/19581/:&lt;/a&gt; Tagged build only&lt;br/&gt;
&lt;a href=&quot;http://review.whamcloud.com/#/c/19582/:&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/#/c/19582/:&lt;/a&gt; Tagged build only&lt;br/&gt;
&lt;a href=&quot;http://review.whamcloud.com/#/c/19583/:&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/#/c/19583/:&lt;/a&gt; Tagged build only&lt;br/&gt;
&lt;a href=&quot;http://review.whamcloud.com/#/c/19584/:&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/#/c/19584/:&lt;/a&gt; Tagged build only&lt;br/&gt;
&lt;a href=&quot;http://review.whamcloud.com/#/c/19585/:&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/#/c/19585/:&lt;/a&gt; abandoned&lt;/p&gt;

&lt;p&gt;&lt;a href=&quot;http://review.whamcloud.com/#/c/19886/:&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/#/c/19886/:&lt;/a&gt; Not containing build only tag&lt;/p&gt;

&lt;p&gt;1. The final &lt;a href=&quot;http://review.whamcloud.com/#/c/19886/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;patch&lt;/a&gt; dependent on all the patches mentioned above with &lt;b&gt;Tagged&lt;/b&gt; &lt;b&gt;build&lt;/b&gt; &lt;b&gt;only&lt;/b&gt; except &lt;b&gt;abandoned&lt;/b&gt; one. Am I right ?&lt;/p&gt;

&lt;p&gt;2. Also please let me know if I can help you in verification process or anything which will boost review process. &lt;/p&gt;

&lt;p&gt;3. Is it not a good idea to submit the single patch aggregate of all to avoid series of conflicts ?&lt;/p&gt;

&lt;p&gt;Thanks,&lt;/p&gt;
</comment>
                            <comment id="151495" author="fzago" created="Mon, 9 May 2016 15:50:28 +0000"  >&lt;p&gt;Hi Vinayak,&lt;/p&gt;

&lt;blockquote&gt;&lt;p&gt;1. The final patch dependent on all the patches mentioned above with Tagged build only except abandoned one. Am I right ?&lt;/p&gt;&lt;/blockquote&gt;

&lt;p&gt;Yes.&lt;/p&gt;

&lt;blockquote&gt;&lt;p&gt;2. Also please let me know if I can help you in verification process or anything which will boost review process.&lt;/p&gt;&lt;/blockquote&gt;

&lt;p&gt;As this series changes the way HSM requests are processed, care should be made to ensure the changes make sense, and no undesired behaviour is introduced.&lt;/p&gt;

&lt;blockquote&gt;&lt;p&gt;3. Is it not a good idea to submit the single patch aggregate of all to avoid series of conflicts ?&lt;/p&gt;&lt;/blockquote&gt;

&lt;p&gt;It is much easier to review a series a patches that do separate things that a big one that does many things.&lt;/p&gt;</comment>
                            <comment id="151599" author="vinayakh" created="Tue, 10 May 2016 05:43:27 +0000"  >&lt;blockquote&gt;&lt;p&gt;As this series changes the way HSM requests are processed, care should be made to ensure the changes make sense, and no undesired behaviour is introduced.&lt;/p&gt;&lt;/blockquote&gt;
&lt;p&gt;Okay Frank. I will port the changes to our local branch, create dependency, run the hsm tests and report you.&lt;/p&gt;</comment>
                            <comment id="152599" author="gerrit" created="Tue, 17 May 2016 18:28:45 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/20272&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/20272&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: split mdt_hsm_add_actions()&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: b37bd734053f281a54f8eee6a3e9a589d64823dd&lt;/p&gt;</comment>
                            <comment id="154862" author="vinayakh" created="Tue, 7 Jun 2016 03:27:32 +0000"  >&lt;p&gt;Hello Frank,&lt;/p&gt;

&lt;p&gt;Sorry, I am heavily occupied with other work. I did not get time to test the patch set.&lt;/p&gt;

&lt;p&gt;Thanks,&lt;/p&gt;</comment>
                            <comment id="156841" author="vinayakh" created="Fri, 24 Jun 2016 06:01:40 +0000"  >&lt;p&gt;Hello Frank,&lt;/p&gt;

&lt;p&gt;I have tried to reproduce this issue on master(without the patch) using following script by creating 1000 files (I thought 1000 files would take some big time) but it did not take much time for me. Finished in few fraction seconds. btw, I am trying it on single node set up with 1 MDS and 2 OSTs. Please let me know if I am missing anything.&lt;/p&gt;
&lt;blockquote&gt;
&lt;ol&gt;
	&lt;li&gt;time wc -l /proc/fs/lustre/mdt/lustre-MDT0000/hsm/actions&lt;br/&gt;
real	0m0.013s&lt;br/&gt;
user	0m0.000s&lt;br/&gt;
sys	0m0.012s&lt;/li&gt;
&lt;/ol&gt;
&lt;/blockquote&gt;

&lt;div class=&quot;preformatted panel&quot; style=&quot;border-width: 1px;&quot;&gt;&lt;div class=&quot;preformattedContent panelContent&quot;&gt;
&lt;pre&gt;        numfiles=1000

        echo &quot;Creating $numfiles files&quot;

        for i in $(seq 1 $numfiles); do
                make_small $DIR/f501.sanity-hsm$i
        done

        local fid=$(path2fid $DIR/f501.sanity-hsm$numfiles)

        echo &quot;fid of f501.sanity-hsm$numfiles = $fid&quot; 

        echo &quot;Archiving the files&quot;

        for i in $(seq 1 $numfiles); do
                $LFS hsm_archive $DIR/f501.sanity-hsm$i
        done
      
        sleep 5

        local time=$(time wc -l /proc/fs/lustre/mdt/lustre-MDT0000/hsm/actions)
    
        wait_request_state $fid ARCHIVE SUCCEED

&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;

&lt;p&gt;Thanks,&lt;/p&gt;</comment>
                            <comment id="156867" author="fzago" created="Fri, 24 Jun 2016 15:59:42 +0000"  >&lt;p&gt;Hi Vinayak,&lt;/p&gt;

&lt;p&gt;You  need a lot more files. Go with at least 100000. I usually test with 250000, and got to 1M files. Let the queue fill up then start the copytool. Then you&apos;ll see how slow files are being processed. Use small files (eg 1 byte) so the copytool isn&apos;t the choking point.&lt;/p&gt;

&lt;p&gt;Here&apos;s a script I use:&lt;/p&gt;
&lt;div class=&quot;preformatted panel&quot; style=&quot;border-width: 1px;&quot;&gt;&lt;div class=&quot;preformattedContent panelContent&quot;&gt;
&lt;pre&gt;#!/bin/sh

mkdir /mnt/lustre/foo

start=$(date &quot;+%s&quot;)

echo Start $start

for x in {1..6250}; do

        echo loop $x $(($(date &quot;+%s&quot;) - start)) seconds since start  $((x * 40 / ($(date &quot;+%s&quot;) - start + 1) ))

        mkdir /mnt/lustre/foo/d$x
        cd /mnt/lustre/foo/d$x

        touch f-{1..40}

        lfs hsm_archive f-{1..40}
done

echo Done $start to $(date &quot;+%s&quot;)
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;

&lt;p&gt;With that kind of setup, you may run into &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-5228&quot; title=&quot;HSM: posix copytool can (and do) run out of file descriptors&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-5228&quot;&gt;LU-5228&lt;/a&gt; too.&lt;/p&gt;</comment>
                            <comment id="156964" author="vinayakh" created="Mon, 27 Jun 2016 04:03:54 +0000"  >&lt;p&gt;Thanks Frank. I will try that.&lt;/p&gt;</comment>
                            <comment id="157235" author="vinayakh" created="Wed, 29 Jun 2016 08:49:52 +0000"  >&lt;p&gt;Hello Frank,&lt;/p&gt;

&lt;p&gt;Update: &lt;/p&gt;

&lt;p&gt;With the reproducer I could able to experience the delay of 2 mins 33.750 secs with 250,000 files.&lt;/p&gt;

&lt;blockquote&gt;
&lt;ol&gt;
	&lt;li&gt;time wc -l  /proc/fs/lustre/mdt/lustre-MDT0000/hsm/actions&lt;br/&gt;
145678 /proc/fs/lustre/mdt/lustre-MDT0000/hsm/actions&lt;/li&gt;
&lt;/ol&gt;


&lt;p&gt;real    2m33.750s&lt;br/&gt;
user    0m0.021s&lt;br/&gt;
sys    0m0.795s&lt;/p&gt;&lt;/blockquote&gt;

&lt;p&gt;Tried the same reproducer with &lt;a href=&quot;http://review.whamcloud.com/#/c/19341/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;patch1&lt;/a&gt; , &lt;a href=&quot;http://review.whamcloud.com/#/c/19585/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;patch2&lt;/a&gt; and observed the delay of 56.946 secs which is better value.&lt;/p&gt;

&lt;blockquote&gt;
&lt;ol&gt;
	&lt;li&gt;time wc -l /proc/fs/lustre/mdt/lustre-MDT0000/hsm/actions&lt;br/&gt;
163911 /proc/fs/lustre/mdt/lustre-MDT0000/hsm/actions&lt;/li&gt;
&lt;/ol&gt;


&lt;p&gt;real    0m56.946s&lt;br/&gt;
user    0m0.026s&lt;br/&gt;
sys    0m0.936s&lt;/p&gt;&lt;/blockquote&gt;

&lt;p&gt;Thanks,&lt;/p&gt;</comment>
                            <comment id="157283" author="fzago" created="Wed, 29 Jun 2016 16:21:06 +0000"  >&lt;p&gt;Hi Vinayak,&lt;br/&gt;
I dropped the patch2 you mentioned above a while ago because of incorrect assumptions I made. The new patch is &lt;a href=&quot;http://review.whamcloud.com/#/c/19886&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/#/c/19886&lt;/a&gt;&lt;/p&gt;</comment>
                            <comment id="157351" author="vinayakh" created="Thu, 30 Jun 2016 04:29:55 +0000"  >&lt;p&gt;Hello Frank,&lt;/p&gt;

&lt;p&gt;Sorry for the wrong link tagged in my last comment.&lt;/p&gt;

&lt;p&gt;Yes, I am using the same patch you indicated there i.e &lt;a href=&quot;http://review.whamcloud.com/#/c/19886&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/#/c/19886&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;Just a thought, would it be better to combine these two patches (&lt;a href=&quot;http://review.whamcloud.com/#/c/19886&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;patch1&lt;/a&gt;, &lt;a href=&quot;http://review.whamcloud.com/#/c/19341/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;patch2&lt;/a&gt;) and make it one as these two mainly contribute to the cause and remaining patches can be pushed over this. &lt;/p&gt;

&lt;p&gt;copied from your comment:&lt;/p&gt;
&lt;blockquote&gt;
&lt;p&gt;It is much easier to review a series a patches that do separate things that a big one that does many things.&lt;/p&gt;&lt;/blockquote&gt;

&lt;p&gt;I agree but merge conflicts cause other patches also to be re-based as they are dependent on each other. &lt;/p&gt;

&lt;p&gt;IMO, we can reduce the number of patches, may be to 3. &lt;/p&gt;

&lt;p&gt;Thanks,&lt;/p&gt;

</comment>
                            <comment id="157825" author="gerrit" created="Wed, 6 Jul 2016 16:19:37 +0000"  >&lt;p&gt;Jean-Baptiste Riaux (riaux.jb@intel.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/21173&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/21173&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: defer all new archive hals&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: dd9a0e96a71c69f1862494f32f76da65553b9359&lt;/p&gt;</comment>
                            <comment id="157826" author="gerrit" created="Wed, 6 Jul 2016 16:19:38 +0000"  >&lt;p&gt;Jean-Baptiste Riaux (riaux.jb@intel.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/21174&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/21174&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: defer all new archive hals&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 37d8a8bcf4865198ef1b8ec932f9ea17d5683f43&lt;/p&gt;</comment>
                            <comment id="157941" author="riauxjb" created="Thu, 7 Jul 2016 09:15:39 +0000"  >&lt;p&gt;Sorry, made a mistake while merging patches. Patches  &lt;a href=&quot;http://review.whamcloud.com/21173&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/21173&lt;/a&gt; and  &lt;a href=&quot;http://review.whamcloud.com/21174&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/21174&lt;/a&gt; abandoned.&lt;/p&gt;</comment>
                            <comment id="160395" author="gerrit" created="Mon, 1 Aug 2016 02:10:26 +0000"  >&lt;p&gt;Oleg Drokin (oleg.drokin@intel.com) merged in patch &lt;a href=&quot;http://review.whamcloud.com/19578/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19578/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: remove hsd.request_sz from struct hsm_scan_data&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 3061a7cc5a3a3e0c7d73d8f5c308635fce69aa66&lt;/p&gt;</comment>
                            <comment id="161546" author="gerrit" created="Thu, 11 Aug 2016 05:50:29 +0000"  >&lt;p&gt;Oleg Drokin (oleg.drokin@intel.com) merged in patch &lt;a href=&quot;http://review.whamcloud.com/19579/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19579/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: Fix possible out of bounds reference in message&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 526ae2f6a383b326da4b7be1623d9fb6563d0d37&lt;/p&gt;</comment>
                            <comment id="161547" author="gerrit" created="Thu, 11 Aug 2016 05:50:39 +0000"  >&lt;p&gt;Oleg Drokin (oleg.drokin@intel.com) merged in patch &lt;a href=&quot;http://review.whamcloud.com/19581/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19581/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: only browse entries of hsd.request when needed&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 1a10d9219e966daaf61ea9c42610c22126acd836&lt;/p&gt;</comment>
                            <comment id="161548" author="gerrit" created="Thu, 11 Aug 2016 05:50:47 +0000"  >&lt;p&gt;Oleg Drokin (oleg.drokin@intel.com) merged in patch &lt;a href=&quot;http://review.whamcloud.com/19583/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19583/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: remove compound_id from mdt_hsm_add_actions proto&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 250e826cd3902cd8e06cb7ee5167f2814219f63f&lt;/p&gt;</comment>
                            <comment id="161626" author="fzago" created="Thu, 11 Aug 2016 15:45:48 +0000"  >&lt;p&gt;Series rebased: Current patches, in order, are:&lt;/p&gt;

&lt;p&gt;  &lt;a href=&quot;http://review.whamcloud.com/19341&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19341&lt;/a&gt;&lt;br/&gt;
  &lt;a href=&quot;http://review.whamcloud.com/19582&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19582&lt;/a&gt;&lt;br/&gt;
  &lt;a href=&quot;http://review.whamcloud.com/19584&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19584&lt;/a&gt;&lt;br/&gt;
  &lt;a href=&quot;http://review.whamcloud.com/20272&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/20272&lt;/a&gt;&lt;br/&gt;
  &lt;a href=&quot;http://review.whamcloud.com/19886&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/19886&lt;/a&gt;&lt;/p&gt;</comment>
                            <comment id="166806" author="fzago" created="Wed, 21 Sep 2016 20:48:26 +0000"  >&lt;p&gt;Adding a new patch to the series as the remaining issue seems to be around the shutdown of the coordinator, which the new code appears to trigger.&lt;/p&gt;</comment>
                            <comment id="166812" author="gerrit" created="Wed, 21 Sep 2016 20:51:11 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/22667&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/22667&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: change coordinator start/stop mechanisms&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 13edea48863f849b19f4fd59e462cea794c1b112&lt;/p&gt;</comment>
                            <comment id="166907" author="bfaccini" created="Thu, 22 Sep 2016 15:05:20 +0000"  >&lt;p&gt;Frank, shutdown of CDT is a long time known issue (particularly the fact that CDT stuff/structs cleanup could occur from both MDT umount thread and/or CDT thread itself, and I have missed time to fix it myself) and your last #22667 seems to be a right way to fix it. I only have an additional concern about if cdt_state tests/updates should not need to be protected to fully avoid racy cases ?&lt;/p&gt;</comment>
                            <comment id="167076" author="gerrit" created="Fri, 23 Sep 2016 16:31:54 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/22704&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/22704&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: make mdt_hsm_cdt_start a static function&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: e271b1a669cc7c19c8de597c3b2f01b0ab05e0bc&lt;/p&gt;</comment>
                            <comment id="167509" author="gerrit" created="Tue, 27 Sep 2016 20:19:14 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/22761&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/22761&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: don&apos;t use a ptlrpc_thread struct for the cdt&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 622e168f23541917f6cdc3e04e40bd78109c79ab&lt;/p&gt;</comment>
                            <comment id="167510" author="gerrit" created="Tue, 27 Sep 2016 20:19:14 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/22762&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/22762&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: mark the cdt as stopped when its thread exits&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: f8ade6d5f931640ea10b1541681e603a0e3c2182&lt;/p&gt;</comment>
                            <comment id="167640" author="gerrit" created="Wed, 28 Sep 2016 20:58:24 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/22789&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/22789&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: change the cdt event flags to a simple boolean&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 25b347477bfb2cd09250dd09c82dbe5fde106b1d&lt;/p&gt;</comment>
                            <comment id="167802" author="gerrit" created="Thu, 29 Sep 2016 19:52:25 +0000"  >&lt;p&gt;Frank Zago (fzago@cray.com) uploaded a new patch: &lt;a href=&quot;http://review.whamcloud.com/22823&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/22823&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: replace l_wait_event with wait_event_timeout&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 2bbb447b1b6922bfb6b596d9b1c76b6968e6df6f&lt;/p&gt;</comment>
                            <comment id="168189" author="fzago" created="Tue, 4 Oct 2016 15:20:02 +0000"  >&lt;p&gt;This series now passes the tests again.&lt;/p&gt;</comment>
                            <comment id="168812" author="gerrit" created="Sat, 8 Oct 2016 16:39:09 +0000"  >&lt;p&gt;Oleg Drokin (oleg.drokin@intel.com) merged in patch &lt;a href=&quot;http://review.whamcloud.com/22704/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/22704/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: make mdt_hsm_cdt_start a static function&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 243880c0c6f380591f71b85847f57d45bd441c2d&lt;/p&gt;</comment>
                            <comment id="178913" author="gerrit" created="Fri, 23 Dec 2016 05:05:12 +0000"  >&lt;p&gt;Oleg Drokin (oleg.drokin@intel.com) merged in patch &lt;a href=&quot;https://review.whamcloud.com/22761/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/22761/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: don&apos;t use a ptlrpc_thread struct for the cdt&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 9811edb26d3bff66c61acc39a72554461445298f&lt;/p&gt;</comment>
                            <comment id="193250" author="spitzcor" created="Mon, 24 Apr 2017 17:24:47 +0000"  >&lt;p&gt;Some stuff has landed for 2.10, other stuff remains pending.  What&apos;s the current list of changes that is still pending?&lt;/p&gt;</comment>
                            <comment id="196222" author="gerrit" created="Wed, 17 May 2017 18:07:45 +0000"  >&lt;p&gt;Oleg Drokin (oleg.drokin@intel.com) merged in patch &lt;a href=&quot;https://review.whamcloud.com/22667/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/22667/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: change coordinator start/stop mechanisms&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 958198e3938120040e93fc2aa255151fa6a81907&lt;/p&gt;</comment>
                            <comment id="196513" author="pjones" created="Fri, 19 May 2017 19:53:44 +0000"  >&lt;p&gt;As discussed in the LWG this week, work in this area is ongoing and those ready before code freeze will still land but the prospects of all landing in the remaining time look unlikely so they will end up in a subsequent maintenance release&lt;/p&gt;</comment>
                            <comment id="196849" author="gerrit" created="Wed, 24 May 2017 07:33:29 +0000"  >&lt;p&gt;Oleg Drokin (oleg.drokin@intel.com) merged in patch &lt;a href=&quot;https://review.whamcloud.com/22789/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/22789/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: change the cdt event flags to a simple boolean&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: d185bf805b71079343b42521fafdd734d52d052b&lt;/p&gt;</comment>
                            <comment id="202567" author="gerrit" created="Wed, 19 Jul 2017 03:27:56 +0000"  >&lt;p&gt;Oleg Drokin (oleg.drokin@intel.com) merged in patch &lt;a href=&quot;https://review.whamcloud.com/22762/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/22762/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: mark the cdt as stopped when its thread exits&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: f11a5022fc129fec797adb155e5553331f224ecc&lt;/p&gt;</comment>
                            <comment id="203601" author="gerrit" created="Wed, 26 Jul 2017 17:17:40 +0000"  >&lt;p&gt;Minh Diep (minh.diep@intel.com) uploaded a new patch: &lt;a href=&quot;https://review.whamcloud.com/28219&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/28219&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: mark the cdt as stopped when its thread exits&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: b2_10&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 11cfae630ff68cd2f83af6e38223a8256c718661&lt;/p&gt;</comment>
                            <comment id="204017" author="gerrit" created="Tue, 1 Aug 2017 05:32:53 +0000"  >&lt;p&gt;Oleg Drokin (oleg.drokin@intel.com) merged in patch &lt;a href=&quot;https://review.whamcloud.com/19341/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/19341/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: run HSM coordinator once per second at most&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: cc6ef11d2f972ebc440013bddda87a536a09750c&lt;/p&gt;</comment>
                            <comment id="204382" author="gerrit" created="Thu, 3 Aug 2017 21:40:45 +0000"  >&lt;p&gt;John L. Hammond (john.hammond@intel.com) merged in patch &lt;a href=&quot;https://review.whamcloud.com/28219/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/28219/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: mark the cdt as stopped when its thread exits&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: b2_10&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 58da5f41d5db9818ae1a1f148331eaad6fc63918&lt;/p&gt;</comment>
                            <comment id="204532" author="gerrit" created="Fri, 4 Aug 2017 22:22:41 +0000"  >&lt;p&gt;Minh Diep (minh.diep@intel.com) uploaded a new patch: &lt;a href=&quot;https://review.whamcloud.com/28368&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/28368&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: run HSM coordinator once per second at most&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: b2_10&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 39cebbd76c55c9673502a4caa3863840d9f4597d&lt;/p&gt;</comment>
                            <comment id="205039" author="gerrit" created="Thu, 10 Aug 2017 16:26:16 +0000"  >&lt;p&gt;John L. Hammond (john.hammond@intel.com) merged in patch &lt;a href=&quot;https://review.whamcloud.com/28368/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/28368/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: run HSM coordinator once per second at most&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: b2_10&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 3223073d68647607d53bb6b4c7447648306e14b6&lt;/p&gt;</comment>
                            <comment id="207109" author="gerrit" created="Thu, 31 Aug 2017 19:15:35 +0000"  >&lt;p&gt;Oleg Drokin (oleg.drokin@intel.com) merged in patch &lt;a href=&quot;https://review.whamcloud.com/19582/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/19582/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: added coordinator housekeeping flag&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: afc9ff6caff7d572041cabf0a957dc8749fce49d&lt;/p&gt;</comment>
                            <comment id="207901" author="gerrit" created="Fri, 8 Sep 2017 14:49:27 +0000"  >&lt;p&gt;Minh Diep (minh.diep@intel.com) uploaded a new patch: &lt;a href=&quot;https://review.whamcloud.com/28908&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/28908&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: added coordinator housekeeping flag&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: b2_10&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 10c2083bc68309b64309e922c27d67c814ce00c5&lt;/p&gt;</comment>
                            <comment id="208194" author="gerrit" created="Wed, 13 Sep 2017 03:36:53 +0000"  >&lt;p&gt;Oleg Drokin (oleg.drokin@intel.com) merged in patch &lt;a href=&quot;https://review.whamcloud.com/19584/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/19584/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: update many cookie status at once&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: f3a415289b560b5f422efe2bd08b3b7cff113cf0&lt;/p&gt;</comment>
                            <comment id="208250" author="gerrit" created="Wed, 13 Sep 2017 15:24:13 +0000"  >&lt;p&gt;Minh Diep (minh.diep@intel.com) uploaded a new patch: &lt;a href=&quot;https://review.whamcloud.com/28973&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/28973&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: update many cookie status at once&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: b2_10&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 207f81b700adff52bc58685efa8724ed534ae0cf&lt;/p&gt;</comment>
                            <comment id="210588" author="gerrit" created="Mon, 9 Oct 2017 03:49:32 +0000"  >&lt;p&gt;Oleg Drokin (oleg.drokin@intel.com) merged in patch &lt;a href=&quot;https://review.whamcloud.com/20272/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/20272/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: split mdt_hsm_add_actions()&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 3c0b677cdffae7d329e6f0ab73784b20af2f11f5&lt;/p&gt;</comment>
                            <comment id="210943" author="gerrit" created="Thu, 12 Oct 2017 14:40:33 +0000"  >&lt;p&gt;Minh Diep (minh.diep@intel.com) uploaded a new patch: &lt;a href=&quot;https://review.whamcloud.com/29583&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/29583&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: split mdt_hsm_add_actions()&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: b2_10&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 80ce8998119a9bf125c014c94f8fa712a47e0b47&lt;/p&gt;</comment>
                            <comment id="211826" author="gerrit" created="Tue, 24 Oct 2017 15:38:52 +0000"  >&lt;p&gt;Ben Evans (bevans@cray.com) uploaded a new patch: &lt;a href=&quot;https://review.whamcloud.com/29742&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/29742&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: wake up cdt when requests are empty&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 0556c7c9bf289ac34cda099819d23ce0063302a2&lt;/p&gt;</comment>
                            <comment id="212083" author="gerrit" created="Thu, 26 Oct 2017 16:08:15 +0000"  >&lt;p&gt;John L. Hammond (john.hammond@intel.com) merged in patch &lt;a href=&quot;https://review.whamcloud.com/28973/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/28973/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: update many cookie status at once&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: b2_10&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 019c48b649d0f668563cf72495d1d1b02f4f69c0&lt;/p&gt;</comment>
                            <comment id="212822" author="gerrit" created="Mon, 6 Nov 2017 03:42:56 +0000"  >&lt;p&gt;Oleg Drokin (oleg.drokin@intel.com) merged in patch &lt;a href=&quot;https://review.whamcloud.com/29742/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/29742/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: wake up cdt when requests are empty&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 7251fea8dc3c4d29e30c5a3f763c4c33d35f90a7&lt;/p&gt;</comment>
                            <comment id="212852" author="pjones" created="Mon, 6 Nov 2017 13:28:24 +0000"  >&lt;p&gt;As we finally seem to be at an end of patches queued up for this ticket let&apos;s close it and open a new ticket to track any new fixes identified in this area of code in the future.&lt;/p&gt;</comment>
                            <comment id="216308" author="gerrit" created="Thu, 14 Dec 2017 16:33:10 +0000"  >&lt;p&gt;Minh Diep (minh.diep@intel.com) uploaded a new patch: &lt;a href=&quot;https://review.whamcloud.com/30538&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/30538&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: wake up cdt when requests are empty&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: b2_10&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 7121b1d335453f31766f2f59909d20b7cde3e948&lt;/p&gt;</comment>
                            <comment id="216802" author="gerrit" created="Tue, 19 Dec 2017 22:16:26 +0000"  >&lt;p&gt;John L. Hammond (john.hammond@intel.com) merged in patch &lt;a href=&quot;https://review.whamcloud.com/30538/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/30538/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: wake up cdt when requests are empty&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: b2_10&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: d5a7563a46acf15252929b6a045eef8c58022314&lt;/p&gt;</comment>
                            <comment id="221688" author="gerrit" created="Mon, 26 Feb 2018 18:42:54 +0000"  >&lt;p&gt;John L. Hammond (john.hammond@intel.com) merged in patch &lt;a href=&quot;https://review.whamcloud.com/29583/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/29583/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: split mdt_hsm_add_actions()&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: b2_10&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 03fc1a015f5869b68d21eee69531a201045532e1&lt;/p&gt;</comment>
                            <comment id="238608" author="adegremont" created="Fri, 14 Dec 2018 12:08:50 +0000"  >&lt;p&gt;It seems this ticket was wrongly flagged to 2.10.6.&lt;/p&gt;

&lt;p&gt;The last patch being landed on b2_10 from this series was landed for 2.10.4&lt;/p&gt;

&lt;p&gt;There is still one backport which was not landed and is interesting: &lt;a href=&quot;https://review.whamcloud.com/#/c/28908/2&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/#/c/28908/2&lt;/a&gt;&lt;/p&gt;</comment>
                            <comment id="240380" author="gerrit" created="Sat, 19 Jan 2019 06:29:26 +0000"  >&lt;p&gt;Oleg Drokin (green@whamcloud.com) merged in patch &lt;a href=&quot;https://review.whamcloud.com/28908/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/28908/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-7988&quot; title=&quot;HSM: high lock contention for cdt_llog_lock&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-7988&quot;&gt;&lt;del&gt;LU-7988&lt;/del&gt;&lt;/a&gt; hsm: added coordinator housekeeping flag&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: b2_10&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 387dca70723affc0b45b87730e994395b9627eef&lt;/p&gt;</comment>
                    </comments>
                <issuelinks>
                            <issuelinktype id="10011">
                    <name>Related</name>
                                            <outwardlinks description="is related to ">
                                                        </outwardlinks>
                                                                <inwardlinks description="is related to">
                                        <issuelink>
            <issuekey id="48231">LU-9959</issuekey>
        </issuelink>
            <issuelink>
            <issuekey id="39907">LU-8626</issuekey>
        </issuelink>
                            </inwardlinks>
                                    </issuelinktype>
                    </issuelinks>
                <attachments>
                            <attachment id="21018" name="mdt-cdt-250000.svg" size="144235" author="fzago" created="Wed, 6 Apr 2016 20:20:36 +0000"/>
                    </attachments>
                <subtasks>
                    </subtasks>
                <customfields>
                                                                                                                                                                                            <customfield id="customfield_10890" key="com.atlassian.jira.plugins.jira-development-integration-plugin:devsummary">
                        <customfieldname>Development</customfieldname>
                        <customfieldvalues>
                            
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                        <customfield id="customfield_10390" key="com.pyxis.greenhopper.jira:gh-lexo-rank">
                        <customfieldname>Rank</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>1|hzy6u7:</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_10090" key="com.pyxis.greenhopper.jira:gh-global-rank">
                        <customfieldname>Rank (Obsolete)</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>9223372036854775807</customfieldvalue>
                        </customfieldvalues>
                    </customfield>
                                                                                            <customfield id="customfield_10060" key="com.atlassian.jira.plugin.system.customfieldtypes:select">
                        <customfieldname>Severity</customfieldname>
                        <customfieldvalues>
                                <customfieldvalue key="10022"><![CDATA[3]]></customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                        </customfields>
    </item>
</channel>
</rss>