Am 09.06.2017 um 13:59 schrieb horchen:
Under VF environment, the ucode would be settled to the visible VRAM,
As it would be pinned to the visible VRAM, it's better to add
contiguous flag,otherwise it need to move gpu address during the pin
process. This movement is not necessary.
Signed-off-by: horchen <horace.c...@amd.com>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c
index cd6d3d0..3b1ee05 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c
@@ -379,7 +379,8 @@ int amdgpu_ucode_init_bo(struct amdgpu_device *adev)
err = amdgpu_bo_create(adev, adev->firmware.fw_size, PAGE_SIZE, true,
amdgpu_sriov_vf(adev) ? AMDGPU_GEM_DOMAIN_VRAM
: AMDGPU_GEM_DOMAIN_GTT,
- 0, NULL, NULL, bo);
+ amdgpu_sriov_vf(adev) ?
AMDGPU_GEM_CREATE_VRAM_CONTIGUOUS : 0,
+ NULL, NULL, bo);
Mhm, did you missed my comment? You can add the
AMDGPU_GEM_CREATE_VRAM_CONTIGUOUS unconditionally here, e.g. without
checking amdgpu_sriov_vf(adev).
With that fixed the patch is Reviewed-by: Christian König
<christian.koe...@amd.com>.
if (err) {
dev_err(adev->dev, "(%d) Firmware buffer allocate failed\n",
err);
goto failed;
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
