Whamcloud Community JIRA https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 [LU-5577] comparison between signed and unsigned integer expressions https://jira.whamcloud.com/browse/LU-5577 Lustre <p>This ticket is for cleanup patches that should remove warnings about comparison between signed and unsigned integer expressions.</p> <p>Some of them cause real issue in code.</p> LU-5577 comparison between signed and unsigned integer expressions Bug Minor Resolved Fixed Dmitry Eremin Dmitry Eremin Wed, 3 Sep 2014 09:20:41 +0000 Mon, 31 Jan 2022 03:54:55 +0000 Thu, 26 Feb 2015 22:36:43 +0000 Lustre 2.7.0 Lustre 2.7.0 0 4 <p>Patches are: <a href="http://review.whamcloud.com/#/q/message:LU-5577+is:open,n,z" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/#/q/message:LU-5577+is:open,n,z</a></p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/12385/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12385/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-5577" title="comparison between signed and unsigned integer expressions" class="issue-link" data-issue-key="LU-5577"><del>LU-5577</del></a> obdclass: lu_htable_order() return type to long<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 8475dda6d6d96ff021f045e15c3a65fbd7736de8</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/12387/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12387/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-5577" title="comparison between signed and unsigned integer expressions" class="issue-link" data-issue-key="LU-5577"><del>LU-5577</del></a> obdclass: change loop indexes to unsigned<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: c1352f54dba21d36581001043f7182f561c87f95</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/12378/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12378/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-5577" title="comparison between signed and unsigned integer expressions" class="issue-link" data-issue-key="LU-5577"><del>LU-5577</del></a> obd: change brw_page-&gt;count to unsigned<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 34c03a21c044af07ea3dd9008c154b77107b55fb</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/12379/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12379/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-5577" title="comparison between signed and unsigned integer expressions" class="issue-link" data-issue-key="LU-5577"><del>LU-5577</del></a> libcfs: fix warnings in libcfs/curproc.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: d39408943a56f452d68b3c8b170ee4ce10e416fa</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/12380/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12380/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-5577" title="comparison between signed and unsigned integer expressions" class="issue-link" data-issue-key="LU-5577"><del>LU-5577</del></a> obdclass: change cl_fault_io-&gt;ft_nob to size_t<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: ca58021173bbfe465a13cd34cc42b12fe5af31e4</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/12382/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12382/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-5577" title="comparison between signed and unsigned integer expressions" class="issue-link" data-issue-key="LU-5577"><del>LU-5577</del></a> obdclass: (*dt_index_page_build_t) arg nob to size_t<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: a2245cbf99d15a24605e355f26b7612895ae2287</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/12388/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12388/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-5577" title="comparison between signed and unsigned integer expressions" class="issue-link" data-issue-key="LU-5577"><del>LU-5577</del></a> obdclass: do_body_ops-&gt;dbo_</p> {read,write} <p> return ssize_t<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 83efcf39e2f88ecd3cf5162c0f5cead2c39a5de3</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/12475/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12475/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-5577" title="comparison between signed and unsigned integer expressions" class="issue-link" data-issue-key="LU-5577"><del>LU-5577</del></a> ptlrpc: fix comparison between signed and unsigned<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 24c4f3a53b9ca5ec2c6f8c71e166f579d8838137</p> <p>Dmitry Eremin (dmitry.eremin@intel.com) uploaded a new patch: <a href="http://review.whamcloud.com/13125" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13125</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-5577" title="comparison between signed and unsigned integer expressions" class="issue-link" data-issue-key="LU-5577"><del>LU-5577</del></a> obd: change type of cl_conn_count to size_t<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 08889454c636be25f4151ae322af0c1312d6fb08</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/12386/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12386/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-5577" title="comparison between signed and unsigned integer expressions" class="issue-link" data-issue-key="LU-5577"><del>LU-5577</del></a> osc: change cl_extent_tax and *grants to unsigned<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: a2cbc7d543a54f33d4e84bcd0465b7029aa1c82d</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/13125/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13125/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-5577" title="comparison between signed and unsigned integer expressions" class="issue-link" data-issue-key="LU-5577"><del>LU-5577</del></a> obd: change type of cl_conn_count to size_t<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 5e17af16a920c56eba438faf7cb66f55c70e0b79</p> <p>All patches have been merged.</p> <p>Unfortunately there are many warnings still exist. I'm going to continue this work. So, new patches will be coming...</p> <p>All patches under this bug have been landed for 2.7.0. Closing it and opened <a href="https://jira.whamcloud.com/browse/LU-6303" title="comparison between signed and unsigned integer expressions" class="issue-link" data-issue-key="LU-6303"><del>LU-6303</del></a> for future patches.</p> Related LU-6303 LU-2753 LU-6215 LU-5439 Development Rank 1|hzwv7j: Rank (Obsolete) 15557 Severity