On Sun, Jul 20, 2025 at 11:40:36AM -0700, Keith Busch wrote: > static inline bool blk_rq_dma_unmap(struct request *req, struct device *dma_dev, > - struct dma_iova_state *state, size_t mapped_len) > + struct dma_iova_state *state, size_t mapped_len, bool p2p) Nit: can we names this is_p2p as that makes reading these boolean flags a bit easier? Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>