[LU-11942] OSC page & grant simplification/batching Created: 07/Feb/19 Updated: 16/Aug/19 Resolved: 16/Aug/19 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Minor |
| Reporter: | Patrick Farrell (Inactive) | Assignee: | Patrick Farrell (Inactive) |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Severity: | 3 | ||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||
| Description |
|
This is a ticket to capture what I expect will be a different OSC grant code simplifications/efficiency improvements that don't really need separate LUs for each. The basic goal is to do certain operations less often - Various grant (and related page update) operations are done per page, and for some of them this is either totally unnecessary or can be avoided with some care. The goal is to do at least a bit less work for each page submission in the write path, and potentially prepare some of the ground for write containers (aggressively simplified write path dealing with pages in large batches). |
| Comments |
| Comment by James A Simmons [ 07/Feb/19 ] |
|
Will this cover the changes Neil did upstream? |
| Comment by Patrick Farrell (Inactive) [ 07/Feb/19 ] |
|
Dunno, what did Neil do upstream? Can you shoot me a link or two? |
| Comment by James A Simmons [ 07/Feb/19 ] |
|
https://patchwork.kernel.org/project/lustre-devel/list/?series=63793 |
| Comment by Patrick Farrell (Inactive) [ 07/Feb/19 ] |
|
Ah, no, it will not. I wasn't aware of those. This is rather different - It's more performance oriented, though there may be some other stuff. Are you planning to merge those in? |
| Comment by James A Simmons [ 08/Feb/19 ] |
|
Yes I can merge them in. The question is should I do it right away to prevent collisions with you? |
| Comment by Patrick Farrell (Inactive) [ 08/Feb/19 ] |
|
If you're ready, that would be great. My first patch (which I didn't push yet because of a Gerrit issue, I'll push it soon) is like, a one liner, so it won't matter. But I am contemplating some bigger changes. I think a different LU for Neil's stuff would be good though. |
| Comment by James A Simmons [ 08/Feb/19 ] |
|
Peter do you want a new ticket or should |
| Comment by Patrick Farrell (Inactive) [ 08/Feb/19 ] |
|
Peter isn't on this ticket, but looking at |
| Comment by James A Simmons [ 09/Feb/19 ] |
|
Also Li is work on similar improvements. |
| Comment by Patrick Farrell (Inactive) [ 16/Aug/19 ] |
|
The specific work intended for this ticket wound up not being of benefit, and there are several other tickets where related work was done, so no reason to leave this open. |