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

change lru_max_age to units of ms

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

          is related to

          Improvement - An improvement or enhancement to an existing feature or task. LU-9019 Migrate lustre to standard 64 bit time kernel API

          • Major - Major loss of function.
          • Resolved

          Activity

            [LU-8541] change lru_max_age to units of ms
            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

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

            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
            mdiep Minh Diep made changes -
            Labels Original: LTS
            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

            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
            mdiep Minh Diep made changes -
            Fix Version/s New: Lustre 2.10.2 [ 13494 ]
            Fix Version/s Original: Lustre 2.10.1 [ 13090 ]
            mdiep Minh Diep made changes -
            Fix Version/s New: Lustre 2.10.1 [ 13090 ]
            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

            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
            jgmitter Joseph Gmitter (Inactive) made changes -
            Fix Version/s Original: Lustre 2.10.1 [ 13090 ]
            jgmitter Joseph Gmitter (Inactive) made changes -
            Labels New: LTS
            pjones Peter Jones made changes -
            Resolution New: Fixed [ 1 ]
            Status Original: Open [ 1 ] New: Resolved [ 5 ]

            People

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

              Dates

                Created:
                Updated:
                Resolved: