[LU-11089] Performance improvements for lu_object locking

• Type: Improvement
• Resolution: Fixed
• Priority: Major
• Fix Version/s: Lustre 2.13.0, Lustre 2.12.5
• Affects Version/s: Lustre 2.12.0
• Assignee: James A Simmons
• Reporter: James A Simmons

Description

While porting the LU-6800 work upstream the reaction to the approach was disliked since it wasn't a real improvement. Neil has created a patch series to break up the global lock to increase its performance.

Issue Links

• is related to: LU-8346 conf-sanity test_93: test failed to respond and timed out
• is related to: LU-12565 Use bit locking in obd_echo
• is related to: LU-6800 Significant performance regression with patch LU-5264
• is related to: LU-9679 Prepare lustre for adoption into the linux kernel

Comments

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/32713/
Subject: LU-11089 obdclass: remove locking from lu_context_exit()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 62f6496f81ff5896ecc778c9e57b6f84d0f83da9

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/32712/
Subject: LU-11089 obdclass: use an rwsem instead of lu_key_initing_cnt.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 99bb9f91f5c5ca6a380b22efa04a3c00c8f520ca

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/32711/
Subject: LU-11089 obdclass: make key_set_version an atomic_t
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: e9213217691ae78d15237b0c5ecd3ba0b0416652

[James A Simmons]

Sigh, RHEL 7.6 did a port wrong from upstream  Need to download RHEL kernel source and see how they botched the port.

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33674
Subject: LU-11089 lu_object: fix possible hang waiting for LCS_LEAVING
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 28bd00dbcada616bcb5cf14899c3feac784bc6c1

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33673
Subject: LU-11089 obd: rename lu_keys_guard to lu_context_remembered_guard
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0c790df73f57138e554326d95566de04618c1e93

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33668
Subject: LU-11089 obd: remove lock from key register/degister
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 837c5d44d10daf04353e900939d9977feed064c5

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33667
Subject: LU-11089 obd: use wait_event_var() in lu_context_key_degister()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 37bb534e3779c4cfcf46a0206583ce3a88be69d1

[Shuichi Ihara]

Patrick, sorry! forgot to update this ticket. Here is new ticket. LU-11624.

[Patrick Farrell (Inactive)]

Ihara,

Could you link that ticket here?  I'm interested in tracking it.  Our MDSses running 2.12 are crashing when we fail them over under load.  Pretty reliably.

[James A Simmons]

Thanks. I have a patch based on LU-8130 work that should fix this.