NAK, that is clearly incorrect. The gpu_offset are MC addresses while the vram_base_offset is a bud address.

Most of the time they are both zero, except for APUs which would break with this change.

Regards,
Christian.

Am 09.09.2016 um 08:30 schrieb Flora Cui:
Change-Id: I8112e9d85866104559ecef7449f50fbb94167382
Signed-off-by: Flora Cui <flora....@amd.com>
---
  drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index a549abd..3d7a3ab 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -1151,7 +1151,7 @@ int amdgpu_vm_bo_update(struct amdgpu_device *adev,
                        break;
case TTM_PL_VRAM:
-                       addr += adev->vm_manager.vram_base_offset;
+                       addr += 
bo_va->bo->tbo.bdev->man[mem->mem_type].gpu_offset;
                        break;
default:


_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to