Re: [PATCH 01/19] drm: Pass pixel_format+modifier to .get_format_info()

2025-04-10 Thread Ville Syrjälä
On Thu, Apr 10, 2025 at 10:27:45PM +0300, Laurent Pinchart wrote: > Hi Ville, > > Thank you for the patch. > > On Thu, Apr 10, 2025 at 07:32:00PM +0300, Ville Syrjala wrote: > > From: Ville Syrjälä > > > > Decouple .get_format_info() from struct drm_mode_fb_cmd2 and just > > pass the pixel form

[PATCH] drm/amdgpu: Set RAS EEPROM table version to v3 for umc v12_5

2025-04-10 Thread Candice Li
Set RAS EEPROM table version to v3 for umc v12_5. Signed-off-by: Candice Li --- drivers/gpu/drm/amd/amdgpu/amdgpu_ras_eeprom.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras_eeprom.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras_eeprom.c index c985d58fdd7ddc

Re: [PATCH 2/2 v2] drm/amdgpu: Add fw minimum version check for usermode queue

2025-04-10 Thread Khatri, Sunil
Series LGTM, Reviewed-by: Sunil Khatri On 4/11/2025 10:25 AM, Deucher, Alexander wrote: [AMD Official Use Only - AMD Internal Distribution Only] Acked-by: Alex Deucher *From:* Yadav, Arvind *Sent:* Friday, April 11, 2

Re: [PATCH 2/2 v2] drm/amdgpu: Add fw minimum version check for usermode queue

2025-04-10 Thread Deucher, Alexander
[AMD Official Use Only - AMD Internal Distribution Only] Acked-by: Alex Deucher From: Yadav, Arvind Sent: Friday, April 11, 2025 12:54 AM To: Yadav, Arvind ; Koenig, Christian ; Deucher, Alexander ; Khatri, Sunil ; Sharma, Shashank Cc: amd-gfx@lists.freedeskto

Re: [PATCH 2/2 v2] drm/amdgpu: Add fw minimum version check for usermode queue

2025-04-10 Thread Yadav, Arvind
Alex, This is v2 of 2/2 patch. Please review this. ~arvind On 4/10/2025 8:27 PM, Arvind Yadav wrote: This patch is load usermode queue based on FW support for gfx12. CP Ucode FW Vesion: [PFP = 2840, ME = 2780, MEC = 3050, MES = 123] v2: Addressed review comments from Alex - Just check the

Re: [v4 1/7] drm/amd/amdgpu: Simplify SDMA reset mechanism by removing dynamic callbacks

2025-04-10 Thread Alex Deucher
On Tue, Apr 8, 2025 at 4:47 AM jesse.zh...@amd.com wrote: > > Since KFD no longer registers its own callbacks for SDMA resets, and only KGD > uses the reset mechanism, > we can simplify the SDMA reset flow by directly calling the ring's > `stop_queue` and `start_queue` functions. > This patch re

RE: [v2 1/2] drm/amdgpu: Enable per-queue reset for SDMA v4.4.2 on IP v9.5.0

2025-04-10 Thread Huang, Tim
[AMD Official Use Only - AMD Internal Distribution Only] Hi Jesse, > -Original Message- > From: jesse.zh...@amd.com > Sent: Tuesday, April 8, 2025 2:33 PM > To: amd-gfx@lists.freedesktop.org > Cc: Deucher, Alexander ; Koenig, Christian > ; Huang, Tim ; Lazar, > Lijo ; Zhang, Jesse(Jie) ;

RE: [PATCH 13/13] drm/amdgpu/userq: enable support for secure queues

2025-04-10 Thread Zhang, Jesse(Jie)
[AMD Official Use Only - AMD Internal Distribution Only] Test-by: Jesse.Zhang , Series is Reviewed-by: Jesse.Zhang -Original Message- From: amd-gfx On Behalf Of Alex Deucher Sent: Friday, April 11, 2025 2:54 AM To: amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander Subject: [PATCH

RE: [PATCH] drm/amdgpu: Clear overflow for SRIOV

2025-04-10 Thread Deng, Emily
[AMD Official Use Only - AMD Internal Distribution Only] Ping.. Best Wishes Emily Deng >-Original Message- >From: Chen, Horace >Sent: Thursday, April 10, 2025 4:00 PM >To: Deng, Emily ; amd-gfx@lists.freedesktop.org >Cc: Deng, Emily >Subject: RE: [PATCH] drm/amdgpu: Clear overflo

Re: [PATCH v2] drm/amdgpu: make mes_userq_unmap as int from void

2025-04-10 Thread Alex Deucher
On Wed, Apr 2, 2025 at 8:11 AM Sunil Khatri wrote: > > mes_userq_unmap could fail due to MES fw unable to > unmap the queue and the return value needs is not > to be ignored and handled on first step itself. > > Also queue->queue_active set to false in this function > but only when the queue is re

[PATCH 6/6] drm/amdgpu: fix typo in bios_parser.c

2025-04-10 Thread Alexandre Demers
Probably a cut and paste error from using get_integrated_info_v8's comment. This has to be get_integrated_info_v9 Signed-off-by: Alexandre Demers --- drivers/gpu/drm/amd/display/dc/bios/bios_parser.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/disp

[PATCH 2/5] drm/amdgpu/userq: Add lock before accessing dma_fence_is_signaled_locked

2025-04-10 Thread Arunpravin Paneer Selvam
Add lock before accessing dma_fence_is_signaled_locked. Signed-off-by: Arunpravin Paneer Selvam --- drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fen

[PATCH 1/2] drm/amdgpu: Update vcn doorbell range in NBIO 7.9

2025-04-10 Thread Sathishkumar S
From: Shiwu Zhang Increase vcn doorbell range for gfx950 to 11. Signed-off-by: Shiwu Zhang --- drivers/gpu/drm/amd/amdgpu/nbio_v7_9.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/nbio_v7_9.c b/drivers/gpu/drm/amd/amdgpu/nbio_v7_9.c ind

Re: [PATCH RFC 4/9] drm/amd/pm: Avoid open-coded use of ratelimit_state structure's internals

2025-04-10 Thread Paul E. McKenney
On Mon, Apr 07, 2025 at 02:35:59PM +, Deucher, Alexander wrote: > [Public] > > > -Original Message- > > From: Paul E. McKenney > > Sent: Thursday, April 3, 2025 5:15 PM > > To: linux-ker...@vger.kernel.org > > Cc: kernel-t...@meta.com; Paul E. McKenney ; kernel > > test robot ; Feng,

Re: [PATCH V8 06/43] drm/colorop: Add 1D Curve subtype

2025-04-10 Thread Simon Ser
On Tuesday, April 8th, 2025 at 18:40, Daniel Stone wrote: > > > 5. For a given colorop property, is it an invariant that the colorop > > > will only appear in one color pipeline at a time? (I believe so, but > > > this probably needs documenting and/or igt.) > > > > I don't really understand why

Re: [PATCH] drm/amdgpu: fix warning of drm_mm_clean

2025-04-10 Thread Alex Deucher
On Tue, Apr 8, 2025 at 5:53 AM Christian König wrote: > > Am 08.04.25 um 10:23 schrieb ZhenGuo Yin: > > Kernel doorbell BOs needs to be freed before ttm_fini. > > > > Fixes: 54c30d2a8def ("drm/amdgpu: create kernel doorbell pages") > > Signed-off-by: ZhenGuo Yin > > At least from my point that pa

Re: [lvc-project] [PATCH] drm/amdgpu: check a user-provided number of BOs in list

2025-04-10 Thread Christian König
Am 09.04.25 um 04:39 schrieb Linus Torvalds: > On Tue, 8 Apr 2025 at 09:07, Fedor Pchelkin wrote: >>> Linus comment is about kvmalloc(), but the code here is using >>> kvmalloc_array() which as far as I know is explicitly made to safely >>> allocate arrays with parameters provided by userspace. >

Re: [v3 7/7] drm/amd/amdgpu: Remove deprecated SDMA reset callback mechanism

2025-04-10 Thread Alex Deucher
On Wed, Apr 2, 2025 at 5:15 AM jesse.zh...@amd.com wrote: > > From: "jesse.zh...@amd.com" > > This patch removes the deprecated SDMA reset callback mechanism, which was > previously used to register pre-reset and post-reset callbacks for SDMA > engine resets. > The callback mechanism has been

Pipeline vs. no pipeline (Re: [PATCH V8 06/43] drm/colorop: Add 1D Curve subtype)

2025-04-10 Thread Pekka Paalanen
On Tue, 8 Apr 2025 13:30:46 -0400 Harry Wentland wrote: > On 2025-04-08 12:40, Daniel Stone wrote: > > Hi there, > > > > On Tue, 1 Apr 2025 at 20:53, Simon Ser wrote: > >> On Tuesday, April 1st, 2025 at 17:14, Daniel Stone > >> wrote: ... > >>> 1. Is it guaranteed that, if any plane on

Re: [PATCH 1/2] dma-fence: Rename dma_fence_is_signaled()

2025-04-10 Thread Boris Brezillon
Hi Philipp, On Wed, 9 Apr 2025 14:06:37 +0200 Philipp Stanner wrote: > dma_fence_is_signaled()'s name strongly reads as if this function were > intended for checking whether a fence is already signaled. Also the > boolean it returns hints at that. > > The function's behavior, however, is more

[PATCH] drm/amdgpu: Move read of snoop register from guest to host

2025-04-10 Thread Masha Grinman
From: Masha Grinman Guest is reading/writing to snoop register which is a security violation We moved the code to the host driver And also added a validation on the guest side to check if it's guest Change-Id: I1d5773ffa6187a961994b3403d4cde5b1641369f Signed-off-by: Masha Grinman --- drivers/g

RE: [PATCH] drm/amdgpu: Move read of snoop register from guest to host

2025-04-10 Thread Kasiviswanathan, Harish
[Public] One minor comment, with that fixed. Reviewed-by: Harish Kasiviswanathan -Original Message- From: amd-gfx On Behalf Of Masha Grinman Sent: Monday, April 7, 2025 1:14 PM To: amd-gfx@lists.freedesktop.org Cc: Grinman, Masha ; Grinman, Masha Subject: [PATCH] drm/amdgpu: Move re

Re: [v3 1/7] drm/amd/amdgpu: Simplify SDMA reset mechanism by removing dynamic callbacks

2025-04-10 Thread Alex Deucher
On Wed, Apr 2, 2025 at 5:28 AM jesse.zh...@amd.com wrote: > > Since KFD no longer registers its own callbacks for SDMA resets, and only KGD > uses the reset mechanism, > we can simplify the SDMA reset flow by directly calling the ring's > `stop_queue` and `start_queue` functions. > This patch re

RE: [PATCH] drm/amdgpu: Direct ret in ras_reset_err_cnt on VF

2025-04-10 Thread Rehman, Ahmad
[AMD Official Use Only - AMD Internal Distribution Only] Reviewed-by: Ahmad Rehman Thanks, Ahmad -Original Message- From: Pan, Ellen Sent: Thursday, April 3, 2025 10:40 AM To: amd-gfx@lists.freedesktop.org Cc: Skvortsov, Victor ; Rehman, Ahmad ; Gande, Shravan kumar ; Pan, Ellen Sub

[PATCH] drm/amd/display/dml2: use vzalloc rather than kzalloc

2025-04-10 Thread Alex Deucher
The structures are large and they do not require continuous memory so use vzalloc. Fixes: 70839da63605 ("drm/amd/display: Add new DCN401 sources") Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/4126 Cc: Aurabindo Pillai Signed-off-by: Alex Deucher --- .../gpu/drm/amd/display/dc/dml2/dm

Re: [RFC] PCI: add CONFIG_MMU dependency

2025-04-10 Thread Martin K. Petersen
Arnd, > It turns out that there are no platforms that have PCI but don't have > an MMU, so adding a Kconfig dependency on CONFIG_PCI simplifies build > testing kernels for those platforms a lot, and avoids a lot of > inadvertent build regressions. > > Add a dependency for CONFIG_PCI and remove a

[PATCH 1/2] drm/amdgpu: use a dummy owner for sysfs triggered cleaner shaders v3

2025-04-10 Thread Christian König
Otherwise triggering sysfs multiple times without other submissions in between only runs the shader once. v2: add some comment v3: re-add missing cast Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-)

[PATCH 4/6] drm/radeon: fix typo in atombios.h

2025-04-10 Thread Alexandre Demers
"aligned" not "aligend" Signed-off-by: Alexandre Demers --- drivers/gpu/drm/radeon/atombios.h | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/radeon/atombios.h b/drivers/gpu/drm/radeon/atombios.h index 2db40789235c..c4abd1f78410 100644 --- a/drivers/gpu/

Re: [PATCH 05/19] drm: Allow the caller to pass in the format info to drm_helper_mode_fill_fb_struct()

2025-04-10 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Thu, Apr 10, 2025 at 07:32:04PM +0300, Ville Syrjala wrote: > From: Ville Syrjälä > > Soon all drivers should have the format info already available in the > places where they call drm_helper_mode_fill_fb_struct(). Allow it to > be passed along into drm_hel

Re: [PATCH 04/19] drm: Pass the format info to .fb_create()

2025-04-10 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Thu, Apr 10, 2025 at 07:32:03PM +0300, Ville Syrjala wrote: > From: Ville Syrjälä > > Pass long the format information from the top to .fb_create() > so that we can avoid redundant (and somewhat expensive) lookups > in the drivers. > > Done with cocci (wit

Re: [PATCH 02/19] drm: Pass pixel_format+modifier directly to drm_get_format_info()

2025-04-10 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Thu, Apr 10, 2025 at 07:32:01PM +0300, Ville Syrjala wrote: > From: Ville Syrjälä > > Decouple drm_get_format_info() from struct drm_mode_fb_cmd2 and just > pass the pixel format+modifier combo in by hand. > > We may want to use drm_get_format_info() outsi

Re: [PATCH 01/19] drm: Pass pixel_format+modifier to .get_format_info()

2025-04-10 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Thu, Apr 10, 2025 at 07:32:00PM +0300, Ville Syrjala wrote: > From: Ville Syrjälä > > Decouple .get_format_info() from struct drm_mode_fb_cmd2 and just > pass the pixel format+modifier combo in by hand. > > We may want to use .get_format_info() outside of

Re: [PATCH] drm/amdgpu/gfx11: Add Cleaner Shader Support for GFX11.5.2/11.5.3 GPUs

2025-04-10 Thread Alex Deucher
On Thu, Apr 10, 2025 at 10:18 AM Srinivasan Shanmugam wrote: > > Enable the cleaner shader for additional GFX11.5.2/11.5.3 series GPUs to > ensure data isolation among GPU tasks. The cleaner shader is tasked with > clearing the Local Data Store (LDS), Vector General Purpose Registers > (VGPRs), an

[PATCH 09/13] drm/amdgpu: add tmz queue parameter to mqd props

2025-04-10 Thread Alex Deucher
Use this to track the whether we want TMZ for queues. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h index b818ad63dc84f..364a65524cfdb 100644 --- a

[PATCH 11/13] drm/amdgpu/gfx12: add support for TMZ queues to mqd_init

2025-04-10 Thread Alex Deucher
Set up TMZ for queues. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/gfx_v12_0.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v12_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v12_0.c index 2474006b1a340..da67f27d65a33 100644 --- a/drivers/gpu/drm/a

[PATCH 10/13] drm/amdgpu/gfx11: add support for TMZ queues to mqd_init

2025-04-10 Thread Alex Deucher
Set up TMZ for queues. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c index 91d29f482c3ca..b204d0e6e816d 100644 --- a/drivers/gpu/drm/a

[PATCH 05/13] drm/amdgpu/user: add priorty to user queue structure

2025-04-10 Thread Alex Deucher
So we can track this when we create user queues. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.h index fd0542f60433b..e

[PATCH 12/13] drm/amdgpu/userq/mes: pass the secure flag to mqd init

2025-04-10 Thread Alex Deucher
So that we initialize the MQD as a secure queue. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/mes_userqueue.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/mes_userqueue.c b/drivers/gpu/drm/amd/amdgpu/mes_userqueue.c index f406a9a29bda5..e175f0

[PATCH 13/13] drm/amdgpu/userq: enable support for secure queues

2025-04-10 Thread Alex Deucher
Enable users to create secure GFX/compute queues. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq

[PATCH 07/13] drm/amdgpu/userq: enable support for queue priorities

2025-04-10 Thread Alex Deucher
Enable users to create queues at different priority levels. The highest level is restricted to drm master. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c | 26 ++- 1 file changed, 25 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amd

[PATCH 06/13] drm/amdgpu/userq/mes: handle user queue priority

2025-04-10 Thread Alex Deucher
Handle the queue priority set by the user. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/mes_userqueue.c | 17 - 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/mes_userqueue.c b/drivers/gpu/drm/amd/amdgpu/mes_userqueue.c ind

[PATCH 08/13] drm/amdgpu/userq: add UAPI for setting up secure queues

2025-04-10 Thread Alex Deucher
If the queues needs to access TMZ surfaces, it must be set up as secure. Signed-off-by: Alex Deucher --- include/uapi/drm/amdgpu_drm.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/uapi/drm/amdgpu_drm.h b/include/uapi/drm/amdgpu_drm.h index 8719754c777b4..0ca4b3b961eb3 100644 ---

[PATCH 04/13] drm/amdgpu/mes12: add conversion for priority levels

2025-04-10 Thread Alex Deucher
Convert driver priority levels to MES11 priority levels. At the moment they are the same, but they may not always be. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/mes_v12_0.c | 21 +++-- 1 file changed, 19 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/

[PATCH 01/13] drm/amdgpu: convert userq UAPI _pad to flags

2025-04-10 Thread Alex Deucher
Reuse the _pad field for flags. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c | 4 ++-- include/uapi/drm/amdgpu_drm.h | 5 - 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c b/drivers

[PATCH 03/13] drm/amdgpu/mes11: add conversion for priority levels

2025-04-10 Thread Alex Deucher
Convert driver priority levels to MES11 priority levels. At the moment they are the same, but they may not always be. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/mes_v11_0.c | 21 +++-- 1 file changed, 19 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/

[PATCH 02/13] drm/amdgpu/userq: add UAPI for setting queue priority

2025-04-10 Thread Alex Deucher
Allow the user to set a queue priority levels: 0 - normal low - most apps (maps to MES AMD_PRIORITY_LEVEL_NORMAL) 1 - low - background jobs (maps to MES AMD_PRIORITY_LEVEL_LOW) 2 - normal high - apps that need relative high (maps to MES AMD_PRIORITY_LEVEL_MEDIUM) 3 - high (admin only - for composi

[PATCH 3/9] drm/amdgpu/userq: move some code around

2025-04-10 Thread Alex Deucher
Move some userq fence handling code into amdgpu_userq_fence.c. This matches the other code in that file. Signed-off-by: Alex Deucher --- .../gpu/drm/amd/amdgpu/amdgpu_userq_fence.c | 26 +++ .../gpu/drm/amd/amdgpu/amdgpu_userq_fence.h | 1 + drivers/gpu/drm/amd/amdgpu/amdgpu

[PATCH 9/9] drm/amdgpu/userq: integrate with enforce isolation

2025-04-10 Thread Alex Deucher
Enforce isolation serializes access to the GFX IP. User queues are isolated in the MES scheduler, but we still need to serialize between kernel queues and user queues. For enforce isolation, group KGD user queues with KFD user queues. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/a

[PATCH 1/9] drm/amdgpu/userq: rename suspend/resume callbacks

2025-04-10 Thread Alex Deucher
Rename to map and umap to better align with what is happening at the firmware level and remove the extra level of indirection in the MES userq code. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c | 10 ++-- drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.h | 8 ++-- d

[PATCH] drm/amdgpu/userq/mes: remove unused header

2025-04-10 Thread Alex Deucher
This is unused so remove it. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/mes_userqueue.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/mes_userqueue.c b/drivers/gpu/drm/amd/amdgpu/mes_userqueue.c index abd32415d7343..e3c3fc160b799 100644 --- a/driv

[PATCH 5/9] drm/amdgpu/userq: add suspend and resume helpers

2025-04-10 Thread Alex Deucher
Add helpers to unmap and map user queues on suspend and resume. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c | 39 +++ drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.h | 3 ++ 2 files changed, 42 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgp

[PATCH 4/9] drm/amdgpu/userq: properly clean up userq fence driver on failure

2025-04-10 Thread Alex Deucher
If userq creation fails, we need to properly unwind and free the user queue fence driver. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c b/drivers/gpu/drm/amd/amdg

[PATCH 2/9] drm/amdgpu/userq: rework front end call sequence

2025-04-10 Thread Alex Deucher
Split out the queue map from the mqd create call and split out the queue unmap from the mqd destroy call. This splits the queue setup and teardown with the actual enablement in the firmware. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c | 17 +++-- dr

[PATCH 8/9] drm/amdgpu/userq: add helpers to start/stop scheduling

2025-04-10 Thread Alex Deucher
This will be used to stop/start user queue scheduling for example when switching between kernel and user queues when enforce isolation is enabled. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 + drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c | 66

[PATCH 7/9] drm/amdgpu: don't swallow errors in amdgpu_userqueue_resume_all()

2025-04-10 Thread Alex Deucher
since we loop through the queues |= the errors. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c index 4

[PATCH 6/9] drm/amdgpu/userq: handle system suspend and resume

2025-04-10 Thread Alex Deucher
Unmap user queues on suspend and map them on resume. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_devic

[PATCH 5/5] drm/amdgpu: Acquire BO locks only during BO processing

2025-04-10 Thread Arunpravin Paneer Selvam
Fix lockdep warnings. [ +0.24] WARNING: CPU: 10 PID: 1909 at drivers/gpu/drm/drm_syncobj.c:456 drm_syncobj_find_fence+0x58c/0x6e0 [ +0.000519] CPU: 10 UID: 1000 PID: 1909 Comm: gnome-shel:cs0 Tainted: G W OE 6.12.0+ #18 [ +0.08] Tainted: [W]=WARN, [O]=OOT_MODULE, [E]=UNSI

[v4 5/7] drm/amdgpu: Optimize SDMA v5.2 queue reset and stop logic

2025-04-10 Thread jesse.zh...@amd.com
From: "jesse.zh...@amd.com" This patch refactors the SDMA v5.2 queue reset and stop logic to improve code readability, maintainability, and performance. The key changes include: 1. **Generalized `sdma_v5_2_gfx_stop` Function**: - Added an `inst_mask` parameter to allow stopping specific

Reminder: Seeking nominations for the 2025 X.Org Board of Directors Elections

2025-04-10 Thread Mark Filion
Hello! The nomination period is currently open for the upcoming election to the X.Org Foundation Board of Directors. All X.Org Foundation members are eligible for election to the board. Nominations for the 2025 election are now open and will remain open until 23:59 UTC on 16 April 2025. The Boar

Re: [PATCH 1/2] dma-fence: Rename dma_fence_is_signaled()

2025-04-10 Thread Christian König
Am 09.04.25 um 14:56 schrieb Philipp Stanner: > On Wed, 2025-04-09 at 14:51 +0200, Philipp Stanner wrote: >> On Wed, 2025-04-09 at 14:39 +0200, Boris Brezillon wrote: >>> Hi Philipp, >>> >>> On Wed,  9 Apr 2025 14:06:37 +0200 >>> Philipp Stanner wrote: >>> dma_fence_is_signaled()'s name stron

RE: [PATCH] drm/amdgpu: remove the duplicated mes queue active state setting

2025-04-10 Thread Liang, Prike
[AMD Official Use Only - AMD Internal Distribution Only] Ping .. Regards, Prike > -Original Message- > From: Liang, Prike > Sent: Friday, March 28, 2025 7:52 PM > To: amd-gfx@lists.freedesktop.org > Cc: Deucher, Alexander ; Liang, Prike > > Subject: [PATCH] drm/amdgpu: remove the

Re: [PATCH] drm/amd/display: do not copy invalid CRTC timing info

2025-04-10 Thread Alex Hung
Hi Gergo, Thanks for the patch. I am sending this patch for testing and I will update test result next week. On 4/2/25 11:03, Gergo Koteles wrote: Since b255ce4388e0, it is possible that the CRTC timing information for the preferred mode has not yet been calculated while amdgpu_dm_connector_

[PATCH 4/5] drm/amdgpu/userq: Fix DEBUG_LOCKS_WARN_ON(lock->magic != lock)

2025-04-10 Thread Arunpravin Paneer Selvam
Fix DEBUG_LOCKS_WARN_ON(lock->magic != lock) warning logs. Signed-off-by: Arunpravin Paneer Selvam --- drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_

RE: [RFC] PCI: add CONFIG_MMU dependency

2025-04-10 Thread Deucher, Alexander
[Public] > -Original Message- > From: Arnd Bergmann > Sent: Monday, April 7, 2025 6:38 AM > To: Bjorn Helgaas > Cc: Arnd Bergmann ; Jeff Hugo > ; Carl Vanderlip ; Oded > Gabbay ; Takashi Sakamoto ; > Maarten Lankhorst ; Maxime Ripard > ; Thomas Zimmermann ; David > Airlie ; Simona Vetter

Re: [PATCH v2] drm/amd/display: Add error check for avi and vendor infoframe setup function

2025-04-10 Thread Alex Hung
On 4/7/25 20:20, Wentao Liang wrote: The function fill_stream_properties_from_drm_display_mode() calls the function drm_hdmi_avi_infoframe_from_display_mode() and the function drm_hdmi_vendor_infoframe_from_display_mode(), but does not check its return value. Log the error messages to prevent

Re: [PATCH] drm/amdgpu: still cleanup sid.h

2025-04-10 Thread Alex Deucher
Applied. Thanks! Alex On Mon, Apr 7, 2025 at 1:52 AM Alexandre Demers wrote: > > The defines, shifts and masks are already available in dce_6_0_d.h, > dce_6_0_sh_mask.h. > > Signed-off-by: Alexandre Demers > --- > drivers/gpu/drm/amd/amdgpu/si.c | 26 +- > drivers/gpu

Re: [PATCH 0/6] Introduce a generic function to get the CSB buffer

2025-04-10 Thread Alex Deucher
On Mon, Apr 7, 2025 at 4:15 PM Rodrigo Siqueira wrote: > > On 04/07, Alex Deucher wrote: > > On Sun, Apr 6, 2025 at 7:07 PM Rodrigo Siqueira wrote: > > > > > > This patchset was inspired and made on top of the below series: > > > > > > https://lore.kernel.org/amd-gfx/20250319162225.3775315-1-alex

Re: [PATCH] Documentation/amdgpu: Add Ryzen AI 350 series processors

2025-04-10 Thread Alex Deucher
On Thu, Apr 10, 2025 at 1:33 PM Mario Limonciello wrote: > > These have been announced so add them to the table. > > Link: > https://www.amd.com/en/products/processors/laptop/ryzen/ai-300-series/amd-ryzen-ai-7-350.html > Signed-off-by: Mario Limonciello Acked-by: Alex Deucher > --- > Documen

[PATCH] Documentation/amdgpu: Add Ryzen AI 350 series processors

2025-04-10 Thread Mario Limonciello
These have been announced so add them to the table. Link: https://www.amd.com/en/products/processors/laptop/ryzen/ai-300-series/amd-ryzen-ai-7-350.html Signed-off-by: Mario Limonciello --- Documentation/gpu/amdgpu/apu-asic-info-table.csv | 1 + 1 file changed, 1 insertion(+) diff --git a/Docum

Re: [PATCH] platform/x86/amd: pmc: Require at least 2.5 seconds between HW sleep cycles

2025-04-10 Thread Mario Limonciello
On 4/10/2025 12:24 PM, Mario Limonciello wrote: When an APU exits HW sleep with no active wake sources the Linux kernel will rapidly assert that the APU can enter back into HW sleep. This happens in a few ms. Contrasting this to Windows, Windows can take 10s of seconds to enter back into the resi

[PATCH] platform/x86/amd: pmc: Require at least 2.5 seconds between HW sleep cycles

2025-04-10 Thread Mario Limonciello
When an APU exits HW sleep with no active wake sources the Linux kernel will rapidly assert that the APU can enter back into HW sleep. This happens in a few ms. Contrasting this to Windows, Windows can take 10s of seconds to enter back into the resiliency phase for Modern Standby. For some situati

[PATCH 1/2 v2] drm/amdgpu: Add fw minimum version check for usermode queue

2025-04-10 Thread Arvind Yadav
This patch is load usermode queue based on FW support for gfx11. CP Ucode FW version: [PFP = 2530, ME = 2390, MEC = 2600, MES = 120] v2: Addressed review comments from Alex. - Just check the firmware versions directly. Cc: Alex Deucher Cc: Christian Koenig Cc: Shashank Sharma Cc: Sunil Kha

Re: [PATCH 1/2 v3] drm/amdgpu: Add fw minimum version check for usermode queue

2025-04-10 Thread Alex Deucher
On Thu, Apr 10, 2025 at 12:52 PM Arvind Yadav wrote: > > This patch is load usermode queue based on FW support for gfx11. > CP Ucode FW version: [PFP = 2530, ME = 2390, MEC = 2600, MES = 120] > > v2: Addressed review comments from Alex. > - Just check the firmware versions directly. > v3: Firm

Re: [PATCH next] drm/amdgpu: Fix double free in amdgpu_userq_fence_driver_alloc()

2025-04-10 Thread Yadav, Arvind
Please change this also instead of 'goto free_fence_drv' just return err.     fence_drv = kzalloc(sizeof(*fence_drv), GFP_KERNEL);     if (!fence_drv) {     DRM_ERROR("Failed to allocate memory for fence driver\n");     r = -ENOMEM;     goto free_fence_

[PATCH 1/2 v3] drm/amdgpu: Add fw minimum version check for usermode queue

2025-04-10 Thread Arvind Yadav
This patch is load usermode queue based on FW support for gfx11. CP Ucode FW version: [PFP = 2530, ME = 2390, MEC = 2600, MES = 120] v2: Addressed review comments from Alex. - Just check the firmware versions directly. v3: Firmware version checks only for Navi3x(by Alex). Cc: Alex Deucher Cc

[PATCH 05/19] drm: Allow the caller to pass in the format info to drm_helper_mode_fill_fb_struct()

2025-04-10 Thread Ville Syrjala
From: Ville Syrjälä Soon all drivers should have the format info already available in the places where they call drm_helper_mode_fill_fb_struct(). Allow it to be passed along into drm_helper_mode_fill_fb_struct() instead of doing yet another redundant lookup. Start by always passing in NULL and

[PATCH 09/19] drm/amdgpu: Pass along the format info from .fb_create() to drm_helper_mode_fill_fb_struct()

2025-04-10 Thread Ville Syrjala
From: Ville Syrjälä Plumb the format info from .fb_create() all the way to drm_helper_mode_fill_fb_struct() to avoid the redundant lookup. Cc: Alex Deucher Cc: amd-gfx@lists.freedesktop.org Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 5 +++-- 1 file changed,

[PATCH 04/19] drm: Pass the format info to .fb_create()

2025-04-10 Thread Ville Syrjala
From: Ville Syrjälä Pass long the format information from the top to .fb_create() so that we can avoid redundant (and somewhat expensive) lookups in the drivers. Done with cocci (with some manual fixups): @@ identifier func =~ ".*create.*"; identifier dev, file, mode_cmd; @@ struct drm_framebuff

[PATCH 02/19] drm: Pass pixel_format+modifier directly to drm_get_format_info()

2025-04-10 Thread Ville Syrjala
From: Ville Syrjälä Decouple drm_get_format_info() from struct drm_mode_fb_cmd2 and just pass the pixel format+modifier combo in by hand. We may want to use drm_get_format_info() outside of the normal addfb paths where we won't have a struct drm_mode_fb_cmd2, and creating a temporary one just fo

[PATCH 01/19] drm: Pass pixel_format+modifier to .get_format_info()

2025-04-10 Thread Ville Syrjala
From: Ville Syrjälä Decouple .get_format_info() from struct drm_mode_fb_cmd2 and just pass the pixel format+modifier combo in by hand. We may want to use .get_format_info() outside of the normal addfb paths where we won't have a struct drm_mode_fb_cmd2, and creating a temporary one just for this

[PATCH 2/2 v2] drm/amdgpu: Add fw minimum version check for usermode queue

2025-04-10 Thread Arvind Yadav
This patch is load usermode queue based on FW support for gfx12. CP Ucode FW Vesion: [PFP = 2840, ME = 2780, MEC = 3050, MES = 123] v2: Addressed review comments from Alex - Just check the firmware versions directly. Cc: Alex Deucher Cc: Christian Koenig Cc: Shashank Sharma Cc: Sunil Khatri

Re: [PATCH V8 43/43] drm/colorop: Add destroy functions for color pipeline

2025-04-10 Thread Simon Ser
On Tuesday, April 1st, 2025 at 04:42, Alex Hung wrote: > On 3/29/25 09:48, Simon Ser wrote: > > > I would prefer these functions to be introduced together with the > > patches adding functions to create objects and adding the new fields. > > That way it's easier to check the symmetry and at no p

Re: [PATCH 1/2 v2] drm/amdgpu: Add fw minimum version check for usermode queue

2025-04-10 Thread Alex Deucher
On Thu, Apr 10, 2025 at 11:56 AM Yadav, Arvind wrote: > > > On 4/10/2025 8:50 PM, Alex Deucher wrote: > > On Thu, Apr 10, 2025 at 10:57 AM Arvind Yadav wrote: > >> This patch is load usermode queue based on FW support for gfx11. > >> CP Ucode FW version: [PFP = 2530, ME = 2390, MEC = 2600, MES =

Re: [PATCH 1/2 v2] drm/amdgpu: Add fw minimum version check for usermode queue

2025-04-10 Thread Yadav, Arvind
On 4/10/2025 8:50 PM, Alex Deucher wrote: On Thu, Apr 10, 2025 at 10:57 AM Arvind Yadav wrote: This patch is load usermode queue based on FW support for gfx11. CP Ucode FW version: [PFP = 2530, ME = 2390, MEC = 2600, MES = 120] v2: Addressed review comments from Alex. - Just check the f

Re: [PATCH] drm/amd/display/dml2: use vzalloc rather than kzalloc

2025-04-10 Thread Pillai, Aurabindo
[AMD Official Use Only - AMD Internal Distribution Only] Reviewed-by: Aurabindo Pillai -- Regards, Jay From: Deucher, Alexander Sent: Wednesday, April 9, 2025 5:41 PM To: amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander ; Pillai, Aurabindo Subject: [PATC

Re: [PATCH 1/2 v2] drm/amdgpu: Add fw minimum version check for usermode queue

2025-04-10 Thread Alex Deucher
On Thu, Apr 10, 2025 at 10:57 AM Arvind Yadav wrote: > > This patch is load usermode queue based on FW support for gfx11. > CP Ucode FW version: [PFP = 2530, ME = 2390, MEC = 2600, MES = 120] > > v2: Addressed review comments from Alex. > - Just check the firmware versions directly. > > Cc: Al

[PATCH] drm/amdgpu/gfx11: Add Cleaner Shader Support for GFX11.5.2/11.5.3 GPUs

2025-04-10 Thread Srinivasan Shanmugam
Enable the cleaner shader for additional GFX11.5.2/11.5.3 series GPUs to ensure data isolation among GPU tasks. The cleaner shader is tasked with clearing the Local Data Store (LDS), Vector General Purpose Registers (VGPRs), and Scalar General Purpose Registers (SGPRs), which helps avoid data leaka

Re: [PATCH] drm/amdgpu: Replace tmp_adev with hive in amdgpu_pci_slot_reset

2025-04-10 Thread Lazar, Lijo
On 4/9/2025 7:26 PM, Dan Carpenter wrote: > On Wed, Apr 09, 2025 at 07:19:25PM +0530, Lazar, Lijo wrote: >> >> >> On 4/9/2025 7:09 PM, Ce Sun wrote: >>> Checking hive is more readable. >>> >>> The following smatch warning: >>> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:6820 amdgpu_pci_slot_reset

Re: [PATCH 2/2] drm/amdgpu: Add fw minimum version check for usermode queue

2025-04-10 Thread Yadav, Arvind
On 4/10/2025 7:01 PM, Alex Deucher wrote: On Thu, Apr 10, 2025 at 7:48 AM Arvind Yadav wrote: This patch is load usermode queue based on FW support for gfx12. CP Ucode FW Vesion: [PFP = 2840, ME = 2780, MEC = 2600, MES = 123] Cc: Alex Deucher Cc: Christian Koenig Cc: Shashank Sharma Cc: S

Re: [PATCH 2/2] drm/amdgpu: Add fw minimum version check for usermode queue

2025-04-10 Thread Khatri, Sunil
Apart from minor comment LGTM.  Also lets wait for Alex to review the series. Reviewed-by: Sunil Khatri On 4/10/2025 4:59 PM, Arvind Yadav wrote: This patch is load usermode queue based on FW support for gfx12. CP Ucode FW Vesion: [PFP = 2840, ME = 2780, MEC = 2600, MES = 123] Cc: Alex Deuche

Re: [PATCH] drm/amdgpu: cleanup amdgpu_vm_flush v5

2025-04-10 Thread Christian König
Am 10.04.25 um 05:52 schrieb SRINIVASAN SHANMUGAM: > > On 4/9/2025 7:16 PM, SRINIVASAN SHANMUGAM wrote: >> >> On 4/9/2025 7:11 PM, SRINIVASAN SHANMUGAM wrote: >>> >>> On 4/9/2025 6:45 PM, SRINIVASAN SHANMUGAM wrote: On 4/9/2025 4:15 PM, Christian König wrote: > This reverts commit c2c

[PATCH 1/1] drm/amd/display: Add quirk to force backlight type on some TUXEDO devices

2025-04-10 Thread Werner Sembach
The display backlight on TUXEDO Polaris AMD Gen2 and Gen3 with panels BOE 2420 and BOE 2423 must be forced to pwn controlled to be able to control the brightness. This could already be archived via a module parameter, but this patch adds a quirk to apply this by default on the mentioned device + p

Re: [PATCH 2/2] drm/amdgpu: Add fw minimum version check for usermode queue

2025-04-10 Thread Alex Deucher
On Thu, Apr 10, 2025 at 7:48 AM Arvind Yadav wrote: > > This patch is load usermode queue based on FW support for gfx12. > CP Ucode FW Vesion: [PFP = 2840, ME = 2780, MEC = 2600, MES = 123] > > Cc: Alex Deucher > Cc: Christian Koenig > Cc: Shashank Sharma > Cc: Sunil Khatri > Signed-off-by: Ar

Re: [PATCH 1/2] dma-fence: Rename dma_fence_is_signaled()

2025-04-10 Thread Philipp Stanner
On Wed, 2025-04-09 at 16:10 +0200, Christian König wrote: > Am 09.04.25 um 16:01 schrieb Philipp Stanner: > > On Wed, 2025-04-09 at 15:14 +0200, Christian König wrote: > > > Am 09.04.25 um 14:56 schrieb Philipp Stanner: > > > > On Wed, 2025-04-09 at 14:51 +0200, Philipp Stanner wrote: > > > > > On

[PATCH] drm/amd/display: drop unimplemented mode_422 code

2025-04-10 Thread Rostislav Nesin
The variable mode_422 is initialized to zero, making if (mode_422) always false. Removing this unimplemented code and the redundant check simplifies the code without affecting functionality. Found by Linux Verification Center (linuxtesting.org) with SVACE. Signed-off-by: Rostislav Nesin --- ..

Re: [PATCH] drm/amdgpu: Replace tmp_adev with hive in amdgpu_pci_slot_reset

2025-04-10 Thread Dan Carpenter
On Wed, Apr 09, 2025 at 07:19:25PM +0530, Lazar, Lijo wrote: > > > On 4/9/2025 7:09 PM, Ce Sun wrote: > > Checking hive is more readable. > > > > The following smatch warning: > > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:6820 amdgpu_pci_slot_reset() > > warn: iterator used outside loop: 'tmp_a

[PATCH 5.15.y] drm/amd/display: Add null checks for 'stream' and 'plane' before dereferencing

2025-04-10 Thread Zhi Yang
From: Srinivasan Shanmugam commit 15c2990e0f0108b9c3752d7072a97d45d4283aea upstream. This commit adds null checks for the 'stream' and 'plane' variables in the dcn30_apply_idle_power_optimizations function. These variables were previously assumed to be null at line 922, but they were used later

Re: [PATCH RESEND RFC v4 3/6] drm/display: dp: use new DCPD access helpers

2025-04-10 Thread Kees Bakker
Op 24-03-2025 om 12:51 schreef Dmitry Baryshkov: From: Dmitry Baryshkov Switch drm_dp_helper.c to use new set of DPCD read / write helpers. Reviewed-by: Lyude Paul Acked-by: Jani Nikula Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/display/drm_dp_helper.c | 296 +

Re: [PATCH 1/2] dma-fence: Rename dma_fence_is_signaled()

2025-04-10 Thread Philipp Stanner
On Wed, 2025-04-09 at 14:51 +0200, Philipp Stanner wrote: > On Wed, 2025-04-09 at 14:39 +0200, Boris Brezillon wrote: > > Hi Philipp, > > > > On Wed,  9 Apr 2025 14:06:37 +0200 > > Philipp Stanner wrote: > > > > > dma_fence_is_signaled()'s name strongly reads as if this function > > > were > > >

[bug report] drm/amdgpu: Multi-GPU DPC recovery support

2025-04-10 Thread Dan Carpenter
Hello Ce Sun, Commit 8ba904f54148 ("drm/amdgpu: Multi-GPU DPC recovery support") from Mar 21, 2025 (linux-next), leads to the following Smatch static checker warning: drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:6820 amdgpu_pci_slot_reset() warn: iterator used outside loop: 'tmp_ade

  1   2   >