Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-5577

comparison between signed and unsigned integer expressions

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.7.0
    • Lustre 2.7.0
    • None
    • 3
    • 15557

    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.

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-5439 Function lustre_msg_buflen() can return -EINVAL but it's never checked

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved
          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-6303 comparison between signed and unsigned integer expressions

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. LU-2753 Tracking bug for static code analysis fixes.

          • Trivial - Cosmetic problem like misspelt words or misaligned text.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. LU-6215 Sync Lustre external tree with lustre linux kernel client

          • Major - Major loss of function.
          • Resolved

          Activity

            [LU-5577] comparison between signed and unsigned integer expressions
            adilger Andreas Dilger made changes -
            Link New: This issue is related to LU-5439 [ LU-5439 ]
            adilger Andreas Dilger made changes -
            Fix Version/s New: Lustre 2.7.0 [ 10631 ]
            Resolution New: Fixed [ 1 ]
            Status Original: In Progress [ 3 ] New: Resolved [ 5 ]
            adilger Andreas Dilger made changes -
            Link New: This issue is related to LU-6303 [ LU-6303 ]
            adilger Andreas Dilger added a comment -

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

            adilger Andreas Dilger added a comment - All patches under this bug have been landed for 2.7.0. Closing it and opened LU-6303 for future patches.
            adilger Andreas Dilger made changes -
            Link New: This issue is related to LU-2753 [ LU-2753 ]
            simmonsja James A Simmons made changes -
            Link New: This issue is related to LU-6215 [ LU-6215 ]
            dmiter Dmitry Eremin (Inactive) added a comment -

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

            dmiter Dmitry Eremin (Inactive) added a comment - Unfortunately there are many warnings still exist. I'm going to continue this work. So, new patches will be coming...
            simmonsja James A Simmons added a comment -

            All patches have been merged.

            simmonsja James A Simmons added a comment - All patches have been merged.
            gerrit Gerrit Updater added a comment -

            Oleg Drokin (oleg.drokin@intel.com) 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 Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) 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 Gerrit Updater added a comment -

            Oleg Drokin (oleg.drokin@intel.com) 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 Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) 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

            People

              dmiter Dmitry Eremin (Inactive)
              dmiter Dmitry Eremin (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: