[LU-9201] reduce llmount.sh startup time

• Status: Resolved
• Project: Lustre
• Fix Version/s: Lustre 2.10.0, Lustre 2.11.0
• Type: Improvement
• Priority: Minor
• Reporter: Andreas Dilger
• Assignee: Andreas Dilger
• Resolution: Fixed

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.

Issue Links

• is related to: LU-14772 split conf-sanity into 2 or 3 parts

Comments

[Gerrit Updater]

Andreas Dilger ([e-mail]) 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

[Jinshan Xiong (Inactive)]

this patch is worth 1m$

[Alex Zhuravlev]

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

[Gerrit Updater]

Andreas Dilger ([e-mail]) 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 Updater]

Oleg Drokin ([e-mail]) 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 Updater]

Oleg Drokin ([e-mail]) 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

[Peter Jones]

Landed for 2.10

[Gerrit Updater]

Andreas Dilger ([e-mail]) 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 Updater]

Oleg Drokin ([e-mail]) 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