<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr">
<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">
<p style="margin-top:0;margin-bottom:0">Felix, </p>
<p style="margin-top:0;margin-bottom:0"><br>
</p>
<p style="margin-top:0;margin-bottom:0">we ended up with this fix. <a href="http://git.amd.com:8080/#/c/122202/" class="OWAAutoLink" id="LPlnk552811" previewremoved="true" style="font-size: 12pt;">http://git.amd.com:8080/#/c/122202/</a></p>
<p style="margin-top:0;margin-bottom:0"><br>
</p>
<p style="margin-top:0;margin-bottom:0">I explored the option to override the VM size only for KFD VMs, but it was not easy as
<span>amdgpu_check_vm_size() and </span><span style="color: rgb(33, 33, 33); font-size: 15px;">amdgpu_vm_adjust_size() come in during loading amdgpu, i. e. before KFD VMs are created. So to fix the problem quickly, we ended up to not overwrite the VM size for
all Raven VMs. </span></p>
<p style="margin-top:0;margin-bottom:0"><br>
</p>
<p style="margin-top:0;margin-bottom:0"><span style="color: rgb(33, 33, 33); font-size: 15px;">Regards,</span></p>
<p style="margin-top:0;margin-bottom:0">Yong</p>
</div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Christian König <ckoenig.leichtzumerken@gmail.com><br>
<b>Sent:</b> Monday, December 18, 2017 4:22:43 AM<br>
<b>To:</b> Felix Kühling; Zhao, Yong; amd-gfx@lists.freedesktop.org<br>
<b>Subject:</b> Re: [PATCH] drm/amdgpu: Fix a bug that vm size is wrong on Raven</font>
<div> </div>
</div>
<div style="background-color:#FFFFFF">
<div class="x_moz-cite-prefix">The problem was a merge conflict.<br>
<br>
No idea what exactly went wrong, but Yong ended up with a branch where the vm_size was always overwritten with the value 64.<br>
<br>
So we can completely drop this patch and yes when an user overrides the vm_size value he/she should know what the consequences are.<br>
<br>
Regards,<br>
Christian.<br>
<br>
Am 17.12.2017 um 00:21 schrieb Felix Kühling:<br>
</div>
<blockquote type="cite">
<pre>Is there a way to override the VM size for KFD VMs only? Only they
depend on ATS, so only they need to be forced to be 48-bits.
On the other hand, it could be argued that a user who manually sets the
VM size with a module parameter knows what they're doing. So let them
break ATS.
Regards,
Felix
Am 14.12.2017 um 17:45 schrieb Yong Zhao:
</pre>
<blockquote type="cite">
<pre>Change-Id: Id522c1cbadb8c069720f4e64a31cff42cd014733
Signed-off-by: Yong Zhao <a class="x_moz-txt-link-rfc2396E" href="mailto:yong.zhao@amd.com"><yong.zhao@amd.com></a>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index 709587d..93500e6 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -2534,7 +2534,7 @@ void amdgpu_vm_adjust_size(struct amdgpu_device *adev, uint32_t vm_size,
uint64_t tmp;
/* adjust vm size first */
- if (amdgpu_vm_size != -1) {
+ if (amdgpu_vm_size != -1 && adev->asic_type != CHIP_RAVEN) {
unsigned max_size = 1 << (max_bits - 30);
vm_size = amdgpu_vm_size;
</pre>
</blockquote>
<pre>
</pre>
<br>
<fieldset class="x_mimeAttachmentHeader"></fieldset> <br>
<pre>_______________________________________________
amd-gfx mailing list
<a class="x_moz-txt-link-abbreviated" href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a>
<a class="x_moz-txt-link-freetext" href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a>
</pre>
</blockquote>
<br>
</div>
</body>
</html>