Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.10.0, Lustre 2.11.0
    • None
    • None
    • 9223372036854775807

    Description

      Loading the libcfs module is getting increasingly slow due to multiple checksum types being speed tested at startup (8 checksum types * 1s each).

      Reduce the number of checksum algorithms checked at module load time to the ones that are actually need the speed, and reduce the amount of time taken to compute the checksum.

      The mount_facet() code also gained a 5s sleep for each target mounted when http://review.whamcloud.com/20194 was landed, which slows down tests that reformat the filesystem a lot (like conf-sanity.sh) significantly, and is in any case an annoyance for developer testing.

      Attachments

        Issue Links

          Activity

            [LU-9201] reduce llmount.sh startup time

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/29011/
            Subject: LU-9201 tests: don't check network for localhost
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: ee2e79bd389c968db43ebe88814e8b24eb096c3c

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/29011/ Subject: LU-9201 tests: don't check network for localhost Project: fs/lustre-release Branch: master Current Patch Set: Commit: ee2e79bd389c968db43ebe88814e8b24eb096c3c

            Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: https://review.whamcloud.com/29011
            Subject: LU-9201 tests: don't check network for localhost
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: a45f13db141a34984ed7f590ece165c36f7f840f

            gerrit Gerrit Updater added a comment - Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: https://review.whamcloud.com/29011 Subject: LU-9201 tests: don't check network for localhost Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: a45f13db141a34984ed7f590ece165c36f7f840f
            pjones Peter Jones added a comment -

            Landed for 2.10

            pjones Peter Jones added a comment - Landed for 2.10

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/26021/
            Subject: LU-9201 test: avoid long sleeps in mount_facet()
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 293da2049c902de97c8cad2dfa17486a36473421

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/26021/ Subject: LU-9201 test: avoid long sleeps in mount_facet() Project: fs/lustre-release Branch: master Current Patch Set: Commit: 293da2049c902de97c8cad2dfa17486a36473421

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/25923/
            Subject: LU-9201 libcfs: reduce libcfs checksum speed test time
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: f470f3a166b3d471e2a3282864af35fa9d83c859

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/25923/ Subject: LU-9201 libcfs: reduce libcfs checksum speed test time Project: fs/lustre-release Branch: master Current Patch Set: Commit: f470f3a166b3d471e2a3282864af35fa9d83c859

            Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: https://review.whamcloud.com/26021
            Subject: LU-9201 test: revert gratuitous sleeps from mount_facet
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: c51a31fb57cbf0de098b2e6357bfebf669c7f92d

            gerrit Gerrit Updater added a comment - Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: https://review.whamcloud.com/26021 Subject: LU-9201 test: revert gratuitous sleeps from mount_facet Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: c51a31fb57cbf0de098b2e6357bfebf669c7f92d

            I was thinking of passing a special argument from t-f to enforce checksum algo with sort-of-random (jiffies % <# of alrorithms>) to 1) fix load time issue 2) ensure all the algorithms are tested

            bzzz Alex Zhuravlev added a comment - I was thinking of passing a special argument from t-f to enforce checksum algo with sort-of-random (jiffies % <# of alrorithms>) to 1) fix load time issue 2) ensure all the algorithms are tested

            this patch is worth 1m$

            jay Jinshan Xiong (Inactive) added a comment - this patch is worth 1m$

            Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: https://review.whamcloud.com/25923
            Subject: LU-9201 libcfs: reduce libcfs checksum speed test time
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 1cf8867d26c5f652f9bf30d809bac49d815bd208

            gerrit Gerrit Updater added a comment - Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: https://review.whamcloud.com/25923 Subject: LU-9201 libcfs: reduce libcfs checksum speed test time Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 1cf8867d26c5f652f9bf30d809bac49d815bd208

            People

              adilger Andreas Dilger
              adilger Andreas Dilger
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: