[LU-14217] better SEEK_HOLE/DATA support in OSD-ZFS

• Type: Improvement
• Resolution: Fixed
• Priority: Major
• Fix Version/s: Lustre 2.15.0
• Assignee: Mikhail Pershin
• Reporter: Mikhail Pershin

Description

After LU-10810 the osd-zfs has support for SEEK_HOLE/DATA but with no access to ZFS dmu_offset_next() function yet. So we had no check how it works actually with ZFS.
With latest ZFS 8.5 this was solved and I am able to run lseek tests on ZFS.
This reveals several problems with ZFS support for SEEK_HOLE/DATA

1. Results for files with dirty data are unreliable, it can be that some offset is reported as HOLE and immediate SEEK_DATA for the same offset finds dirty data, switches to generic approach and reports DATA. This problem exists mostly because HOLE lseek doesn't check if data is dirty like SEEK_DATA does, so HOLE can be reported correctly even with dirty data
Solution is data sync and repeat lseek, or don't report that osd-zfs as supporting SEEK_DATA/HOLE, we cannot accept unreliable results while using SEEK_HOLE/DATA in our tools, especially where data is copied.

2. Current tests doesn't fit well with ZFS bigger block sizes, so either tests should be properly updated to check ZFS big blocksize and uses appropriate offsets or skip tests if blocksize is too big for reliable results.

Issue Links

• is related to: LU-10810 SEEK_HOLE and SEEK_DATA support for lseek

Comments

[Peter Jones]

Landed for 2.15

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/40970/
Subject: LU-14217 osd-zfs: allow SEEK_HOLE/DATA only with sync
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: a3f5aa624b83307caf56ca6d2490040f6e9cf2fc

[Gerrit Updater]

Mike Pershin ([e-mail]) uploaded a new patch: https://review.whamcloud.com/40970
Subject: LU-14217 osd-zfs: allow SEEK_HOLE/DATA only with sync
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 2fe91608147021874782a9ed49396f31f981d391