RE: [PATCH 21/22] drm/amd/pm: fix get dpm level count for yello carp

2024-05-09 Thread Zhang, Yifan
[AMD Official Use Only - General] Reviewed-by: Yifan Zhang Best Regards, Yifan -Original Message- From: amd-gfx On Behalf Of Jesse Zhang Sent: Friday, May 10, 2024 10:51 AM To: amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander ; Koenig, Christian ; Huang, Tim ; Zhang, Jesse(Jie)

RE: [PATCH 20/22] drm/amd/pm: fix get dpm level count for smu13

2024-05-09 Thread Zhang, Yifan
[AMD Official Use Only - General] Reviewed-by: Yifan Zhang Best Regards, Yifan -Original Message- From: amd-gfx On Behalf Of Jesse Zhang Sent: Friday, May 10, 2024 10:51 AM To: amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander ; Koenig, Christian ; Huang, Tim ; Zhang, Jesse(Jie)

[PATCH 2/2] drm/amdkfd: Minor corrections to a previous commit

2024-05-09 Thread Sreekant Somasekharan
This patch rectifies mistake in previous commit: 'drm/amdkfd: mark GFX12 system and peer GPU memory mappings as MTYPE_NC'. Signed-off-by: Sreekant Somasekharan --- drivers/gpu/drm/amd/amdgpu/gmc_v12_0.c | 3 ++- drivers/gpu/drm/amd/amdkfd/kfd_svm.c | 1 + 2 files changed, 3 insertions(+), 1

[PATCH 1/2] drm/amdkfd: Remove bo NULL check in gmc_v12_0_get_vm_pte() function

2024-05-09 Thread Sreekant Somasekharan
Remove bo NULL check in amdgpu/gmc_v12_0.c:gmc_v12_0_get_vm_pte() function to fix smatch warning: 'drivers/gpu/drm/amd/amdgpu/gmc_v12_0.c:518 gmc_v12_0_get_vm_pte() warn: variable dereferenced before check 'bo' (see line 500)' Signed-off-by: Sreekant Somasekharan Suggested-by: Dan Carpenter

RE: [PATCH 13/22] drm/amd/pm: remove logically dead code for renoir

2024-05-09 Thread Wang, Yang(Kevin)
[AMD Official Use Only - General] Please add following information into commit message. Fixes: ca55f459f5ad ("drm/amd/pm: add the fine grain tuning function for Renoir") Reviewed-by: Yang Wang Best Regards, Kevin -Original Message- From: amd-gfx On Behalf Of Jesse Zhang Sent:

Re: [PATCH v3 2/3] drm/amd/amdgpu: Add ISP driver support

2024-05-09 Thread Mario Limonciello
On 5/9/2024 16:50, Pratap Nirujogi wrote: Add the isp driver in amdgpu to support ISP device on the APUs that supports ISP IP block. ISP hw block is used for camera front-end, pre and post processing operations. Signed-off-by: Pratap Nirujogi Reviewed-by: Mario Limonciello --- Changes made

RE: [PATCH 15/22] drm/amd/pm: fix enum feature compared against 0

2024-05-09 Thread Wang, Yang(Kevin)
[AMD Official Use Only - General] Reviewed-by: Yang Wang Best Regards, Kevin -Original Message- From: amd-gfx On Behalf Of Jesse Zhang Sent: Friday, May 10, 2024 10:51 AM To: amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander ; Koenig, Christian ; Huang, Tim ; Zhang, Jesse(Jie) ;

RE: [PATCH 05/22] drm/amd/pm: check specific index for aldebaran

2024-05-09 Thread Wang, Yang(Kevin)
[AMD Official Use Only - General] Reviewed-by: Yang Wang Best Regards, Kevin -Original Message- From: amd-gfx On Behalf Of Jesse Zhang Sent: Friday, May 10, 2024 10:50 AM To: amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander ; Koenig, Christian ; Huang, Tim ; Zhang, Jesse(Jie) ;

RE: [PATCH 09/22] drm/amd/pm: check specific index for smu13

2024-05-09 Thread Wang, Yang(Kevin)
[AMD Official Use Only - General] Reviewed-by: Yang Wang -Original Message- From: amd-gfx On Behalf Of Jesse Zhang Sent: Friday, May 10, 2024 10:50 AM To: amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander ; Koenig, Christian ; Huang, Tim ; Zhang, Jesse(Jie) ; Zhang, Jesse(Jie)

RE: [PATCH] drm/mst: Fix NULL pointer dereference at drm_dp_add_payload_part2

2024-05-09 Thread Lin, Wayne
[Public] > -Original Message- > From: Limonciello, Mario > Sent: Friday, May 10, 2024 3:18 AM > To: Linux regressions mailing list ; Wentland, > Harry > ; Lin, Wayne > Cc: ly...@redhat.com; imre.d...@intel.com; Leon Weiß bochum.de>; sta...@vger.kernel.org;

Re: [pull] amdgpu, amdkfd drm-fixes-6.9

2024-05-09 Thread Dave Airlie
On Thu, 9 May 2024 at 09:00, Alex Deucher wrote: > > Hi Dave, Sima, > > Fixes for 6.9. > > The following changes since commit dd5a440a31fae6e459c0d627162825505361: > > Linux 6.9-rc7 (2024-05-05 14:06:01 -0700) > > are available in the Git repository at: > >

[PATCH 22/22] drm/amdgpu: clear the warning unsigned compared against 0 for xcp_id

2024-05-09 Thread Jesse Zhang
This greater-than-or-equal-to-zero comparison of an unsigned value is always true. fpriv->xcp_id >= 0U Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c

[PATCH 21/22] drm/amd/pm: fix get dpm level count for yello carp

2024-05-09 Thread Jesse Zhang
For invalid clk types, return -EINVAL to check the return. Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/pm/swsmu/smu13/yellow_carp_ppt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/yellow_carp_ppt.c

[PATCH 20/22] drm/amd/pm: fix get dpm level count for smu13

2024-05-09 Thread Jesse Zhang
For invalid clk types, return -EINVAL to check the return. Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_5_ppt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_5_ppt.c

[PATCH 19/22] drm/amdgpu: Fix the warning division or modulo by zero for the variable num_xcc_per_xcp

2024-05-09 Thread Jesse Zhang
Dividing expression num_xcc_per_xcp which may be zero has undefined behavior. Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/amdgpu/aqua_vanjaram.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/aqua_vanjaram.c b/drivers/gpu/drm/amd/amdgpu/aqua_vanjaram.c

[PATCH 18/22] drm/amd/pm: check negtive return for table entries

2024-05-09 Thread Jesse Zhang
Function hwmgr->hwmgr_func->get_num_of_pp_table_entries(hwmgr) returns a negative number Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/pm/powerplay/hwmgr/pp_psm.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/pp_psm.c

[PATCH 17/22] drm/amdgpu: fix the warning bad bit shift operation for aca_error_type type

2024-05-09 Thread Jesse Zhang
Filter invalid aca error types before performing a shift operation. Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/amdgpu/amdgpu_aca.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_aca.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_aca.c index

[PATCH 16/22] drm/amd/pm: fix enum type compared against 0

2024-05-09 Thread Jesse Zhang
This less-than-zero comparison of an unsigned value is never true. type < 0U Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c b/drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c

[PATCH 12/22] drm/amd/pm: remove logically dead code

2024-05-09 Thread Jesse Zhang
Execution cannot reach this statement: case POWER_STATE_TYPE_BALAN. Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/pm/legacy-dpm/legacy_dpm.c | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/gpu/drm/amd/pm/legacy-dpm/legacy_dpm.c

[PATCH 15/22] drm/amd/pm: fix enum feature compared against 0

2024-05-09 Thread Jesse Zhang
This less-than-zero comparison of an unsigned value is never true. feature < 0U Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c

[PATCH 14/22] drm/amdgu: remove unused code

2024-05-09 Thread Jesse Zhang
The same code is executed when the condition err is true or false, because the code in the if-then branch and after the if statement is identical Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c | 2 -- 1 file changed, 2 deletions(-) diff --git

[PATCH 11/22] drm/amdgpu: remove structurally dead code for amd_gmc

2024-05-09 Thread Jesse Zhang
This code cannot be reached: return sysfs_emit(buf, "UNK) Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/amdgpu/amdgpu_gmc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gmc.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gmc.c index

[PATCH 13/22] drm/amd/pm: remove logically dead code for renoir

2024-05-09 Thread Jesse Zhang
The switch governing value clk_type cannot be SMU_GFXCLK and SMU_SCLK. Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/pm/swsmu/smu12/renoir_ppt.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu12/renoir_ppt.c

[PATCH 10/22] drm/amdgpu: remove structurally dead code

2024-05-09 Thread Jesse Zhang
This code cannot be reached: return "UNKNOWN";. Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h index 9a946f0e015c..109f471ff315 100644

[PATCH 08/22] drm/amd/pm: check the return of send smc msg for smu_v13

2024-05-09 Thread Jesse Zhang
Set smu work laod mask may fail, so check return. Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_7_ppt.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_7_ppt.c

[PATCH 09/22] drm/amd/pm: check specific index for smu13

2024-05-09 Thread Jesse Zhang
Check for specific indexes that may be invalid values. Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c

[PATCH 07/22] drm/amd/pm: check the return of send smc msg for navi10

2024-05-09 Thread Jesse Zhang
Set smu work laod mask may fail, so check return. Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c

[PATCH 06/22] drm/amd/pm: check the return of send smc msg for sienna_cichild

2024-05-09 Thread Jesse Zhang
Set smu work laod mask may fail, so check return. Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c

[PATCH 05/22] drm/amd/pm: check specific index for aldebaran

2024-05-09 Thread Jesse Zhang
Check for specific indexes that may be invalid values. Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c

[PATCH 03/22] drm/amdgpu: fix the waring dereferencing hive

2024-05-09 Thread Jesse Zhang
Check the amdgpu_hive_info *hive that maybe is NULL. Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c index 37820dd03cab..5a648a657dc6

[PATCH 04/22] drm/amd: fix the warning unchecking return vaule for sdma_v7

2024-05-09 Thread Jesse Zhang
check ring allocate success before emit preempt ib Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c b/drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c index

[PATCH 02/22] drm/amdgpu: the warning dereferencing obj for nbio_v7_4

2024-05-09 Thread Jesse Zhang
if ras_manager obj null, don't print NBIO err data Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c b/drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c index

[PATCH 01/22] drm/amdgpu: fix dereference after null check

2024-05-09 Thread Jesse Zhang
check the pointer hive before use. Signed-off-by: Jesse Zhang --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c index

Re: [PATCH 1/2] drm/amdgpu/pm: Check input value for power profile setting on smu13 and smu14

2024-05-09 Thread Ma, Jun
On 5/10/2024 9:41 AM, Wang, Yang(Kevin) wrote: > [AMD Official Use Only - General] > > Ok, I miss this patch #2. > > And please merge swsmu parts in patch#1 to patch#2. (patch #1 : powerplay, > patch #2: swsmu) Thanks, will fix this when push. Regards, Ma Jun > > Reviewed-by: Yang Wang >

RE: [PATCH 1/2] drm/amdgpu/pm: Check input value for power profile setting on smu13 and smu14

2024-05-09 Thread Wang, Yang(Kevin)
[AMD Official Use Only - General] Ok, I miss this patch #2. And please merge swsmu parts in patch#1 to patch#2. (patch #1 : powerplay, patch #2: swsmu) Reviewed-by: Yang Wang Best Regards, Kevin -Original Message- From: Ma, Jun Sent: Friday, May 10, 2024 9:00 AM To: Wang,

Re: [PATCH 1/2] drm/amdgpu/pm: Check input value for power profile setting on smu13 and smu14

2024-05-09 Thread Ma, Jun
On 5/9/2024 9:01 PM, Wang, Yang(Kevin) wrote: > [AMD Official Use Only - General] > > please fix similar issues in other xxx_ppt.c file together? e. g: > navi10_ppt.c, etc > Fix codes for navi10,vega20, etc. are in the patch 2 of this serial. Regards, Ma Jun > Best Regards, > Kevin >

RE: [PATCH v2 2/3] drm/amd/amdgpu: Add ISP driver support

2024-05-09 Thread Nirujogi, Pratap
[AMD Official Use Only - General] Hi Mario / Alex, Please see inline for below comment, others I addressed and submitted patch v3. RE: I think this is the wrong place for this isp_ip_block and also it should be named differently. Maybe it should be named isp_ip_block_v4_1 and then in a

Re: [PATCH] drm/amd/amdgpu: update jpeg 5 capability

2024-05-09 Thread Alex Deucher
On Thu, May 9, 2024 at 6:02 PM David (Ming Qiang) Wu wrote: > > Based on the documentation the maximum resolustion should > be 16384x16384. > > Signed-off-by: David (Ming Qiang) Wu Acked-by: Alex Deucher > --- > drivers/gpu/drm/amd/amdgpu/soc24.c | 2 +- > 1 file changed, 1 insertion(+), 1

Re: [PATCH] drm/amdgpu/jpeg: keep drm kernel message consistent

2024-05-09 Thread Alex Deucher
On Thu, May 9, 2024 at 6:03 PM David Wu wrote: > > Hi Alex, > > Well - the dev_info has 2 "amdgpu"s. > > [ 67.227121] amdgpu :43:00.0: amdgpu: JPEG decode is enabled in VM mode > > In practice we use "drm" to grep the GPU info for VCN and JPEG support. > So I believe the DRM_INFO is the

[PATCH] drm/amdkfd: Fix CU Masking for GFX 9.4.3

2024-05-09 Thread Mukul Joshi
We are incorrectly passing the first XCC's MQD when updating CU masks for other XCCs in the partition. Fix this by passing the MQD for the XCC currently being updated with CU mask to update_cu_mask function. Fixes: fc6efed2c728 ("drm/amdkfd: Update CU masking for GFX 9.4.3") Signed-off-by: Mukul

Re: [PATCH] drm/amdgpu/jpeg: keep drm kernel message consistent

2024-05-09 Thread David Wu
Hi Alex, Well - the dev_info has 2 "amdgpu"s. [   67.227121] amdgpu :43:00.0: amdgpu: JPEG decode is enabled in VM mode In practice we use "drm" to grep the GPU info for VCN and JPEG support. So I believe the DRM_INFO is the best and it is used by most of the amdgpu code with

[PATCH v3 2/3] drm/amd/amdgpu: Add ISP driver support

2024-05-09 Thread Pratap Nirujogi
Add the isp driver in amdgpu to support ISP device on the APUs that supports ISP IP block. ISP hw block is used for camera front-end, pre and post processing operations. Signed-off-by: Pratap Nirujogi --- Changes made in v3: - Remove unwanted header files - Remove suprious lines, duplicate

[PATCH v3 1/3] drm/amd/amdgpu: Add ISP support to amdgpu_discovery

2024-05-09 Thread Pratap Nirujogi
ISP hw block is supported in some of the AMD GPU versions, add support to discover ISP IP in amdgpu_discovery. Reviewed-by: Mario Limonciello Signed-off-by: Pratap Nirujogi --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 + drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c | 1 +

[PATCH v3 3/3] drm/amd/amdgpu: Enable ISP in amdgpu_discovery

2024-05-09 Thread Pratap Nirujogi
Enable ISP for ISP V4.1.0 and V4.1.1 in amdgpu_discovery. Reviewed-by: Mario Limonciello Signed-off-by: Pratap Nirujogi --- drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c | 22 +++ 1 file changed, 22 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c

[PATCH v3 0/3] Add ISP driver support in amdgpu

2024-05-09 Thread Pratap Nirujogi
Add Reviewed-by tag and address review comments on patch-2. Changes made: - Add Reviewed-by tag on patch-1 and 3. - Remove unwanted header files - Remove suprious lines, duplicate error prints - Fix multi-line comment style Pratap Nirujogi (3): drm/amd/amdgpu: Add ISP support to

[PATCH] drm/amd/amdgpu: update jpeg 5 capability

2024-05-09 Thread David (Ming Qiang) Wu
Based on the documentation the maximum resolustion should be 16384x16384. Signed-off-by: David (Ming Qiang) Wu --- drivers/gpu/drm/amd/amdgpu/soc24.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/soc24.c b/drivers/gpu/drm/amd/amdgpu/soc24.c

Re: [PATCH] drm/amdgpu/jpeg: keep drm kernel message consistent

2024-05-09 Thread Alex Deucher
On Thu, May 9, 2024 at 5:31 PM David Wu wrote: > > Hi Alex, > > Thanks for the suggestion! > What I am thinking is "DRM_DEV_INFO" should not be the one we want - as it is > more like a debug message. > > [drm:jpeg_v5_0_0_hw_init [amdgpu]] JPEG decode initialized successfully. > > instead I

Re: [PATCH] drm/amdgpu/jpeg: keep drm kernel message consistent

2024-05-09 Thread David Wu
Hi Alex, Thanks for the suggestion! What I am thinking is "DRM_DEV_INFO" should not be the one we want - as it is more like a debug message. [drm:jpeg_v5_0_0_hw_init [amdgpu]] JPEG decode initialized successfully. instead I prefer to use this format: "amdgpu :43:00.0: amdgpu: JPEG

Re: [PATCH] drm/amdgpu/jpeg: keep drm kernel message consistent

2024-05-09 Thread Alex Deucher
On Thu, May 9, 2024 at 4:57 PM David (Ming Qiang) Wu wrote: > > amdgpu jpeg kernel message is different than others such as vcn: > [drm:jpeg_v5_0_0_hw_init [amdgpu]] JPEG decode initialized successfully. > > This patch is to make them consistent. > > The message after the change is: > [drm]

[PATCH] drm/amdgpu/jpeg: keep drm kernel message consistent

2024-05-09 Thread David (Ming Qiang) Wu
amdgpu jpeg kernel message is different than others such as vcn: [drm:jpeg_v5_0_0_hw_init [amdgpu]] JPEG decode initialized successfully. This patch is to make them consistent. The message after the change is: [drm] JPEG decode initialized successfully. Signed-off-by: David (Ming Qiang) Wu

[PATCH v2] drm/amdgpu: Add Ring Hang Events

2024-05-09 Thread Ori Messinger
This patch adds 'ring hang' events to the driver. This is done by adding a 'reset_ring_hang' bool variable to the struct 'amdgpu_reset_context' in the amdgpu_reset.h file. The purpose for this 'reset_ring_hang' variable is whenever a GPU reset is initiated due to a ring hang, the reset_ring_hang

Re: [PATCH] drm/amdgpu/vcn: remove irq disabling in vcn 5 suspend

2024-05-09 Thread Alex Deucher
Acked-by: Alex Deucher On Thu, May 9, 2024 at 2:47 PM David (Ming Qiang) Wu wrote: > > We do not directly enable/disable VCN IRQ in vcn 5.0.0. > And we do not handle the IRQ state as well. So the calls to > disable IRQ and set state are removed. This effectively gets > rid of the warining of >

[PATCH] drm/amdkfd: Ensure gpu_id is unique

2024-05-09 Thread Harish Kasiviswanathan
gpu_id needs to be unique for user space to identify GPUs via KFD interface. In the current implementation there is a very small probability of having non unique gpu_ids. v2: Add check to confirm if gpu_id is unique. If not unique, find one Changed commit header to reflect the above v3: Use

[PATCH] drm/amdgpu/vcn: set VCN5 power gating state to GATE on suspend

2024-05-09 Thread David (Ming Qiang) Wu
On suspend, we need to set power gating state to GATE when VCN5 is busy, otherwise we will get following error on resume: [drm:amdgpu_ring_test_helper [amdgpu]] *ERROR* ring vcn_unified_0 test failed (-110) [drm:amdgpu_device_ip_resume_phase2 [amdgpu]] *ERROR* resume of IP block failed -110

Re: [PATCH v2 3/3] drm/amd/amdgpu: Enable ISP in amdgpu_discovery

2024-05-09 Thread Mario Limonciello
On 5/9/2024 14:35, Pratap Nirujogi wrote: Enable ISP for ISP V4.1.0 and V4.1.1 in amdgpu_discovery. Signed-off-by: Pratap Nirujogi Reviewed-by: Mario Limonciello --- drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c | 22 +++ 1 file changed, 22 insertions(+) diff --git

Re: [PATCH v2 1/3] drm/amd/amdgpu: Add ISP support to amdgpu_discovery

2024-05-09 Thread Mario Limonciello
On 5/9/2024 14:35, Pratap Nirujogi wrote: ISP hw block is supported in some of the AMD GPU versions, add support to discover ISP IP in amdgpu_discovery. Signed-off-by: Pratap Nirujogi Reviewed-by: Mario Limonciello --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 +

Re: [PATCH v2 2/3] drm/amd/amdgpu: Add ISP driver support

2024-05-09 Thread Mario Limonciello
On 5/9/2024 14:35, Pratap Nirujogi wrote: Add the isp driver in amdgpu to support ISP device on the APUs that supports ISP IP block. ISP hw block is used for camera front-end, pre and post processing operations. Signed-off-by: Pratap Nirujogi Change-Id:

RE: [PATCH 2/3] drm/amd/amdgpu: Add ISP driver support

2024-05-09 Thread Nirujogi, Pratap
[AMD Official Use Only - General] Thanks Mario for the inputs and clarifications on the offline chat. I submitted a fresh set of 3 patches addressing the comments, adding the cover-letter and brief change log in the cutlist in patch-2. Hope this is in the right format. Incase this needs to be

[PATCH v2 3/3] drm/amd/amdgpu: Enable ISP in amdgpu_discovery

2024-05-09 Thread Pratap Nirujogi
Enable ISP for ISP V4.1.0 and V4.1.1 in amdgpu_discovery. Signed-off-by: Pratap Nirujogi --- drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c | 22 +++ 1 file changed, 22 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c

[PATCH v2 1/3] drm/amd/amdgpu: Add ISP support to amdgpu_discovery

2024-05-09 Thread Pratap Nirujogi
ISP hw block is supported in some of the AMD GPU versions, add support to discover ISP IP in amdgpu_discovery. Signed-off-by: Pratap Nirujogi --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 + drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c | 1 + drivers/gpu/drm/amd/include/amd_shared.h

[PATCH v2 0/3] Add ISP driver support in amdgpu

2024-05-09 Thread Pratap Nirujogi
Add the isp driver in amdgpu to support ISP device on the APUs that supports ISP IP block. ISP hw block is used for camera front-end, pre and post processing operations. Pratap Nirujogi (3): drm/amd/amdgpu: Add ISP support to amdgpu_discovery drm/amd/amdgpu: Add ISP driver support

[PATCH v2 2/3] drm/amd/amdgpu: Add ISP driver support

2024-05-09 Thread Pratap Nirujogi
Add the isp driver in amdgpu to support ISP device on the APUs that supports ISP IP block. ISP hw block is used for camera front-end, pre and post processing operations. Signed-off-by: Pratap Nirujogi Change-Id: I67ef206e5eca1fe74e495c3262746be495e17d09 --- Changes in v2: - Remove adding

Re: [PATCH] drm/mst: Fix NULL pointer dereference at drm_dp_add_payload_part2

2024-05-09 Thread Mario Limonciello
On 5/9/2024 07:43, Linux regression tracking (Thorsten Leemhuis) wrote: On 18.04.24 21:43, Harry Wentland wrote: On 2024-03-07 01:29, Wayne Lin wrote: [Why] Commit: - commit 5aa1dfcdf0a4 ("drm/mst: Refactor the flow for payload allocation/removement") accidently overwrite the commit - commit

[PATCH] drm/amdgpu/vcn: remove irq disabling in vcn 5 suspend

2024-05-09 Thread David (Ming Qiang) Wu
We do not directly enable/disable VCN IRQ in vcn 5.0.0. And we do not handle the IRQ state as well. So the calls to disable IRQ and set state are removed. This effectively gets rid of the warining of "WARN_ON(!amdgpu_irq_enabled(adev, src, type))" in amdgpu_irq_put(). Signed-off-by: David

Re: [PATCH] drm/radeon: Delay Connector detecting when HPD singals is unstable

2024-05-09 Thread Alex Deucher
On Thu, May 9, 2024 at 4:58 AM oushixiong wrote: > > From: Shixiong Ou > > In some causes, HPD signals will jitter when plugging in > or unplugging HDMI. > > Rescheduling the hotplug work for a second when EDID may still be > readable but HDP is disconnected, and fixes this issue. > >

RE: [PATCH 2/3] drm/amd/amdgpu: Add ISP driver support

2024-05-09 Thread Nirujogi, Pratap
[AMD Official Use Only - General] Hi Alex, Please see inline comments. RE: You should replace 162 with whatever the src id you need for your IP is. 162 was the src id for ACP. I think it's something else for ISP. Also, if you need both the client id and the src id, you should update

RE: [PATCH 2/3] drm/amd/amdgpu: Add ISP driver support

2024-05-09 Thread Limonciello, Mario
[AMD Official Use Only - General] Thanks, I'll take a look. In general make sure that you prefix new versions with "PATCH v2", "PATCH v3" etc and include a changelog below the cutlist or in the cover letter. There is a --subject-prefix argument for git send-email you can use. It's okay this

RE: [PATCH 2/3] drm/amd/amdgpu: Add ISP driver support

2024-05-09 Thread Nirujogi, Pratap
[AMD Official Use Only - General] Hi Mario, I addressed the comments, submitted the updated patchset, please review and let us know your comments. Thanks, Pratap -Original Message- From: Limonciello, Mario Sent: Thursday, May 9, 2024 10:15 AM To: Nirujogi, Pratap ;

[PATCH 2/3] drm/amd/amdgpu: Add ISP driver support

2024-05-09 Thread Pratap Nirujogi
Add the isp driver in amdgpu to support ISP device on the APUs that supports ISP IP block. ISP hw block is used for camera front-end, pre and post processing operations. Signed-off-by: Pratap Nirujogi Change-Id: I67ef206e5eca1fe74e495c3262746be495e17d09 --- drivers/gpu/drm/amd/amdgpu/Makefile

Re: [PATCH] drm/amd/pm: remove deprecated I2C_CLASS_SPD support from newly added SMU_14_0_2

2024-05-09 Thread Alex Deucher
On Thu, May 9, 2024 at 8:02 AM Heiner Kallweit wrote: > > Support for I2C_CLASS_SPD is currently being removed from the kernel. > Only remaining step is to remove the definition of I2C_CLASS_SPD. > Setting I2C_CLASS_SPD in a driver is a no-op meanwhile, so remove it > here. > > Signed-off-by:

[PATCH] drm/amd/display: Don't register panel_power_savings on OLED panels

2024-05-09 Thread Mario Limonciello
OLED panels don't support the ABM, they shouldn't offer the panel_power_savings attribute to the user. Check whether aux BL control support was enabled to decide whether to offer it. Reported-by: Gergo Koteles Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/3359 Signed-off-by: Mario

Re: [PATCH 2/3] drm/amd/amdgpu: Add ISP driver support

2024-05-09 Thread Alex Deucher
On Wed, May 8, 2024 at 11:17 AM Pratap Nirujogi wrote: > > Add the isp driver in amdgpu to support ISP device on the APUs that > supports ISP IP block. ISP hw block is used for camera front-end, pre > and post processing operations. > > Signed-off-by: Pratap Nirujogi > --- >

Re: [PATCH] drm/amdgpu: enable unmapped doorbell handling basic mode on mes 12

2024-05-09 Thread Alex Deucher
On Wed, May 8, 2024 at 9:52 PM shaoyunl wrote: > > This reverts commit 9606c08e178f953d22e50b05c64b4b1a48051f3e. > > Signed-off-by: shaoyunl Acked-by: Alex Deucher > --- > drivers/gpu/drm/amd/amdgpu/mes_v12_0.c| 14 ++ > drivers/gpu/drm/amd/include/mes_v12_api_def.h | 3

Re: [PATCH 2/3] drm/amd/amdgpu: Add ISP driver support

2024-05-09 Thread Mario Limonciello
On 5/8/2024 09:50, Pratap Nirujogi wrote: Add the isp driver in amdgpu to support ISP device on the APUs that supports ISP IP block. ISP hw block is used for camera front-end, pre and post processing operations. Signed-off-by: Pratap Nirujogi --- drivers/gpu/drm/amd/amdgpu/Makefile |

Re: [PATCH] drm/mst: Fix NULL pointer dereference at drm_dp_add_payload_part2

2024-05-09 Thread Linux regression tracking (Thorsten Leemhuis)
On 18.04.24 21:43, Harry Wentland wrote: > On 2024-03-07 01:29, Wayne Lin wrote: >> [Why] >> Commit: >> - commit 5aa1dfcdf0a4 ("drm/mst: Refactor the flow for payload >> allocation/removement") >> accidently overwrite the commit >> - commit 54d217406afe ("drm: use mgr->dev in drm_dbg_kms in >>

Re: [RFC 1/5] drm/amdgpu: Fix migration rate limiting accounting

2024-05-09 Thread Tvrtko Ursulin
On 08/05/2024 20:08, Friedrich Vock wrote: On 08.05.24 20:09, Tvrtko Ursulin wrote: From: Tvrtko Ursulin The logic assumed any migration attempt worked and therefore would over- account the amount of data migrated during buffer re-validation. As a consequence client can be unfairly

[PATCH] drm/amd/pm: remove deprecated I2C_CLASS_SPD support from newly added SMU_14_0_2

2024-05-09 Thread Heiner Kallweit
Support for I2C_CLASS_SPD is currently being removed from the kernel. Only remaining step is to remove the definition of I2C_CLASS_SPD. Setting I2C_CLASS_SPD in a driver is a no-op meanwhile, so remove it here. Signed-off-by: Heiner Kallweit ---

RE: [PATCH 1/2] drm/amdgpu/pm: Check input value for power profile setting on smu13 and smu14

2024-05-09 Thread Wang, Yang(Kevin)
[AMD Official Use Only - General] please fix similar issues in other xxx_ppt.c file together? e. g: navi10_ppt.c, etc Best Regards, Kevin -Original Message- From: Ma, Jun Sent: Thursday, May 9, 2024 5:37 PM To: amd-gfx@lists.freedesktop.org Cc: Feng, Kenneth ; Deucher, Alexander ;

RE: [PATCH] drm/amdgpu/pm: Fix code alignment issue

2024-05-09 Thread Wang, Yang(Kevin)
[AMD Official Use Only - General] Reviewed-by: Yang Wang Best Regards, Kevin -Original Message- From: Ma, Jun Sent: Thursday, May 9, 2024 5:37 PM To: amd-gfx@lists.freedesktop.org Cc: Feng, Kenneth ; Deucher, Alexander ; Wang, Yang(Kevin) ; Koenig, Christian ; Ma, Jun Subject:

RE: [PATCH] drm/amd/pm: Fix aldebaran pcie speed reporting

2024-05-09 Thread Wang, Yang(Kevin)
[AMD Official Use Only - General] Reviewed-by: Yang Wang Best Regards, Kevin -Original Message- From: Lazar, Lijo Sent: Thursday, May 9, 2024 4:59 PM To: amd-gfx@lists.freedesktop.org Cc: Zhang, Hawking ; Deucher, Alexander ; Wang, Yang(Kevin) Subject: [PATCH] drm/amd/pm: Fix

RE: [PATCH] drm/amd/pm: Fix aldebaran pcie speed reporting

2024-05-09 Thread Kamal, Asad
[AMD Official Use Only - General] Reviewed-by: Asad Kamal Thanks & Regards Asad -Original Message- From: amd-gfx On Behalf Of Lijo Lazar Sent: Thursday, May 9, 2024 2:29 PM To: amd-gfx@lists.freedesktop.org Cc: Zhang, Hawking ; Deucher, Alexander ; Wang, Yang(Kevin) Subject: [PATCH]

Re: [RFC 0/5] Discussion around eviction improvements

2024-05-09 Thread Tvrtko Ursulin
On 08/05/2024 19:09, Tvrtko Ursulin wrote: From: Tvrtko Ursulin Last few days I was looking at the situation with VRAM over subscription, what happens versus what perhaps should happen. Browsing through the driver and running some simple experiments. I ended up with this patch series which,

[PATCH v1 3/4] drm/amdgpu: add support to dump gfx10 queue registers

2024-05-09 Thread Sunil Khatri
Add gfx queue register for all instances in ip dump for gfx10. Signed-off-by: Sunil Khatri --- drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h | 1 + drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 83 + 2 files changed, 84 insertions(+) diff --git

[PATCH v1 4/4] drm/amdgpu: add prints while ip registr dump

2024-05-09 Thread Sunil Khatri
add prints before and after during ip registers dump. It avoids user to think of system being stuck/hung as register dump takes time after a gpu hang. Signed-off-by: Sunil Khatri --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCH v1 2/4] drm/amdgpu: Add support to dump gfx10 cp registers

2024-05-09 Thread Sunil Khatri
add support to dump registers of all instances of cp registers in gfx10 Signed-off-by: Sunil Khatri --- drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h | 1 + drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 117 +++- 2 files changed, 114 insertions(+), 4 deletions(-) diff --git

[PATCH v1 1/4] drm/amdgpu: update the ip_dump to ipdump_core

2024-05-09 Thread Sunil Khatri
Update the memory pointer from ip_dump to ipdump_core to make it specific to core registers and rest other registers to be dumped in their respective memories. Signed-off-by: Sunil Khatri --- drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h | 2 +- drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 14

[PATCH v1 0/4] Add CP and GFX Queue register in ip Dump for

2024-05-09 Thread Sunil Khatri
*** BLURB HERE *** Sunil Khatri (4): drm/amdgpu: update the ip_dump to ipdump_core drm/amdgpu: Add support to dump gfx10 cp registers drm/amdgpu: add support to dump gfx10 queue registers drm/amdgpu: add prints while ip registr dump drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 +

[PATCH 2/2] drm/amdgpu/pm: Check input value for CUSTOM profile mode setting on legacy SOCs

2024-05-09 Thread Ma Jun
Check the input value for CUSTOM profile mode setting on legacy SOCs. Otherwise it may cause out-of-bouds read error. Signed-off-by: Ma Jun --- drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu7_hwmgr.c | 2 +- drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega20_hwmgr.c | 8 ++--

[PATCH 1/2] drm/amdgpu/pm: Check input value for power profile setting on smu13 and smu14

2024-05-09 Thread Ma Jun
Check the input value for CUSTOM profile mode setting on smu13 and smu14. Otherwise it may cause out-of-bouds read error. Signed-off-by: Ma Jun --- drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c | 5 + drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_7_ppt.c | 4

[PATCH] drm/amdgpu/pm: Fix code alignment issue

2024-05-09 Thread Ma Jun
Fix code alignment issue Signed-off-by: Ma Jun Reported-by: Yang Wang --- drivers/gpu/drm/amd/pm/amdgpu_pm.c | 4 ++-- drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git

Re: [PATCH 1/3] drm/amdgpu: Add amdgpu_bo_is_vm_bo helper

2024-05-09 Thread Tvrtko Ursulin
On 08/05/2024 06:42, Christian König wrote: Am 06.05.24 um 18:26 schrieb Tvrtko Ursulin: On 03/05/2024 10:14, Tvrtko Ursulin wrote: From: Tvrtko Ursulin Help code readability by replacing a bunch of: bo->tbo.base.resv == vm->root.bo->tbo.base.resv With: amdgpu_vm_is_bo_always_valid(vm,

[PATCH v3 4/6] sfc: falcon: Make I2C terminology more inclusive

2024-05-09 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C 1.1.1 specifications have replaced "master/slave" with more appropriate terms. Inspired by Wolfram's series to fix drivers/i2c/, fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the specification. Compile

[PATCH v3 6/6] fbdev/viafb: Make I2C terminology more inclusive

2024-05-09 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C 1.1.1 specifications have replaced "master/slave" with more appropriate terms. Inspired by Wolfram's series to fix drivers/i2c/, fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the specification. Compile

2024 X.Org Developers Conference - October 9-11, Montréal, Canada

2024-05-09 Thread Mark Filion
Hello! We're delighted to announce that the 2024 X.Org Developers Conference (XDC) will be taking place on October 9 to 11 in Montréal, Canada, co- located with the GStreamer Conference & Hackfest 2024 which will be running from October 7 to 10. Join us for a freedesktop week in Montréal! XDC is

Re: [PATCH v8 00/35] fix CONFIG_DRM_USE_DYNAMIC_DEBUG=y regression

2024-05-09 Thread jim . cromie
On Mon, Apr 29, 2024 at 1:32 PM Jim Cromie wrote: > > hi Greg, Jason, DRM-folk, > > This patchset fixes the CONFIG_DRM_USE_DYNAMIC_DEBUG=y regression, > Fixes: bb2ff6c27bc9 ("drm: Disable dynamic debug as broken") > > this is v8. > Its also here: >

[PATCH v3 5/6] fbdev/smscufx: Make I2C terminology more inclusive

2024-05-09 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C 1.1.1 specifications have replaced "master/slave" with more appropriate terms. Inspired by Wolfram's series to fix drivers/i2c/, fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the specification. Compile

[PATCH v3 0/6] Make I2C terminology more inclusive for I2C Algobit and consumers

2024-05-09 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C 1.1.1 specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c/[1], fix the terminology for users of the I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the

[PATCH v3 2/6] drm/gma500: Make I2C terminology more inclusive

2024-05-09 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C 1.1.1 specifications have replaced "master/slave" with more appropriate terms. Inspired by Wolfram's series to fix drivers/i2c/, fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the specification. Compile

[PATCH v3 3/6] drm/i915: Make I2C terminology more inclusive

2024-05-09 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C 1.1.1 specifications have replaced "master/slave" with more appropriate terms. Inspired by Wolfram's series to fix drivers/i2c/, fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the specification. Compile

Re: [patch] problems with "fix visible VRAM handling during faults"

2024-05-09 Thread Jeremy Day
Christian König wrote: > Am 08.05.24 um 12:17 schrieb Michel Dänzer: > > Does this instead of your patch help by any chance? > > > > > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c > > b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c > > index 109fe557a02b..29c197c00018 100644 > > ---

  1   2   >