Re: [PATCH] drm/amdgpu: pin the csb buffer on hw init for gfx v8

2019-08-04 Thread Yuan, Xiaojie
Reviewed-by: Xiaojie Yuan BR, Xiaojie > On Aug 5, 2019, at 10:20 AM, Xu, Feifei wrote: > > Reviewed-by: Feifei Xu > > -Original Message- > From: amd-gfx On Behalf Of Gao Likun > Sent: Monday, August 5, 2019 10:07 AM > To: amd-gfx@lists.freedesktop.org > Cc: Deucher, Alexander ;

RE: [PATCH] drm/amdgpu: pin the csb buffer on hw init for gfx v8

2019-08-04 Thread Xu, Feifei
Reviewed-by: Feifei Xu -Original Message- From: amd-gfx On Behalf Of Gao Likun Sent: Monday, August 5, 2019 10:07 AM To: amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander ; Gao, Likun ; Paul Gover ; Yuan, Xiaojie Subject: [PATCH] drm/amdgpu: pin the csb buffer on hw init for gfx

RE: [PATCH libdrm 1/1] tests/amdgpu: add the missing deactivation case for dispatch test

2019-08-04 Thread Xu, Feifei
Reviewed-by: Feifei Xu -Original Message- From: amd-gfx On Behalf Of Le Ma Sent: Monday, August 5, 2019 10:14 AM To: amd-gfx@lists.freedesktop.org Cc: Ma, Le Subject: [PATCH libdrm 1/1] tests/amdgpu: add the missing deactivation case for dispatch test Change-Id:

[PATCH libdrm 1/1] tests/amdgpu: add the missing deactivation case for dispatch test

2019-08-04 Thread Le Ma
Change-Id: I502cc5fde7f00e41d496bfba0963d4db20459e00 Signed-off-by: Le Ma --- tests/amdgpu/amdgpu_test.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/tests/amdgpu/amdgpu_test.c b/tests/amdgpu/amdgpu_test.c index a4ce8ce..dc54155 100644 ---

[PATCH] drm/amdgpu: pin the csb buffer on hw init for gfx v8

2019-08-04 Thread Gao Likun
From: Likun Gao Without this pin, the csb buffer will be filled with inconsistent data after S3 resume. And that will causes gfx hang on gfxoff exit since this csb will be executed then. Signed-off-by: Likun Gao --- drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 40

RE: [PATCH] drm/amd/powerplay: Zero initialize some variables

2019-08-04 Thread Quan, Evan
Thanks Nathan. The patch is reviewed-by: Evan Quan > -Original Message- > From: Nathan Chancellor > Sent: Monday, August 05, 2019 4:37 AM > To: Quan, Evan ; Deucher, Alexander > ; Koenig, Christian > ; Zhou, David(ChunMing) > > Cc: amd-gfx@lists.freedesktop.org;

Re: between commits d7d170a8e357 and 22051d9c4a57 begins issue page allocation failure in gnome-shell process

2019-08-04 Thread Dave Airlie
On Wed, 31 Jul 2019 at 05:34, Mikhail Gavrilov wrote: > > Is anyone here? Is everyone so busy what is wrong? > RC2 is still affected by this issue and unusable for every day because > opening a video in totem player cause DE a hang with a lot of > messages: > This looks like dc_state got a size

Re: The issue with page allocation 5.3 rc1-rc2 (seems drm culprit here)

2019-08-04 Thread Dave Airlie
On Mon, 5 Aug 2019 at 08:23, Mikhail Gavrilov wrote: > > Hi folks, > Two weeks ago when commit 22051d9c4a57 coming to my system. > Started happen randomly errors: > "gnome-shell: page allocation failure: order:4, > mode:0x40cc0(GFP_KERNEL|__GFP_COMP), > nodemask=(null),cpuset=/,mems_allowed=0" >

Re: [PATCH hmm] drm/amdkfd: fix a use after free race with mmu_notififer unregister

2019-08-04 Thread Kuehling, Felix
On 2019-08-02 16:07, Jason Gunthorpe wrote: > When using mmu_notififer_unregister_no_release() the caller must ensure > there is a SRCU synchronize before the mn memory is freed, otherwise use > after free races are possible, for instance: > > CPU0 CPU1 >

[PATCH] drm/amd/powerplay: Zero initialize some variables

2019-08-04 Thread Nathan Chancellor
Clang warns (only Navi warning shown but Arcturus warns as well): drivers/gpu/drm/amd/amdgpu/../powerplay/navi10_ppt.c:1534:4: warning: variable 'asic_default_power_limit' is used uninitialized whenever '?:' condition is false [-Wsometimes-uninitialized]

Re: [PATCH v6 00/24] Associate ddc adapters with connectors

2019-08-04 Thread Laurent Pinchart
On Tue, Jul 30, 2019 at 04:01:23PM +0100, Emil Velikov wrote: > On 2019/07/26, Andrzej Pietrasiewicz wrote: > > It is difficult for a user to know which of the i2c adapters is for which > > drm connector. This series addresses this problem. > > > > The idea is to have a symbolic link in

Re: [PATCH v6 00/24] Associate ddc adapters with connectors

2019-08-04 Thread Laurent Pinchart
Hi Andrzej, On Fri, Jul 26, 2019 at 07:22:54PM +0200, Andrzej Pietrasiewicz wrote: > It is difficult for a user to know which of the i2c adapters is for which > drm connector. This series addresses this problem. > > The idea is to have a symbolic link in connector's sysfs directory, e.g.: > >

Re: [PATCH v6 01/24] drm: Add ddc link in sysfs created by drm_connector

2019-08-04 Thread Laurent Pinchart
Hi Andrzej, On Sun, Aug 04, 2019 at 03:04:37PM +0300, Laurent Pinchart wrote: > Hi Andrzej, > > Thank you for the patch, and sorry for the late review (I've been > travelling for the past few weeks). > > On Fri, Jul 26, 2019 at 07:22:55PM +0200, Andrzej Pietrasiewicz wrote: > > Add generic code

Re: [PATCH v6 01/24] drm: Add ddc link in sysfs created by drm_connector

2019-08-04 Thread Laurent Pinchart
Hi Andrzej, Thank you for the patch, and sorry for the late review (I've been travelling for the past few weeks). On Fri, Jul 26, 2019 at 07:22:55PM +0200, Andrzej Pietrasiewicz wrote: > Add generic code which creates symbolic links in sysfs, pointing to ddc > interface used by a particular