RE: [PATCH] drm/amdgpu: For virtual_display feature, the vblank_get_counter hook is always return 0 when there's no hardware frame counter which can be used.

2016-08-16 Thread Deng, Emily
> -Original Message- > From: Michel Dänzer [mailto:mic...@daenzer.net] > Sent: Wednesday, August 17, 2016 11:50 AM > To: Deng, Emily > Cc: amd-gfx@lists.freedesktop.org > Subject: Re: [PATCH] drm/amdgpu: For virtual_display feature, the > vblank_get_counter hook is

Re: [PATCH] drm/amdgpu: For virtual_display feature, the vblank_get_counter hook is always return 0 when there's no hardware frame counter which can be used.

2016-08-16 Thread Michel Dänzer
On 16/08/16 07:15 PM, Emily Deng wrote: > Signed-off-by: Emily Deng Please change the shortlog to be no longer than ~72 characters. Maybe something like this for the commit log: drm/amdgpu: Hardcode virtual DCE vblank / scanout position return values By hardcoding 0 for the

RE: [PATCH] drm/amdgpu: For virtual_display feature, the vblank_get_counter hook is always return 0 when there's no hardware frame counter which can be used.

2016-08-16 Thread Deng, Emily
Ping ... Hi Michel, Could you please help to review this? Thanks very much. Best Wishes, Emily Deng > -Original Message- > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of > Emily Deng > Sent: Tuesday, August 16, 2016 6:15 PM > To:

RE: Random short freezes due to TTM buffer migrations

2016-08-16 Thread Zhou, David(ChunMing)
Add his email. > -Original Message- > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > Of Zhou, David(ChunMing) > Sent: Wednesday, August 17, 2016 9:57 AM > To: Kuehling, Felix ; Christian König > ; Marek Olšák

Re: Random short freezes due to TTM buffer migrations

2016-08-16 Thread Felix Kuehling
Very nice. I'm looking forward to this for KFD as well. One question: Will it be possible to share these split BOs as dmabufs? Regards, Felix On 16-08-16 11:27 AM, Christian König wrote: > Hi Marek, > > I'm already working on this. > > My current approach is to use a custom BO manager for

Re: Reverted another change to fix buffer move hangs (was Re: [PATCH] drm/ttm: partial revert "cleanup ttm_tt_(unbind|destroy)" v2)

2016-08-16 Thread Felix Kuehling
Thank you. Sorry, I already pushed it with Alex's R-B, without yours. On 16-08-16 03:53 AM, Christian König wrote: > Am 15.08.2016 um 23:03 schrieb Alex Deucher: >> On Mon, Aug 15, 2016 at 3:06 PM, Felix Kuehling >> wrote: >>> Patch against current amd-staging-4.6 is

underclocking support rx480

2016-08-16 Thread Jarkko Korpi
I haven't tried yet the overclocking feature that's limited 20% at command line. But please make it possible to downlock too. ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx

RE: underclocking support rx480

2016-08-16 Thread Deucher, Alexander
You can already limit the clock levels as I described previously. Alex From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Jarkko Korpi Sent: Tuesday, August 16, 2016 2:37 PM To: amd-gfx@lists.freedesktop.org Subject: underclocking support rx480 I haven't tried yet the

Re: Random short freezes due to TTM buffer migrations

2016-08-16 Thread Alex Deucher
On Tue, Aug 16, 2016 at 11:58 AM, Christian König wrote: > Am 16.08.2016 um 17:50 schrieb Alex Deucher: >> >> On Tue, Aug 16, 2016 at 11:27 AM, Christian König >> wrote: >>> >>> Hi Marek, >>> >>> I'm already working on this. >>> >>> My current

Re: Random short freezes due to TTM buffer migrations

2016-08-16 Thread Christian König
Am 16.08.2016 um 17:56 schrieb Marek Olšák: On Tue, Aug 16, 2016 at 5:27 PM, Christian König wrote: Hi Marek, I'm already working on this. My current approach is to use a custom BO manager for VRAM with TTM and so split allocations into chunks of 4MB. Large BOs are

Re: Random short freezes due to TTM buffer migrations

2016-08-16 Thread Marek Olšák
On Tue, Aug 16, 2016 at 5:27 PM, Christian König wrote: > Hi Marek, > > I'm already working on this. > > My current approach is to use a custom BO manager for VRAM with TTM and so > split allocations into chunks of 4MB. > > Large BOs are still swapped out as one, but it

Random short freezes due to TTM buffer migrations

2016-08-16 Thread Marek Olšák
Hi, I'm seeing random temporary freezes (up to 2 seconds) under memory pressure. Before I describe the exact circumstances, I'd like to say that this is a serious issue affecting playability of certain AAA Linux games. In order to reproduce this, an application should: - allocate a few very

Re: DCE wait for idle

2016-08-16 Thread Alex Deucher
On Tue, Aug 16, 2016 at 7:53 AM, StDenis, Tom wrote: > In these functions > > > static bool dce_v11_0_is_idle(void *handle) > { > return true; > } > > static int dce_v11_0_wait_for_idle(void *handle) > { > return 0; > } > > Shouldn't they wait on the GUI bit of the

[PATCH] drm/amdgpu: For virtual_display feature, the vblank_get_counter hook is always return 0 when there's no hardware frame counter which can be used.

2016-08-16 Thread Emily Deng
Signed-off-by: Emily Deng --- drivers/gpu/drm/amd/amdgpu/dce_virtual.c | 16 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/dce_virtual.c b/drivers/gpu/drm/amd/amdgpu/dce_virtual.c index 2ce5f90..85f14a6 100644 ---

Re: fix possible bad kref_put in amdgpu_uvd_ring_end_use

2016-08-16 Thread Christian König
NAK, we already merged a patch to avoid the fence_put() in general when the ring test fails. Regards, Christian. Am 16.08.2016 um 08:33 schrieb Matthew Macy: Clang identified this when I was merging up 4.8-rc1/rc2. I usually just disable warnings as they pop up as I treat the drivers as

Re: [PATCH] drm/amdgpu: For virtual_display feature, define a variable for vblank count of cpu vsync timer.

2016-08-16 Thread Michel Dänzer
On 16/08/16 03:28 PM, Deng, Emily wrote: >> From: Michel Dänzer [mailto:mic...@daenzer.net] >> Sent: Tuesday, August 16, 2016 12:01 PM >> On 16/08/16 12:49 PM, Deng, Emily wrote: >>> Hi Michel, Thanks, I still couldn't see the issue that use a variable >>> to calculate the vblank count when vsync

RE: [PATCH] drm/amdgpu: For virtual_display feature, define a variable for vblank count of cpu vsync timer.

2016-08-16 Thread Deng, Emily
> -Original Message- > From: Michel Dänzer [mailto:mic...@daenzer.net] > Sent: Tuesday, August 16, 2016 12:01 PM > To: Deng, Emily > Cc: dri-de...@lists.freedesktop.org; amd-gfx@lists.freedesktop.org > Subject: Re: [PATCH] drm/amdgpu: For virtual_display feature,