[LU-12014] check correct size in ll_dom_finish_open()

• Type: Bug
• Resolution: Fixed
• Priority: Major
• Fix Version/s: Lustre 2.13.0, Lustre 2.12.6
• Affects Version/s: Lustre 2.12.0
• Labels: arm
• Environment: Lustre Build: https://build.whamcloud.com/job/lustre-master/3811 ; Distro/Arch: RHEL7.5/aarch64 (client), RHEL7.5/x86_64 (server)
• Severity: 3
• Assignee: Mikhail Pershin
• Reporter: Joseph Gmitter (Inactive)

Description

Cloned from LU-11595 to capture a related patch being added to that already resolved ticket.

 

The check in ll_dom_finish_open() for data end shouldn't use i_size for comparision because it may be not updated yet with just returned data from server. Use size value in mdt_body from reply for that check.

Issue Links

• Clones: LU-11595 sanity-dom sanityn test 11: LBUG: (file.c:470:ll_dom_finish_open()) ASSERTION( lnb.lnb_file_offset % ((1UL) << 16) == 0 ) failed
• is duplicated by: LU-11627 LustreError: 86028:0:(file.c:454:ll_dom_finish_open()) ASSERTION( lnb.lnb_file_offset % (1UL << 16) == 0 ) failed:
• is related to: LU-10300 Can the Lustre 2.10.x clients support 64K kernel page?

Comments

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/40301/
Subject: LU-12014 llite: check correct size in ll_dom_finish_open()
Project: fs/lustre-release
Branch: b2_12
Current Patch Set:
Commit: af49e96b95909f470757890f34da760b8af3c9a8

[Gerrit Updater]

Mike Pershin ([e-mail]) uploaded a new patch: https://review.whamcloud.com/40301
Subject: LU-12014 llite: check correct size in ll_dom_finish_open()
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: 6c1bf839a206189deae00bb3e10ab1e740ca3fa9

[Stephane Thiell]

Could this patch be considered for LTS12? thanks

[Peter Jones]

Landed for 2.13

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/33895/
Subject: LU-12014 llite: check correct size in ll_dom_finish_open()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 7b9fd576f7de7d4bfa40c85d06bb224e7a29c829

[James A Simmons]

Is this ticket resolved?

[Joseph Gmitter (Inactive)]

Mike Pershin ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33895
Subject: LU-11595 llite: check correct size in ll_dom_finish_open()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7140c356de05e93834ec2f1f7156f17cc771826f

[Joseph Gmitter (Inactive)]

Cloned from LU-11595 to capture the new patch pushed there after the ticket was closed for 2.12.