diff options
author | Chunming Zhou <david1.zhou@amd.com> | 2018-03-29 22:23:48 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2018-04-03 14:08:44 -0400 |
commit | 073997ccd9dd8eb8015f5371ff0ea85a48814bb4 (patch) | |
tree | ef95f43d251aa13783aafe5d2d06c8584e50c5d1 /drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | |
parent | 46bce67f62e7daa00ad908c4c17e9750df2acf52 (diff) |
Revert "drm/amdgpu: Don't change preferred domian when fallback GTT v5"
This reverts commit bbaf1871ea1d7b7021e350a3e8bb1ec8408c2030.
Felix reports this commit causes regression.
Signed-off-by: Chunming Zhou <david1.zhou@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_object.c')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 18 |
1 files changed, 3 insertions, 15 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c index 5ad985c40d49..6d08cde8443c 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | |||
@@ -356,7 +356,6 @@ static int amdgpu_bo_do_create(struct amdgpu_device *adev, unsigned long size, | |||
356 | struct amdgpu_bo *bo; | 356 | struct amdgpu_bo *bo; |
357 | unsigned long page_align; | 357 | unsigned long page_align; |
358 | size_t acc_size; | 358 | size_t acc_size; |
359 | u32 domains; | ||
360 | int r; | 359 | int r; |
361 | 360 | ||
362 | page_align = roundup(byte_align, PAGE_SIZE) >> PAGE_SHIFT; | 361 | page_align = roundup(byte_align, PAGE_SIZE) >> PAGE_SHIFT; |
@@ -418,23 +417,12 @@ static int amdgpu_bo_do_create(struct amdgpu_device *adev, unsigned long size, | |||
418 | #endif | 417 | #endif |
419 | 418 | ||
420 | bo->tbo.bdev = &adev->mman.bdev; | 419 | bo->tbo.bdev = &adev->mman.bdev; |
421 | domains = bo->preferred_domains; | 420 | amdgpu_ttm_placement_from_domain(bo, domain); |
422 | retry: | 421 | |
423 | amdgpu_ttm_placement_from_domain(bo, domains); | ||
424 | r = ttm_bo_init_reserved(&adev->mman.bdev, &bo->tbo, size, type, | 422 | r = ttm_bo_init_reserved(&adev->mman.bdev, &bo->tbo, size, type, |
425 | &bo->placement, page_align, &ctx, acc_size, | 423 | &bo->placement, page_align, &ctx, acc_size, |
426 | NULL, resv, &amdgpu_ttm_bo_destroy); | 424 | NULL, resv, &amdgpu_ttm_bo_destroy); |
427 | 425 | if (unlikely(r != 0)) | |
428 | if (unlikely(r && r != -ERESTARTSYS)) { | ||
429 | if (bo->flags & AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED) { | ||
430 | bo->flags &= ~AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED; | ||
431 | goto retry; | ||
432 | } else if (domains != bo->allowed_domains) { | ||
433 | domains = bo->allowed_domains; | ||
434 | goto retry; | ||
435 | } | ||
436 | } | ||
437 | if (unlikely(r)) | ||
438 | return r; | 426 | return r; |
439 | 427 | ||
440 | if (adev->gmc.visible_vram_size < adev->gmc.real_vram_size && | 428 | if (adev->gmc.visible_vram_size < adev->gmc.real_vram_size && |