Hi all, Here's v2 of the fiemap delalloc processing patch. The main feedback to v1 was around the organization of the code, so this tries to clean that up a bit. Otherwise the functionality/behavior is intended to be pretty much the same as v1. The main difference is that bch2_fiemap() is broken down into helpers to clarify between when it's processing extents vs. holes, and to generally separate extent key processing from the high level scan.
The series is currently posted to my CI branch. I don't _think_ there are any immediate regressions after having a look through the failures on the head commit (as well as via my own local testing), but ISTM that the variance/failure count on the baseline branches has increased a bit from where we were at a few weeks or so ago... Brian v2: - Clean up bch2_fiemap() a bit to use processing helpers. - Added patch for sync flag fixup. v1: https://lore.kernel.org/linux-bcachefs/[email protected]/ Brian Foster (4): bcachefs: drop duplicate fiemap sync flag bcachefs: track current fiemap offset in start variable bcachefs: refactor fiemap processing into extent helper and struct bcachefs: add fiemap delalloc extent detection fs/bcachefs/fs.c | 156 ++++++++++++++++++++++++++++++++++------------- 1 file changed, 113 insertions(+), 43 deletions(-) -- 2.42.0
