<!-- 
RSS generated by JIRA (9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c) at Sat Feb 10 01:20:36 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-1892] In-tree documentation improvements</title>
                <link>https://jira.whamcloud.com/browse/LU-1892</link>
                <project id="10000" key="LU">Lustre</project>
                    <description>&lt;p&gt;The Lustre code base has a number undocumented areas, making it difficult for new developers to get their bearings. This issue is to track patches that add comments and documentation to the Lustre source tree, and to discuss how to approach the problem of documenting the code better. In particular, I have in mind the following types of improvements.&lt;/p&gt;

&lt;ul&gt;
	&lt;li&gt;Doxygen comment headers on all non-trivial functions (See &lt;a href=&quot;http://wiki.lustre.org/index.php/Documenting_Code&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;guidelines&lt;/a&gt;)&lt;/li&gt;
	&lt;li&gt;More thorough high-level description at the top of each file stating its reason for existing&lt;/li&gt;
	&lt;li&gt;Description of how the code is organized, i.e. what each directory is for&lt;/li&gt;
	&lt;li&gt;High level overview of Lustre internal architecture&lt;/li&gt;
	&lt;li&gt;Deep-dives into specific subsystems (i.e. &lt;a href=&quot;http://wiki.lustre.org/images/3/37/CLIO-TOI-notes.pdf&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://wiki.lustre.org/images/3/37/CLIO-TOI-notes.pdf&lt;/a&gt; for CLIO)&lt;/li&gt;
&lt;/ul&gt;


&lt;p&gt;While one could argue that the architectural content belongs in an external developers manual, developing it in-tree has several benefits&lt;/p&gt;

&lt;ul&gt;
	&lt;li&gt;It is easier for someone poking around in the code to quickly come up to speed&lt;/li&gt;
	&lt;li&gt;Code changes can include corresponding documentation updates in the same patch&lt;/li&gt;
	&lt;li&gt;Plain text documentation can be evolved rapidly and informally and later incorporated into a formal manual&lt;/li&gt;
	&lt;li&gt;It is consistent with how the Documentation directory is used in the Linux kernel&lt;/li&gt;
&lt;/ul&gt;
</description>
                <environment></environment>
        <key id="15881">LU-1892</key>
            <summary>In-tree documentation improvements</summary>
                <type id="4" iconUrl="https://jira.whamcloud.com/secure/viewavatar?size=xsmall&amp;avatarId=11310&amp;avatarType=issuetype">Improvement</type>
                                            <priority id="4" iconUrl="https://jira.whamcloud.com/images/icons/priorities/minor.svg">Minor</priority>
                        <status id="1" iconUrl="https://jira.whamcloud.com/images/icons/statuses/open.png" description="The issue is open and ready for the assignee to start work on it.">Open</status>
                    <statusCategory id="2" key="new" colorName="default"/>
                                    <resolution id="-1">Unresolved</resolution>
                                        <assignee username="pjones">Peter Jones</assignee>
                                    <reporter username="nedbass">Ned Bass</reporter>
                        <labels>
                            <label>documentation</label>
                    </labels>
                <created>Tue, 11 Sep 2012 12:18:21 +0000</created>
                <updated>Tue, 7 Jun 2016 15:38:58 +0000</updated>
                                                                                <due></due>
                            <votes>0</votes>
                                    <watches>4</watches>
                                                                            <comments>
                            <comment id="44614" author="adilger" created="Tue, 11 Sep 2012 17:35:13 +0000"  >&lt;p&gt;Ned, I was poking around Bugzilla last week, and in fact there appear to be a number of bugs with patches for DOxygen comments in them that are still open.  It would probably make a lot of sense to take a look at those patches, and either determine they have already landed and close the bugs, or update the patches for master under this patch (and still close those bugs).&lt;/p&gt;</comment>
                            <comment id="44615" author="nedbass" created="Tue, 11 Sep 2012 17:47:50 +0000"  >&lt;p&gt;Good idea, I&apos;ll search for them and start evaluating.&lt;/p&gt;</comment>
                            <comment id="44630" author="nedbass" created="Tue, 11 Sep 2012 20:10:52 +0000"  >&lt;p&gt;Here are the open tickets I found and their status.&lt;/p&gt;

&lt;div class=&apos;table-wrap&apos;&gt;
&lt;table class=&apos;confluenceTable&apos;&gt;&lt;tbody&gt;
&lt;tr&gt;
&lt;th class=&apos;confluenceTh&apos;&gt;bz &lt;/th&gt;
&lt;th class=&apos;confluenceTh&apos;&gt; summary &lt;/th&gt;
&lt;th class=&apos;confluenceTh&apos;&gt; patch status &lt;/th&gt;
&lt;th class=&apos;confluenceTh&apos;&gt; Assignee/Author &lt;/th&gt;
&lt;/tr&gt;
&lt;tr&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; &lt;a href=&quot;https://bugzilla.lustre.org/show_bug.cgi?id=20943&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;20943&lt;/a&gt; &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Doxygen comments - ldlm module              &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; inspection+/not landed &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Oleg &lt;/td&gt;
&lt;/tr&gt;
&lt;tr&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; &lt;a href=&quot;https://bugzilla.lustre.org/show_bug.cgi?id=20969&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;20969&lt;/a&gt; &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Doxygen comments - ptlrpc module            &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; landed &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Oleg &lt;/td&gt;
&lt;/tr&gt;
&lt;tr&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; &lt;a href=&quot;https://bugzilla.lustre.org/show_bug.cgi?id=20973&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;20973&lt;/a&gt; &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Doxygen comments - lnet module              &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; landed &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Isaac Huang &lt;/td&gt;
&lt;/tr&gt;
&lt;tr&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; &lt;a href=&quot;https://bugzilla.lustre.org/show_bug.cgi?id=20974&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;20974&lt;/a&gt; &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Doxygen comments - libcfs module            &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; inspection+/not landed &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Maxim Patlasov &lt;/td&gt;
&lt;/tr&gt;
&lt;tr&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; &lt;a href=&quot;https://bugzilla.lustre.org/show_bug.cgi?id=20976&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;20976&lt;/a&gt; &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Doxygen comments - liblustre module         &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; nonexistent &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Oleg &lt;/td&gt;
&lt;/tr&gt;
&lt;tr&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; &lt;a href=&quot;https://bugzilla.lustre.org/show_bug.cgi?id=21833&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;21833&lt;/a&gt; &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Doxygen comments - lustre target code       &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; inspection-/not landed &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Mikhail &lt;/td&gt;
&lt;/tr&gt;
&lt;tr&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; &lt;a href=&quot;https://bugzilla.lustre.org/show_bug.cgi?id=23011&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;23011&lt;/a&gt; &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Doxygen comments - OSD module               &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; nonexistent &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Mikhail &lt;/td&gt;
&lt;/tr&gt;
&lt;tr&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; &lt;a href=&quot;https://bugzilla.lustre.org/show_bug.cgi?id=23718&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;23718&lt;/a&gt; &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Doxygen style comments on gss/kerberos code &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; landed &lt;/td&gt;
&lt;td class=&apos;confluenceTd&apos;&gt; Eric Mei &lt;/td&gt;
&lt;/tr&gt;
&lt;/tbody&gt;&lt;/table&gt;
&lt;/div&gt;
</comment>
                            <comment id="82805" author="spitzcor" created="Tue, 29 Apr 2014 21:37:13 +0000"  >&lt;p&gt;FYI: Hugo F. took an action item from the CDWG LUG meeting to work out whether we can start building on the LID (&lt;a href=&quot;http://wiki.lustre.org/lid&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://wiki.lustre.org/lid&lt;/a&gt;) as part of the lustre.org asset assignment to OpenSFS and EOFS.&lt;/p&gt;</comment>
                            <comment id="89863" author="nedbass" created="Wed, 23 Jul 2014 19:25:16 +0000"  >&lt;p&gt;After reviewing many of the patches in the sub-task issues, I think we could standardize and improve how we use the doxygen command &lt;tt&gt;\retval&lt;/tt&gt;.  The format should be as follows (see &lt;a href=&quot;http://www.stack.nl/~dimitri/doxygen/manual/commands.html#cmdretval&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;manual&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;\retval &amp;lt;return value&amp;gt; { description }
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;

&lt;p&gt;Where the first two fields are separated by a single space, &amp;lt;return value&amp;gt; is a single word (positive, negative, pointer, size, etc.), and the description is tab-indented.  For example,&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; * \retval 0                 FID lookup succeeded
 * \retval negative          negated errno if lookup failed
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;

&lt;p&gt;This convention would improve code readability and the quality of automatically generated documentation.&lt;/p&gt;</comment>
                            <comment id="100660" author="gerrit" created="Thu, 4 Dec 2014 13:47:26 +0000"  >&lt;p&gt;Oleg Drokin (oleg.drokin@intel.com) merged in patch &lt;a href=&quot;http://review.whamcloud.com/12659/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://review.whamcloud.com/12659/&lt;/a&gt;&lt;br/&gt;
Subject: &lt;a href=&quot;https://jira.whamcloud.com/browse/LU-1892&quot; title=&quot;In-tree documentation improvements&quot; class=&quot;issue-link&quot; data-issue-key=&quot;LU-1892&quot;&gt;LU-1892&lt;/a&gt; osp: Fix Doxygen warnings for osp_trans.c&lt;br/&gt;
Project: fs/lustre-release&lt;br/&gt;
Branch: master&lt;br/&gt;
Current Patch Set: &lt;br/&gt;
Commit: 99a05bdd378be62c31e0e2320310871eb9c9b68b&lt;/p&gt;</comment>
                    </comments>
                    <attachments>
                    </attachments>
                <subtasks>
                            <subtask id="15915">LU-1914</subtask>
                            <subtask id="15969">LU-1939</subtask>
                            <subtask id="16418">LU-2220</subtask>
                            <subtask id="24483">LU-4974</subtask>
                            <subtask id="24484">LU-4975</subtask>
                            <subtask id="24485">LU-4976</subtask>
                            <subtask id="24303">LU-4932</subtask>
                    </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_10490" key="com.atlassian.jira.plugin.system.customfieldtypes:datepicker">
                        <customfieldname>End date</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>Wed, 10 Dec 2014 12:31:24 +0000</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                            <customfield id="customfield_10390" key="com.pyxis.greenhopper.jira:gh-lexo-rank">
                        <customfieldname>Rank</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>1|hzw0uf:</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_10090" key="com.pyxis.greenhopper.jira:gh-global-rank">
                        <customfieldname>Rank (Obsolete)</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>10217</customfieldvalue>
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                <customfield id="customfield_10493" key="com.atlassian.jira.plugin.system.customfieldtypes:datepicker">
                        <customfieldname>Start date</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>Fri, 14 Sep 2012 12:31:24 +0000</customfieldvalue>

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