[LU-14286] fallocate() does not zero on-disk data

• Type: Bug
• Resolution: Fixed
• Priority: Blocker
• Fix Version/s: Lustre 2.14.0
• Affects Version/s: Lustre 2.14.0
• Severity: 3
• Assignee: Arshad Hussain
• Reporter: Arshad Hussain

Description

The ldiskfs osd_fallocate() does not correctly zero the allocated blocks. This is not noticed during autotest and local developer testing because the underlying storage is zero-filled due to using a sparse backing file and will usually read back as zero. Also, the fsx data correctness test was not properly detecting fallocate() support, and as a result this behavior was not detected until after fsx was fixed.

fsx uses test_fallocate() to determine if fallocate() along with its various FLAGS is enabled and could be called. This was discovered while working on LU-14160 that test_fallocate() always returned false as a bug. Resulting is fallocate() being skipped always for fsx runs.

Issue Links

• is related to: LU-14326 sanity-dom test_fsx: crash in osc_extent_make_ready()
• is related to: LU-14333 ASSERTION( oinfo->loi_lvb.lvb_size >= oinfo->loi_kms ) failed
• is related to: LU-14324 fsx fallocate detection is buggy
• is related to: LUDOC-487 Document llite.*.fallocate_zero_blocks tunable

Comments

[Peter Jones]

Finally I think that everything tracked here has landed

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/41315/
Subject: LU-14286 osd-ldiskfs: enable fallocate by default
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 51d2a79ac93d31f969baaa5f10da707762acf3f2

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: https://review.whamcloud.com/41315
Subject: LU-14286 osd-ldiskfs: enable fallocate by default
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0ba7b99f41593cdedcf6b1340b73932edd3fa3e0

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/41204/
Subject: LU-14286 osd-ldiskfs: fallocate with unwritten extents
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 4f18e08099e51b682f6acb1cf9fea6d7d45f5fd7

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/41216/
Subject: LU-14286 osd-ldiskfs: don't read unwritten blocks
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 42c52069cada6837df34a02638717dff709e5a0e

[Gerrit Updater]

Alex Zhuravlev ([e-mail]) uploaded a new patch: https://review.whamcloud.com/41216
Subject: LU-14286 osd: don't read unwritten blocks
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 065e10770fc082ba8ec80aed21a797a34faaaab4

[Gerrit Updater]

Andreas Dilger ([e-mail]) merged in patch https://review.whamcloud.com/41211/
Subject: LU-14286 tests: reduce fallocate space sanity 150e
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: b9274179adf0939909ff2dd78d077515a64fd25f

[Gerrit Updater]

James Nunez ([e-mail]) uploaded a new patch: https://review.whamcloud.com/41211
Subject: LU-14286 tests: reduce fallocate space sanity 150e
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4be24e98c0d1f1a08c9ca03444a3cb682017303c

[Andreas Dilger]

Still not fixed.

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: https://review.whamcloud.com/41204
Subject: LU-14286 osd-ldiskfs: fallocate with unwritten extents
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 44c56a496d80a8607d29681ade498fe4fbf85a59