<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> </head> <body text="#000000" bgcolor="#FFFFFF"> <div class="moz-cite-prefix">Hi Horace, yeah that looks good to me, feel free to add my reviewed-by line and push it. Regards, Christian. Am 09.06.2017 um 15:45 schrieb Chen, Horace: </div> <blockquote type="cite" cite="mid:MWHPR12MB1629B8FDE5C086109492DCB1E1CE0@MWHPR12MB1629.namprd12.prod.outlook.com"> <pre wrap="">Hi Christian, Sorry, it's my first time to submit code to the staging. I have done the fix. Attached the fixed review. Thank you for the review. Regards, Horace. -----Original Message----- From: Christian König [<a class="moz-txt-link-freetext" href="mailto:deathsimple@vodafone.de">mailto:deathsimple@vodafone.de</a>] Sent: Friday, June 09, 2017 8:25 PM To: Chen, Horace; <a class="moz-txt-link-abbreviated" href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> Cc: Huang, Ray Subject: Re: [PATCH] drm/amdgpu: add contiguous flag in ucode bo create Am 09.06.2017 um 13:59 schrieb horchen: </pre> <blockquote type="cite"> <pre wrap="">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 <a class="moz-txt-link-rfc2396E" href="mailto:horace.chen@amd.com"><horace.chen@amd.com></a> --- 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); </pre> </blockquote> <pre wrap=""> 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 <a class="moz-txt-link-rfc2396E" href="mailto:christian.koenig@amd.com"><christian.koenig@amd.com></a>. </pre> <blockquote type="cite"> <pre wrap=""> if (err) { dev_err(adev->dev, "(%d) Firmware buffer allocate failed\n", err); goto failed; </pre> </blockquote> <pre wrap=""> </pre> <fieldset class="mimeAttachmentHeader"></fieldset> <pre wrap="">_______________________________________________ amd-gfx mailing list <a class="moz-txt-link-abbreviated" href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> <a class="moz-txt-link-freetext" href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a> </pre> </blockquote> </body> </html>