Commit c0dd8a92 authored by Matthew Auld's avatar Matthew Auld Committed by Alex Deucher
Browse files

drm/amdgpu/dma_buf: fix page_link check 



The page_link lower bits of the first sg could contain something like
SG_END, if we are mapping a single VRAM page or contiguous blob which
fits into one sg entry. Rather pull out the struct page, and use that in
our check to know if we mapped struct pages vs VRAM.

Fixes: f44ffd67 ("drm/amdgpu: add support for exporting VRAM using DMA-buf v3")
Signed-off-by: default avatarMatthew Auld <matthew.auld@intel.com>
Cc: Christian König <christian.koenig@amd.com>
Cc: amd-gfx@lists.freedesktop.org
Cc: <stable@vger.kernel.org> # v5.8+
Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent a755906f

drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c

+1 −1
Original line number Diff line number Diff line
@@ -184,7 +184,7 @@ static void amdgpu_dma_buf_unmap(struct dma_buf_attachment *attach, 
				 struct sg_table *sgt,

				 enum dma_data_direction dir)

{

	if (sgt->sgl->page_link) {

	if (sg_page(sgt->sgl)) {

		dma_unmap_sgtable(attach->dev, sgt, dir, 0);

		sg_free_table(sgt);

		kfree(sgt);