[LU-5577] comparison between signed and unsigned integer expressions

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.7.0
• Affects Version/s: Lustre 2.7.0
• Severity: 3
• Assignee: Dmitry Eremin (Inactive)
• Reporter: Dmitry Eremin (Inactive)

Description

This ticket is for cleanup patches that should remove warnings about comparison between signed and unsigned integer expressions.

Some of them cause real issue in code.

Issue Links

• is related to: LU-5439 Function lustre_msg_buflen() can return -EINVAL but it's never checked
• is related to: LU-6303 comparison between signed and unsigned integer expressions
• is related to: LU-2753 Tracking bug for static code analysis fixes.
• is related to: LU-6215 Sync Lustre external tree with lustre linux kernel client

Comments

[Andreas Dilger]

All patches under this bug have been landed for 2.7.0. Closing it and opened LU-6303 for future patches.

[Dmitry Eremin (Inactive)]

Unfortunately there are many warnings still exist. I'm going to continue this work. So, new patches will be coming...

[James A Simmons]

All patches have been merged.

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/13125/
Subject: LU-5577 obd: change type of cl_conn_count to size_t
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 5e17af16a920c56eba438faf7cb66f55c70e0b79

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/12386/
Subject: LU-5577 osc: change cl_extent_tax and *grants to unsigned
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: a2cbc7d543a54f33d4e84bcd0465b7029aa1c82d

[Gerrit Updater]

Dmitry Eremin ([e-mail]) uploaded a new patch: http://review.whamcloud.com/13125
Subject: LU-5577 obd: change type of cl_conn_count to size_t
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 08889454c636be25f4151ae322af0c1312d6fb08

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/12475/
Subject: LU-5577 ptlrpc: fix comparison between signed and unsigned
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 24c4f3a53b9ca5ec2c6f8c71e166f579d8838137

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/12388/
Subject: LU-5577 obdclass: do_body_ops->dbo_

{read,write}

return ssize_t
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 83efcf39e2f88ecd3cf5162c0f5cead2c39a5de3

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/12382/
Subject: LU-5577 obdclass: (*dt_index_page_build_t) arg nob to size_t
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: a2245cbf99d15a24605e355f26b7612895ae2287

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/12380/
Subject: LU-5577 obdclass: change cl_fault_io->ft_nob to size_t
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: ca58021173bbfe465a13cd34cc42b12fe5af31e4