RE: [PATCH 2/3] drm/amdgpu: drop the sched_sync

2018-11-04 Thread Zhou, David(ChunMing)
> -Original Message- > From: Koenig, Christian > Sent: Monday, November 05, 2018 3:48 PM > To: Liu, Monk ; amd-gfx@lists.freedesktop.org; Zhou, > David(ChunMing) > Subject: Re: [PATCH 2/3] drm/amdgpu: drop the sched_sync > > Am 05.11.18 um 08:24 schrieb Liu, Monk: > >> David Zhou had an

Re: [PATCH 2/3] drm/amdgpu: drop the sched_sync

2018-11-04 Thread Koenig, Christian
Am 05.11.18 um 08:24 schrieb Liu, Monk: >> David Zhou had an use case which saw a >10% performance drop the last time >> he tried it. > I really don't believe that, because if you insert a WAIT_MEM on an already > signaled fence, it only cost GPU couple clocks to move on, right ? no reason > to

RE: [PATCH 2/3] drm/amdgpu: drop the sched_sync

2018-11-04 Thread Liu, Monk
> David Zhou had an use case which saw a >10% performance drop the last time he > tried it. I really don't believe that, because if you insert a WAIT_MEM on an already signaled fence, it only cost GPU couple clocks to move on, right ? no reason to slow down up to 10% ... with 3dmark vulkan ver

RE: [PATCH 1/4] drm/amdgpu: remove nonsense in_interrupt() checks

2018-11-04 Thread Deng, Emily
The series are Reviewed-by: Emily Deng Best wishes Emily Deng >-Original Message- >From: Christian König >Sent: Tuesday, October 30, 2018 10:39 PM >To: Deng, Emily ; amd-gfx@lists.freedesktop.org; Lou, >Wentao >Subject: Re: [PATCH 1/4] drm/amdgpu: remove nonsense in_interrupt() >check

Re: [PATCH] drm/amdkfd: fix interrupt spin lock

2018-11-04 Thread Christian König
Am 02.11.18 um 19:59 schrieb Kuehling, Felix: On 2018-11-02 9:48 a.m., Christian König wrote: Vega10 has multiple interrupt rings, I don't think I've seen your code that implements multiple interrupt rings. So it's a bit hard to comment. It's already a while ago, but you actually reviewed it

Re: [PATCH 2/3] drm/amdgpu: drop the sched_sync

2018-11-04 Thread Koenig, Christian
> Can you tell me which game/benchmark will have performance drop with this fix > by your understanding ? When you sync between submission things like composing X windows are slowed down massively. David Zhou had an use case which saw a >10% performance drop the last time he tried it. > The pr

Re: Freezes back with Raven Ridge 2500u and 4.19

2018-11-04 Thread Bráulio Bhavamitra
Hi Alex, It might be the new firmware from 18.40. I'm back to linux 4.18.16 and having some freezes, although it seems less frequent than with 4.19. Regards, bráulio On Thu, Nov 1, 2018 at 11:24 AM Alex Deucher wrote: > On Thu, Nov 1, 2018 at 8:34 AM Bráulio Bhavamitra > wrote: > > > > Hi all

radeon vs radeonfb Mobility quirks (Thinkpad X32)

2018-11-04 Thread Eric Wong
My Thinkpad X32 (r100, Mobility M6) can't suspend or hibernate with KMS using the "radeon" driver. "radeonfb" and the VESA fallback (no KMS) are both fine. It seems to be the same bug as: https://bugs.freedesktop.org/show_bug.cgi?id=38554 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=583120 T