From: "Christian König" <christian.koenig-5C7GfCeVMHo@public.gmane.org>
To: Yong Zhao <yong.zhao-5C7GfCeVMHo@public.gmane.org>,
amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Subject: Re: [PATCH] drm/amdgpu: Fix a bug that vm size is wrong on Raven
Date: Thu, 14 Dec 2017 18:23:03 +0100 [thread overview]
Message-ID: <952df2e0-a5dd-64e0-e4b3-241796bc0105@amd.com> (raw)
In-Reply-To: <9fc57983-8af9-789d-6e60-e3716f846d6c-5C7GfCeVMHo@public.gmane.org>
Hi Yong,
> How should deal with that or can you come up with a fix?
well that is the expected effect. So I don't see much we can do here.
Support for the parameter was added to be able to intentionally break
support for HMM/SVM for testing the fall back paths.
Didn't thought about ATC while enabling this, but it certainly falls
into the same category. Could be that we can still keep ATC working
while reducing the GPUVM size, but that would require further testing.
Regards,
Christian.
Am 14.12.2017 um 16:56 schrieb Yong Zhao:
> Hi Christian,
>
> I don't know much about the background. But according to my
> experiments, as long as we change the vm size to 64G, ATC memory
> access on Raven will fall apart. How should deal with that or can you
> come up with a fix?
>
> Regards,
>
> Yong
>
>
> On 2017-12-14 03:47 AM, Christian König wrote:
>> NAK, that really circumvents the intention of the patch to adjust the
>> number of levels based on the vm_size.
>>
>> Christian.
>>
>> Am 14.12.2017 um 03:25 schrieb Yong Zhao:
>>> Change-Id: Id522c1cbadb8c069720f4e64a31cff42cd014733
>>> Signed-off-by: Yong Zhao <yong.zhao@amd.com>
>>> ---
>>> 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..3b9eb1a 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 && max_level == 1) {
>>> unsigned max_size = 1 << (max_bits - 30);
>>> vm_size = amdgpu_vm_size;
>>
>
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
next prev parent reply other threads:[~2017-12-14 17:23 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-14 2:25 [PATCH] drm/amdgpu: Fix a bug that vm size is wrong on Raven Yong Zhao
[not found] ` <1513218350-29030-1-git-send-email-yong.zhao-5C7GfCeVMHo@public.gmane.org>
2017-12-14 8:47 ` Christian König
[not found] ` <24c4613c-ed30-2ac3-cca2-4aec5dd707d5-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-12-14 15:56 ` Yong Zhao
[not found] ` <9fc57983-8af9-789d-6e60-e3716f846d6c-5C7GfCeVMHo@public.gmane.org>
2017-12-14 17:23 ` Christian König [this message]
2017-12-14 16:45 Yong Zhao
[not found] ` <1513269904-3385-1-git-send-email-yong.zhao-5C7GfCeVMHo@public.gmane.org>
2017-12-16 23:21 ` Felix Kühling
[not found] ` <9ebd38b5-2b12-8797-65e5-3a920ce697aa-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-12-18 9:22 ` Christian König
[not found] ` <e3d20771-df70-b9e5-345b-9914c54db292-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-12-18 17:00 ` Zhao, Yong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=952df2e0-a5dd-64e0-e4b3-241796bc0105@amd.com \
--to=christian.koenig-5c7gfcevmho@public.gmane.org \
--cc=amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=yong.zhao-5C7GfCeVMHo@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.