[PATCH V2 3/3] drm/amd/pm: disable cstate feature during smu cleanup

2022-10-11 Thread Evan Quan
This can address some reset failures observed on SMU13.0.0. Signed-off-by: Evan Quan Reviewed-by: Hawking Zhang Change-Id: Ieb4e204c49abd405b1dce559c2ff75bb3887b6f9 -- v1->v2: - revise the code comments(Alex) - limit this to SMU13.0.0 and 13.0.7 --- drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c |

[PATCH V2 2/3] drm/amd/pm: fulfill SMU13.0.7 cstate control interface

2022-10-11 Thread Evan Quan
Fulfill the functionality for cstate control. Signed-off-by: Evan Quan Reviewed-by: Hawking Zhang Change-Id: I3e7ec7d0434d3b34855a9267ab1989a3b85d5206 --- drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_7_ppt.c | 12 1 file changed, 12 insertions(+) diff --git a/drivers/gpu/drm/amd/p

[PATCH V2 1/3] drm/amd/pm: fulfill SMU13.0.0 cstate control interface

2022-10-11 Thread Evan Quan
Fulfill the functionality for cstate control. Signed-off-by: Evan Quan Reviewed-by: Hawking Zhang Change-Id: I2b7253898503b8051bbf90f429b46221410146b0 --- drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/drivers/gpu/drm/amd/pm

Re: [PATCH 1/4] drm/amdgpu: extend HWIP_MAX_INSTANCE to 28

2022-10-11 Thread Christian König
Am 11.10.22 um 22:56 schrieb Alex Deucher: From: Hawking Zhang more ip instances are available Signed-off-by: Hawking Zhang Signed-off-by: Alex Deucher Acked-by: Christian König for the entire series. --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 2 +- 1 file changed, 1 insertion(+), 1 d

RE: [PATCH 1/2] drm/amdgpu: add tmz support for GC 11.0.1

2022-10-11 Thread Huang, Tim
[AMD Official Use Only - General] Series is Reviewed-by: Tim Huang Best Regards, Tim Huang -Original Message- From: amd-gfx On Behalf Of Yifan Zhang Sent: Sunday, October 9, 2022 2:40 PM To: amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander ; Huang, Tim ; Du, Xiaojian ; Zhang, Y

RE: [PATCH] drm/amdkfd: correct the cache info for gfx1036

2022-10-11 Thread Zhang, Yifan
[Public] This patch is Reviewed-by: Yifan Zhang From: Zhang, Jesse(Jie) Sent: Tuesday, October 11, 2022 1:23 PM To: amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander ; Zhang, Yifan ; Kuehling, Felix Subject: RE: [PATCH] drm/amdkfd: correct the cache info for gfx1036 [AMD Official Use On

RE: [PATCH] drm/amdgpu: Fix SDMA engine resume issue under SRIOV

2022-10-11 Thread Liu, Monk
[AMD Official Use Only - General] Hi Bokun Can you elaborate more on this reenabling SDMA engine during suspend ? Why VF need the SDMA engine alive there ? > - > + /* > +* Under SRIOV, the VF cannot single-mindedly stop SDMA engine > +* However, we still need to clean up th

RE: [PATCH 3/4] drm/amdgpu: convert vega20_ih.c to IP version checks

2022-10-11 Thread Zhang, Hawking
Reviewed-by: Hawking Zhang Regards, Hawking -Original Message- From: amd-gfx On Behalf Of Alex Deucher Sent: Wednesday, October 12, 2022 04:56 To: amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander Subject: [PATCH 3/4] drm/amdgpu: convert vega20_ih.c to IP version checks For consiste

RE: [PATCH 4/4] drm/amdgpu: convert amdgpu_amdkfd_gpuvm.c to IP version checks

2022-10-11 Thread Zhang, Hawking
Reviewed-by: Hawking Zhang Regards, Hawking -Original Message- From: amd-gfx On Behalf Of Alex Deucher Sent: Wednesday, October 12, 2022 04:56 To: amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander Subject: [PATCH 4/4] drm/amdgpu: convert amdgpu_amdkfd_gpuvm.c to IP version checks F

Re: [PATCH 2/3] drm/amd/display: Enable new commit sequence only for DCN32x

2022-10-11 Thread Harry Wentland
On 2022-10-11 14:11, Rodrigo Siqueira wrote: > Change commit sequence might impact multiple ASICs; for this reason, it > is better to run it only on DCN32x for the first set of validation. > After we get more confident about this change, we should expand it to > all ASICs. > > Cc: Nicholas Kazlaus

Re: [PATCH 3/3] drm/amd/display: Move dc_state copy in commit_tail after dc_commit_state

2022-10-11 Thread Harry Wentland
On 2022-10-11 14:11, Rodrigo Siqueira wrote: > From: Aurabindo Pillai > > [Why&How] > With the new commit sequence, we do not want the state to be copied before > the call to dc_commit_state() since this leaks the phantom streams into > new state. > What are phantom streams? Why are they nee

[PATCH] drm/amdgpu: switch to select_se_sh wrapper for gfx v9_0

2022-10-11 Thread Alex Deucher
From: Hawking Zhang To allow invoking ip specific callbacks Signed-off-by: Hawking Zhang Reviewed-by: Le Ma Signed-off-by: Alex Deucher --- .../gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v9.c | 4 +-- drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 28 +-- 2 files changed, 16 inse

[PATCH 4/4] drm/amdgpu: convert amdgpu_amdkfd_gpuvm.c to IP version checks

2022-10-11 Thread Alex Deucher
For consistency with the rest of the code. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpu

[PATCH 3/4] drm/amdgpu: convert vega20_ih.c to IP version checks

2022-10-11 Thread Alex Deucher
For consistency with newer asics. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/vega20_ih.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/vega20_ih.c b/drivers/gpu/drm/amd/amdgpu/vega20_ih.c index 59dfca093155..1706081d054d 100

[PATCH 2/4] drm/amdgpu: update psp_fw_type enum in amdgpu_ucode header

2022-10-11 Thread Alex Deucher
From: Hawking Zhang To match with the definition in psp firmware Signed-off-by: Hawking Zhang Reviewed-by: Le Ma Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h b/drivers/

[PATCH 1/4] drm/amdgpu: extend HWIP_MAX_INSTANCE to 28

2022-10-11 Thread Alex Deucher
From: Hawking Zhang more ip instances are available Signed-off-by: Hawking Zhang Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h i

RE: [PATCH 0/3] Update DC commit sequence

2022-10-11 Thread Wheeler, Daniel
[Public] Hi all, I tested these patches as well, they were added to the most recent patchset. Tested-by: Daniel Wheeler Thank you, Dan Wheeler Sr. Technologist  |  AMD SW Display -

Re: [PATCH 1/3] drm/amd/display: Add functions for new DC commit sequence

2022-10-11 Thread Harry Wentland
On 2022-10-11 14:11, Rodrigo Siqueira wrote: > Since DCN32, our DC commit become more complex and requires more > adjustments to work well with some features. This is a preparation work > for replacing dc_commit_state and dc_commit_updates_for_stream with > dc_update_planes_and_stream and dc_upda

[PATCH 3/3] drm/amd/display: Move dc_state copy in commit_tail after dc_commit_state

2022-10-11 Thread Rodrigo Siqueira
From: Aurabindo Pillai [Why&How] With the new commit sequence, we do not want the state to be copied before the call to dc_commit_state() since this leaks the phantom streams into new state. Fix this by doing the dc state copy right after the dc_commit_state() call. Cc: Nicholas Kazlauskas Cc:

[PATCH 1/3] drm/amd/display: Add functions for new DC commit sequence

2022-10-11 Thread Rodrigo Siqueira
Since DCN32, our DC commit become more complex and requires more adjustments to work well with some features. This is a preparation work for replacing dc_commit_state and dc_commit_updates_for_stream with dc_update_planes_and_stream and dc_update_planes_and_stream respectively. Cc: Nicholas Kazlau

[PATCH 2/3] drm/amd/display: Enable new commit sequence only for DCN32x

2022-10-11 Thread Rodrigo Siqueira
Change commit sequence might impact multiple ASICs; for this reason, it is better to run it only on DCN32x for the first set of validation. After we get more confident about this change, we should expand it to all ASICs. Cc: Nicholas Kazlauskas Cc: Harry Wentland Signed-off-by: Rodrigo Siqueira

[PATCH 0/3] Update DC commit sequence

2022-10-11 Thread Rodrigo Siqueira
This patchset introduces the basic infrastructure to change our display commit sequence to make it more robust and flexible for handling some display features. Current, amdgpu_dm tries to commit a validation state directly into update_planes and commit_streams. However, we want to avoid committing

Re: [PATCH] drm/amd/display: Add HUBP surface flip interrupt handler

2022-10-11 Thread Alex Deucher
On Tue, Oct 11, 2022 at 1:32 PM Rodrigo Siqueira wrote: > > From: Aurabindo Pillai > > Add the hubp surface flip handler. This fixes some flip timeout issues. > > Reviewed-by: Rodrigo Siqueira > Signed-off-by: Aurabindo Pillai Acked-by: Alex Deucher > --- > drivers/gpu/drm/amd/display/dc/dc

Re: [PATCH v6 00/21] Move all drivers to a common dma-buf locking convention

2022-10-11 Thread Dmitry Osipenko
On 9/28/22 22:15, Dmitry Osipenko wrote: > Hello, > > This series moves all drivers to a dynamic dma-buf locking specification. > From now on all dma-buf importers are made responsible for holding > dma-buf's reservation lock around all operations performed over dma-bufs > in accordance to the loc

[PATCH] drm/amd/display: Add HUBP surface flip interrupt handler

2022-10-11 Thread Rodrigo Siqueira
From: Aurabindo Pillai Add the hubp surface flip handler. This fixes some flip timeout issues. Reviewed-by: Rodrigo Siqueira Signed-off-by: Aurabindo Pillai --- drivers/gpu/drm/amd/display/dc/dcn32/dcn32_hubp.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/amd/display/dc

RE: [PATCH 00/23] DC Patches October 17th, 2022

2022-10-11 Thread Wheeler, Daniel
[Public] Hi all, This week this patchset was tested on the following systems: HP Envy 360, with Ryzen 5 4500U Lenovo Thinkpad T14s Gen2, with AMD Ryzen 5 5650U Sapphire Pulse RX5700XT Reference AMD RX6800 Engineering board with Ryzen 9 5900H These systems were tested on the following displ

Re: [PATCH -next] drm/amd/display: Simplify bool conversion

2022-10-11 Thread Alex Deucher
Applied. Thanks! Alex On Tue, Oct 11, 2022 at 11:27 AM Harry Wentland wrote: > > On 2022-10-10 03:38, Yang Li wrote: > > The result of 'pwr_status == 0' is Boolean, and the question mark > > expression is redundant. > > > > Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=2354 > > Reported-

Re: [PATCH -next] drm/amd/display: Simplify bool conversion

2022-10-11 Thread Harry Wentland
On 2022-10-10 03:38, Yang Li wrote: > The result of 'pwr_status == 0' is Boolean, and the question mark > expression is redundant. > > Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=2354 > Reported-by: Abaci Robot > Signed-off-by: Yang Li Reviewed-by: Harry Wentland Harry > --- > driv

Re: [PATCH] drm/amd/display: make virtual_disable_link_output static

2022-10-11 Thread Harry Wentland
On 2022-10-10 10:41, Alex Deucher wrote: > It's not used outside of virtual_link_hwss.c. Fixes > a -Wmissing-prototypes warning. > > Signed-off-by: Alex Deucher Reviewed-by: Harry Wentland Harry > --- > drivers/gpu/drm/amd/display/dc/virtual/virtual_link_hwss.c | 2 +- > 1 file changed, 1 i

Re: [PATCH] drm/amd/display: fix transfer function passed to build_coefficients()

2022-10-11 Thread Harry Wentland
On 2022-09-19 12:43, Alex Deucher wrote: > The default argument should be enum TRANSFER_FUNCTION_SRGB rather than > the current boolean value which improperly maps to > TRANSFER_FUNCTION_BT709. > > Commit 9b3d76527f6e ("drm/amd/display: Revert adding degamma coefficients") > looks to have improper

[linux-next:master] BUILD REGRESSION b9f85101cad3397ef1e509909602a90e257ab9d8

2022-10-11 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master branch HEAD: b9f85101cad3397ef1e509909602a90e257ab9d8 Add linux-next specific files for 20221011 Error/Warning reports: https://lore.kernel.org/linux-doc/202209201326.sy9kholm-...@intel.com https

Re: [PATCH] drm/amd/display: add a license to cursor_reg_cache.h

2022-10-11 Thread Harry Wentland
On 2022-10-10 17:36, Alex Deucher wrote: > It's MIT. > > Fixes: b73353f7f3d434 ("drm/amd/display: Use the same cursor info across > features") > Signed-off-by: Alex Deucher Reviewed-by: Harry Wentland Harry > --- > drivers/gpu/drm/amd/display/dc/inc/hw/cursor_reg_cache.h | 1 + > 1 file cha

Re: [PATCH] drm/amd/display: fix indentation in dc.c

2022-10-11 Thread Harry Wentland
On 2022-10-06 11:32, Alex Deucher wrote: > Fixes a warning in dc.c. > > Signed-off-by: Alex Deucher Reviewed-by: Harry Wentland Harry > --- > drivers/gpu/drm/amd/display/dc/core/dc.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/amd/display/d

Re: [PATCH] drm/amd/display: fix indentation in dc.c

2022-10-11 Thread Alex Deucher
Ping? On Thu, Oct 6, 2022 at 11:32 AM Alex Deucher wrote: > > Fixes a warning in dc.c. > > Signed-off-by: Alex Deucher > --- > drivers/gpu/drm/amd/display/dc/core/dc.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c > b/dr

Re: [PATCH] drm/amd/display: make dcn32_split_stream_for_mpc_or_odm static

2022-10-11 Thread Harry Wentland
On 2022-10-11 09:41, Alex Deucher wrote: > It's not used outside of dcn32_fpu.c. > > Reported-by: kernel test robot > Fixes: 20dad3813b3c15 ("drm/amd/display: Add a helper to map > ODM/MPC/Multi-Plane resources") > Signed-off-by: Alex Deucher Reviewed-by: Harry Wentland Harry > --- > drive

Re: [PATCH 03/15] drm/amd/display: stop using connector->override_edid

2022-10-11 Thread Alex Deucher
On Tue, Oct 11, 2022 at 10:12 AM Jani Nikula wrote: > > On Tue, 11 Oct 2022, Harry Wentland wrote: > > On 2022-10-11 09:49, Jani Nikula wrote: > >> The connector->override_edid flag is strictly for EDID override debugfs > >> management, and drivers have no business using it. > >> > >> Cc: Alex De

Re: [PATCH 03/15] drm/amd/display: stop using connector->override_edid

2022-10-11 Thread Jani Nikula
On Tue, 11 Oct 2022, Harry Wentland wrote: > On 2022-10-11 09:49, Jani Nikula wrote: >> The connector->override_edid flag is strictly for EDID override debugfs >> management, and drivers have no business using it. >> >> Cc: Alex Deucher >> Cc: Christian König >> Cc: Xinhui Pan >> Cc: amd-gfx@l

Re: [PATCH 03/15] drm/amd/display: stop using connector->override_edid

2022-10-11 Thread Harry Wentland
On 2022-10-11 09:49, Jani Nikula wrote: > The connector->override_edid flag is strictly for EDID override debugfs > management, and drivers have no business using it. > > Cc: Alex Deucher > Cc: Christian König > Cc: Xinhui Pan > Cc: amd-gfx@lists.freedesktop.org > Signed-off-by: Jani Nikula

[PATCH 03/15] drm/amd/display: stop using connector->override_edid

2022-10-11 Thread Jani Nikula
The connector->override_edid flag is strictly for EDID override debugfs management, and drivers have no business using it. Cc: Alex Deucher Cc: Christian König Cc: Xinhui Pan Cc: amd-gfx@lists.freedesktop.org Signed-off-by: Jani Nikula --- I really have no idea what the functional impact her

Re: [PATCH -next] drm/amd/display: fix build error on arm64

2022-10-11 Thread Alex Deucher
Applied. Thanks. Alex On Tue, Oct 11, 2022 at 8:41 AM Yang Yingliang wrote: > > dcn20_build_mapped_resource() and dcn20_acquire_dsc() is not defined, > if CONFIG_DRM_AMD_DC_DCN is disabled. > > Fix the following build error on arm64: > > ERROR: modpost: "dcn20_build_mapped_resource" > [drive

[PATCH] drm/amd/display: make dcn32_split_stream_for_mpc_or_odm static

2022-10-11 Thread Alex Deucher
It's not used outside of dcn32_fpu.c. Reported-by: kernel test robot Fixes: 20dad3813b3c15 ("drm/amd/display: Add a helper to map ODM/MPC/Multi-Plane resources") Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/display/dc/dml/dcn32/dcn32_fpu.c | 2 +- 1 file changed, 1 insertion(+), 1 delet

Re: [PATCH] drm/amdkfd: correct the cache info for gfx1036

2022-10-11 Thread Deucher, Alexander
[Public] Acked-by: Alex Deucher From: Zhang, Jesse(Jie) Sent: Tuesday, October 11, 2022 1:23 AM To: amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander ; Zhang, Yifan ; Kuehling, Felix Subject: RE: [PATCH] drm/amdkfd: correct the cache info for gfx1036 [AMD

[PATCH -next] drm/amd/display: fix build error on arm64

2022-10-11 Thread Yang Yingliang
dcn20_build_mapped_resource() and dcn20_acquire_dsc() is not defined, if CONFIG_DRM_AMD_DC_DCN is disabled. Fix the following build error on arm64: ERROR: modpost: "dcn20_build_mapped_resource" [drivers/gpu/drm/amd/amdgpu/amdgpu.ko] undefined! ERROR: modpost: "dcn20_acquire_dsc" [drivers/gpu

Re: [PATCH] drm/amd/display: fix array-bounds error in dc_stream_remove_writeback()

2022-10-11 Thread Thorsten Leemhuis
[removed a lot of people from the list of recipients, as this is mainly for Guenter] Hi Guenter! On 06.10.22 19:23, Guenter Roeck wrote: > On Wed, Oct 05, 2022 at 11:46:15PM -0700, Guenter Roeck wrote: >> On Tue, Sep 27, 2022 at 03:12:00PM -0400, Hamza Mahfooz wrote: >>> Address the following err

Re: [regression][6.1] After commit e4dc45b1848bc6bcac31eb1b4ccdd7f6718b3c86 system randomly hungs

2022-10-11 Thread Christian König
Yeah, that's a known issue. Dave already reverted the problematic patch in drm-next. But it will probably take a while until the revert propagates down into the drm-misc-next or whatever you uses. Regards, Christian. Am 11.10.22 um 14:02 schrieb Mikhail Gavrilov: Hi! The hungs occurs random

[regression][6.1] After commit e4dc45b1848bc6bcac31eb1b4ccdd7f6718b3c86 system randomly hungs

2022-10-11 Thread Mikhail Gavrilov
Hi! The hungs occurs randomly, but I found good reproductive scenario (This is running the campaign in the game Halo Infinite) The backtrace is look like this: [ 147.260971] BUG: kernel NULL pointer dereference, address: 0088 [ 147.260987] [ cut here ] [ 147.