[LU-11552] improper FMR/FastReg pool cleanup

• Type: Improvement
• Resolution: Unresolved
• Priority: Minor
• Affects Version/s: Lustre 2.12.0, Lustre 2.10.5
• Environment: Any lustre system using IB for LNet
• Severity: 3
• Assignee: James A Simmons
• Reporter: James A Simmons

Description

While porting the map_on_demand changes to the linux lustre client it was noticed that the cleanup for the FMR / FastReg pool handling was wrong. Even when using FMR if the fmr pool is already deleted it will attempt to cleanup the FastReg pool which shouldn't even exist.

Issue Links

• is related to: LU-11152 sanity test_133g: ost1 find /proc/fs/lustre/ /proc/sys/lnet/ /proc/sys/lustre/ failed
• is related to: LU-11735 o2iblnd - bad check for fmr

Comments

[Andreas Dilger]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/33802/
Subject: Revert "LU-11152 lnd: test fpo_fmr_poool pointer instead of special bool"
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: a65d072fa45fc90c2fc74b61d214de79c0bf33e5

[Andreas Dilger]

Amir Shehata ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33802
Subject: Revert "LU-11152 lnd: test fpo_fmr_poool pointer instead of special bool"
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: b169980b805b78df9b05e10079d27c4a6d3dbbc1

[Cory Spitz]

I'm reopening this issue due to the mix-up described at comment-260120. There was a typo in the commit header.

Even though code landed for this ticket, it was later reverted because it introduced LU-11735 (comment-238218) and so we need to re-open this ticket accordingly.

[Amir Shehata (Inactive)]

Just to clarify, while the patch cleans up the code. There was no bug.

[James A Simmons]

Patch at https://review.whamcloud.com/#/c/33408. Fixed the LU number.