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

Replace lprocfs_str_with_units_to_s64 with string_get_size()

Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.14.0
    • None
    • 9223372036854775807

    Description

      The latest kernels have introduced a function that is very similar to what was done in LU-7334. The difference is the unit labels i.e M instead of MB. Upstream will be wanting us to use this standard function.

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-10517 setting llite.*.checksum_pages doesn't actually work

          • 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-11157 sanity test_42e: invalid arithmetic operator (error token is ".9")

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

          Improvement - An improvement or enhancement to an existing feature or task. LU-12511 Prepare lustre for adoption into the linux kernel

          • Major - Major loss of function.
          • Open
          is related to

          Improvement - An improvement or enhancement to an existing feature or task. LU-8066 Move lustre procfs handling to sysfs and debugfs.

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

          Improvement - An improvement or enhancement to an existing feature or task. LU-7334 Refactor lprocfs helpers

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

          Activity

            [LU-9091] Replace lprocfs_str_with_units_to_s64 with string_get_size()
            pjones Peter Jones made changes -
            Link New: This issue is related to DDN-1583 [ DDN-1583 ]
            simmonsja James A Simmons made changes -
            Status Original: Resolved [ 5 ] New: Closed [ 6 ]
            simmonsja James A Simmons added a comment -

            Yep its finished!!!

            simmonsja James A Simmons added a comment - Yep its finished!!!
            pjones Peter Jones made changes -
            Resolution New: Fixed [ 1 ]
            Status Original: Reopened [ 4 ] New: Resolved [ 5 ]
            pjones Peter Jones added a comment -

            Finally?

            pjones Peter Jones added a comment - Finally?
            gerrit Gerrit Updater added a comment -

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37275/
            Subject: LU-9091 procfs: remove old string parsing routines
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 48242f98c3add59d9e084d73057e64265230a19f

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37275/ Subject: LU-9091 procfs: remove old string parsing routines Project: fs/lustre-release Branch: master Current Patch Set: Commit: 48242f98c3add59d9e084d73057e64265230a19f
            simmonsja James A Simmons made changes -
            Resolution Original: Fixed [ 1 ]
            Status Original: Resolved [ 5 ] New: Reopened [ 4 ]
            simmonsja James A Simmons added a comment -

            One last cleanup patch to land - https://review.whamcloud.com/#/c/37275/

            simmonsja James A Simmons added a comment - One last cleanup patch to land -  https://review.whamcloud.com/#/c/37275/
            pjones Peter Jones made changes -
            Resolution New: Fixed [ 1 ]
            Status Original: Reopened [ 4 ] New: Resolved [ 5 ]
            pjones Peter Jones added a comment -

            Seems to have all landed for 2.14

            pjones Peter Jones added a comment - Seems to have all landed for 2.14

            People

              simmonsja James A Simmons
              simmonsja James A Simmons
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: