<!-- 
RSS generated by JIRA (9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c) at Sat Feb 10 03:19:10 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-15538] DLC doesn&apos;t initialize default LND tunables correctly</title>
                <link>https://jira.whamcloud.com/browse/LU-15538</link>
                <project id="10000" key="LU">Lustre</project>
                    <description>&lt;p&gt;LND tunables are not initialized correctly when any net or lnd tunable parameter is specified via either the CLI or via the yaml config.&lt;/p&gt;

&lt;p&gt;I noticed this with map_on_demand, but the bug potentially impacts any LND tunable where &apos;0&apos; is an accepted value:&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;/home/hornc/lustre-wc-rel/lnet/utils/lnetctl lnet configure
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net add --net o2ib --if enp137s0f0
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net show -v 5 | egrep -e o2ib -e  map
    - net type: o2ib
        - nid: 172.18.4.4@o2ib
              map_on_demand: 1

/home/hornc/lustre-wc-rel/lnet/utils/lnetctl lnet configure
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net add --net o2ib --if enp137s0f0 --peer-credits=32
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net show -v 5 | egrep -e o2ib -e  map
    - net type: o2ib
        - nid: 172.18.4.4@o2ib
              map_on_demand: 0
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;

&lt;p&gt;Issue is that when any NET or LND tunable is specified via CLI or yaml, then the whole tunables struct gets memset to 0, or in the case of yaml config, 0 gets assigned to any tunable that isn&apos;t specified in the yaml. LND then thinks that &apos;0&apos; was specified by user and will use that value (if it is valid).&lt;/p&gt;</description>
                <environment></environment>
        <key id="68590">LU-15538</key>
            <summary>DLC doesn&apos;t initialize default LND tunables correctly</summary>
                <type id="1" iconUrl="https://jira.whamcloud.com/secure/viewavatar?size=xsmall&amp;avatarId=11303&amp;avatarType=issuetype">Bug</type>
                                            <priority id="4" iconUrl="https://jira.whamcloud.com/images/icons/priorities/minor.svg">Minor</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="hornc">Chris Horn</assignee>
                                    <reporter username="hornc">Chris Horn</reporter>
                        <labels>
                    </labels>
                <created>Tue, 8 Feb 2022 22:08:04 +0000</created>
                <updated>Fri, 23 Sep 2022 16:41:54 +0000</updated>
                            <resolved>Sat, 11 Jun 2022 15:41:23 +0000</resolved>
                                                    <fixVersion>Lustre 2.16.0</fixVersion>
                                        <due></due>
                            <votes>0</votes>
                                    <watches>3</watches>
                                                                            <comments>
                            <comment id="325880" author="gerrit" created="Thu, 10 Feb 2022 16:50:25 +0000"  >&lt;p&gt;&quot;Chris Horn &amp;lt;chris.horn@hpe.com&amp;gt;&quot; uploaded a new patch: &lt;a href=&quot;https://review.whamcloud.com/46492&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/46492&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-15538&quot; title=&quot;DLC doesn&amp;#39;t initialize default LND tunables correctly&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-15538&quot;&gt;&lt;del&gt;LU-15538&lt;/del&gt;&lt;/a&gt; lnet: DLC sets map_on_demand incorrectly&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: 1&lt;br/&gt;
Commit: 6d5b0308cae0db67daa7df626a60fd0825d252cc&lt;/p&gt;</comment>
                            <comment id="325893" author="hornc" created="Thu, 10 Feb 2022 17:21:51 +0000"  >&lt;p&gt;Test notes for &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-15538&quot; title=&quot;DLC doesn&amp;#39;t initialize default LND tunables correctly&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-15538&quot;&gt;&lt;del&gt;LU-15538&lt;/del&gt;&lt;/a&gt;:&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;[hornc@s-lmo-gaz38b lustre-wc-rel]$ git reset --hard origin/master
HEAD is now at 450d10c362 New RC 2.15.0-RC2
[hornc@s-lmo-gaz38b lustre-wc-rel]$ make -j 32
...
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&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;[root@s-lmo-gaz38b hornc]# bash start.sh
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl lnet configure
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net add --net o2ib --if enp137s0f0
    - net type: o2ib
        - nid: 172.18.4.4@o2ib
              map_on_demand: 1
[root@s-lmo-gaz38b hornc]# bash clean.sh
[root@s-lmo-gaz38b hornc]# bash start.sh map_on_demand=0
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl lnet configure
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net add --net o2ib --if enp137s0f0
    - net type: o2ib
        - nid: 172.18.4.4@o2ib
              map_on_demand: 0
[root@s-lmo-gaz38b hornc]# bash clean.sh
[root@s-lmo-gaz38b hornc]# bash start.sh map_on_demand=1
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl lnet configure
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net add --net o2ib --if enp137s0f0
    - net type: o2ib
        - nid: 172.18.4.4@o2ib
              map_on_demand: 1
[root@s-lmo-gaz38b hornc]#
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;

&lt;p&gt;We can see map_on_demand has expected value (note, the default is 1) in each case.&lt;/p&gt;

&lt;p&gt;Repeat test, but specify &apos;--peer-credits=32&apos;:&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;[root@s-lmo-gaz38b hornc]# bash start.sh2
debug=+net
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl lnet configure
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net add --net o2ib --if enp137s0f0 --peer-credits=32
    - net type: o2ib
        - nid: 172.18.4.4@o2ib
              map_on_demand: 0
[root@s-lmo-gaz38b hornc]# bash clean.sh
[root@s-lmo-gaz38b hornc]# bash start.sh2 map_on_demand=0
debug=+net
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl lnet configure
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net add --net o2ib --if enp137s0f0 --peer-credits=32
    - net type: o2ib
        - nid: 172.18.4.4@o2ib
              map_on_demand: 0
[root@s-lmo-gaz38b hornc]# bash clean.sh
[root@s-lmo-gaz38b hornc]# bash start.sh2 map_on_demand=1
debug=+net
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl lnet configure
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net add --net o2ib --if enp137s0f0 --peer-credits=32
    - net type: o2ib
        - nid: 172.18.4.4@o2ib
              map_on_demand: 0
[root@s-lmo-gaz38b hornc]#
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;

&lt;p&gt;We can see the wrong value is used in 2/3 cases.&lt;/p&gt;

&lt;p&gt;Apply fix:&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;[hornc@s-lmo-gaz38b lustre-wc-rel]$ git fetch https://review.whamcloud.com/fs/lustre-release refs/changes/92/46492/1 &amp;amp;&amp;amp; git cherry-pick FETCH_HEAD
remote: Counting objects: 2010, done
remote: Finding sources: 100% (10/10)
remote: Total 10 (delta 9), reused 10 (delta 9)
Unpacking objects: 100% (10/10), 1.30 KiB | 665.00 KiB/s, done.
From https://review.whamcloud.com/fs/lustre-release
 * branch                  refs/changes/92/46492/1 -&amp;gt; FETCH_HEAD
[tmp4 c2c7abe9b6] LU-15538 lnet: DLC sets map_on_demand incorrectly
 Date: Sat Feb 5 23:15:30 2022 +0000
 3 files changed, 7 insertions(+), 1 deletion(-)
[hornc@s-lmo-gaz38b lustre-wc-rel]$ make -j 32
...
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;

&lt;p&gt;Repeat test cases:&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;[root@s-lmo-gaz38b hornc]# bash start.sh
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl lnet configure
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net add --net o2ib --if enp137s0f0
    - net type: o2ib
        - nid: 172.18.4.4@o2ib
              map_on_demand: 1
[root@s-lmo-gaz38b hornc]# bash clean.sh
[root@s-lmo-gaz38b hornc]# bash start.sh map_on_demand=0
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl lnet configure
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net add --net o2ib --if enp137s0f0
    - net type: o2ib
        - nid: 172.18.4.4@o2ib
              map_on_demand: 0
[root@s-lmo-gaz38b hornc]# bash clean.sh
[root@s-lmo-gaz38b hornc]# bash start.sh map_on_demand=1
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl lnet configure
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net add --net o2ib --if enp137s0f0
    - net type: o2ib
        - nid: 172.18.4.4@o2ib
              map_on_demand: 1
[root@s-lmo-gaz38b hornc]# bash clean.sh
[root@s-lmo-gaz38b hornc]# bash start.sh2
debug=+net
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl lnet configure
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net add --net o2ib --if enp137s0f0 --peer-credits=32
    - net type: o2ib
        - nid: 172.18.4.4@o2ib
              map_on_demand: 1
[root@s-lmo-gaz38b hornc]# bash clean.sh
[root@s-lmo-gaz38b hornc]# bash start.sh2 map_on_demand=0
debug=+net
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl lnet configure
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net add --net o2ib --if enp137s0f0 --peer-credits=32
    - net type: o2ib
        - nid: 172.18.4.4@o2ib
              map_on_demand: 0
[root@s-lmo-gaz38b hornc]# bash clean.sh
[root@s-lmo-gaz38b hornc]# bash start.sh2 map_on_demand=1
debug=+net
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl lnet configure
/home/hornc/lustre-wc-rel/lnet/utils/lnetctl net add --net o2ib --if enp137s0f0 --peer-credits=32
    - net type: o2ib
        - nid: 172.18.4.4@o2ib
              map_on_demand: 1
[root@s-lmo-gaz38b hornc]#
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;

&lt;p&gt;We can see map_on_demand gets correct value in every case.&lt;/p&gt;</comment>
                            <comment id="337418" author="gerrit" created="Sat, 11 Jun 2022 05:34:54 +0000"  >&lt;p&gt;&quot;Oleg Drokin &amp;lt;green@whamcloud.com&amp;gt;&quot; merged in patch &lt;a href=&quot;https://review.whamcloud.com/46492/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://review.whamcloud.com/46492/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-15538&quot; title=&quot;DLC doesn&amp;#39;t initialize default LND tunables correctly&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-15538&quot;&gt;&lt;del&gt;LU-15538&lt;/del&gt;&lt;/a&gt; lnet: DLC sets map_on_demand incorrectly&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 896f4a082b93453f5e7168f685faff4fba594ff3&lt;/p&gt;</comment>
                            <comment id="337510" author="pjones" created="Sat, 11 Jun 2022 15:41:23 +0000"  >&lt;p&gt;Landed for 2.16&lt;/p&gt;</comment>
                    </comments>
                    <attachments>
                    </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|i02hvj:</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>