Re: [PATCH] drm/amdgpu: Place new CPU-accessbile BOs in GTT if visible VRAM is full

2017-05-21 Thread Michel Dänzer
On 20/05/17 12:21 PM, John Brooks wrote: > On Sat, May 20, 2017 at 10:27:14AM +0900, Michel Dänzer wrote: >> On 20/05/17 04:23 AM, John Brooks wrote: >>> On Fri, May 19, 2017 at 04:03:28PM +0900, Michel Dänzer wrote: On 19/05/17 12:04 PM, John Brooks wrote: > Set GTT as the busy placement

Re: [PATCH] drm/amdgpu: Place new CPU-accessbile BOs in GTT if visible VRAM is full

2017-05-19 Thread Michel Dänzer
On 20/05/17 04:23 AM, John Brooks wrote: > On Fri, May 19, 2017 at 04:03:28PM +0900, Michel Dänzer wrote: >> On 19/05/17 12:04 PM, John Brooks wrote: >>> Set GTT as the busy placement for newly created BOs that have the >>> AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED flag, so that they don't cause >>>

Re: [PATCH] drm/amdgpu: Place new CPU-accessbile BOs in GTT if visible VRAM is full

2017-05-19 Thread Michel Dänzer
On 20/05/17 12:52 AM, Marek Olšák wrote: > On Fri, May 19, 2017 at 9:03 AM, Michel Dänzer wrote: >> On 19/05/17 12:04 PM, John Brooks wrote: >>> Set GTT as the busy placement for newly created BOs that have the >>> AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED flag, so that they don't

Re: [PATCH] drm/amdgpu: Place new CPU-accessbile BOs in GTT if visible VRAM is full

2017-05-19 Thread John Brooks
On Fri, May 19, 2017 at 04:03:28PM +0900, Michel Dänzer wrote: > On 19/05/17 12:04 PM, John Brooks wrote: > > Set GTT as the busy placement for newly created BOs that have the > > AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED flag, so that they don't cause > > established BOs to be evicted from visible

Re: [PATCH] drm/amdgpu: Place new CPU-accessbile BOs in GTT if visible VRAM is full

2017-05-19 Thread Marek Olšák
On Fri, May 19, 2017 at 9:03 AM, Michel Dänzer wrote: > On 19/05/17 12:04 PM, John Brooks wrote: >> Set GTT as the busy placement for newly created BOs that have the >> AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED flag, so that they don't cause >> established BOs to be evicted from

Re: [PATCH] drm/amdgpu: Place new CPU-accessbile BOs in GTT if visible VRAM is full

2017-05-19 Thread Michel Dänzer
On 19/05/17 12:04 PM, John Brooks wrote: > Set GTT as the busy placement for newly created BOs that have the > AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED flag, so that they don't cause > established BOs to be evicted from visible VRAM. This is an interesting idea, but there are some issues with this

Re: [PATCH] drm/amdgpu: Place new CPU-accessbile BOs in GTT if visible VRAM is full

2017-05-18 Thread zhoucm1
On 2017年05月19日 11:04, John Brooks wrote: Set GTT as the busy placement for newly created BOs that have the AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED flag, so that they don't cause established BOs to be evicted from visible VRAM. You this patch is consistent with my opinion I mentioned in Marek

[PATCH] drm/amdgpu: Place new CPU-accessbile BOs in GTT if visible VRAM is full

2017-05-18 Thread John Brooks
Set GTT as the busy placement for newly created BOs that have the AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED flag, so that they don't cause established BOs to be evicted from visible VRAM. Signed-off-by: John Brooks --- drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 18