Re: [PATCH 1/4] drm/edid: Pass connector to AVI inforframe functions

2018-12-04 Thread Andrzej Hajda
On 04.12.2018 20:02, Ville Syrjälä wrote: > On Tue, Dec 04, 2018 at 08:03:53AM +0100, Andrzej Hajda wrote: >> On 03.12.2018 22:48, Ville Syrjälä wrote: >>> On Thu, Nov 29, 2018 at 09:46:16AM +0100, Andrzej Hajda wrote: Quite late, hopefully not too late. On 21.11.2018 12:51,

RE: [PATCH] drm/amdgpu: both support PCO real/fake rlc fw

2018-12-04 Thread Huang, Ray
> -Original Message- > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > Of Aaron Liu > Sent: Wednesday, December 05, 2018 11:12 AM > To: amd-gfx@lists.freedesktop.org > Cc: Liu, Aaron > Subject: [PATCH] drm/amdgpu: both support PCO real/fake rlc fw > > For Picasso

[PATCH 2/3] drm/amdgpu/psp: Add support VMR ring for VF

2018-12-04 Thread Xiangliang Yu
PSP only support VMR ring for SRIOV vf since v45 and all commands will be send to VMR ring for executing. VMR ring use C2PMSG 101 ~ 103 instead of C2PMSG 64 ~ 71. Signed-off-by: Xiangliang Yu --- drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 18 +-- drivers/gpu/drm/amd/amdgpu/amdgpu_psp.h | 1

[PATCH 3/3] drm/amdgpu/psp: Destroy psp ring when doing gpu reset

2018-12-04 Thread Xiangliang Yu
PSP ring need to be destroy before starting reinit for vf. This patche move it from hypervisor driver into guest. Signed-off-by: Xiangliang Yu Signed-off-by: Frank Min --- drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git

[PATCH 1/3] drm/amdgpu/psp: Get psp fw version through reading register

2018-12-04 Thread Xiangliang Yu
If PSP FW is running already, driver will not load PSP FW again and skip it. So psp fw version is not correct if reading it from FW binary file, need to get right version from register. Signed-off-by: Xiangliang Yu --- drivers/gpu/drm/amd/amdgpu/psp_v11_0.c | 4 +++- 1 file changed, 3

Re: [PATCH 1/4] drm/edid: Pass connector to AVI inforframe functions

2018-12-04 Thread Laurent Pinchart
Hi Ville, On Tuesday, 4 December 2018 21:13:20 EET Ville Syrjälä wrote: > On Tue, Dec 04, 2018 at 08:46:53AM +0100, Andrzej Hajda wrote: > > On 03.12.2018 22:38, Ville Syrjälä wrote: > >> On Thu, Nov 29, 2018 at 10:08:07AM +0100, Andrzej Hajda wrote: > >>> On 21.11.2018 19:19, Laurent Pinchart

[PATCH] drm/amdgpu: both support PCO real/fake rlc fw

2018-12-04 Thread Aaron Liu
For Picasso && AM4 SOCKET board, we use picasso_rlc_fake.bin For Picasso && FP5 SOCKET board, we use picasso_rlc.bin Judgment method: PCO AM4: revision >= 0xC8 && revision <= 0xCF or revision >= 0xD8 && revision <= 0xDF otherwise is PCO FP5 Change-Id:

Re: [PATCH 1/2] drm/amdgpu: use HMM mirror callback to replace mmu notifier v5

2018-12-04 Thread Yang, Philip
On 2018-12-04 4:06 a.m., Christian König wrote: > Am 03.12.18 um 21:19 schrieb Yang, Philip: >> Replace our MMU notifier with hmm_mirror_ops.sync_cpu_device_pagetables >> callback. Enable CONFIG_HMM and CONFIG_HMM_MIRROR as a dependency in >> DRM_AMDGPU_USERPTR Kconfig. >> >> It supports both KFD

RE: [PATCH] drm/amdgpu/si: fix SI after doorbell rework

2018-12-04 Thread Zeng, Oak
Reviewed-by: Oak Zeng Regards, Oak -Original Message- From: Alex Deucher Sent: Tuesday, December 4, 2018 2:01 PM To: amd-gfx list ; Zeng, Oak Cc: Deucher, Alexander Subject: Re: [PATCH] drm/amdgpu/si: fix SI after doorbell rework Ping? On Sun, Dec 2, 2018 at 9:50 PM Alex Deucher

Re: [PATCH 1/4] drm/edid: Pass connector to AVI inforframe functions

2018-12-04 Thread Ville Syrjälä
On Tue, Dec 04, 2018 at 08:46:53AM +0100, Andrzej Hajda wrote: > On 03.12.2018 22:38, Ville Syrjälä wrote: > > On Thu, Nov 29, 2018 at 10:08:07AM +0100, Andrzej Hajda wrote: > >> On 21.11.2018 19:19, Laurent Pinchart wrote: > >>> Hi Ville, > >>> > >>> Thank you for the patch. > >>> > >>> On

Re: [PATCH] drm/amdgpu/si: fix SI after doorbell rework

2018-12-04 Thread sylvain . bertrand
On Tue, Dec 04, 2018 at 02:00:54PM -0500, Alex Deucher wrote: > Ping? Does fix the SI issue (done in bugzilla). -- Sylvain ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Re: [PATCH 1/4] drm/edid: Pass connector to AVI inforframe functions

2018-12-04 Thread Ville Syrjälä
On Tue, Dec 04, 2018 at 08:03:53AM +0100, Andrzej Hajda wrote: > On 03.12.2018 22:48, Ville Syrjälä wrote: > > On Thu, Nov 29, 2018 at 09:46:16AM +0100, Andrzej Hajda wrote: > >> Quite late, hopefully not too late. > >> > >> > >> On 21.11.2018 12:51, Ville Syrjälä wrote: > >>> On Wed, Nov 21, 2018

Re: [PATCH] drm/amdgpu/si: fix SI after doorbell rework

2018-12-04 Thread Alex Deucher
Ping? On Sun, Dec 2, 2018 at 9:50 PM Alex Deucher wrote: > > SI does not use doorbells, move asic doorbell init later > asic check. > > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=108920 > Signed-off-by: Alex Deucher > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 3 ++- > 1 file

Re: [PATCH 2/2] drm/amdgpu/powerplay: fix clock stretcher limits on polaris

2018-12-04 Thread Alex Deucher
On Mon, Dec 3, 2018 at 9:39 PM Zhang, Jerry(Junwei) wrote: > > On 12/4/18 12:21 AM, Alex Deucher wrote: > > Adjust limits for newer polaris variants. > > > > Signed-off-by: Alex Deucher > > --- > > drivers/gpu/drm/amd/powerplay/smumgr/polaris10_smumgr.c | 17 > > +++-- > > 1 file

Re: regression in ttm

2018-12-04 Thread Koenig, Christian
Thanks, going to take a look tomorrow. Christian. Am 04.12.2018 18:28 schrieb "StDenis, Tom" : This commit causes a regression on my Carrizo running piglit (dmesg attached) commit 5786b66c9e3b7b18f3c24566e70cae450969cb14 Refs: v4.20-rc3-498-g5786b66c9e3b Author: Christian König AuthorDate:

regression in ttm

2018-12-04 Thread StDenis, Tom
This commit causes a regression on my Carrizo running piglit (dmesg attached) commit 5786b66c9e3b7b18f3c24566e70cae450969cb14 Refs: v4.20-rc3-498-g5786b66c9e3b Author: Christian König AuthorDate: Mon Sep 24 13:35:53 2018 +0200 Commit: Christian König CommitDate: Tue Dec 4 10:22:22 2018

Re: [PATCH] drm/amdgpu/vcn:Fixed S3 hung issue

2018-12-04 Thread Liu, Leo
Please re-state the patch subject instead of "Fixed S3 hung issue" With that fixed, the patch is Reviewed-by: Leo Liu On 12/4/18 10:31 AM, Zhu, James wrote: > Replace vcn_v1_0_stop with vcn_v1_0_set_powergating_state during suspend, > to keep adev->vcn.cur_state update. It will fix VCN S3

[PATCH] drm/amdgpu/vcn:Fixed S3 hung issue

2018-12-04 Thread Zhu, James
Replace vcn_v1_0_stop with vcn_v1_0_set_powergating_state during suspend, to keep adev->vcn.cur_state update. It will fix VCN S3 hung issue. Signed-off-by: James Zhu --- drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

Re: [PATCH 0/4] KFD upstreaming Nov 2018, part 2

2018-12-04 Thread Alex Deucher
For the series: Acked-by: Alex Deucher On Mon, Dec 3, 2018 at 7:05 PM Kuehling, Felix wrote: > > Ping. Any comments, R-b, A-b? > > On 2018-11-20 10:07 p.m., Kuehling, Felix wrote: > > This round adds support for more ROCm memory manager features: > > * VRAM limit checking to avoid overcommitment

Re: [PATCH 01/10] dma-buf: make fence sequence numbers 64 bit v2

2018-12-04 Thread Chris Wilson
Quoting Chris Wilson (2018-12-04 12:52:15) > Quoting Christian König (2018-12-04 11:59:39) > > -static inline bool __dma_fence_is_later(u32 f1, u32 f2) > > +static inline bool __dma_fence_is_later(u64 f1, u64 f2) > > { > > - return (int)(f1 - f2) > 0; > > + /* This is for backward

Re: [PATCH 01/10] dma-buf: make fence sequence numbers 64 bit v2

2018-12-04 Thread Chris Wilson
Quoting Christian König (2018-12-04 11:59:39) > -static inline bool __dma_fence_is_later(u32 f1, u32 f2) > +static inline bool __dma_fence_is_later(u64 f1, u64 f2) > { > - return (int)(f1 - f2) > 0; > + /* This is for backward compatibility with drivers which can only > handle > +

Re: [PATCH 08/10] drm/syncobj: use the timeline point in drm_syncobj_find_fence v3

2018-12-04 Thread Chunming Zhou
在 2018/12/4 19:59, Christian König 写道: > Implement finding the right timeline point in drm_syncobj_find_fence. > > v2: return -EINVAL when the point is not submitted yet. > v3: fix reference counting bug, add flags handling as well > > Signed-off-by: Christian König > --- >

Re: [PATCH 05/10] drm/syncobj: add new drm_syncobj_add_point interface v2

2018-12-04 Thread Chunming Zhou
在 2018/12/4 19:59, Christian König 写道: > Use the dma_fence_chain object to create a timeline of fence objects > instead of just replacing the existing fence. > > v2: rebase and cleanup > > Signed-off-by: Christian König Reviewed-by: Chunming Zhou > --- > drivers/gpu/drm/drm_syncobj.c | 37

Re: [PATCH 04/10] drm/syncobj: remove drm_syncobj_cb and cleanup

2018-12-04 Thread Chunming Zhou
在 2018/12/4 19:59, Christian König 写道: > This completes "drm/syncobj: Drop add/remove_callback from driver > interface" and cleans up the implementation a bit. > > Signed-off-by: Christian König Reviewed-by: Chunming Zhou > --- > drivers/gpu/drm/drm_syncobj.c | 91 >

Re: [PATCH 03/10] drm: revert "expand replace_fence to support timeline point v2"

2018-12-04 Thread Chunming Zhou
在 2018/12/4 19:59, Christian König 写道: > This reverts commit 9a09a42369a4a37a959c051d8e1a1f948c1529a4. > > The whole interface isn't thought through. Since this function can't > fail we actually can't allocate an object to store the sync point. > > Sorry, I should have taken the lead on this

Re: [PATCH 02/10] dma-buf: add new dma_fence_chain container v3

2018-12-04 Thread Chunming Zhou
在 2018/12/4 19:59, Christian König 写道: > Lockless container implementation similar to a dma_fence_array, but with > only two elements per node and automatic garbage collection. > > v2: properly document dma_fence_chain_for_each, add > dma_fence_chain_find_seqno, > drop prev reference

Re: [PATCH 01/10] dma-buf: make fence sequence numbers 64 bit v2

2018-12-04 Thread Chunming Zhou
在 2018/12/4 19:59, Christian König 写道: > For a lot of use cases we need 64bit sequence numbers. Currently drivers > overload the dma_fence structure to store the additional bits. > > Stop doing that and make the sequence number in the dma_fence always > 64bit. > > For compatibility with hardware

[PATCH 06/10] drm/syncobj: add support for timeline point wait v7

2018-12-04 Thread Christian König
From: Chunming Zhou points array is one-to-one match with syncobjs array. v2: add seperate ioctl for timeline point wait, otherwise break uapi. v3: userspace can specify two kinds waits:: a. Wait for time point to be completed. b. and wait for time point to become available v4: rebase v5: add

[PATCH 05/10] drm/syncobj: add new drm_syncobj_add_point interface v2

2018-12-04 Thread Christian König
Use the dma_fence_chain object to create a timeline of fence objects instead of just replacing the existing fence. v2: rebase and cleanup Signed-off-by: Christian König --- drivers/gpu/drm/drm_syncobj.c | 37 + include/drm/drm_syncobj.h | 5 + 2

[PATCH 10/10] drm/amdgpu: update version for timeline syncobj support in amdgpu

2018-12-04 Thread Christian König
From: Chunming Zhou Signed-off-by: Chunming Zhou --- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c index 90f474f98b6e..316bfc1a6a75 100644 ---

[PATCH 08/10] drm/syncobj: use the timeline point in drm_syncobj_find_fence v3

2018-12-04 Thread Christian König
Implement finding the right timeline point in drm_syncobj_find_fence. v2: return -EINVAL when the point is not submitted yet. v3: fix reference counting bug, add flags handling as well Signed-off-by: Christian König --- drivers/gpu/drm/drm_syncobj.c | 43

[PATCH 09/10] drm/amdgpu: add timeline support in amdgpu CS v2

2018-12-04 Thread Christian König
From: Chunming Zhou syncobj wait/signal operation is appending in command submission. v2: separate to two kinds in/out_deps functions Signed-off-by: Chunming Zhou Cc: Daniel Rakos Cc: Jason Ekstrand Cc: Bas Nieuwenhuizen Cc: Dave Airlie Cc: Christian König Cc: Chris Wilson ---

[PATCH 07/10] drm/syncobj: add timeline payload query ioctl v4

2018-12-04 Thread Christian König
From: Chunming Zhou user mode can query timeline payload. v2: check return value of copy_to_user v3: handle querying entry by entry v4: rebase on new chain container, simplify interface Signed-off-by: Chunming Zhou Cc: Daniel Rakos Cc: Jason Ekstrand Cc: Bas Nieuwenhuizen Cc: Dave Airlie

[PATCH 04/10] drm/syncobj: remove drm_syncobj_cb and cleanup

2018-12-04 Thread Christian König
This completes "drm/syncobj: Drop add/remove_callback from driver interface" and cleans up the implementation a bit. Signed-off-by: Christian König --- drivers/gpu/drm/drm_syncobj.c | 91 ++- include/drm/drm_syncobj.h | 21 -- 2 files changed,

[PATCH 03/10] drm: revert "expand replace_fence to support timeline point v2"

2018-12-04 Thread Christian König
This reverts commit 9a09a42369a4a37a959c051d8e1a1f948c1529a4. The whole interface isn't thought through. Since this function can't fail we actually can't allocate an object to store the sync point. Sorry, I should have taken the lead on this from the very beginning and reviewed it more

[PATCH 02/10] dma-buf: add new dma_fence_chain container v3

2018-12-04 Thread Christian König
Lockless container implementation similar to a dma_fence_array, but with only two elements per node and automatic garbage collection. v2: properly document dma_fence_chain_for_each, add dma_fence_chain_find_seqno, drop prev reference during garbage collection if it's not a chain fence. v3:

[PATCH 01/10] dma-buf: make fence sequence numbers 64 bit v2

2018-12-04 Thread Christian König
For a lot of use cases we need 64bit sequence numbers. Currently drivers overload the dma_fence structure to store the additional bits. Stop doing that and make the sequence number in the dma_fence always 64bit. For compatibility with hardware which can do only 32bit sequences the comparisons in

Re: [PATCH v3 1/3] drm/connector: Add generic underscan properties

2018-12-04 Thread Daniel Vetter
On Mon, Dec 03, 2018 at 07:50:53AM -0800, Eric Anholt wrote: > Boris Brezillon writes: > > > On Mon, 3 Dec 2018 16:40:11 +0200 > > Ville Syrjälä wrote: > > > >> On Thu, Nov 22, 2018 at 12:23:29PM +0100, Boris Brezillon wrote: > >> > @@ -924,6 +978,29 @@ struct drm_connector { > >> >

答复: [PATCH] drm/amd/powerplay: improve OD code robustness

2018-12-04 Thread Yin, Tianci (Rico)
thanks Rex, done! I only push it to amd-staging-drm-next, is this enough? 发件人: Zhu, Rex 发送时间: 2018年12月4日 17:26:36 收件人: Yin, Tianci (Rico); Zhang, Jerry; Li, Pauline; Teng, Rui; Liang, Prike; Zhu, Changfeng; Wang, Kevin(Yang); amd-gfx@lists.freedesktop.org 主题:

Re: [PATCH] drm/amd/powerplay: improve OD code robustness

2018-12-04 Thread Zhu, Rex
Please add Signed-off-by in the patch commit. Except that, Patch is Reviewed-by: Rex Zhu Best Regards Rex From: Yin, Tianci (Rico) Sent: Tuesday, December 4, 2018 5:01 PM To: Zhu, Rex; Zhang, Jerry; Li, Pauline; Teng, Rui; Liang, Prike; Zhu, Changfeng;

Re: [PATCH 1/2] drm/amdgpu: use HMM mirror callback to replace mmu notifier v5

2018-12-04 Thread Christian König
Am 03.12.18 um 21:19 schrieb Yang, Philip: Replace our MMU notifier with hmm_mirror_ops.sync_cpu_device_pagetables callback. Enable CONFIG_HMM and CONFIG_HMM_MIRROR as a dependency in DRM_AMDGPU_USERPTR Kconfig. It supports both KFD userptr and gfx userptr paths. The depdent HMM patchsets from

答复: [PATCH] drm/amd/powerplay: improve OD code robustness

2018-12-04 Thread Yin, Tianci (Rico)
hi , a lower request system clock may cause gpu hang, add protection code to avoid this kind of issue, pls help to review. thanks! Rico From e8f0a05ae172f6e2988148dc925b0bde9943be9a Mon Sep 17 00:00:00 2001 From: tianci yin Date: Tue, 4 Dec 2018 16:07:18 +0800 Subject: [PATCH]

[PATCH -next] drm/amdgpu: Fix return value check in amdgpu_allocate_static_csa()

2018-12-04 Thread Wei Yongjun
Fix the return value check which testing the wrong variable in amdgpu_allocate_static_csa(). Fixes: 7946340fa389 ("drm/amdgpu: Move csa related code to separate file") Signed-off-by: Wei Yongjun --- drivers/gpu/drm/amd/amdgpu/amdgpu_csa.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)