[LU-11552] improper FMR/FastReg pool cleanup Created: 20/Oct/18 Updated: 19/Dec/19 |
|
| Status: | Reopened |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.12.0, Lustre 2.10.5 |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Minor |
| Reporter: | James A Simmons | Assignee: | James A Simmons |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Environment: |
Any lustre system using IB for LNet |
||
| Issue Links: |
|
||||||||||||
| Severity: | 3 | ||||||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||||||
| 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. |
| Comments |
| Comment by James A Simmons [ 20/Oct/18 ] |
|
Patch at https://review.whamcloud.com/#/c/33408. Fixed the LU number. |
| Comment by Amir Shehata (Inactive) [ 24/Oct/18 ] |
|
Just to clarify, while the patch cleans up the code. There was no bug. |
| Comment by Cory Spitz [ 18/Dec/19 ] |
|
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. |
| Comment by Andreas Dilger [ 19/Dec/19 ] |
|
Amir Shehata (ashehata@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33802 |
| Comment by Andreas Dilger [ 19/Dec/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33802/ |