On Thu, Aug 28, 2025 at 01:10:32PM -0600, Keith Busch wrote: > On Thu, Aug 28, 2025 at 03:41:15PM -0300, Jason Gunthorpe wrote: > > On Thu, Aug 28, 2025 at 11:15:20AM -0600, Keith Busch wrote: > > > > > > I don't think that was ever the case. Metadata is allocated > > > independently of the data payload, usually by the kernel in > > > bio_integrity_prep() just before dispatching the request. The bio may > > > have a p2p data payload, but the integrity metadata is just a kmalloc > > > buf in that path. > > > > Then you should do two dma mapping operations today, that is how the > > API was built. You shouldn't mix P2P and non P2P within a single > > operation right now.. > > Data and metadata are mapped as separate operations. They're just > different parts of one blk-mq request.
In that case the new bit leon proposes should only be used for the unmap of the data pages and the metadata unmap should always be unmapped as CPU? Jason