summaryrefslogtreecommitdiff
path: root/certs
diff options
context:
space:
mode:
authorChuck Lever <chuck.lever@oracle.com>2020-02-12 11:12:30 -0500
committerAnna Schumaker <Anna.Schumaker@Netapp.com>2020-02-13 15:35:33 -0500
commitca1c671302825182629d3c1a60363cee6f5455bb (patch)
tree3d73e0b5bab64840f7c824f8469ec9464a99e12a /certs
parentefeda80da38d0b4afd77a12bd4a44f657567d26c (diff)
xprtrdma: Fix DMA scatter-gather list mapping imbalance
The @nents value that was passed to ib_dma_map_sg() has to be passed to the matching ib_dma_unmap_sg() call. If ib_dma_map_sg() choses to concatenate sg entries, it will return a different nents value than it was passed. The bug was exposed by recent changes to the AMD IOMMU driver, which enabled sg entry concatenation. Looking all the way back to commit 4143f34e01e9 ("xprtrdma: Port to new memory registration API") and reviewing other kernel ULPs, it's not clear that the frwr_map() logic was ever correct for this case. Reported-by: Andre Tomt <andre@tomt.net> Suggested-by: Robin Murphy <robin.murphy@arm.com> Signed-off-by: Chuck Lever <chuck.lever@oracle.com> Cc: stable@vger.kernel.org Reviewed-by: Jason Gunthorpe <jgg@mellanox.com> Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
Diffstat (limited to 'certs')
0 files changed, 0 insertions, 0 deletions