[PATCH] drm/amdgpu:ucode bo init add contiguous flag

[Chen, Horace]

Sorry, branch wrong, please ignore this.

Regards,
Horace.

-----Original Message-----
From: horchen [mailto:horace.chen at amd.com] 
Sent: Friday, June 09, 2017 5:19 PM
To: amd-gfx at lists.freedesktop.org
Cc: Huang, Ray; Chen, Horace
Subject: [PATCH] drm/amdgpu:ucode bo init add contiguous flag

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.chen at amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)  mode change 100644 => 100755 drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c
old mode 100644
new mode 100755
index f2c6c6d..563ceb7
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c
@@ -348,7 +348,8 @@ int amdgpu_ucode_init_bo(struct amdgpu_device *adev)
 	if (!amdgpu_sriov_vf(adev) || !adev->in_reset ) {
 		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);
 		if (err) {
 			dev_err(adev->dev, "(%d) Firmware buffer allocate failed\n", err);
 			goto failed;
--
2.7.4