[LU-17194] parallelize DIO submit Created: 13/Oct/23 Updated: 19/Oct/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 | ||
| Issue Links: |
|
||||||||||||
| Severity: | 3 | ||||||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||||||
| Description |
|
This is a ticket to capture some early work on parallelizing DIO submission. This is required to speed up unaligned DIO writes, which otherwise do all the buffer allocation and copying in a single thread, which is rather painful.
This is effectively a follow on to LU-13805.
The idea is we will parallelize IO submission for each cl_sub_dio. This leverages the IO simplifications from LU-13814. This parallelization has a few requirements: The io submit process needs to not use the cl_io, because this is a shared structure. For sure we can never write to the cl_io, but we also can't safely use it because, for example, the OSC and LOV io in the cl_io are overwritten when we go to the next stripe. So the first step is to remove all usage of cl_io in the DIO submission path. This may require partly breaking DIO ENOSPC handling. We'll see. |
| Comments |
| Comment by Gerrit Updater [ 13/Oct/23 ] |
|
"Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52692 |
| Comment by Gerrit Updater [ 19/Oct/23 ] |
|
"Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52762 |
| Comment by Gerrit Updater [ 19/Oct/23 ] |
|
"Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52763 |
| Comment by Gerrit Updater [ 19/Oct/23 ] |
|
"Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52764 |
| Comment by Gerrit Updater [ 19/Oct/23 ] |
|
"Patrick Farrell <pfarrell@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52765 |