Details

    • 3
    • 9223372036854775807

    Description

      Currently the ldlm.namespaces.*.lru_max_age parameter is printed in units of HZ, but that is not a useful unit of measure. Instead this should be exported to userspace in units of ms (which matches the HZ=1000 used by default on RHEL systems, so it won't be a user-visible change). The parameter read/written should be translated into units of HZ in lprocfs.

      It would actually be better to have this in units of seconds, but I don't think it is possible to change lru_max_age in a compatible way. Instead, a second parameter could be added like lru_max_seconds that converts from seconds to HZ to set the same parameter, but that is not very critical compared to having constant units between clients.

      Attachments

        Issue Links

          Activity

            [LU-8541] change lru_max_age to units of ms

            Joseph Gmitter (jgmitter@whamcloud.com) merged in patch https://review.whamcloud.com/33406/
            Subject: LU-8541 ldlm: describe lru_max_age better
            Project: doc/manual
            Branch: master
            Current Patch Set:
            Commit: 58e196f6c0e20bd70231e18171902dc42a4a1850

            gerrit Gerrit Updater added a comment - Joseph Gmitter (jgmitter@whamcloud.com) merged in patch https://review.whamcloud.com/33406/ Subject: LU-8541 ldlm: describe lru_max_age better Project: doc/manual Branch: master Current Patch Set: Commit: 58e196f6c0e20bd70231e18171902dc42a4a1850

            Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33406
            Subject: LU-8541 ldlm: describe lru_max_age
            Project: doc/manual
            Branch: master
            Current Patch Set: 1
            Commit: 7a78cba1cd13506ec0af02cb45d78c0107a3a860

            gerrit Gerrit Updater added a comment - Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33406 Subject: LU-8541 ldlm: describe lru_max_age Project: doc/manual Branch: master Current Patch Set: 1 Commit: 7a78cba1cd13506ec0af02cb45d78c0107a3a860

            John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/29265/
            Subject: LU-8541 ldlm: don't use jiffies as sysfs parameter
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set:
            Commit: 5ace209644a8c2ef021052967d528d66119647e2

            gerrit Gerrit Updater added a comment - John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/29265/ Subject: LU-8541 ldlm: don't use jiffies as sysfs parameter Project: fs/lustre-release Branch: b2_10 Current Patch Set: Commit: 5ace209644a8c2ef021052967d528d66119647e2

            Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/29265
            Subject: LU-8541 ldlm: don't use jiffies as sysfs parameter
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set: 1
            Commit: 0d78c50b74f36285d44b529b76fdc0131ca7d42c

            gerrit Gerrit Updater added a comment - Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/29265 Subject: LU-8541 ldlm: don't use jiffies as sysfs parameter Project: fs/lustre-release Branch: b2_10 Current Patch Set: 1 Commit: 0d78c50b74f36285d44b529b76fdc0131ca7d42c
            pjones Peter Jones added a comment -

            Landed for 2.11

            pjones Peter Jones added a comment - Landed for 2.11

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/28370/
            Subject: LU-8541 ldlm: don't use jiffies as sysfs parameter
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 800ffd4711863e1067a693f1283bccf4edddf2a2

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/28370/ Subject: LU-8541 ldlm: don't use jiffies as sysfs parameter Project: fs/lustre-release Branch: master Current Patch Set: Commit: 800ffd4711863e1067a693f1283bccf4edddf2a2

            James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/28370
            Subject: LU-8541 ldlm: don't use jiffies as sysfs parameter
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: ebb17919cd1098d6d12e51f2171120ec033fd826

            gerrit Gerrit Updater added a comment - James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/28370 Subject: LU-8541 ldlm: don't use jiffies as sysfs parameter Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: ebb17919cd1098d6d12e51f2171120ec033fd826

            Because of the HZ issue being different between platforms we should look to moving the code to ktime instead of jiffies.

            simmonsja James A Simmons added a comment - Because of the HZ issue being different between platforms we should look to moving the code to ktime instead of jiffies.

            People

              niu Niu Yawei (Inactive)
              adilger Andreas Dilger
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: