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

comparison between signed and unsigned integer expressions

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.9.0
    • Lustre 2.7.0
    • None
    • 3
    • 17647

    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

          Activity

            [LU-6303] comparison between signed and unsigned integer expressions
            pjones Peter Jones added a comment -

            Landed for 2.9

            pjones Peter Jones added a comment - Landed for 2.9

            Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/23155/
            Subject: LU-6303 osc: remove handling cl_avail_grant less than zero
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 6936a19b9c2e6b8c2a9ccff1357d07ba86008510

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/23155/ Subject: LU-6303 osc: remove handling cl_avail_grant less than zero Project: fs/lustre-release Branch: master Current Patch Set: Commit: 6936a19b9c2e6b8c2a9ccff1357d07ba86008510

            James Simmons (uja.ornl@yahoo.com) uploaded a new patch: http://review.whamcloud.com/23155
            Subject: LU-6303 osc: remove handling cl_avail_grant less than zero
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 972a6cdefb50bf8f33ff7f3dd104c1ef779adb91

            gerrit Gerrit Updater added a comment - James Simmons (uja.ornl@yahoo.com) uploaded a new patch: http://review.whamcloud.com/23155 Subject: LU-6303 osc: remove handling cl_avail_grant less than zero Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 972a6cdefb50bf8f33ff7f3dd104c1ef779adb91

            yes, there are many warnings still exist. Not sure I can fix them fast in nearest future. This is long term project. My assumption is we should clean them time to time during code cleanup.

            dmiter Dmitry Eremin (Inactive) added a comment - yes, there are many warnings still exist. Not sure I can fix them fast in nearest future. This is long term project. My assumption is we should clean them time to time during code cleanup.

            Dmitry is any work left for this?

            simmonsja James A Simmons added a comment - Dmitry is any work left for this?

            People

              dmiter Dmitry Eremin (Inactive)
              adilger Andreas Dilger
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: