[LU-16649] EIO is possible on race with page reclaim/deletion Created: 20/Mar/23 Updated: 20/May/23 Resolved: 12/May/23 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.16.0, Lustre 2.15.3 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Patrick Farrell | Assignee: | Patrick Farrell |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||
| Severity: | 3 | ||||
| Rank (Obsolete): | 9223372036854775807 | ||||
| Description |
|
Careful examination shows that it's possible for the bug fixed for mmap in https://review.whamcloud.com/49647 can also occur in the standard read() path. This means unless we can figure out a more graceful way to handle our page deletion, we need the same check and retry code in the normal read() path as we have in the page fault path. |
| Comments |
| Comment by Gerrit Updater [ 20/Mar/23 ] |
|
"Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50344 |
| Comment by Gerrit Updater [ 23/Mar/23 ] |
|
"Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50392 |
| Comment by Gerrit Updater [ 23/Mar/23 ] |
|
"Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50393 |
| Comment by Gerrit Updater [ 23/Mar/23 ] |
|
"Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50394 |
| Comment by Gerrit Updater [ 11/Apr/23 ] |
|
"Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50600 |
| Comment by Gerrit Updater [ 11/Apr/23 ] |
|
"Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50602 |
| Comment by Gerrit Updater [ 09/May/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50344/ |
| Comment by Peter Jones [ 12/May/23 ] |
|
Landed for 2.16 |
| Comment by Gerrit Updater [ 20/May/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50600/ |