[LU-10357] ll_ioc_copy_{start,end}() depend on search_inode_for_lustre() which is bad

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.11.0, Lustre 2.10.4
• Labels: hsm
• Severity: 3
• Assignee: John Hammond
• Reporter: John Hammond

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.

Issue Links

• is related to: LU-10302 hsm: obscure bug with multi-mountpoints and ldlm

Comments

[Gerrit Updater]

John L. Hammond ([e-mail]) merged in patch https://review.whamcloud.com/30721/
Subject: LU-10357 hsm: open file to be archived before starting
Project: fs/lustre-release
Branch: b2_10
Current Patch Set:
Commit: 56d0f3df87f4fecae028d7aace73f8979f54ae94

[Gerrit Updater]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/30721
Subject: LU-10357 hsm: open file to be archived before starting
Project: fs/lustre-release
Branch: b2_10
Current Patch Set: 1
Commit: 169b17f5c68a2b637ea921d7a9812ad102f167ef

[Peter Jones]

Landed for 2.11

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/30456/
Subject: LU-10357 hsm: open file to be archived before starting
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 77cd1a21e312ca00b37b33689091734a12eec048

[Gerrit Updater]

John L. Hammond ([e-mail]) uploaded a new patch: https://review.whamcloud.com/30456
Subject: LU-10357 hsm: open file to be archived before starting
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 13b8102c912e48cbec2d01416678f7165883610b