Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-25 Thread Michel Dänzer
On 25/05/17 08:24 PM, Marek Olšák wrote: > On Thu, May 25, 2017 at 5:31 AM, Michel Dänzer wrote: >> On 24/05/17 08:27 PM, Christian König wrote: >>> Am 24.05.2017 um 13:03 schrieb Marek Olšák: > I think the final solution (done in fault_reserve_notify) should be:

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-25 Thread Marek Olšák
On Thu, May 25, 2017 at 5:31 AM, Michel Dänzer wrote: > On 24/05/17 08:27 PM, Christian König wrote: >> Am 24.05.2017 um 13:03 schrieb Marek Olšák: >>> I think the final solution (done in fault_reserve_notify) should be: >>> if (bo->num_cpu_page_faults++ > 20) >>>

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-24 Thread Michel Dänzer
On 24/05/17 08:27 PM, Christian König wrote: > Am 24.05.2017 um 13:03 schrieb Marek Olšák: >>> >> I think the final solution (done in fault_reserve_notify) should be: >> if (bo->num_cpu_page_faults++ > 20) >> bo->preferred_domain = GTT_WC; I agree something like that will probably be part of

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-24 Thread Christian König
Am 24.05.2017 um 13:03 schrieb Marek Olšák: On Wed, May 24, 2017 at 9:56 AM, Michel Dänzer wrote: On 23/05/17 07:38 PM, Marek Olšák wrote: On Tue, May 23, 2017 at 2:45 AM, Michel Dänzer wrote: On 22/05/17 07:09 PM, Marek Olšák wrote: On Mon, May 22,

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-24 Thread Marek Olšák
On Wed, May 24, 2017 at 9:56 AM, Michel Dänzer wrote: > On 23/05/17 07:38 PM, Marek Olšák wrote: >> On Tue, May 23, 2017 at 2:45 AM, Michel Dänzer wrote: >>> On 22/05/17 07:09 PM, Marek Olšák wrote: On Mon, May 22, 2017 at 12:00 PM, Michel Dänzer

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-24 Thread Michel Dänzer
On 23/05/17 07:38 PM, Marek Olšák wrote: > On Tue, May 23, 2017 at 2:45 AM, Michel Dänzer wrote: >> On 22/05/17 07:09 PM, Marek Olšák wrote: >>> On Mon, May 22, 2017 at 12:00 PM, Michel Dänzer wrote: On 20/05/17 06:26 PM, Marek Olšák wrote: > On

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-23 Thread Marek Olšák
On Tue, May 23, 2017 at 2:45 AM, Michel Dänzer wrote: > On 22/05/17 07:09 PM, Marek Olšák wrote: >> On Mon, May 22, 2017 at 12:00 PM, Michel Dänzer wrote: >>> On 20/05/17 06:26 PM, Marek Olšák wrote: On May 20, 2017 3:26 AM, "Michel Dänzer"

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-22 Thread Michel Dänzer
On 22/05/17 07:09 PM, Marek Olšák wrote: > On Mon, May 22, 2017 at 12:00 PM, Michel Dänzer wrote: >> On 20/05/17 06:26 PM, Marek Olšák wrote: >>> On May 20, 2017 3:26 AM, "Michel Dänzer" >> > wrote: >>> >>> On 20/05/17 01:14

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-22 Thread John Brooks
On Mon, May 22, 2017 at 12:09:21PM +0200, Marek Olšák wrote: > On Mon, May 22, 2017 at 12:00 PM, Michel Dänzer wrote: > > On 20/05/17 06:26 PM, Marek Olšák wrote: > >> On May 20, 2017 3:26 AM, "Michel Dänzer" >> > wrote: > >> >

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-22 Thread Marek Olšák
On Mon, May 22, 2017 at 12:00 PM, Michel Dänzer wrote: > On 20/05/17 06:26 PM, Marek Olšák wrote: >> On May 20, 2017 3:26 AM, "Michel Dänzer" > > wrote: >> >> On 20/05/17 01:14 AM, Marek Olšák wrote: >> > Hi Michel, >>

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-20 Thread Marek Olšák
On May 20, 2017 3:26 AM, "Michel Dänzer" wrote: On 20/05/17 01:14 AM, Marek Olšák wrote: > Hi Michel, > > I've applied your series Thanks for testing it. > and it doesn't help with low Dirt Rally performance on Fiji. I see TTM > buffer moves at 800MB/s and many VRAM page

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-19 Thread Michel Dänzer
On 20/05/17 01:14 AM, Marek Olšák wrote: > Hi Michel, > > I've applied your series Thanks for testing it. > and it doesn't help with low Dirt Rally performance on Fiji. I see TTM > buffer moves at 800MB/s and many VRAM page faults. Did you see this: >> Note that there's only little if any

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-19 Thread Marek Olšák
Hi Michel, I've applied your series and it doesn't help with low Dirt Rally performance on Fiji. I see TTM buffer moves at 800MB/s and many VRAM page faults. Marek On Thu, May 18, 2017 at 11:08 AM, Michel Dänzer wrote: > From: Michel Dänzer > > This

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-19 Thread Marek Olšák
On Fri, May 19, 2017 at 5:27 PM, John Brooks wrote: > On Fri, May 19, 2017 at 05:24:36PM +0200, Marek Olšák wrote: >> Where is your "attached" patch? >> >> Marek > > It's actually a reply to my message. Sorry if that was unclear. That's OK, but I don't see any patch from you

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-19 Thread John Brooks
On Fri, May 19, 2017 at 05:24:36PM +0200, Marek Olšák wrote: > Where is your "attached" patch? > > Marek It's actually a reply to my message. Sorry if that was unclear. > > On Fri, May 19, 2017 at 5:04 AM, John Brooks wrote: > > I'm glad this is being worked on. However,

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-19 Thread Marek Olšák
Where is your "attached" patch? Marek On Fri, May 19, 2017 at 5:04 AM, John Brooks wrote: > I'm glad this is being worked on. However, somewhat to my surprise, this patch > series didn't help Dying Light's BO eviction problem. For those who don't > know, > that game

Re: [PATCH 0/3] drm/amdgpu: Tweaks for high pressure on CPU visible VRAM

2017-05-18 Thread John Brooks
I'm glad this is being worked on. However, somewhat to my surprise, this patch series didn't help Dying Light's BO eviction problem. For those who don't know, that game performs very badly in certain areas, and it is correlated with increased TTM eviction rates. Relevant screenshots of gallium HUD