Re: [PATCH v3 4/4] bcachefs: add fiemap delalloc extent detection

2024-04-19 Thread Kent Overstreet
On Thu, Apr 18, 2024 at 10:46:21AM -0400, Brian Foster wrote: > So Kent had pointed out in an earlier chat that this needs to be a > little more robust wrt to transaction restart because there is the > potential for spinning under folio lock contention. I think there are a > couple ways to do this

Re: [PATCH v3 4/4] bcachefs: add fiemap delalloc extent detection

2024-04-18 Thread Brian Foster
On Mon, Apr 08, 2024 at 10:48:46AM -0400, Brian Foster wrote: > bcachefs currently populates fiemap data from the extents btree. > This works correctly when the fiemap sync flag is provided, but if > not, it skips all delalloc extents that have not yet been flushed. > This is because delalloc