RE: [PATCH 1/3] drm/amdgpu: add RAS poison creation handler (v2)

2022-04-21 Thread Zhang, Hawking
Hi Tao, I'm thinking of checking ip block first because we might want to leverage this interrupt handler as a common entry for other RAS interrupt as well. In such case, it looks more clearer if we check the ip block first. I agree with you either way looks good. You can have my RB for

RE: [PATCH 1/3] drm/amdgpu: add RAS poison creation handler (v2)

2022-04-21 Thread Zhou1, Tao
[AMD Official Use Only] Hi Hawking, The logic in my patch is: if (poison) if (umc) poison_creation_handler else poison_consumption_handler else if (umc) umc_handler else not supported I think your suggestion is: if (umc) if (poison)

Re: [PATCH] drm/amdkfd: use kvcalloc() instead of kvmalloc() in kfd_migrate

2022-04-21 Thread Wang, Yang(Kevin)
[AMD Official Use Only] From: Kuehling, Felix Sent: Thursday, April 21, 2022 10:55 PM To: Wang, Yang(Kevin) ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdkfd: use kvcalloc() instead of kvmalloc() in kfd_migrate Am 2022-04-21 um 08:33 schrieb

Re: [PATCH 03/15] dma-buf & drm/amdgpu: remove dma_resv workaround

2022-04-21 Thread Zack Rusin
On Thu, 2022-04-21 at 12:17 +0200, Christian König wrote: > ⚠ External Email > > Am 20.04.22 um 21:28 schrieb Zack Rusin: > > [SNIP] > > > To figure out what it is could you try the following code > > > fragment: > > > > > > diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_validation.c > > >

[PATCH v2] drm/amdkfd: Ignore bogus signals from MEC efficiently

2022-04-21 Thread Felix Kuehling
MEC firmware sometimes sends signal interrupts without a valid context ID on end of pipe events that don't intend to signal any HSA signals. This triggers the slow path in kfd_signal_event_interrupt that scans the entire event page for signaled events. Detect these signals in the top half

Re: [PATCH 2/2] drm/amdkfd: Update mapping if range attributes changed

2022-04-21 Thread Felix Kuehling
Am 2022-04-19 um 20:47 schrieb Philip Yang: Change SVM range mapping flags or access attributes don't trigger migration, if range is already mapped on GPUs we should update GPU mapping, and pass flush_tlb flag to amdgpu vm. Signed-off-by: Philip Yang --- drivers/gpu/drm/amd/amdkfd/kfd_svm.c

Re: AMD display drivers handling DRM CRTC color mgmt props

2022-04-21 Thread Melissa Wen
On 04/21, Harry Wentland wrote: > > > On 2022-04-21 10:37, Melissa Wen wrote: > > Hi all, > > > > I'm examining how DRM color management properties (degamma, ctm, gamma) > > are applied to AMD display drivers. As far I could understand thanks > > Nicholas documentation on

Re: [PATCH] drm/amdgpu/display: make hubp31_program_extended_blank static

2022-04-21 Thread Harry Wentland
On 2022-04-21 10:10, Alex Deucher wrote: It's not used outside of dcn31_hubp.c. Signed-off-by: Alex Deucher Reviewed-by: Harry Wentland Harry --- drivers/gpu/drm/amd/display/dc/dcn31/dcn31_hubp.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

Re: [PATCH 2/3 v3] drm/amd/amdgpu: Properly indent PF2VF header

2022-04-21 Thread Alex Deucher
On Wed, Apr 20, 2022 at 6:47 PM Bokun Zhang wrote: > > - Clean up the identation in the header file > > Signed-off-by: Bokun Zhang > --- > drivers/gpu/drm/amd/amdgpu/amdgv_sriovmsg.h | 70 ++--- > 1 file changed, 34 insertions(+), 36 deletions(-) > > diff --git

Re: [PATCH] drm/amdkfd: use kvcalloc() instead of kvmalloc() in kfd_migrate

2022-04-21 Thread Felix Kuehling
Am 2022-04-21 um 08:33 schrieb Yang Wang: simplify programming with existing functions. Signed-off-by: Yang Wang Reviewed-by: Felix Kuehling There is one more kvmalloc_array call with GFP_ZERO that could be replaced with kvcalloc in svm_range_dma_map_dev in kfd_svm.c. Maybe fix that one

RE: [PATCH 1/3] drm/amdgpu: add RAS poison creation handler (v2)

2022-04-21 Thread Zhang, Hawking
[AMD Official Use Only] Hi Tao, I was thinking more aggressive change - current amdgpu_ras_interrupt_handler only serves as umc poison (poison mode) or uncorrectable error handler (fue mode). We can still keep it as a unified entry point, but how about check ip block first, then if it is

Re: [PATCH] drm/amdkfd: use kvcalloc() instead of kvmalloc() in kfd_migrate

2022-04-21 Thread Alex Deucher
On Thu, Apr 21, 2022 at 8:33 AM Yang Wang wrote: > > simplify programming with existing functions. > > Signed-off-by: Yang Wang Reviewed-by: Alex Deucher > --- > drivers/gpu/drm/amd/amdkfd/kfd_migrate.c | 15 +++ > 1 file changed, 7 insertions(+), 8 deletions(-) > > diff --git

AMD display drivers handling DRM CRTC color mgmt props

2022-04-21 Thread Melissa Wen
Hi all, I'm examining how DRM color management properties (degamma, ctm, gamma) are applied to AMD display drivers. As far I could understand thanks Nicholas documentation on amdgpu_dm/amdgpu_dm_color, DC drivers have per-plane color correction features: * - Input gamma LUT (de-normalized) * -

RE: [PATCH v2 2/2] drm/amdgpu: Add debugfs TA load/unload/invoke support

2022-04-21 Thread Li, Candice
[Public] Thanks, Candice -Original Message- From: Lazar, Lijo Sent: Thursday, April 21, 2022 7:01 PM To: Li, Candice ; amd-gfx@lists.freedesktop.org Cc: Clements, John Subject: Re: [PATCH v2 2/2] drm/amdgpu: Add debugfs TA load/unload/invoke support On 4/21/2022 4:08 PM, Li,

Re: [PATCH] drm/amd/display: Fix memory leak in dcn21_clock_source_create

2022-04-21 Thread Alex Deucher
Applied. Thanks! Alex On Thu, Apr 21, 2022 at 5:03 AM Miaoqian Lin wrote: > > When dcn20_clk_src_construct() fails, we need to release clk_src. > > Fixes: 6f4e6361c3ff ("drm/amd/display: Add Renoir resource (v2)") > Signed-off-by: Miaoqian Lin > --- >

Re: [PATCH] drm/amdgpu: don't runtime suspend if there are displays attached (v2)

2022-04-21 Thread Alex Deucher
On Thu, Apr 21, 2022 at 9:06 AM Thorsten Leemhuis wrote: > > On 21.04.22 05:16, Alex Deucher wrote: > > We normally runtime suspend when there are displays attached if they > > are in the DPMS off state, however, if something wakes the GPU > > we send a hotplug event on resume (in case any

Re: [PATCH] drm/amd/display: Remove useless code

2022-04-21 Thread Alex Deucher
Applied. Thanks! Alex On Thu, Apr 21, 2022 at 6:29 AM Haowen Bai wrote: > > aux_rep only memset but no use at all, so we drop it. > > Signed-off-by: Haowen Bai > --- > drivers/gpu/drm/amd/display/dc/dce/dce_aux.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git

[PATCH] drm/amdgpu/display: make hubp31_program_extended_blank static

2022-04-21 Thread Alex Deucher
It's not used outside of dcn31_hubp.c. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/display/dc/dcn31/dcn31_hubp.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/display/dc/dcn31/dcn31_hubp.c b/drivers/gpu/drm/amd/display/dc/dcn31/dcn31_hubp.c

[PATCH] drm/amd/display: Remove useless code

2022-04-21 Thread Haowen Bai
aux_rep only memset but no use at all, so we drop it. Signed-off-by: Haowen Bai --- drivers/gpu/drm/amd/display/dc/dce/dce_aux.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/amd/display/dc/dce/dce_aux.c b/drivers/gpu/drm/amd/display/dc/dce/dce_aux.c index

Re: [PATCH] drm/amdgpu: don't runtime suspend if there are displays attached (v2)

2022-04-21 Thread Thorsten Leemhuis
On 21.04.22 05:16, Alex Deucher wrote: > We normally runtime suspend when there are displays attached if they > are in the DPMS off state, however, if something wakes the GPU > we send a hotplug event on resume (in case any displays were connected > while the GPU was in suspend) which can cause

[PATCH] drm/amd/display: Fix memory leak in dcn21_clock_source_create

2022-04-21 Thread Miaoqian Lin
When dcn20_clk_src_construct() fails, we need to release clk_src. Fixes: 6f4e6361c3ff ("drm/amd/display: Add Renoir resource (v2)") Signed-off-by: Miaoqian Lin --- drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [PATCH V2] drm/amd/pm: fix the deadlock issue observed on SI

2022-04-21 Thread Thorsten Leemhuis
On 11.04.22 10:54, Evan Quan wrote: > The adev->pm.mutx is already held at the beginning of > amdgpu_dpm_compute_clocks/amdgpu_dpm_enable_uvd/amdgpu_dpm_enable_vce. > But on their calling path, amdgpu_display_bandwidth_update will be > called and thus its sub functions amdgpu_dpm_get_sclk/mclk.

Re: [PATCH] drm/ttm: use kvcalloc() instead of kvmalloc_array() in ttm_tt

2022-04-21 Thread Wang, Yang(Kevin)
[AMD Official Use Only] From: Koenig, Christian Sent: Thursday, April 21, 2022 8:48 PM To: Wang, Yang(Kevin) ; amd-gfx@lists.freedesktop.org ; dri-de...@lists.freedesktop.org Cc: Kuehling, Felix ; Lazar, Lijo Subject: Re: [PATCH] drm/ttm: use kvcalloc()

Re: [PATCH] drm/ttm: use kvcalloc() instead of kvmalloc_array() in ttm_tt

2022-04-21 Thread Christian König
Am 21.04.22 um 14:34 schrieb Yang Wang: simplify programming with existing functions. Signed-off-by: Yang Wang With a minimal style change reviewed and pushed to drm-misc-next. Thanks, Christian. --- drivers/gpu/drm/ttm/ttm_tt.c | 21 - 1 file changed, 12

[PATCH] drm/ttm: use kvcalloc() instead of kvmalloc_array() in ttm_tt

2022-04-21 Thread Yang Wang
simplify programming with existing functions. Signed-off-by: Yang Wang --- drivers/gpu/drm/ttm/ttm_tt.c | 21 - 1 file changed, 12 insertions(+), 9 deletions(-) diff --git a/drivers/gpu/drm/ttm/ttm_tt.c b/drivers/gpu/drm/ttm/ttm_tt.c index 79c870a3bef8..cbb3d1fb4caf 100644

[PATCH] drm/amdkfd: use kvcalloc() instead of kvmalloc() in kfd_migrate

2022-04-21 Thread Yang Wang
simplify programming with existing functions. Signed-off-by: Yang Wang --- drivers/gpu/drm/amd/amdkfd/kfd_migrate.c | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c b/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c index

Re: [pull] amdgpu, amdkfd, radeon drm-next-5.19

2022-04-21 Thread Christian König
Hi Dave, just a gentle ping on that one here since it looks like it was missed because of the eastern holidays. I have general DRM/TTM changes I want to publish and need the amdgpu changes in drm-next so that we don't run into any merge conflicts. Thanks, Christian. Am 15.04.22 um 15:51

Re: [PATCH v2 2/2] drm/amdgpu: Add debugfs TA load/unload/invoke support

2022-04-21 Thread Lazar, Lijo
On 4/21/2022 4:08 PM, Li, Candice wrote: [Public] Thanks, Candice -Original Message- From: Lazar, Lijo Sent: Thursday, April 21, 2022 6:03 PM To: Li, Candice ; amd-gfx@lists.freedesktop.org Cc: Clements, John Subject: Re: [PATCH v2 2/2] drm/amdgpu: Add debugfs TA

RE: [PATCH v2 2/2] drm/amdgpu: Add debugfs TA load/unload/invoke support

2022-04-21 Thread Li, Candice
[Public] Thanks, Candice -Original Message- From: Lazar, Lijo Sent: Thursday, April 21, 2022 6:03 PM To: Li, Candice ; amd-gfx@lists.freedesktop.org Cc: Clements, John Subject: Re: [PATCH v2 2/2] drm/amdgpu: Add debugfs TA load/unload/invoke support On 4/21/2022 3:28 PM, Li,

Re: [PATCH 03/15] dma-buf & drm/amdgpu: remove dma_resv workaround

2022-04-21 Thread Christian König
Am 20.04.22 um 21:28 schrieb Zack Rusin: [SNIP] To figure out what it is could you try the following code fragment: diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_validation.c b/drivers/gpu/drm/vmwgfx/vmwgfx_validation.c index f46891012be3..a36f89d3f36d 100644 ---

Re: [PATCH v2 2/2] drm/amdgpu: Add debugfs TA load/unload/invoke support

2022-04-21 Thread Lazar, Lijo
On 4/21/2022 3:28 PM, Li, Candice wrote: [Public] Thanks, Candice -Original Message- From: Lazar, Lijo Sent: Thursday, April 21, 2022 5:23 PM To: Li, Candice ; amd-gfx@lists.freedesktop.org Cc: Clements, John Subject: Re: [PATCH v2 2/2] drm/amdgpu: Add debugfs TA

RE: [PATCH v2 2/2] drm/amdgpu: Add debugfs TA load/unload/invoke support

2022-04-21 Thread Li, Candice
[Public] Thanks, Candice -Original Message- From: Lazar, Lijo Sent: Thursday, April 21, 2022 5:23 PM To: Li, Candice ; amd-gfx@lists.freedesktop.org Cc: Clements, John Subject: Re: [PATCH v2 2/2] drm/amdgpu: Add debugfs TA load/unload/invoke support On 4/20/2022 4:43 PM,

Re: [PATCH v2 2/2] drm/amdgpu: Add debugfs TA load/unload/invoke support

2022-04-21 Thread Lazar, Lijo
On 4/20/2022 4:43 PM, Candice Li wrote: v1: Add debugfs support to load/unload/invoke TA in runtime. v2: 1. Update some variables to static. 2. Use PAGE_ALIGN to calculate shared buf size directly. 3. Remove fp check. 4. Update debugfs from read to write. Signed-off-by: John Clements

RE: [PATCH v2 2/2] drm/amdgpu: Add debugfs TA load/unload/invoke support

2022-04-21 Thread Zhang, Hawking
[AMD Official Use Only] Series is Reviewed-by: Hawking Zhang Regards, Hawking -Original Message- From: amd-gfx On Behalf Of Candice Li Sent: Wednesday, April 20, 2022 19:14 To: amd-gfx@lists.freedesktop.org Cc: Li, Candice ; Clements, John Subject: [PATCH v2 2/2] drm/amdgpu: Add

RE: [PATCH v2 2/2] drm/amdgpu: Add debugfs TA load/unload/invoke support

2022-04-21 Thread Zhou1, Tao
The series is: Reviewed-by: Tao Zhou Make sure the series is checked by checkpatch.pl. > -Original Message- > From: amd-gfx On Behalf Of Candice > Li > Sent: Wednesday, April 20, 2022 7:14 PM > To: amd-gfx@lists.freedesktop.org > Cc: Li, Candice ; Clements, John > > Subject: [PATCH

Re: [PATCH] drm/ttm: fix ttm tt init fail when size exceeds kmalloc limit

2022-04-21 Thread Christian König
Am 21.04.22 um 04:15 schrieb Wang, Yang(Kevin): [AMD Official Use Only] *From:* Kuehling, Felix *Sent:* Thursday, April 21, 2022 5:21 AM *To:* Lazar, Lijo ; Koenig, Christian ; Wang, Yang(Kevin) ; Christian König ;