[LU-16656] Improve location of 'out:' in ll_file_io_generic Created: 22/Mar/23 Updated: 22/Mar/23 |
|
| Status: | Open |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Minor |
| Reporter: | Patrick Farrell | Assignee: | Patrick Farrell |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Rank (Obsolete): | 9223372036854775807 |
| Description |
|
The location of the 'out:' label in ll_file_io_generic is a bit scary, because it skips part of the setup for io_restart, where partial io is taken in to account. This is safe today because 'out:' is only used before calls to cl_io_loop, so a partial IO is impossible, but if 'out;' is ever used later in the function, it will be incorrect. Let's move it now rather than leave a trap for the unwary. |
| Comments |
| Comment by Gerrit Updater [ 22/Mar/23 ] |
|
"Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50379 |