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

          Activity

            [LU-9091] Replace lprocfs_str_with_units_to_s64 with string_get_size()

            Yep its finished!!!

            simmonsja James A Simmons added a comment - Yep its finished!!!
            pjones Peter Jones added a comment -

            Finally?

            pjones Peter Jones added a comment - Finally?

            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

            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 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

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37620/
            Subject: LU-9091 obdclass: allow bare KMGTPE param suffix
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: d105cc4a6448b3c842790d0e1f0380b94d087f6f

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37620/ Subject: LU-9091 obdclass: allow bare KMGTPE param suffix Project: fs/lustre-release Branch: master Current Patch Set: Commit: d105cc4a6448b3c842790d0e1f0380b94d087f6f

            James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/37622
            Subject: LU-9091 sysfs: add memparse format support for defunit
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 63b1825928c5f7d6ed8bc906d5983c71a9d74c54

            gerrit Gerrit Updater added a comment - James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/37622 Subject: LU-9091 sysfs: add memparse format support for defunit Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 63b1825928c5f7d6ed8bc906d5983c71a9d74c54
            simmonsja James A Simmons added a comment - https://review.whamcloud.com/#/c/34849 https://review.whamcloud.com/#/c/37275

            James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/37274
            Subject: LU-9091 obd: move remaining sysfs that needs units to sysfs_memparse()
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 5a13c7eccc3df9f34d2088cdaa0a9439b4286c05

            gerrit Gerrit Updater added a comment - James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/37274 Subject: LU-9091 obd: move remaining sysfs that needs units to sysfs_memparse() Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 5a13c7eccc3df9f34d2088cdaa0a9439b4286c05

            Another patch is needed as well as a final cleanup patch.

            simmonsja James A Simmons added a comment - Another patch is needed as well as a final cleanup patch.

            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: