RE: [PATCH] drm/amdgpu: fix Null pointer dereference error in amdgpu_device_recover_vram

2023-05-30 Thread Xu, Feifei
[AMD Official Use Only - General] Acked-by: Feifei Xu -Original Message- From: Horatio Zhang Sent: Tuesday, May 30, 2023 2:53 AM To: amd-gfx@lists.freedesktop.org Cc: Xu, Feifei ; Yao, Longlong ; Zhang, Horatio ; Pan, Xinhui Subject: [PATCH] drm/amdgpu: fix Null pointer dereference

[PATCH v2] drm/ttm: Remove redundant code in ttm_tt_init_fields

2023-05-30 Thread Ma Jun
Remove redundant assignment code for ttm->caching as it's overwritten just a few lines later. v2: - Update the commit message. Signed-off-by: Ma Jun --- drivers/gpu/drm/ttm/ttm_tt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/ttm/ttm_tt.c b/drivers/gpu/drm/ttm/ttm_tt.c

[PATCH v2] drm/amd/display: Fix up missing 'dc' & 'pipe_ctx' kdoc parameters in delay_cursor_until_vupdate()

2023-05-30 Thread Srinivasan Shanmugam
Fixes the following gcc with W=1: drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:1904: warning: Function parameter or member 'dc' not described in 'delay_cursor_until_vupdate' drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:1904: warning: Function

Re: [PATCH] drm/amdkfd: Fix MEC pipe interrupt enablement

2023-05-30 Thread Felix Kuehling
This quirk of for_each_inst has caused problems more than once. Why not use for_each_set_bit to implement it? That one doesn't have side effects. Regards,   Felix Am 2023-05-29 um 09:58 schrieb Lijo Lazar: for_each_inst modifies xcc_mask and therefore the loop doesn't initialize properly

[PATCH 13/14] drm/amd/display: Skip DPP DTO update if root clock is gated

2023-05-30 Thread Stylon Wang
From: Nicholas Kazlauskas [Why] Hardware implements root clock gating by utilizing the DPP DTO registers with a special case of DTO enabled, phase = 0, modulo = 1. This conflicts with our policy to always update the DPPDTO for cases where it's expected to be disabled. The pipes unexpectedly

[PATCH 14/14] drm/amd/display: Enable dcn314 DPP RCO

2023-05-30 Thread Stylon Wang
From: Daniel Miess [Why and How] Add back debug bits enabling RCO for dcn314 as underflow associated with this change has been resolved Acked-by: Stylon Wang Signed-off-by: Daniel Miess Reviewed-by: Jun Lei --- .../drm/amd/display/dc/dcn314/dcn314_resource.c | 16 1 file

[PATCH 12/14] drm/amd/display: Filter out AC mode frequencies on DC mode systems

2023-05-30 Thread Stylon Wang
From: Austin Zheng Why: Limit maximum clock speeds to DC mode limits for DC mode systems How: Store DC mode limits when individual clocks are initialized and cap the values when building the clock table Acked-by: Stylon Wang Signed-off-by: Austin Zheng Reviewed-by: Alvin Lee ---

[PATCH 11/14] drm/amd/display: DSC Programming Deltas

2023-05-30 Thread Stylon Wang
From: Sridevi [Why] Programming register delta for DSC sub-block [How] Change DSC, resource files for programming register delta. Acked-by: Stylon Wang Signed-off-by: Sridevi Reviewed-by: Chris Park --- .../gpu/drm/amd/display/dc/dcn20/dcn20_dsc.c | 29 +++

[PATCH 10/14] Revert "drm/amd/display: cache trace buffer size"

2023-05-30 Thread Stylon Wang
From: Leo Ma Revert commit 9caa026e4e65 ("drm/amd/display: cache trace buffer size") to fix regression found in tests. Acked-by: Stylon Wang Signed-off-by: Leo Ma Reviewed-by: Josip Pavic --- drivers/gpu/drm/amd/display/dmub/dmub_srv.h | 1 -

[PATCH 09/14] drm/amd/display: add NULL pointer check

2023-05-30 Thread Stylon Wang
From: Charlene Liu [why] check dmub_Srv exist or not before accessing dmub. Acked-by: Stylon Wang Signed-off-by: Charlene Liu Reviewed-by: Zhan Liu --- drivers/gpu/drm/amd/display/dc/core/dc_stream.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 08/14] drm/amd/display: Enable Freesync Video Mode by default

2023-05-30 Thread Stylon Wang
From: Aurabindo Pillai Revert commit d54f66bc9c37 ("Revert drm/amd/display: Enable Freesync Video Mode by default") Enables freesync video by default, since the hang and corruption issue on eDP panels are now fixed. Acked-by: Stylon Wang Signed-off-by: Aurabindo Pillai Reviewed-by: Rodrigo

[PATCH 07/14] drm/amd/display: Reduce sdp bw after urgent to 90%

2023-05-30 Thread Stylon Wang
From: Alvin Lee [Description] Reduce expected SDP bandwidth due to poor QoS and arbitration issues on high bandwidth configs Cc: Mario Limonciello Cc: Alex Deucher Cc: sta...@vger.kernel.org Acked-by: Stylon Wang Signed-off-by: Alvin Lee Reviewed-by: Nevenko Stupar ---

[PATCH 06/14] drm/amd/display: Add control flag to dc_stream_state to skip eDP BL off/link off

2023-05-30 Thread Stylon Wang
From: Max Tseng Add control flag to dc_stream_state to skip eDP BL off/link off. Acked-by: Stylon Wang Signed-off-by: Max Tseng Reviewed-by: Anthony Koo --- drivers/gpu/drm/amd/display/dc/dc_stream.h | 1 + .../gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c| 3 ++-

[PATCH 04/14] drm/amd/display: Refactor fast update to use new HWSS build sequence

2023-05-30 Thread Stylon Wang
From: Alvin Lee [Description] - Refactor HW sequencer to use a build / execute sequence - Also move gamma updates to become fast Acked-by: Stylon Wang Signed-off-by: Alvin Lee Reviewed-by: Jun Lei --- drivers/gpu/drm/amd/display/dc/core/dc.c | 271 --

[PATCH 05/14] drm/amd/display: Wrong index type for pipe iterator

2023-05-30 Thread Stylon Wang
From: Saaem Rizvi [Why and How] Type mismatch in index and pipe count might cause an infinite loop. code Change should resolve this issue. Acked-by: Stylon Wang Signed-off-by: Saaem Rizvi Reviewed-by: Josip Pavic --- drivers/gpu/drm/amd/display/dc/dcn314/dcn314_hwseq.c | 2 +-

[PATCH 03/14] drm/amd/display: fix dcn315 single stream crb allocation

2023-05-30 Thread Stylon Wang
From: Dmytro Laktyushkin Change to improve avoiding asymetric crb calculations for single stream scenarios. Cc: Mario Limonciello Cc: Alex Deucher Cc: sta...@vger.kernel.org Acked-by: Stylon Wang Signed-off-by: Dmytro Laktyushkin Reviewed-by: Charlene Liu ---

[PATCH 01/14] drm/amd/display: add ODM case when looking for first split pipe

2023-05-30 Thread Stylon Wang
From: Samson Tam [Why] When going from ODM 2:1 single display case to max displays, second odm pipe needs to be repurposed for one of the new single displays. However, acquire_first_split_pipe() only handles MPC case and not ODM case [How] Add ODM conditions in acquire_first_split_pipe() Add

[PATCH 02/14] drm/amd/display: fix seamless odm transitions

2023-05-30 Thread Stylon Wang
From: Dmytro Laktyushkin Add missing programming and function pointers Cc: Mario Limonciello Cc: Alex Deucher Cc: sta...@vger.kernel.org Acked-by: Stylon Wang Signed-off-by: Dmytro Laktyushkin Reviewed-by: Charlene Liu --- drivers/gpu/drm/amd/display/dc/dcn20/dcn20_hwseq.c | 11

[PATCH 00/14] DC Patches June 2, 2023

2023-05-30 Thread Stylon Wang
This DC patchset brings improvements in multiple areas. In summary, we have: * Clock optimiation for DCN 3.1.4 * Performance improvements * Improvements on power saving * Fix screen flash in high resolution displays * Enable Freesync video mode by default * Bug fixed on hang or crashes in various

[PATCH] drm/amdgpu: skip force complete fences when ring is uninitialized

2023-05-30 Thread Bob Zhou
uvd ring in uvd_v7_0_sw_init only initializes ring in bare metal case, so when executing amdgpu_uvd_resume to restore fence seq in SRIOV, a null pointer dereference will occur. This patch correct this. Fixes: 043f2271e2d0a ("drm/amdgpu: mark force completed fences with -ECANCELED") BUG: kernel

[PATCH] drm/amd/display: Fix up missing 'dc' & 'pipe' kdoc parameters in delay_cursor_until_vupdate()

2023-05-30 Thread Srinivasan Shanmugam
Fixes the following gcc with W=1: drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:1904: warning: Function parameter or member 'dc' not described in 'delay_cursor_until_vupdate' drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:1904: warning: Function

[PATCH] drm/amd/display: Fix up kdoc formatting in dcn32_resource_helpers.c

2023-05-30 Thread Srinivasan Shanmugam
Fixes the following W=1 kernel build warning: drivers/gpu/drm/amd/amdgpu/../display/dc/dcn32/dcn32_resource_helpers.c:97: warning: Cannot understand * **

[PATCH] drm/dp_mst: Clear MSG_RDY flag before sending new message

2023-05-30 Thread Wayne Lin
[Why] The sequence for collecting down_reply from source perspective should be: Request_n->repeat (get partial reply of Request_n->clear message ready flag to ack DPRX that the message is received) till all partial replies for Request_n are received->new Request_n+1. Now there is chance that

[PATCH] drm/amd/display: Correct kdoc formatting for DCN32_CRB_SEGMENT_SIZE_KB in dcn32_hubbub.c

2023-05-30 Thread Srinivasan Shanmugam
Fixes the following W=1 kernel build warning: drivers/gpu/drm/amd/amdgpu/../display/dc/dcn32/dcn32_hubbub.c:45: warning: Cannot understand * @DCN32_CRB_SEGMENT_SIZE_KB: Maximum Configurable Return Buffer size for on line 45 - I thought it was a doc line Cc: Hamza Mahfooz Cc: Rodrigo

[PATCH] drm/amd/display: Drop unused DCN_BASE variable in dcn314_resource.c

2023-05-30 Thread Srinivasan Shanmugam
Fixes the following W=1 kernel build warning: drivers/gpu/drm/amd/amdgpu/../display/dc/dcn314/dcn314_resource.c:128:29: warning: ‘DCN_BASE’ defined but not used [-Wunused-const-variable=] 128 | static const struct IP_BASE DCN_BASE = { { { { 0x0012, 0x00C0, 0x34C0, 0x9000,

Re: [PATCH] drm/ttm: Remove redundant code in ttm_tt_init_fields

2023-05-30 Thread Ma, Jun
On 5/30/2023 4:59 PM, Christian König wrote: > Am 29.05.23 um 11:28 schrieb Ma Jun: >> Remove redundant assignment code for ttm->caching > > The explanation is missing why this is redundant, e.g. something like > "this is overwritten just a few lines later".. > Thanks for review. Will

RE: [PATCH] drm/amdxcp: fix Makefile to build amdxcp module

2023-05-30 Thread Chen, Guchun
[Public] Reviewed-by: Guchun Chen Regards, Guchun > -Original Message- > From: Bob Zhou > Sent: Tuesday, May 30, 2023 5:52 PM > To: amd-gfx@lists.freedesktop.org; Zhu, James > Cc: Cui, Flora ; Chen, Guchun > ; Shi, Leslie ; Ma, Jun > ; Song, Asher ; Zhou, Bob > > Subject: [PATCH]

[PATCH v2 2/2] drm/amd: Make lack of `ACPI_FADT_LOW_POWER_S0` or `CONFIG_AMD_PMC` louder during suspend path

2023-05-30 Thread Mario Limonciello
Users have reported that s2idle wasn't working on OEM Phoenix systems, but it was root caused to be because `CONFIG_AMD_PMC` wasn't set in the distribution kernel config. To make this more apparent, raise the messaging to err instead of warn. Link:

[PATCH v2 1/2] drm/amd: Disallow s0ix without BIOS support again

2023-05-30 Thread Mario Limonciello
commit cf488dcd0ab7 ("drm/amd: Allow s0ix without BIOS support") showed improvements to power consumption over suspend when s0ix wasn't enabled in BIOS and the system didn't support S3. This patch however was misguided because the reason the system didn't support S3 was because SMT was disabled

Re: [PATCH 1/2] drm/amd: Disallow s0ix without BIOS support again

2023-05-30 Thread Limonciello, Mario
On 5/30/2023 4:34 PM, Alex Deucher wrote: On Tue, May 30, 2023 at 2:19 PM Limonciello, Mario wrote: [AMD Official Use Only - General] -Original Message- From: Alex Deucher Sent: Tuesday, May 30, 2023 1:16 PM To: Limonciello, Mario Cc: amd-gfx@lists.freedesktop.org; Rafael Ávila

Re: [PATCH 1/2] drm/amd: Disallow s0ix without BIOS support again

2023-05-30 Thread Alex Deucher
On Tue, May 30, 2023 at 2:19 PM Limonciello, Mario wrote: > > [AMD Official Use Only - General] > > > -Original Message- > > From: Alex Deucher > > Sent: Tuesday, May 30, 2023 1:16 PM > > To: Limonciello, Mario > > Cc: amd-gfx@lists.freedesktop.org; Rafael Ávila de Espíndola > > > >

[PATCH] drm/amd/display: Add gnu_printf format attribute for snprintf_count()

2023-05-30 Thread Srinivasan Shanmugam
Fix the following W=1 kernel build warning: display/dc/dcn10/dcn10_hw_sequencer_debug.c: In function ‘snprintf_count’: display/dc/dcn10/dcn10_hw_sequencer_debug.c:56:2: warning: function ‘snprintf_count’ might be a candidate for ‘gnu_printf’ format attribute [-Wsuggest-attribute=format] Use

Re: [PATCH 32/33] drm/amdkfd: add debug device snapshot operation

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: Similar to queue snapshot, return an array of device information using an entry_size check and return. Unlike queue snapshots, the debugger needs to pass to correct number of devices that exist. If it fails to do so, the KFD will return the number of

Re: [PATCH 28/33] drm/amdkfd: add debug set flags operation

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: Allow the debugger to set single memory and single ALU operations. Some exceptions are imprecise (memory violations, address watch) in the sense that a trap occurs only when the exception interrupt occurs and not at the non-halting faulty

Re: [PATCH 27/33] drm/amdkfd: add debug set and clear address watch points operation

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: Shader read, write and atomic memory operations can be alerted to the debugger as an address watch exception. Allow the debugger to pass in a watch point to a particular memory address per device. Note that there exists only 4 watch points per

Re: [PATCH 26/33] drm/amdkfd: add debug suspend and resume process queues operation

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: In order to inspect waves from the saved context at any point during a debug session, the debugger must be able to preempt queues to trigger context save by suspending them. On queue suspend, the KFD will copy the context save header information so

Re: [PATCH 25/33] drm/amdkfd: add debug wave launch mode operation

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: Allow the debugger to set wave behaviour on to either normally operate, halt at launch, trap on every instruction, terminate immediately or stall on allocation. v2: fixup with new kfd_node struct reference for mes check Signed-off-by: Jonathan Kim

Re: [PATCH 24/33] drm/amdkfd: add debug wave launch override operation

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: This operation allows the debugger to override the enabled HW exceptions on the device. On debug devices that only support the debugging of a single process, the HW exceptions are global and set through the SPI_GDBG_TRAP_MASK register. Because they

Re: [PATCH 22/33] drm/amdkfd: update process interrupt handling for debug events

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: The debugger must be notified by any debugger subscribed exception that comes from hardware interrupts. If a debugger session exits, any exceptions it subscribed to may still have interrupts in the interrupt ring buffer or KGD/KFD pipeline. To

Re: [PATCH 20/33] drm/amdkfd: add runtime enable operation

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: The debugger can attach to a process prior to HSA enablement (i.e. inferior is spawned by the debugger and attached to immediately before target process has been enabled for HSA dispatches) or it can attach to a running target that is already HSA

Re: [PATCH 18/33] drm/amdkfd: add raise exception event function

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: Exception events can be generated from interrupts or queue activitity. The raise event function will save exception status of a queue, device or process then notify the debugger of the status change by writing to a debugger polled file descriptor

Re: [PATCH 16/33] drm/amdkfd: add per process hw trap enable and disable functions

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: To enable HW debug mode per process, all devices must be debug enabled successfully. If a failure occures, rewind the enablement of debug mode on the enabled devices. A power management scenario that needs to be considered is HW debug mode setting

RE: [PATCH 14/33] drm/amdgpu: prepare map process for multi-process debug devices

2023-05-30 Thread Kim, Jonathan
[Public] > -Original Message- > From: Kuehling, Felix > Sent: Tuesday, May 30, 2023 3:56 PM > To: Kim, Jonathan ; amd- > g...@lists.freedesktop.org; dri-de...@lists.freedesktop.org > Cc: Huang, JinHuiEric > Subject: Re: [PATCH 14/33] drm/amdgpu: prepare map process for multi- > process

Re: [PATCH 14/33] drm/amdgpu: prepare map process for multi-process debug devices

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: Unlike single process debug devices, multi-process debug devices allow debug mode setting per-VMID (non-device-global). Because the HWS manages PASID-VMID mapping, the new MAP_PROCESS API allows the KFD to forward the required SPI debug register

Re: [PATCH 9/9] drm/amd/pm: enable Wifi RFI mitigation feature support for SMU13.0.7

2023-05-30 Thread kernel test robot
Hi Evan, kernel test robot noticed the following build errors: [auto build test ERROR on drm-misc/drm-misc-next] [also build test ERROR on kvalo-ath/ath-next wireless-next/main wireless/main] [cannot apply to linus/master v6.4-rc4 next-20230530] [If your patch is applied to the wrong git tree

Re: [PATCH 13/33] drm/amdkfd: prepare map process for single process debug devices

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: Older HW only supports debugging on a single process because the SPI debug mode setting registers are device global. The HWS has supplied a single pinned VMID (0xf) for MAP_PROCESS for debug purposes. To pin the VMID, the KFD will remove the VMID

Re: [PATCH 12/33] drm/amdgpu: add configurable grace period for unmap queues

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: |diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_packet_manager_vi.c b/drivers/gpu/drm/amd/amdkfd/kfd_packet_manager_vi.c index faf4772ed317..a0cfd57ea84a 100644 --- a/drivers/gpu/drm/amd/amdkfd/kfd_packet_manager_vi.c +++

Re: [PATCH 05/33] drm/amdgpu: setup hw debug registers on driver initialization

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: Add missing debug trap registers references and initialize all debug registers on boot by clearing the hardware exception overrides and the wave allocation ID index. The debugger requires that TTMPs 6 & 7 save the dispatch ID to map waves onto

Re: [PATCH 02/33] drm/amdkfd: display debug capabilities

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: Expose debug capabilities in the KFD topology node's HSA capabilities and debug properties flags. Ensure correct capabilities are exposed based on firmware support. Flag definitions can be referenced in uapi/linux/kfd_sysfs.h. v2: rebase topology

Re: [PATCH 01/33] drm/amdkfd: add debug and runtime enable interface

2023-05-30 Thread Felix Kuehling
Am 2023-05-25 um 13:27 schrieb Jonathan Kim: Introduce the GPU debug operations interface. For ROCm-GDB to extend the GNU Debugger's ability to inspect the AMD GPU instruction set, provide the necessary interface to allow the debugger to HW debug-mode set and query exceptions per HSA queue,

[PATCH] drm/amd/display: Fix up kdoc formatting in display_mode_vba.c

2023-05-30 Thread Srinivasan Shanmugam
Fixes the following W=1 kernel build warning: drivers/gpu/drm/amd/amdgpu/../display/dc/dml/display_mode_vba.c:936: warning: Cannot understand * * Cc: Aurabindo Pillai Signed-off-by: Srinivasan Shanmugam ---

RE: [PATCH 1/2] drm/amd: Disallow s0ix without BIOS support again

2023-05-30 Thread Limonciello, Mario
[AMD Official Use Only - General] > -Original Message- > From: Limonciello, Mario > Sent: Tuesday, May 30, 2023 1:38 PM > To: Rafael Ávila de Espíndola ; Alex Deucher > > Cc: amd-gfx@lists.freedesktop.org > Subject: RE: [PATCH 1/2] drm/amd: Disallow s0ix without BIOS support again > > >

[PATCH] drm/amdgpu: Fix up kdoc 'ring' parameter in sdma_v6_0_ring_pad_ib

2023-05-30 Thread Srinivasan Shanmugam
Fix this warning by adding 'ring' arguments to kdoc. gcc with W=1 drivers/gpu/drm/amd/amdgpu/sdma_v6_0.c:1128: warning: Function parameter or member 'ring' not described in 'sdma_v6_0_ring_pad_ib' Cc: Felix Kuehling Cc: Christian König Cc: Alex Deucher Signed-off-by: Srinivasan Shanmugam

RE: [PATCH 1/2] drm/amd: Disallow s0ix without BIOS support again

2023-05-30 Thread Limonciello, Mario
[AMD Official Use Only - General] > As far as I know the "no S3 if SMT off" is just an oddity of the > particular BIOS I got on the "B550I AORUS PRO AX". In that case, maybe the message should be downgraded to INFO, and only shown in the case that s3 is not supported on APUs. This will narrow

RE: [PATCH] drm/amdgpu: add the accelerator pcie class

2023-05-30 Thread Deucher, Alexander
[Public] > -Original Message- > From: Christoph Hellwig > Sent: Friday, May 26, 2023 2:55 AM > To: Deucher, Alexander > Cc: Christoph Hellwig ; Alex Deucher > ; bhelg...@google.com; amd- > g...@lists.freedesktop.org; Zhang, Morris ; linux- > p...@vger.kernel.org > Subject: Re: [PATCH]

RE: [PATCH 1/2] drm/amd: Disallow s0ix without BIOS support again

2023-05-30 Thread Limonciello, Mario
[AMD Official Use Only - General] > -Original Message- > From: Alex Deucher > Sent: Tuesday, May 30, 2023 1:16 PM > To: Limonciello, Mario > Cc: amd-gfx@lists.freedesktop.org; Rafael Ávila de Espíndola > > Subject: Re: [PATCH 1/2] drm/amd: Disallow s0ix without BIOS support again > >

Re: [PATCH 1/2] drm/amd: Disallow s0ix without BIOS support again

2023-05-30 Thread Alex Deucher
On Tue, May 30, 2023 at 1:53 PM Mario Limonciello wrote: > > commit cf488dcd0ab7 ("drm/amd: Allow s0ix without BIOS support") showed > improvements to power consumption over suspend when s0ix wasn't enabled in > BIOS and the system didn't support S3. > > This patch however was misguided because

Re: [PATCH] drm/amdgpu: Add a low priority scheduler for VRAM clearing

2023-05-30 Thread Christian König
Am 19.05.23 um 03:09 schrieb Chen, Xiaogang: On 5/17/2023 5:10 PM, Felix Kuehling wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. On 2023-05-17 17:40, Mukul Joshi wrote: Add a low priority DRM

Re: [PATCH 2/2] drm/scheduler: avoid infinite loop if entity's dependency is a scheduled error fence

2023-05-30 Thread Christian König
Am 17.05.23 um 23:01 schrieb Alex Deucher: On Wed, May 17, 2023 at 11:02 AM Alex Deucher wrote: + dri-devel for scheduler On Tue, May 9, 2023 at 6:23 AM ZhenGuo Yin wrote: [Why] drm_sched_entity_add_dependency_cb ignores the scheduled fence and return false. If entity's dependency is a

[PATCH 2/2] drm/amd: Make lack of `ACPI_FADT_LOW_POWER_S0` or `CONFIG_AMD_PMC` louder during suspend path

2023-05-30 Thread Mario Limonciello
Users have reported that s2idle wasn't working on OEM Phoenix systems, but it was root caused to be because `CONFIG_AMD_PMC` wasn't set in the distribution kernel config. To make this more apparent, raise the messaging to err instead of warn. Link:

[PATCH 1/2] drm/amd: Disallow s0ix without BIOS support again

2023-05-30 Thread Mario Limonciello
commit cf488dcd0ab7 ("drm/amd: Allow s0ix without BIOS support") showed improvements to power consumption over suspend when s0ix wasn't enabled in BIOS and the system didn't support S3. This patch however was misguided because the reason the system didn't support S3 was because SMT was disabled

Re: [PATCH] drm/amdgpu: Fix missing parameter desc for 'xcp_id' in amdgpu_amdkfd_reserve_mem_limit

2023-05-30 Thread Felix Kuehling
Am 2023-05-29 um 10:56 schrieb Srinivasan Shanmugam: Fix these warnings by adding 'xcp_id' argument. gcc with W=1 drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c:160: warning: Function parameter or member 'xcp_id' not described in 'amdgpu_amdkfd_reserve_mem_limit' Cc: Christian König Cc:

Re: [PATCH] drm/amd/amdgpu: introduce DRM_AMDGPU_WERROR

2023-05-30 Thread Ho, Kenny
[Public] Reviewed-by: Kenny Ho

Re: [PATCH] drm/amd/amdgpu: introduce DRM_AMDGPU_WERROR

2023-05-30 Thread Hamza Mahfooz
On 5/30/23 11:50, Ho, Kenny wrote: [Public] On 5/30/23 11:24, Hamza Mahfooz wrote: I am able to get clean builds with this enabled on GCC 11-13 and Clang 15, at least as of commit e786aef0869c ("drm/amd/display: remove unused definition") on amd-staging-drm-next. Did you try intentionally

Re: [PATCH] drm/amdgpu: Fix up missing parameters kdoc in svm_migrate_vma_to_ram

2023-05-30 Thread Felix Kuehling
Am 2023-05-30 um 00:54 schrieb Srinivasan Shanmugam: Fix these warnings by adding & deleting the deviant arguments. gcc with W=1 drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_migrate.c:671: warning: Function parameter or member 'node' not described in 'svm_migrate_vma_to_ram'

Re: [PATCH] drm/amd/amdgpu: introduce DRM_AMDGPU_WERROR

2023-05-30 Thread Ho, Kenny
[Public] On 5/30/23 11:24, Hamza Mahfooz wrote: > I am able to get clean builds with this enabled on GCC 11-13 and Clang > 15, at least as of commit e786aef0869c ("drm/amd/display: remove unused > definition") on amd-staging-drm-next. Did you try intentionally introducing a warning to see if the

Re: [PATCH v4 13/13] drm/i915: Implement dedicated fbdev I/O helpers

2023-05-30 Thread Sam Ravnborg
Hi Thomas, > > > - if (helper->funcs->fb_dirty) { > > > - drm_fb_helper_memory_range_to_clip(info, pos, ret, > > > _area); > > > - drm_fb_helper_damage(helper, damage_area.x1, damage_area.y1, > > > - drm_rect_width(_area), > > > -

Re: [PATCH 0/9] Support Wifi RFI interference mitigation feature

2023-05-30 Thread Limonciello, Mario
On 5/30/2023 1:22 AM, Felix Fietkau wrote: On 30.05.23 04:42, Evan Quan wrote: Due to electrical and mechanical constraints in certain platform designs there may be likely interference of relatively high-powered harmonics of the (G-)DDR memory clocks with local radio module frequency bands

Re: [PATCH] drm/amdgpu: Fix missing parameter desc for 'xcp_id' in amdgpu_amdkfd_reserve_mem_limit

2023-05-30 Thread Alex Deucher
On Mon, May 29, 2023 at 10:56 AM Srinivasan Shanmugam wrote: > > Fix these warnings by adding 'xcp_id' argument. > > gcc with W=1 > drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c:160: warning: Function > parameter or member 'xcp_id' not described in > 'amdgpu_amdkfd_reserve_mem_limit' > > Cc:

Re: [PATCH] drm/amdgpu: Fix missing parameter desc for 'xcc_id' in gfx_v7_0.c & amdgpu_rlc.c

2023-05-30 Thread Alex Deucher
On Mon, May 29, 2023 at 10:10 AM Srinivasan Shanmugam wrote: > > Fix these warnings by adding 'xcc_id' arguments. > > gcc with W=1 > drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c:1557: warning: Function parameter or > member 'xcc_id' not described in 'gfx_v7_0_select_se_sh' >

Re: [PATCH] drm/amd/amdgpu: introduce DRM_AMDGPU_WERROR

2023-05-30 Thread Hamza Mahfooz
On 5/25/23 12:38, Hamza Mahfooz wrote: We want to do -Werror builds on our CI. However, non-amdgpu breakages have prevented us from doing so thus far. Also, there are a number of additional checks that we should enable, that the community cares about and are hidden behind -Wextra. So, define

Re: [PATCH] drm/amdgpu: Fix up missing parameter in kdoc for 'inst' in gmc_ v7, v8, v9, v10, v11.c

2023-05-30 Thread Alex Deucher
On Tue, May 30, 2023 at 5:17 AM Srinivasan Shanmugam wrote: > > Fix these warnings by adding 'inst' arguments to kdocs. > > gcc with W=1 > drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:428: warning: Function parameter or > member 'inst' not described in 'gmc_v7_0_flush_gpu_tlb_pasid' >

Re: [PATCH 2/2] drm/amdgpu: Implement gfx9 patch functions for resubmission

2023-05-30 Thread Alex Deucher
Series is: Acked-by: Alex Deucher On Thu, May 25, 2023 at 9:19 PM wrote: > > From: Jiadong Zhu > > Patch the packages including CONTEXT_CONTROL and WRITE_DATA for gfx9 > during the resubmission scenario. > > Signed-off-by: Jiadong Zhu > --- > drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 80

[PATCH v5 13/13] drm/i915: Implement dedicated fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Implement dedicated fbdev helpers for framebuffer I/O instead of using DRM's helpers. Use an fbdev generator macro for deferred I/O to create the fbdev callbacks. i915 was the only caller of the DRM helpers, so remove them from the helper module. i915's fbdev emulation is still incomplete as it

[PATCH v5 12/13] drm/fbdev-generic: Implement dedicated fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Implement dedicated fbdev helpers for framebuffer I/O instead of using DRM's helpers. Use an fbdev generator macro for deferred I/O to create the callbacks. Fbdev-generic was the only caller of the DRM helpers, so remove them from the helper module. v4: * generate deferred-I/O helpers

[PATCH v5 06/13] drm/radeon: Use regular fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Use the regular fbdev helpers for framebuffer I/O instead of DRM's helpers. Radeon does not use damage handling, so DRM's fbdev helpers are mere wrappers around the fbdev code. By using fbdev helpers directly within each DRM fbdev emulation, we can eventually remove DRM's wrapper functions

[PATCH v5 02/13] fbdev: Add initializer macros for struct fb_ops

2023-05-30 Thread Thomas Zimmermann
For framebuffers in I/O and system memory, add macros that set struct fb_ops to the respective callback functions. For deferred I/O, add macros that generate callback functions with damage handling. Add initializer macros that set struct fb_ops to the generated callbacks. These macros can remove

[PATCH v5 04/13] drm/exynos: Use regular fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Use the regular fbdev helpers for framebuffer I/O instead of DRM's helpers. Exynos does not use damage handling, so DRM's fbdev helpers are mere wrappers around the fbdev code. By using fbdev helpers directly within each DRM fbdev emulation, we can eventually remove DRM's wrapper functions

[PATCH v5 10/13] drm/fb-helper: Export helpers for marking damage areas

2023-05-30 Thread Thomas Zimmermann
Export drm_fb_helper_damage() and drm_fb_helper_damage_range(), which handle damage areas for fbdev emulation. This is a temporary export that allows to move the DRM I/O helpers for fbdev into drivers. Only fbdev-generic and i915 need them. Both will be updated to implement damage handling by

[PATCH v5 11/13] drm/msm: Use regular fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Use the regular fbdev helpers for framebuffer I/O instead of DRM's helpers. Msm does not use damage handling, so DRM's fbdev helpers are mere wrappers around the fbdev code. By using fbdev helpers directly within each DRM fbdev emulation, we can eventually remove DRM's wrapper functions entirely.

[PATCH v5 08/13] drm/omapdrm: Use regular fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Use the regular fbdev helpers for framebuffer I/O instead of DRM's helpers. Omapdrm does not use damage handling, so DRM's fbdev helpers are mere wrappers around the fbdev code. By using fbdev helpers directly within each DRM fbdev emulation, we can eventually remove DRM's wrapper functions

[PATCH v5 09/13] drm/tegra: Use regular fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Use the regular fbdev helpers for framebuffer I/O instead of DRM's helpers. Tegra does not use damage handling, so DRM's fbdev helpers are mere wrappers around the fbdev code. By using fbdev helpers directly within each DRM fbdev emulation, we can eventually remove DRM's wrapper functions

[PATCH v5 07/13] drm/fbdev-dma: Use regular fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Use the regular fbdev helpers for framebuffer I/O instead of DRM's helpers. Fbdev-dma does not use damage handling, so DRM's fbdev helpers are mere wrappers around the fbdev code. By using fbdev helpers directly within each DRM fbdev emulation, we can eventually remove DRM's wrapper functions

[PATCH v5 05/13] drm/gma500: Use regular fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Use the regular fbdev helpers for framebuffer I/O instead of DRM's helpers. Gma500 does not use damage handling, so DRM's fbdev helpers are mere wrappers around the fbdev code. By using fbdev helpers directly within each DRM fbdev emulation, we can eventually remove DRM's wrapper functions

[PATCH v5 03/13] drm/armada: Use regular fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Use the regular fbdev helpers for framebuffer I/O instead of DRM's helpers. Armada does not use damage handling, so DRM's fbdev helpers are mere wrappers around the fbdev code. By using fbdev helpers directly within each DRM fbdev emulation, we can eventually remove DRM's wrapper functions

[PATCH v5 00/13] drm/fbdev: Remove DRM's helpers for fbdev I/O

2023-05-30 Thread Thomas Zimmermann
DRM provides a number of wrappers around fbdev cfb_() sys_(), fb_io_() and fb_sys_() helpers. The DRM functions don't provide any additional functionality for most DRM drivers. So remove them and call the fbdev I/O helpers directly. The DRM fbdev I/O wrappers were originally added because does

[PATCH v5 01/13] fbdev: Add Kconfig options to select different fb_ops helpers

2023-05-30 Thread Thomas Zimmermann
Many fbdev drivers use the same set of fb_ops helpers. Add Kconfig options to select them at once. This will help with making DRM's fbdev emulation code more modular, but can also be used to simplify fbdev's driver configs. v3: * fix select statement (Jingfeng) Signed-off-by: Thomas

Re: [PATCH] drm/amdgpu: Fix up missing kdoc parameter 'inst' in get_wave_count() & kgd_gfx_v9_get_cu_occupancy()

2023-05-30 Thread Felix Kuehling
Am 2023-05-30 um 08:48 schrieb Srinivasan Shanmugam: Fix these warnings by adding 'inst' arguments to kdocs. gcc with W=1 drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v9.c:692: warning: Function parameter or member 'inst' not described in 'get_wave_count'

Re: [PATCH] drm/amdxcp: fix Makefile to build amdxcp module

2023-05-30 Thread Alex Deucher
Acked-by: Alex Deucher On Tue, May 30, 2023 at 5:52 AM Bob Zhou wrote: > > After drm conduct amdgpu Makefile, amdgpu.ko has been created > and "amdgpu-y +=" in amdxcp Makefile isn't used. > So modify amdgpu-y to amdxcp-y and build amdxcp module. > > Signed-off-by: Bob Zhou > --- >

[PATCH v5 11/13] drm/msm: Use regular fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Use the regular fbdev helpers for framebuffer I/O instead of DRM's helpers. Msm does not use damage handling, so DRM's fbdev helpers are mere wrappers around the fbdev code. By using fbdev helpers directly within each DRM fbdev emulation, we can eventually remove DRM's wrapper functions entirely.

[PATCH v5 12/13] drm/fbdev-generic: Implement dedicated fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Implement dedicated fbdev helpers for framebuffer I/O instead of using DRM's helpers. Use an fbdev generator macro for deferred I/O to create the callbacks. Fbdev-generic was the only caller of the DRM helpers, so remove them from the helper module. v4: * generate deferred-I/O helpers

[PATCH v5 08/13] drm/omapdrm: Use regular fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Use the regular fbdev helpers for framebuffer I/O instead of DRM's helpers. Omapdrm does not use damage handling, so DRM's fbdev helpers are mere wrappers around the fbdev code. By using fbdev helpers directly within each DRM fbdev emulation, we can eventually remove DRM's wrapper functions

[PATCH v5 07/13] drm/fbdev-dma: Use regular fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Use the regular fbdev helpers for framebuffer I/O instead of DRM's helpers. Fbdev-dma does not use damage handling, so DRM's fbdev helpers are mere wrappers around the fbdev code. By using fbdev helpers directly within each DRM fbdev emulation, we can eventually remove DRM's wrapper functions

[PATCH v5 01/13] fbdev: Add Kconfig options to select different fb_ops helpers

2023-05-30 Thread Thomas Zimmermann
Many fbdev drivers use the same set of fb_ops helpers. Add Kconfig options to select them at once. This will help with making DRM's fbdev emulation code more modular, but can also be used to simplify fbdev's driver configs. v3: * fix select statement (Jingfeng) Signed-off-by: Thomas

[PATCH v5 04/13] drm/exynos: Use regular fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Use the regular fbdev helpers for framebuffer I/O instead of DRM's helpers. Exynos does not use damage handling, so DRM's fbdev helpers are mere wrappers around the fbdev code. By using fbdev helpers directly within each DRM fbdev emulation, we can eventually remove DRM's wrapper functions

[PATCH v5 09/13] drm/tegra: Use regular fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Use the regular fbdev helpers for framebuffer I/O instead of DRM's helpers. Tegra does not use damage handling, so DRM's fbdev helpers are mere wrappers around the fbdev code. By using fbdev helpers directly within each DRM fbdev emulation, we can eventually remove DRM's wrapper functions

[PATCH v5 10/13] drm/fb-helper: Export helpers for marking damage areas

2023-05-30 Thread Thomas Zimmermann
Export drm_fb_helper_damage() and drm_fb_helper_damage_range(), which handle damage areas for fbdev emulation. This is a temporary export that allows to move the DRM I/O helpers for fbdev into drivers. Only fbdev-generic and i915 need them. Both will be updated to implement damage handling by

[PATCH v5 13/13] drm/i915: Implement dedicated fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Implement dedicated fbdev helpers for framebuffer I/O instead of using DRM's helpers. Use an fbdev generator macro for deferred I/O to create the fbdev callbacks. i915 was the only caller of the DRM helpers, so remove them from the helper module. i915's fbdev emulation is still incomplete as it

[PATCH v5 05/13] drm/gma500: Use regular fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Use the regular fbdev helpers for framebuffer I/O instead of DRM's helpers. Gma500 does not use damage handling, so DRM's fbdev helpers are mere wrappers around the fbdev code. By using fbdev helpers directly within each DRM fbdev emulation, we can eventually remove DRM's wrapper functions

[PATCH v5 06/13] drm/radeon: Use regular fbdev I/O helpers

2023-05-30 Thread Thomas Zimmermann
Use the regular fbdev helpers for framebuffer I/O instead of DRM's helpers. Radeon does not use damage handling, so DRM's fbdev helpers are mere wrappers around the fbdev code. By using fbdev helpers directly within each DRM fbdev emulation, we can eventually remove DRM's wrapper functions

  1   2   >