When zone device memory is registered in multiple chunks, each chunk
gets its own dev_pagemap. A single bio can contain bvecs from different
pgmaps -- iov_iter_extract_bvecs() breaks at pgmap boundaries but the
outer loop in bio_iov_iter_get_pages() continues filling the same bio.
There are two problems with the current code:
1. biovec_phys_mergeable() has no pgmap check, so the request merge,
DMA mapping, and integrity merge paths can coalesce physically
contiguous bvec segments from different pgmaps. This makes it
impossible to recover the correct pgmap for the merged segment
via page_pgmap().
2. bio_add_page() and bio_integrity_add_page() reject pages from a
different pgmap entirely (returning 0), rather than just skipping
the merge and adding them as new bvec entries. This forces callers
to start a new bio unnecessarily.
Patch 1 fixes the merge-path gap by adding a pgmap check to
biovec_phys_mergeable().
Patch 2 fixes the bio_add_page() and bio_integrity_add_page() API
behavior, moving the pgmap check into the merge conditional so
different-pgmap pages can be added as separate bvec entries. This
depends on patch 1 being in place.
Naman Jain (2):
block: add pgmap check to biovec_phys_mergeable
block: allow different-pgmap pages as separate bvecs in bio_add_page
block/bio-integrity.c | 6 ++----
block/bio.c | 6 ++----
block/blk.h | 2 ++
3 files changed, 6 insertions(+), 8 deletions(-)
--
2.43.0