[LU-10357] ll_ioc_copy_{start,end}() depend on search_inode_for_lustre() which is bad Created: 08/Dec/17 Updated: 09/Feb/18 Resolved: 04/Jan/18 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.11.0, Lustre 2.10.4 |
| Type: | Bug | Priority: | Minor |
| Reporter: | John Hammond | Assignee: | John Hammond |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | hsm | ||
| Issue Links: |
|
||||||||
| Severity: | 3 | ||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||
| Description |
|
In the HSM archive cases of ll_ioc_copy_{start,end}() we depend on search_inode_for_lustre() to get the data version. search_inode_for_lustre() does a MDS_GETATTR using the FID and returns a only partially initialized inode. The CT will later open and fstat the file so we have extra 3 RPCs (getattr, enqueue for layout for data version, enqueue for stat). If the CT would open the file to be archived before the the start ioctl and close it after the end ioctl then we could skip the extra 3 RPCs and just send 1 open. |
| Comments |
| Comment by Gerrit Updater [ 08/Dec/17 ] |
|
John L. Hammond (john.hammond@intel.com) uploaded a new patch: https://review.whamcloud.com/30456 |
| Comment by Gerrit Updater [ 04/Jan/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30456/ |
| Comment by Peter Jones [ 04/Jan/18 ] |
|
Landed for 2.11 |
| Comment by Gerrit Updater [ 04/Jan/18 ] |
|
Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/30721 |
| Comment by Gerrit Updater [ 09/Feb/18 ] |
|
John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/30721/ |