https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 https://jira.whamcloud.com/browse/LU-871 Lustre <p>Tracking bug for warnings produced by the clang-analyzer static code analysis tool.</p> <p>The first one is a trivial printk() format warning due to the use of "%Lu" for LPU64, and other places in the code. The "%L" format specifier is a GCCism that is deprecated in favor of "%ll" from C99.</p> FC13 running clang-analyzer-2.8-10.fc13.x86_64 LU-871

clang: minor code cleanups

Technical task LU-2753 Minor Resolved Fixed Andreas Dilger Andreas Dilger clang Mon, 21 Nov 2011 17:47:18 +0000 Tue, 5 Feb 2013 17:41:43 +0000 Tue, 5 Feb 2013 17:41:24 +0000 Lustre 2.2.0 Lustre 2.2.0 1 5 <p>build: change %L printk format to %ll: <a href="http://review.whamcloud.com/1734" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/1734</a></p> <p>Integrated in <span class="image-wrap" style=""><img src="http://build.whamcloud.com/images/16x16/blue.png" style="border: 0px solid black" /></span> <a href="http://build.whamcloud.com/job/lustre-master/./arch=x86_64,build_type=client,distro=sles11,ib_stack=inkernel/378/" class="external-link" target="_blank" rel="nofollow noopener">lustre-master » x86_64,client,sles11,inkernel #378</a><br/> <a href="https://jira.whamcloud.com/browse/LU-871" title="clang: minor code cleanups" class="issue-link" data-issue-key="LU-871"><del>LU-871</del></a> build: change %L printk format to %ll (Revision 3684de257a7e8932936a27d0fdb3ecb39653e624)</p> <p> Result = SUCCESS<br/> Oleg Drokin : <a href="http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;a=commit&amp;h=3684de257a7e8932936a27d0fdb3ecb39653e624" class="external-link" target="_blank" rel="nofollow noopener">3684de257a7e8932936a27d0fdb3ecb39653e624</a><br/> Files : </p> <ul> <li>lustre/utils/gss/lgss_keyring.c</li> <li>lustre/tests/checkstat.c</li> <li>lustre/llite/rw.c</li> <li>lustre/fid/lproc_fid.c</li> <li>libcfs/include/libcfs/posix/posix-wordsize.h</li> <li>lustre/include/lustre_debug.h</li> <li>lnet/utils/routerstat.c</li> <li>lustre/utils/gss/lgss_utils.c</li> <li>lustre/llite/file.c</li> <li>lustre/tests/truncate.c</li> <li>lustre/liblustre/super.c</li> <li>libcfs/include/libcfs/linux/kp30.h</li> </ul> <p>Integrated in <span class="image-wrap" style=""><img src="http://build.whamcloud.com/images/16x16/blue.png" style="border: 0px solid black" /></span> <a href="http://build.whamcloud.com/job/lustre-master/./arch=x86_64,build_type=client,distro=el6,ib_stack=inkernel/378/" class="external-link" target="_blank" rel="nofollow noopener">lustre-master » x86_64,client,el6,inkernel #378</a><br/> <a href="https://jira.whamcloud.com/browse/LU-871" title="clang: minor code cleanups" class="issue-link" data-issue-key="LU-871"><del>LU-871</del></a> build: change %L printk format to %ll (Revision 3684de257a7e8932936a27d0fdb3ecb39653e624)</p> <p> Result = SUCCESS<br/> Oleg Drokin : <a href="http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;a=commit&amp;h=3684de257a7e8932936a27d0fdb3ecb39653e624" class="external-link" target="_blank" rel="nofollow noopener">3684de257a7e8932936a27d0fdb3ecb39653e624</a><br/> Files : </p> <ul> <li>libcfs/include/libcfs/linux/kp30.h</li> <li>lustre/tests/checkstat.c</li> <li>lustre/utils/gss/lgss_utils.c</li> <li>lustre/tests/truncate.c</li> <li>lustre/llite/file.c</li> <li>lustre/liblustre/super.c</li> <li>libcfs/include/libcfs/posix/posix-wordsize.h</li> <li>lustre/include/lustre_debug.h</li> <li>lustre/llite/rw.c</li> <li>lustre/utils/gss/lgss_keyring.c</li> <li>lustre/fid/lproc_fid.c</li> <li>lnet/utils/routerstat.c</li> </ul> <p>Integrated in <span class="image-wrap" style=""><img src="http://build.whamcloud.com/images/16x16/blue.png" style="border: 0px solid black" /></span> <a href="http://build.whamcloud.com/job/lustre-master/./arch=x86_64,build_type=server,distro=el6,ib_stack=inkernel/378/" class="external-link" target="_blank" rel="nofollow noopener">lustre-master » x86_64,server,el6,inkernel #378</a><br/> <a href="https://jira.whamcloud.com/browse/LU-871" title="clang: minor code cleanups" class="issue-link" data-issue-key="LU-871"><del>LU-871</del></a> build: change %L printk format to %ll (Revision 3684de257a7e8932936a27d0fdb3ecb39653e624)</p> <p> Result = SUCCESS<br/> Oleg Drokin : <a href="http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;a=commit&amp;h=3684de257a7e8932936a27d0fdb3ecb39653e624" class="external-link" target="_blank" rel="nofollow noopener">3684de257a7e8932936a27d0fdb3ecb39653e624</a><br/> Files : </p> <ul> <li>lustre/utils/gss/lgss_utils.c</li> <li>lustre/include/lustre_debug.h</li> <li>lustre/liblustre/super.c</li> <li>libcfs/include/libcfs/posix/posix-wordsize.h</li> <li>lustre/fid/lproc_fid.c</li> <li>lustre/utils/gss/lgss_keyring.c</li> <li>lnet/utils/routerstat.c</li> <li>lustre/tests/truncate.c</li> <li>lustre/llite/file.c</li> <li>libcfs/include/libcfs/linux/kp30.h</li> <li>lustre/llite/rw.c</li> <li>lustre/tests/checkstat.c</li> </ul> <p>Integrated in <span class="image-wrap" style=""><img src="http://build.whamcloud.com/images/16x16/blue.png" style="border: 0px solid black" /></span> <a href="http://build.whamcloud.com/job/lustre-master/./arch=x86_64,build_type=client,distro=el5,ib_stack=ofa/378/" class="external-link" target="_blank" rel="nofollow noopener">lustre-master » x86_64,client,el5,ofa #378</a><br/> <a href="https://jira.whamcloud.com/browse/LU-871" title="clang: minor code cleanups" class="issue-link" data-issue-key="LU-871"><del>LU-871</del></a> build: change %L printk format to %ll (Revision 3684de257a7e8932936a27d0fdb3ecb39653e624)</p> <p> Result = SUCCESS<br/> Oleg Drokin : <a href="http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;a=commit&amp;h=3684de257a7e8932936a27d0fdb3ecb39653e624" class="external-link" target="_blank" rel="nofollow noopener">3684de257a7e8932936a27d0fdb3ecb39653e624</a><br/> Files : </p> <ul> <li>lustre/liblustre/super.c</li> <li>lustre/tests/truncate.c</li> <li>lustre/llite/rw.c</li> <li>lustre/fid/lproc_fid.c</li> <li>libcfs/include/libcfs/posix/posix-wordsize.h</li> <li>libcfs/include/libcfs/linux/kp30.h</li> <li>lustre/tests/checkstat.c</li> <li>lnet/utils/routerstat.c</li> <li>lustre/utils/gss/lgss_utils.c</li> <li>lustre/utils/gss/lgss_keyring.c</li> <li>lustre/llite/file.c</li> <li>lustre/include/lustre_debug.h</li> </ul> <p>Integrated in <span class="image-wrap" style=""><img src="http://build.whamcloud.com/images/16x16/blue.png" style="border: 0px solid black" /></span> <a href="http://build.whamcloud.com/job/lustre-master/./arch=i686,build_type=client,distro=el6,ib_stack=inkernel/378/" class="external-link" target="_blank" rel="nofollow noopener">lustre-master » i686,client,el6,inkernel #378</a><br/> <a href="https://jira.whamcloud.com/browse/LU-871" title="clang: minor code cleanups" class="issue-link" data-issue-key="LU-871"><del>LU-871</del></a> build: change %L printk format to %ll (Revision 3684de257a7e8932936a27d0fdb3ecb39653e624)</p> <p> Result = SUCCESS<br/> Oleg Drokin : <a href="http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;a=commit&amp;h=3684de257a7e8932936a27d0fdb3ecb39653e624" class="external-link" target="_blank" rel="nofollow noopener">3684de257a7e8932936a27d0fdb3ecb39653e624</a><br/> Files : </p> <ul> <li>lustre/tests/checkstat.c</li> <li>lustre/tests/truncate.c</li> <li>libcfs/include/libcfs/linux/kp30.h</li> <li>lustre/liblustre/super.c</li> <li>libcfs/include/libcfs/posix/posix-wordsize.h</li> <li>lustre/utils/gss/lgss_utils.c</li> <li>lustre/fid/lproc_fid.c</li> <li>lustre/llite/rw.c</li> <li>lustre/llite/file.c</li> <li>lustre/utils/gss/lgss_keyring.c</li> <li>lustre/include/lustre_debug.h</li> <li>lnet/utils/routerstat.c</li> </ul> <p>Integrated in <span class="image-wrap" style=""><img src="http://build.whamcloud.com/images/16x16/blue.png" style="border: 0px solid black" /></span> <a href="http://build.whamcloud.com/job/lustre-master/./arch=x86_64,build_type=client,distro=el5,ib_stack=inkernel/378/" class="external-link" target="_blank" rel="nofollow noopener">lustre-master » x86_64,client,el5,inkernel #378</a><br/> <a href="https://jira.whamcloud.com/browse/LU-871" title="clang: minor code cleanups" class="issue-link" data-issue-key="LU-871"><del>LU-871</del></a> build: change %L printk format to %ll (Revision 3684de257a7e8932936a27d0fdb3ecb39653e624)</p> <p> Result = SUCCESS<br/> Oleg Drokin : <a href="http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;a=commit&amp;h=3684de257a7e8932936a27d0fdb3ecb39653e624" class="external-link" target="_blank" rel="nofollow noopener">3684de257a7e8932936a27d0fdb3ecb39653e624</a><br/> Files : </p> <ul> <li>lnet/utils/routerstat.c</li> <li>lustre/tests/truncate.c</li> <li>lustre/fid/lproc_fid.c</li> <li>lustre/liblustre/super.c</li> <li>libcfs/include/libcfs/linux/kp30.h</li> <li>lustre/tests/checkstat.c</li> <li>lustre/utils/gss/lgss_keyring.c</li> <li>lustre/include/lustre_debug.h</li> <li>lustre/llite/rw.c</li> <li>lustre/utils/gss/lgss_utils.c</li> <li>lustre/llite/file.c</li> <li>libcfs/include/libcfs/posix/posix-wordsize.h</li> </ul> <p>Integrated in <span class="image-wrap" style=""><img src="http://build.whamcloud.com/images/16x16/blue.png" style="border: 0px solid black" /></span> <a href="http://build.whamcloud.com/job/lustre-master/./arch=x86_64,build_type=client,distro=ubuntu1004,ib_stack=inkernel/378/" class="external-link" target="_blank" rel="nofollow noopener">lustre-master » x86_64,client,ubuntu1004,inkernel #378</a><br/> <a href="https://jira.whamcloud.com/browse/LU-871" title="clang: minor code cleanups" class="issue-link" data-issue-key="LU-871"><del>LU-871</del></a> build: change %L printk format to %ll (Revision 3684de257a7e8932936a27d0fdb3ecb39653e624)</p> <p> Result = SUCCESS<br/> Oleg Drokin : <a href="http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;a=commit&amp;h=3684de257a7e8932936a27d0fdb3ecb39653e624" class="external-link" target="_blank" rel="nofollow noopener">3684de257a7e8932936a27d0fdb3ecb39653e624</a><br/> Files : </p> <ul> <li>lustre/liblustre/super.c</li> <li>lnet/utils/routerstat.c</li> <li>lustre/utils/gss/lgss_utils.c</li> <li>lustre/tests/checkstat.c</li> <li>lustre/llite/rw.c</li> <li>lustre/tests/truncate.c</li> <li>libcfs/include/libcfs/linux/kp30.h</li> <li>lustre/include/lustre_debug.h</li> <li>lustre/llite/file.c</li> <li>lustre/fid/lproc_fid.c</li> <li>libcfs/include/libcfs/posix/posix-wordsize.h</li> <li>lustre/utils/gss/lgss_keyring.c</li> </ul> <p>Integrated in <span class="image-wrap" style=""><img src="http://build.whamcloud.com/images/16x16/blue.png" style="border: 0px solid black" /></span> <a href="http://build.whamcloud.com/job/lustre-master/./arch=x86_64,build_type=server,distro=el5,ib_stack=inkernel/378/" class="external-link" target="_blank" rel="nofollow noopener">lustre-master » x86_64,server,el5,inkernel #378</a><br/> <a href="https://jira.whamcloud.com/browse/LU-871" title="clang: minor code cleanups" class="issue-link" data-issue-key="LU-871"><del>LU-871</del></a> build: change %L printk format to %ll (Revision 3684de257a7e8932936a27d0fdb3ecb39653e624)</p> <p> Result = SUCCESS<br/> Oleg Drokin : <a href="http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;a=commit&amp;h=3684de257a7e8932936a27d0fdb3ecb39653e624" class="external-link" target="_blank" rel="nofollow noopener">3684de257a7e8932936a27d0fdb3ecb39653e624</a><br/> Files : </p> <ul> <li>lustre/tests/truncate.c</li> <li>lustre/utils/gss/lgss_utils.c</li> <li>lustre/llite/file.c</li> <li>lustre/llite/rw.c</li> <li>lustre/include/lustre_debug.h</li> <li>lustre/fid/lproc_fid.c</li> <li>lustre/utils/gss/lgss_keyring.c</li> <li>libcfs/include/libcfs/posix/posix-wordsize.h</li> <li>libcfs/include/libcfs/linux/kp30.h</li> <li>lustre/tests/checkstat.c</li> <li>lustre/liblustre/super.c</li> <li>lnet/utils/routerstat.c</li> </ul> <p>Integrated in <span class="image-wrap" style=""><img src="http://build.whamcloud.com/images/16x16/blue.png" style="border: 0px solid black" /></span> <a href="http://build.whamcloud.com/job/lustre-master/./arch=i686,build_type=client,distro=el5,ib_stack=ofa/378/" class="external-link" target="_blank" rel="nofollow noopener">lustre-master » i686,client,el5,ofa #378</a><br/> <a href="https://jira.whamcloud.com/browse/LU-871" title="clang: minor code cleanups" class="issue-link" data-issue-key="LU-871"><del>LU-871</del></a> build: change %L printk format to %ll (Revision 3684de257a7e8932936a27d0fdb3ecb39653e624)</p> <p> Result = SUCCESS<br/> Oleg Drokin : <a href="http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;a=commit&amp;h=3684de257a7e8932936a27d0fdb3ecb39653e624" class="external-link" target="_blank" rel="nofollow noopener">3684de257a7e8932936a27d0fdb3ecb39653e624</a><br/> Files : </p> <ul> <li>lnet/utils/routerstat.c</li> <li>libcfs/include/libcfs/linux/kp30.h</li> <li>libcfs/include/libcfs/posix/posix-wordsize.h</li> <li>lustre/llite/file.c</li> <li>lustre/llite/rw.c</li> <li>lustre/liblustre/super.c</li> <li>lustre/tests/checkstat.c</li> <li>lustre/include/lustre_debug.h</li> <li>lustre/utils/gss/lgss_utils.c</li> <li>lustre/utils/gss/lgss_keyring.c</li> <li>lustre/tests/truncate.c</li> <li>lustre/fid/lproc_fid.c</li> </ul> <p>Integrated in <span class="image-wrap" style=""><img src="http://build.whamcloud.com/images/16x16/blue.png" style="border: 0px solid black" /></span> <a href="http://build.whamcloud.com/job/lustre-master/./arch=i686,build_type=client,distro=el5,ib_stack=inkernel/378/" class="external-link" target="_blank" rel="nofollow noopener">lustre-master » i686,client,el5,inkernel #378</a><br/> <a href="https://jira.whamcloud.com/browse/LU-871" title="clang: minor code cleanups" class="issue-link" data-issue-key="LU-871"><del>LU-871</del></a> build: change %L printk format to %ll (Revision 3684de257a7e8932936a27d0fdb3ecb39653e624)</p> <p> Result = SUCCESS<br/> Oleg Drokin : <a href="http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;a=commit&amp;h=3684de257a7e8932936a27d0fdb3ecb39653e624" class="external-link" target="_blank" rel="nofollow noopener">3684de257a7e8932936a27d0fdb3ecb39653e624</a><br/> Files : </p> <ul> <li>lustre/tests/truncate.c</li> <li>lustre/liblustre/super.c</li> <li>libcfs/include/libcfs/posix/posix-wordsize.h</li> <li>lustre/tests/checkstat.c</li> <li>lnet/utils/routerstat.c</li> <li>lustre/include/lustre_debug.h</li> <li>lustre/utils/gss/lgss_utils.c</li> <li>lustre/llite/file.c</li> <li>lustre/utils/gss/lgss_keyring.c</li> <li>libcfs/include/libcfs/linux/kp30.h</li> <li>lustre/fid/lproc_fid.c</li> <li>lustre/llite/rw.c</li> </ul> <p>Integrated in <span class="image-wrap" style=""><img src="http://build.whamcloud.com/images/16x16/red.png" style="border: 0px solid black" /></span> <a href="http://build.whamcloud.com/job/lustre-master/./arch=x86_64,build_type=server,distro=el5,ib_stack=ofa/378/" class="external-link" target="_blank" rel="nofollow noopener">lustre-master » x86_64,server,el5,ofa #378</a><br/> <a href="https://jira.whamcloud.com/browse/LU-871" title="clang: minor code cleanups" class="issue-link" data-issue-key="LU-871"><del>LU-871</del></a> build: change %L printk format to %ll (Revision 3684de257a7e8932936a27d0fdb3ecb39653e624)</p> <p> Result = FAILURE<br/> Oleg Drokin : <a href="http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;a=commit&amp;h=3684de257a7e8932936a27d0fdb3ecb39653e624" class="external-link" target="_blank" rel="nofollow noopener">3684de257a7e8932936a27d0fdb3ecb39653e624</a><br/> Files : </p> <ul> <li>libcfs/include/libcfs/linux/kp30.h</li> <li>lustre/llite/file.c</li> <li>lustre/utils/gss/lgss_utils.c</li> <li>lustre/utils/gss/lgss_keyring.c</li> <li>lustre/tests/truncate.c</li> <li>libcfs/include/libcfs/posix/posix-wordsize.h</li> <li>lustre/llite/rw.c</li> <li>lnet/utils/routerstat.c</li> <li>lustre/tests/checkstat.c</li> <li>lustre/fid/lproc_fid.c</li> <li>lustre/liblustre/super.c</li> <li>lustre/include/lustre_debug.h</li> </ul> <p>Integrated in <span class="image-wrap" style=""><img src="http://build.whamcloud.com/images/16x16/blue.png" style="border: 0px solid black" /></span> <a href="http://build.whamcloud.com/job/lustre-master/./arch=i686,build_type=server,distro=el5,ib_stack=inkernel/378/" class="external-link" target="_blank" rel="nofollow noopener">lustre-master » i686,server,el5,inkernel #378</a><br/> <a href="https://jira.whamcloud.com/browse/LU-871" title="clang: minor code cleanups" class="issue-link" data-issue-key="LU-871"><del>LU-871</del></a> build: change %L printk format to %ll (Revision 3684de257a7e8932936a27d0fdb3ecb39653e624)</p> <p> Result = SUCCESS<br/> Oleg Drokin : <a href="http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;a=commit&amp;h=3684de257a7e8932936a27d0fdb3ecb39653e624" class="external-link" target="_blank" rel="nofollow noopener">3684de257a7e8932936a27d0fdb3ecb39653e624</a><br/> Files : </p> <ul> <li>lustre/utils/gss/lgss_keyring.c</li> <li>lustre/liblustre/super.c</li> <li>lustre/utils/gss/lgss_utils.c</li> <li>lustre/llite/rw.c</li> <li>lustre/tests/checkstat.c</li> <li>lustre/llite/file.c</li> <li>lnet/utils/routerstat.c</li> <li>lustre/tests/truncate.c</li> <li>lustre/fid/lproc_fid.c</li> <li>lustre/include/lustre_debug.h</li> <li>libcfs/include/libcfs/linux/kp30.h</li> <li>libcfs/include/libcfs/posix/posix-wordsize.h</li> </ul> <p>Integrated in <span class="image-wrap" style=""><img src="http://build.whamcloud.com/images/16x16/blue.png" style="border: 0px solid black" /></span> <a href="http://build.whamcloud.com/job/lustre-master/./arch=i686,build_type=server,distro=el6,ib_stack=inkernel/378/" class="external-link" target="_blank" rel="nofollow noopener">lustre-master » i686,server,el6,inkernel #378</a><br/> <a href="https://jira.whamcloud.com/browse/LU-871" title="clang: minor code cleanups" class="issue-link" data-issue-key="LU-871"><del>LU-871</del></a> build: change %L printk format to %ll (Revision 3684de257a7e8932936a27d0fdb3ecb39653e624)</p> <p> Result = SUCCESS<br/> Oleg Drokin : <a href="http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;a=commit&amp;h=3684de257a7e8932936a27d0fdb3ecb39653e624" class="external-link" target="_blank" rel="nofollow noopener">3684de257a7e8932936a27d0fdb3ecb39653e624</a><br/> Files : </p> <ul> <li>lustre/llite/rw.c</li> <li>libcfs/include/libcfs/linux/kp30.h</li> <li>lustre/utils/gss/lgss_utils.c</li> <li>lustre/fid/lproc_fid.c</li> <li>lustre/tests/checkstat.c</li> <li>lustre/liblustre/super.c</li> <li>lustre/tests/truncate.c</li> <li>lustre/utils/gss/lgss_keyring.c</li> <li>lustre/llite/file.c</li> <li>lustre/include/lustre_debug.h</li> <li>lnet/utils/routerstat.c</li> <li>libcfs/include/libcfs/posix/posix-wordsize.h</li> </ul> <p>Integrated in <span class="image-wrap" style=""><img src="http://build.whamcloud.com/images/16x16/blue.png" style="border: 0px solid black" /></span> <a href="http://build.whamcloud.com/job/lustre-master/./arch=i686,build_type=server,distro=el5,ib_stack=ofa/378/" class="external-link" target="_blank" rel="nofollow noopener">lustre-master » i686,server,el5,ofa #378</a><br/> <a href="https://jira.whamcloud.com/browse/LU-871" title="clang: minor code cleanups" class="issue-link" data-issue-key="LU-871"><del>LU-871</del></a> build: change %L printk format to %ll (Revision 3684de257a7e8932936a27d0fdb3ecb39653e624)</p> <p> Result = SUCCESS<br/> Oleg Drokin : <a href="http://git.whamcloud.com/gitweb?p=fs/lustre-release.git;a=shortlog;h=refs/heads/master&amp;a=commit&amp;h=3684de257a7e8932936a27d0fdb3ecb39653e624" class="external-link" target="_blank" rel="nofollow noopener">3684de257a7e8932936a27d0fdb3ecb39653e624</a><br/> Files : </p> <ul> <li>lustre/utils/gss/lgss_utils.c</li> <li>lustre/llite/file.c</li> <li>lustre/utils/gss/lgss_keyring.c</li> <li>lustre/tests/checkstat.c</li> <li>lustre/fid/lproc_fid.c</li> <li>lustre/liblustre/super.c</li> <li>libcfs/include/libcfs/posix/posix-wordsize.h</li> <li>lustre/tests/truncate.c</li> <li>lnet/utils/routerstat.c</li> <li>lustre/llite/rw.c</li> <li>lustre/include/lustre_debug.h</li> <li>libcfs/include/libcfs/linux/kp30.h</li> </ul> <p>FWIW I've compiled Lustre 2.1.54 patchless client on CentOS 6.2 with Clang's static analyzer (LLVM 3.0).</p> <p>I decided to disable the static analyzer check "core.NullDereference" because it detects lots of bugs (521). However, based on a first short glance most of them are dubious because the analyzer seems to get confused by Lustre's LASSERT(). (And including all of these 521 bugs would make the tarball larger than the Jira upload file limit.)</p> <p>Here's the bug summary:</p> <div class='table-wrap'> <table class='confluenceTable'><tbody> <tr> <th class='confluenceTh'>Bug Type</th> <th class='confluenceTh'>Quantity</th> </tr> <tr> <td class='confluenceTd'>All Bugs</td> <td class='confluenceTd'>270</td> </tr> <tr> <td class='confluenceTd'><b>API</b></td> <td class='confluenceTd'>&nbsp;</td> </tr> <tr> <td class='confluenceTd'>Argument with 'nonnull' attribute passed null</td> <td class='confluenceTd'>16</td> </tr> <tr> <td class='confluenceTd'><b>Dead code</b></td> <td class='confluenceTd'>&nbsp;</td> </tr> <tr> <td class='confluenceTd'>Idempotent operation</td> <td class='confluenceTd'>30</td> </tr> <tr> <td class='confluenceTd'><b>Dead store</b></td> <td class='confluenceTd'>&nbsp;</td> </tr> <tr> <td class='confluenceTd'>Dead assignment</td> <td class='confluenceTd'>120</td> </tr> <tr> <td class='confluenceTd'>Dead increment</td> <td class='confluenceTd'>6</td> </tr> <tr> <td class='confluenceTd'>Dead initialization</td> <td class='confluenceTd'>20</td> </tr> <tr> <td class='confluenceTd'><b>Logic error</b></td> <td class='confluenceTd'>&nbsp;</td> </tr> <tr> <td class='confluenceTd'>Assigned value is garbage or undefined</td> <td class='confluenceTd'>12</td> </tr> <tr> <td class='confluenceTd'>Called function pointer is null (null dereference)</td> <td class='confluenceTd'>31</td> </tr> <tr> <td class='confluenceTd'>Division by zero</td> <td class='confluenceTd'>4</td> </tr> <tr> <td class='confluenceTd'>Function call argument is an uninitialized value</td> <td class='confluenceTd'>14</td> </tr> <tr> <td class='confluenceTd'>Garbage return value</td> <td class='confluenceTd'>6</td> </tr> <tr> <td class='confluenceTd'>Result of operation is garbage or undefined</td> <td class='confluenceTd'>8</td> </tr> <tr> <td class='confluenceTd'><b>Unix API</b></td> <td class='confluenceTd'>&nbsp;</td> </tr> <tr> <td class='confluenceTd'>Undefined allocation of 0 bytes</td> <td class='confluenceTd'>3</td> </tr> </tbody></table> </div> <p>Check out the annotated source in the attached tarball with a web browser.</p> <p>This is how the source was compiled:</p> <p>touch META<br/> sh autogen.sh <br/> scan-build ./configure --disable-server \<br/> --with-linux=/usr/src/kernels/2.6.32-220.2.1.el6.x86_64 \<br/> --with-linux-obj=/lib/modules/2.6.32-220.2.1.el6.x86_64/build/ \<br/> --with-downstream-release=wc1<br/> mkdir ./lustre-2.1.54-scan-build<br/> scan-build -o ./lustre-2.1.54-scan-build/ \<br/> -disable-checker core.NullDereference \<br/> make -j 24</p> <p>Karsten, thanks for submitting these patches. Sorry I didn't reply to this when it was first posted, as it slipped off my radar. Could you please submit these patches to our Gerrit (code review) git repository? Please see <a href="http://wiki.whamcloud.com/display/PUB/Using+Gerrit" class="external-link" target="_blank" rel="nofollow noopener">http://wiki.whamcloud.com/display/PUB/Using+Gerrit</a> for details.</p> <p>This will allow us to review, test, and land the patches to Lustre, and not have them sitting in an attachment here.</p> Related LU-2675 Development Rank 1|hzv7vz: Rank (Obsolete) 4812