]> www.infradead.org Git - users/hch/dma-mapping.git/commit
btrfs: fix the delalloc range locking if sector size < page size
authorQu Wenruo <wqu@suse.com>
Tue, 8 Oct 2024 23:07:03 +0000 (09:37 +1030)
committerDavid Sterba <dsterba@suse.com>
Tue, 22 Oct 2024 14:09:44 +0000 (16:09 +0200)
commitf10f59f91a6278e9637327d1206140d28e2d5004
treec0ea22b5b8986b52b3dda95d223f8a980d130993
parent5f9062a48db260fd6b53d86ecfb4d5dc59266316
btrfs: fix the delalloc range locking if sector size < page size

Inside lock_delalloc_folios(), there are several problems related to
sector size < page size handling:

- Set the writer locks without checking if the folio is still valid
  We call btrfs_folio_start_writer_lock() just like it's folio_lock().
  But since the folio may not even be the folio of the current mapping,
  we can easily screw up the folio->private.

- The range is not clamped inside the page
  This means we can over write other bitmaps if the start/len is not
  properly handled, and trigger the btrfs_subpage_assert().

- @processed_end is always rounded up to page end
  If the delalloc range is not page aligned, and we need to retry
  (returning -EAGAIN), then we will unlock to the page end.

  Thankfully this is not a huge problem, as now
  btrfs_folio_end_writer_lock() can handle range larger than the locked
  range, and only unlock what is already locked.

Fix all these problems by:

- Lock and check the folio first, then call
  btrfs_folio_set_writer_lock()
  So that if we got a folio not belonging to the inode, we won't
  touch folio->private.

- Properly truncate the range inside the page

- Update @processed_end to the locked range end

Fixes: 1e1de38792e0 ("btrfs: make process_one_page() to handle subpage locking")
CC: stable@vger.kernel.org # 6.1+
Signed-off-by: Qu Wenruo <wqu@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/extent_io.c