<html><head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
Am 07.12.22 um 08:27 schrieb Zhang, Jesse(Jie):<br>
<blockquote type="cite" cite="mid:DM4PR12MB51524585CEC4E2B4B8370874E31A9@DM4PR12MB5152.namprd12.prod.outlook.com">
<meta name="Generator" content="Microsoft Word 15 (filtered
medium)">
<style>@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}@font-face
{font-family:DengXian;
panose-1:2 1 6 0 3 1 1 1 1 1;}@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}@font-face
{font-family:"\@DengXian";
panose-1:2 1 6 0 3 1 1 1 1 1;}p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}a:link, span.MsoHyperlink
{mso-style-priority:99;
color:#0563C1;
text-decoration:underline;}span.EmailStyle17
{mso-style-type:personal-compose;
font-family:"Calibri",sans-serif;
color:windowtext;}.MsoChpDefault
{mso-style-type:export-only;
font-family:"Calibri",sans-serif;}div.WordSection1
{page:WordSection1;}</style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
<p class="msipheaderdf3d92d6" style="margin:0" align="Left"><span style="font-size:10.0pt;font-family:Arial;color:#0000FF">[AMD
Official Use Only - General]</span></p>
<br>
<div class="WordSection1">
<p class="MsoNormal"> drm/amdgpu: try allowed domain when pin
framebuffer failed.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"> [WHY&HOW]<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"> in some scenarios, the allocate memory
often failed. such as do hot plug or play games.<o:p></o:p></p>
<p class="MsoNormal"> so we can try allowed domain, if the
preferred domain cannot allocate memory.</p>
</div>
</blockquote>
<br>
Not sure why you send it again, but this patch is still a pretty
clear NAK.<br>
<br>
Christian.<br>
<br>
<blockquote type="cite" cite="mid:DM4PR12MB51524585CEC4E2B4B8370874E31A9@DM4PR12MB5152.namprd12.prod.outlook.com">
<div class="WordSection1">
<p class="MsoNormal"><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"> Signed-off-by: jie1zhan <a href="mailto:jesse.zhang@amd.com" moz-do-not-send="true" class="moz-txt-link-freetext">
jesse.zhang@amd.com</a><o:p></o:p></p>
<p class="MsoNormal"> Change-Id:
I4b62e2ff072d02c515f901000a5789339d481273<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">diff --git
a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c<o:p></o:p></p>
<p class="MsoNormal">index 1ae0c8723348..05fcaf7f9d92 100644<o:p></o:p></p>
<p class="MsoNormal">---
a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c<o:p></o:p></p>
<p class="MsoNormal">+++
b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c<o:p></o:p></p>
<p class="MsoNormal">@@ -39,6 +39,7 @@<o:p></o:p></p>
<p class="MsoNormal">#include "amdgpu.h"<o:p></o:p></p>
<p class="MsoNormal">#include "amdgpu_trace.h"<o:p></o:p></p>
<p class="MsoNormal">#include "amdgpu_amdkfd.h"<o:p></o:p></p>
<p class="MsoNormal">+#include "amdgpu_display.h"<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">/**<o:p></o:p></p>
<p class="MsoNormal"> * DOC: amdgpu_object<o:p></o:p></p>
<p class="MsoNormal">@@ -942,8 +943,14 @@ int
amdgpu_bo_pin_restricted(struct amdgpu_bo *bo, u32 domain,<o:p></o:p></p>
<p class="MsoNormal">
bo->placements[i].lpfn = lpfn;<o:p></o:p></p>
<p class="MsoNormal"> }<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">+ retry:<o:p></o:p></p>
<p class="MsoNormal"> r =
ttm_bo_validate(&bo->tbo, &bo->placement,
&ctx);<o:p></o:p></p>
<p class="MsoNormal"> if (unlikely(r)) {<o:p></o:p></p>
<p class="MsoNormal">+ //try allowed domain when
pin failed. just a workaround.<o:p></o:p></p>
<p class="MsoNormal">+ if (unlikely(r == -ENOMEM)
&& domain != bo->allowed_domains) {<o:p></o:p></p>
<p class="MsoNormal">+
amdgpu_bo_placement_from_domain(bo, bo->allowed_domains);<o:p></o:p></p>
<p class="MsoNormal">+ goto retry;<o:p></o:p></p>
<p class="MsoNormal">+ }<o:p></o:p></p>
<p class="MsoNormal"> dev_err(adev->dev, "%p
pin failed\n", bo);<o:p></o:p></p>
<p class="MsoNormal"> goto error;<o:p></o:p></p>
<p class="MsoNormal"> }<o:p></o:p></p>
</div>
</blockquote>
<br>
</body>
</html>