[LU-6741] OSP should use inline data for small requests Created: 18/Jun/15 Updated: 09/Sep/16 Resolved: 06/Oct/15 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.8.0 |
| Fix Version/s: | Lustre 2.8.0 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Alex Zhuravlev | Assignee: | Di Wang |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | dne2 | ||
| Issue Links: |
|
||||||||||||
| Severity: | 3 | ||||||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||||||
| Description |
|
we shouldn't use bulks to transfer updates unless it's needed. otherwise all "interactive" requests (like lookups, getattrs, etc) need 2 RTTs instead of one. |
| Comments |
| Comment by Andreas Dilger [ 19/Jun/15 ] |
|
Alex, what is the impact of this issue? It seems like only a potential performance improvement, but if this is a protocol change for the Async Commit code then I guess it needs to be in 2.8.0? |
| Comment by Alex Zhuravlev [ 19/Jun/15 ] |
|
well, the older versions don't support bulks, right? so, i'd think it's a protocol change. then... I'd say this is a performance regression because in the current release we don't need 2*RTT to fetch attributes (e.g., used all the time by LFSCK), it's a single RTT currently.. |
| Comment by Di Wang [ 14/Jul/15 ] |
|
This seems important, so I will work on a patch before 2.8 release. |
| Comment by Oleg Drokin [ 23/Jul/15 ] |
|
So is the perf regression only for DNE, DNE2? Or would single MDT configs be affected as well? |
| Comment by Alex Zhuravlev [ 23/Jul/15 ] |
|
LFSCK uses this protocol to access data on OST, so this will be affected too. |
| Comment by Gerrit Updater [ 06/Aug/15 ] |
|
wangdi (di.wang@intel.com) uploaded a new patch: http://review.whamcloud.com/15899 |
| Comment by Gerrit Updater [ 10/Sep/15 ] |
|
wangdi (di.wang@intel.com) uploaded a new patch: http://review.whamcloud.com/16353 |
| Comment by Gerrit Updater [ 02/Oct/15 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/16353/ |
| Comment by Gerrit Updater [ 06/Oct/15 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/15899/ |
| Comment by Peter Jones [ 06/Oct/15 ] |
|
Landed for 2.8 |