<!-- 
RSS generated by JIRA (9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c) at Sat Feb 10 01:04:41 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-191] mdd_close will always start transaction even atime is not changed</title>
                <link>https://jira.whamcloud.com/browse/LU-191</link>
                <project id="10000" key="LU">Lustre</project>
                    <description>&lt;p&gt;I&apos;ve fixed this in bug 23289 (attachment 32253), however it&apos;s back after landing of another patch (bug 23291, attachment 31108):&lt;/p&gt;

&lt;ul class=&quot;alternate&quot; type=&quot;square&quot;&gt;
	&lt;li&gt;if (la-&amp;gt;la_atime &amp;lt;= tmp_la-&amp;gt;la_atime +&lt;/li&gt;
	&lt;li&gt;mdd_obj2mdd_dev(obj)-&amp;gt;mdd_atime_diff)&lt;br/&gt;
+                if (la-&amp;gt;la_atime &amp;gt; tmp_la-&amp;gt;la_atime &amp;amp;&amp;amp;&lt;br/&gt;
+                    la-&amp;gt;la_atime &amp;lt;= (tmp_la-&amp;gt;la_atime +&lt;br/&gt;
+                                     mdd_obj2mdd_dev(obj)-&amp;gt;mdd_atime_diff))&lt;br/&gt;
                         la-&amp;gt;la_valid &amp;amp;= ~LA_ATIME;&lt;/li&gt;
&lt;/ul&gt;


&lt;p&gt;with this patch, the condition is always false if &quot;la-&amp;gt;la_atime == tmp_la-&amp;gt;la_atime&quot;, which means even atime is exactly same as before, we still start a transaction and setattr, this will give a big impact on performance for those shortly open+close applications/tests.&lt;/p&gt;</description>
                <environment></environment>
        <key id="10543">LU-191</key>
            <summary>mdd_close will always start transaction even atime is not changed</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="liang">Liang Zhen</assignee>
                                    <reporter username="liang">Liang Zhen</reporter>
                        <labels>
                    </labels>
                <created>Mon, 4 Apr 2011 05:35:28 +0000</created>
                <updated>Thu, 8 Dec 2011 23:27:53 +0000</updated>
                            <resolved>Thu, 8 Dec 2011 23:27:53 +0000</resolved>
                                    <version>Lustre 2.1.0</version>
                                    <fixVersion>Lustre 2.1.0</fixVersion>
                                        <due></due>
                            <votes>0</votes>
                                    <watches>0</watches>
                                                                            <comments>
                            <comment id="12029" author="tappro" created="Mon, 4 Apr 2011 05:58:09 +0000"  >&lt;p&gt;so that is enough to use &amp;gt;= in first check and maybe just &amp;lt; in second, right?&lt;/p&gt;</comment>
                            <comment id="12030" author="liang" created="Mon, 4 Apr 2011 06:02:51 +0000"  >&lt;p&gt;exactly, &lt;img class=&quot;emoticon&quot; src=&quot;https://jira.whamcloud.com/images/icons/emoticons/smile.png&quot; height=&quot;16&quot; width=&quot;16&quot; align=&quot;absmiddle&quot; alt=&quot;&quot; border=&quot;0&quot;/&gt;&lt;/p&gt;</comment>
                            <comment id="12048" author="hudson" created="Mon, 4 Apr 2011 09:47:53 +0000"  >&lt;p&gt;Integrated in &lt;span class=&quot;image-wrap&quot; style=&quot;&quot;&gt;&lt;img src=&quot;http://build.whamcloud.com/images/16x16/blue.png&quot; style=&quot;border: 0px solid black&quot; /&gt;&lt;/span&gt; &lt;a href=&quot;http://build.whamcloud.com/job/reviews-centos5/650/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;reviews-centos5 #650&lt;/a&gt;&lt;br/&gt;
     &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-191&quot; title=&quot;mdd_close will always start transaction even atime is not changed&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-191&quot;&gt;&lt;del&gt;LU-191&lt;/del&gt;&lt;/a&gt; mdd_close always set atime even it&apos;s same&lt;/p&gt;

&lt;p&gt;Liang Zhen : &lt;a href=&quot;http://git.whamcloud.com/gitweb/?p=fs/lustre-release.git&amp;amp;a=commit&amp;amp;h=505ae6ff13b72df8d6db1627ee509398b91c8fd2&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;505ae6ff13b72df8d6db1627ee509398b91c8fd2&lt;/a&gt;&lt;br/&gt;
Files : &lt;/p&gt;
&lt;ul&gt;
	&lt;li&gt;lustre/mdd/mdd_object.c&lt;/li&gt;
&lt;/ul&gt;
</comment>
                            <comment id="12941" author="hudson" created="Mon, 18 Apr 2011 11:47:16 +0000"  >&lt;p&gt;Integrated in &lt;span class=&quot;image-wrap&quot; style=&quot;&quot;&gt;&lt;img src=&quot;http://build.whamcloud.com/images/16x16/blue.png&quot; style=&quot;border: 0px solid black&quot; /&gt;&lt;/span&gt; &lt;a href=&quot;http://build.whamcloud.com/job/lustre-master-centos5/201/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;lustre-master-centos5 #201&lt;/a&gt;&lt;br/&gt;
     &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-191&quot; title=&quot;mdd_close will always start transaction even atime is not changed&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-191&quot;&gt;&lt;del&gt;LU-191&lt;/del&gt;&lt;/a&gt; mdd_close always set atime even it&apos;s same&lt;/p&gt;

&lt;p&gt;Oleg Drokin : &lt;a href=&quot;http://git.whamcloud.com/gitweb/?p=fs/lustre-release.git&amp;amp;a=commit&amp;amp;h=d65f789f433f2d52e82116c374d88906321f9d28&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;d65f789f433f2d52e82116c374d88906321f9d28&lt;/a&gt;&lt;br/&gt;
Files : &lt;/p&gt;
&lt;ul&gt;
	&lt;li&gt;lustre/mdd/mdd_object.c&lt;/li&gt;
&lt;/ul&gt;
</comment>
                            <comment id="12948" author="hudson" created="Mon, 18 Apr 2011 13:02:34 +0000"  >&lt;p&gt;Integrated in &lt;span class=&quot;image-wrap&quot; style=&quot;&quot;&gt;&lt;img src=&quot;http://newbuild.whamcloud.com/images/16x16/red.png&quot; style=&quot;border: 0px solid black&quot; /&gt;&lt;/span&gt; &lt;a href=&quot;http://newbuild.whamcloud.com/job/lustre-master/./build_type=client,platform=ubuntu-x86_64/29/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;lustre-master &#187; client,ubuntu-x86_64 #29&lt;/a&gt;&lt;br/&gt;
     &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-191&quot; title=&quot;mdd_close will always start transaction even atime is not changed&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-191&quot;&gt;&lt;del&gt;LU-191&lt;/del&gt;&lt;/a&gt; mdd_close always set atime even it&apos;s same&lt;/p&gt;

&lt;p&gt;Oleg Drokin : &lt;a href=&quot;http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;amp;a=commit&amp;amp;h=d65f789f433f2d52e82116c374d88906321f9d28&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;d65f789f433f2d52e82116c374d88906321f9d28&lt;/a&gt;&lt;br/&gt;
Files : &lt;/p&gt;
&lt;ul&gt;
	&lt;li&gt;lustre/mdd/mdd_object.c&lt;/li&gt;
&lt;/ul&gt;
</comment>
                            <comment id="12953" author="hudson" created="Mon, 18 Apr 2011 13:13:30 +0000"  >&lt;p&gt;Integrated in &lt;span class=&quot;image-wrap&quot; style=&quot;&quot;&gt;&lt;img src=&quot;http://newbuild.whamcloud.com/images/16x16/blue.png&quot; style=&quot;border: 0px solid black&quot; /&gt;&lt;/span&gt; &lt;a href=&quot;http://newbuild.whamcloud.com/job/lustre-master/./build_type=client,platform=el5-x86_64/29/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;lustre-master &#187; client,el5-x86_64 #29&lt;/a&gt;&lt;br/&gt;
     &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-191&quot; title=&quot;mdd_close will always start transaction even atime is not changed&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-191&quot;&gt;&lt;del&gt;LU-191&lt;/del&gt;&lt;/a&gt; mdd_close always set atime even it&apos;s same&lt;/p&gt;

&lt;p&gt;Oleg Drokin : &lt;a href=&quot;http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;amp;a=commit&amp;amp;h=d65f789f433f2d52e82116c374d88906321f9d28&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;d65f789f433f2d52e82116c374d88906321f9d28&lt;/a&gt;&lt;br/&gt;
Files : &lt;/p&gt;
&lt;ul&gt;
	&lt;li&gt;lustre/mdd/mdd_object.c&lt;/li&gt;
&lt;/ul&gt;
</comment>
                            <comment id="12958" author="hudson" created="Mon, 18 Apr 2011 13:17:18 +0000"  >&lt;p&gt;Integrated in &lt;span class=&quot;image-wrap&quot; style=&quot;&quot;&gt;&lt;img src=&quot;http://newbuild.whamcloud.com/images/16x16/blue.png&quot; style=&quot;border: 0px solid black&quot; /&gt;&lt;/span&gt; &lt;a href=&quot;http://newbuild.whamcloud.com/job/lustre-master/./build_type=client,platform=el5-i686/29/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;lustre-master &#187; client,el5-i686 #29&lt;/a&gt;&lt;br/&gt;
     &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-191&quot; title=&quot;mdd_close will always start transaction even atime is not changed&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-191&quot;&gt;&lt;del&gt;LU-191&lt;/del&gt;&lt;/a&gt; mdd_close always set atime even it&apos;s same&lt;/p&gt;

&lt;p&gt;Oleg Drokin : &lt;a href=&quot;http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;amp;a=commit&amp;amp;h=d65f789f433f2d52e82116c374d88906321f9d28&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;d65f789f433f2d52e82116c374d88906321f9d28&lt;/a&gt;&lt;br/&gt;
Files : &lt;/p&gt;
&lt;ul&gt;
	&lt;li&gt;lustre/mdd/mdd_object.c&lt;/li&gt;
&lt;/ul&gt;
</comment>
                            <comment id="12964" author="hudson" created="Mon, 18 Apr 2011 13:19:51 +0000"  >&lt;p&gt;Integrated in &lt;span class=&quot;image-wrap&quot; style=&quot;&quot;&gt;&lt;img src=&quot;http://newbuild.whamcloud.com/images/16x16/blue.png&quot; style=&quot;border: 0px solid black&quot; /&gt;&lt;/span&gt; &lt;a href=&quot;http://newbuild.whamcloud.com/job/lustre-master/./build_type=server,platform=el6-x86_64/29/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;lustre-master &#187; server,el6-x86_64 #29&lt;/a&gt;&lt;br/&gt;
     &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-191&quot; title=&quot;mdd_close will always start transaction even atime is not changed&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-191&quot;&gt;&lt;del&gt;LU-191&lt;/del&gt;&lt;/a&gt; mdd_close always set atime even it&apos;s same&lt;/p&gt;

&lt;p&gt;Oleg Drokin : &lt;a href=&quot;http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;amp;a=commit&amp;amp;h=d65f789f433f2d52e82116c374d88906321f9d28&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;d65f789f433f2d52e82116c374d88906321f9d28&lt;/a&gt;&lt;br/&gt;
Files : &lt;/p&gt;
&lt;ul&gt;
	&lt;li&gt;lustre/mdd/mdd_object.c&lt;/li&gt;
&lt;/ul&gt;
</comment>
                            <comment id="12969" author="hudson" created="Mon, 18 Apr 2011 13:20:10 +0000"  >&lt;p&gt;Integrated in &lt;span class=&quot;image-wrap&quot; style=&quot;&quot;&gt;&lt;img src=&quot;http://newbuild.whamcloud.com/images/16x16/blue.png&quot; style=&quot;border: 0px solid black&quot; /&gt;&lt;/span&gt; &lt;a href=&quot;http://newbuild.whamcloud.com/job/lustre-master/./build_type=client,platform=el6-i686/29/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;lustre-master &#187; client,el6-i686 #29&lt;/a&gt;&lt;br/&gt;
     &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-191&quot; title=&quot;mdd_close will always start transaction even atime is not changed&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-191&quot;&gt;&lt;del&gt;LU-191&lt;/del&gt;&lt;/a&gt; mdd_close always set atime even it&apos;s same&lt;/p&gt;

&lt;p&gt;Oleg Drokin : &lt;a href=&quot;http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;amp;a=commit&amp;amp;h=d65f789f433f2d52e82116c374d88906321f9d28&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;d65f789f433f2d52e82116c374d88906321f9d28&lt;/a&gt;&lt;br/&gt;
Files : &lt;/p&gt;
&lt;ul&gt;
	&lt;li&gt;lustre/mdd/mdd_object.c&lt;/li&gt;
&lt;/ul&gt;
</comment>
                            <comment id="12974" author="hudson" created="Mon, 18 Apr 2011 13:29:03 +0000"  >&lt;p&gt;Integrated in &lt;span class=&quot;image-wrap&quot; style=&quot;&quot;&gt;&lt;img src=&quot;http://newbuild.whamcloud.com/images/16x16/blue.png&quot; style=&quot;border: 0px solid black&quot; /&gt;&lt;/span&gt; &lt;a href=&quot;http://newbuild.whamcloud.com/job/lustre-master/./build_type=server,platform=el5-x86_64/29/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;lustre-master &#187; server,el5-x86_64 #29&lt;/a&gt;&lt;br/&gt;
     &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-191&quot; title=&quot;mdd_close will always start transaction even atime is not changed&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-191&quot;&gt;&lt;del&gt;LU-191&lt;/del&gt;&lt;/a&gt; mdd_close always set atime even it&apos;s same&lt;/p&gt;

&lt;p&gt;Oleg Drokin : &lt;a href=&quot;http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;amp;a=commit&amp;amp;h=d65f789f433f2d52e82116c374d88906321f9d28&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;d65f789f433f2d52e82116c374d88906321f9d28&lt;/a&gt;&lt;br/&gt;
Files : &lt;/p&gt;
&lt;ul&gt;
	&lt;li&gt;lustre/mdd/mdd_object.c&lt;/li&gt;
&lt;/ul&gt;
</comment>
                            <comment id="12980" author="hudson" created="Mon, 18 Apr 2011 13:34:05 +0000"  >&lt;p&gt;Integrated in &lt;span class=&quot;image-wrap&quot; style=&quot;&quot;&gt;&lt;img src=&quot;http://newbuild.whamcloud.com/images/16x16/blue.png&quot; style=&quot;border: 0px solid black&quot; /&gt;&lt;/span&gt; &lt;a href=&quot;http://newbuild.whamcloud.com/job/lustre-master/./build_type=client,platform=el6-x86_64/29/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;lustre-master &#187; client,el6-x86_64 #29&lt;/a&gt;&lt;br/&gt;
     &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-191&quot; title=&quot;mdd_close will always start transaction even atime is not changed&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-191&quot;&gt;&lt;del&gt;LU-191&lt;/del&gt;&lt;/a&gt; mdd_close always set atime even it&apos;s same&lt;/p&gt;

&lt;p&gt;Oleg Drokin : &lt;a href=&quot;http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;amp;a=commit&amp;amp;h=d65f789f433f2d52e82116c374d88906321f9d28&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;d65f789f433f2d52e82116c374d88906321f9d28&lt;/a&gt;&lt;br/&gt;
Files : &lt;/p&gt;
&lt;ul&gt;
	&lt;li&gt;lustre/mdd/mdd_object.c&lt;/li&gt;
&lt;/ul&gt;
</comment>
                            <comment id="12986" author="hudson" created="Mon, 18 Apr 2011 13:37:11 +0000"  >&lt;p&gt;Integrated in &lt;span class=&quot;image-wrap&quot; style=&quot;&quot;&gt;&lt;img src=&quot;http://newbuild.whamcloud.com/images/16x16/blue.png&quot; style=&quot;border: 0px solid black&quot; /&gt;&lt;/span&gt; &lt;a href=&quot;http://newbuild.whamcloud.com/job/lustre-master/./build_type=server,platform=el5-i686/29/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;lustre-master &#187; server,el5-i686 #29&lt;/a&gt;&lt;br/&gt;
     &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-191&quot; title=&quot;mdd_close will always start transaction even atime is not changed&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-191&quot;&gt;&lt;del&gt;LU-191&lt;/del&gt;&lt;/a&gt; mdd_close always set atime even it&apos;s same&lt;/p&gt;

&lt;p&gt;Oleg Drokin : &lt;a href=&quot;http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;amp;a=commit&amp;amp;h=d65f789f433f2d52e82116c374d88906321f9d28&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;d65f789f433f2d52e82116c374d88906321f9d28&lt;/a&gt;&lt;br/&gt;
Files : &lt;/p&gt;
&lt;ul&gt;
	&lt;li&gt;lustre/mdd/mdd_object.c&lt;/li&gt;
&lt;/ul&gt;
</comment>
                            <comment id="23980" author="liang" created="Thu, 8 Dec 2011 23:27:53 +0000"  >&lt;p&gt;patch already landed in master&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|hzv9hb:</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_10090" key="com.pyxis.greenhopper.jira:gh-global-rank">
                        <customfieldname>Rank (Obsolete)</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>5070</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>