From: "Christian König" <ckoenig.leichtzumerken-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: "Michel Dänzer" <michel-otUistvHUpPR7s880joybQ@public.gmane.org>,
"Christian König" <christian.koenig-5C7GfCeVMHo@public.gmane.org>,
"Felix Kuehling" <felix.kuehling-5C7GfCeVMHo@public.gmane.org>
Cc: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Subject: Re: [PATCH 01/11] drm/amdgpu: try allocating VRAM as power of two
Date: Tue, 11 Sep 2018 10:20:29 +0200 [thread overview]
Message-ID: <a3c91610-e5e2-c666-cfda-344f1f40485c@gmail.com> (raw)
In-Reply-To: <5fd331d0-24ab-7540-2d0a-b227230743c0-otUistvHUpPR7s880joybQ@public.gmane.org>
Am 11.09.2018 um 09:55 schrieb Michel Dänzer:
> On 2018-09-11 9:46 a.m., Christian König wrote:
>> Am 11.09.2018 um 09:37 schrieb Michel Dänzer:
>>> On 2018-09-11 8:49 a.m., Christian König wrote:
>>>> But another question: Why do you want to clear VRAM on allocation? We
>>>> perfectly support allocating VRAM without clearing it.
>>> Which is a problem of its own, as it can leak information from one
>>> process to another.
>> How about adding a AMDGPU_GEM_CREATE_VRAM_SECRED flag which triggers
>> clearing VRAM when the BO is freed?
> Doesn't sound like much better API than AMDGPU_GEM_CREATE_VRAM_CLEARED
> to me. Why would any process be okay with leaking its BO contents to
> other processes?
Well the content of most BOs is uninteresting to other processes, e.g.
textures.
Only things like page tables and maybe some security related
calculations need extra protection as far as I can see.
> Could that flag even prevent it in all cases, e.g. when
> BOs are evicted or otherwise migrated?
Yeah, sure that shouldn't be a problem. Clearing a BO after it is moved
should be trivial.
Christian.
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
next prev parent reply other threads:[~2018-09-11 8:20 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-09 18:03 Optimize VM handling a bit more Christian König
[not found] ` <20180909180339.1910-1-christian.koenig-5C7GfCeVMHo@public.gmane.org>
2018-09-09 18:03 ` [PATCH 01/11] drm/amdgpu: try allocating VRAM as power of two Christian König
[not found] ` <20180909180339.1910-2-christian.koenig-5C7GfCeVMHo@public.gmane.org>
2018-09-11 0:08 ` Felix Kuehling
[not found] ` <60b9c96a-d31d-cd61-df22-c8414f0166dc-5C7GfCeVMHo@public.gmane.org>
2018-09-11 6:49 ` Christian König
[not found] ` <9b61bb4c-4721-c8af-0810-4ecc18ed1ea4-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2018-09-11 7:37 ` Michel Dänzer
[not found] ` <46b7da24-4b04-7daf-e6c3-07e10ecb6fd6-otUistvHUpPR7s880joybQ@public.gmane.org>
2018-09-11 7:46 ` Christian König
[not found] ` <d0af0309-e4f9-9825-fd30-9cf79ed5f823-5C7GfCeVMHo@public.gmane.org>
2018-09-11 7:55 ` Michel Dänzer
[not found] ` <5fd331d0-24ab-7540-2d0a-b227230743c0-otUistvHUpPR7s880joybQ@public.gmane.org>
2018-09-11 8:20 ` Christian König [this message]
[not found] ` <a3c91610-e5e2-c666-cfda-344f1f40485c-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2018-09-11 8:25 ` Michel Dänzer
2018-09-11 14:27 ` Kuehling, Felix
[not found] ` <DM5PR12MB17073F8B673AEFC092FBA5B392040-2J9CzHegvk9TCtO+SvGBKwdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2018-09-11 14:48 ` Christian König
[not found] ` <2361b55f-ed56-bd42-b628-fc001d9ea300-5C7GfCeVMHo@public.gmane.org>
2018-09-11 22:25 ` Felix Kuehling
2018-09-11 9:50 ` Huang Rui
2018-09-11 11:16 ` Christian König
2018-09-09 18:03 ` [PATCH 02/11] drm/amdgpu: add amdgpu_vm_pt_parent helper Christian König
2018-09-09 18:03 ` [PATCH 03/11] drm/amdgpu: add amdgpu_vm_update_func Christian König
2018-09-09 18:03 ` [PATCH 04/11] drm/amdgpu: add some VM PD/PT iterators Christian König
[not found] ` <20180909180339.1910-5-christian.koenig-5C7GfCeVMHo@public.gmane.org>
2018-09-11 1:51 ` Felix Kuehling
2018-09-11 2:38 ` Zhang, Jerry (Junwei)
2018-09-09 18:03 ` [PATCH 05/11] drm/amdgpu: use leaf iterator for allocating PD/PT Christian König
2018-09-09 18:03 ` [PATCH 06/11] drm/amdgpu: use dfs iterator to free PDs/PTs Christian König
2018-09-09 18:03 ` [PATCH 07/11] drm/amdgpu: use the DFS iterator in amdgpu_vm_invalidate_level Christian König
[not found] ` <20180909180339.1910-8-christian.koenig-5C7GfCeVMHo@public.gmane.org>
2018-09-11 0:19 ` Felix Kuehling
2018-09-09 18:03 ` [PATCH 08/11] drm/amdgpu: use leaf iterator for filling PTs Christian König
2018-09-09 18:03 ` [PATCH 09/11] drm/amdgpu: meld together VM fragment and huge page handling Christian König
2018-09-09 18:03 ` [PATCH 10/11] drm/amdgpu: use the maximum possible fragment size on Vega/Raven Christian König
[not found] ` <20180909180339.1910-11-christian.koenig-5C7GfCeVMHo@public.gmane.org>
2018-09-11 0:45 ` Felix Kuehling
2018-09-09 18:03 ` [PATCH 11/11] drm/amdgpu: allow fragment processing for invalid PTEs Christian König
2018-09-11 2:17 ` Optimize VM handling a bit more Felix Kuehling
2018-09-11 2:39 ` Zhang, Jerry (Junwei)
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=a3c91610-e5e2-c666-cfda-344f1f40485c@gmail.com \
--to=ckoenig.leichtzumerken-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
--cc=amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=christian.koenig-5C7GfCeVMHo@public.gmane.org \
--cc=felix.kuehling-5C7GfCeVMHo@public.gmane.org \
--cc=michel-otUistvHUpPR7s880joybQ@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox