On 10/17/2024 11:53 AM, Kenneth Feng wrote:
> disable deep sleep during the compute workload for the
> potential performance loss on smu v14.0.2/3
>
> V2: consider the case when the compute workload is set twice
>
> Signed-off-by: Kenneth Feng
> ---
> drivers/gpu/drm/amd/pm/swsmu/smu14/smu_v
On a hive, NPS request is placed by the first one for all devices in the
hive. If the request fails, mark the mode as UNKNOWN so that subsequent
devices on unload don't request it. Also, fix the mutex double lock
issue in error condition, should have been mutex_unlock.
Signed-off-by: Lijo Lazar
There is a spelling mistake in a dm_error message. Fix it.
Signed-off-by: Colin Ian King
---
.../gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c
b/d
On Mon, Sep 30, 2024 at 01:08:41PM +0530, Raag Jadav wrote:
> Introduce device wedged event, which will notify userspace of wedged
> (hanged/unusable) state of the DRM device through a uevent. This is
> useful especially in cases where the device is no longer operating as
> expected even after a ha
On 10/17/2024 5:16 AM, Feng, Kenneth wrote:
> [AMD Official Use Only - AMD Internal Distribution Only]
>
> [AMD Official Use Only - AMD Internal Distribution Only]
>
> The logic is that when entering the compute workload, enable deep sleep.
> When exiting the compute workload, disable deep sle
Am 17.10.24 um 04:47 schrieb Raag Jadav:
On Mon, Sep 30, 2024 at 01:08:41PM +0530, Raag Jadav wrote:
Introduce device wedged event, which will notify userspace of wedged
(hanged/unusable) state of the DRM device through a uevent. This is
useful especially in cases where the device is no longer o
make sure KFD_FENCE_INIT write to fence_addr before pm_send_query_status
called, to avoid qcm fence timeout caused by incorrect ordering.
Signed-off-by: Victor Zhao
---
drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c | 1 +
drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h | 2 +-
2 fi
[Public]
one comment belon, other than that, looks good to me. Feel free to add my RB if
you send a v2.
Regards,
Rajneesh
From: Lazar, Lijo
Sent: Thursday, October 17, 2024 5:10 AM
To: amd-gfx@lists.freedesktop.org
Cc: Zhang, Hawking ; Deucher, Alexander
; Bha
Need to dereference the atcs acpi buffer after
the method is executed, otherwise it will result in
a memory leak.
Signed-off-by: Prike Liang
---
drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c
b/drivers/gpu/dr
Remove the dummy suspend functions for all
ip blocks.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c | 6 --
drivers/gpu/drm/amd/amdgpu/amdgpu_vkms.c | 3 ++-
drivers/gpu/drm/amd/amdgpu/cik.c | 6 --
drivers/gpu/drm/amd/amdgpu/si.c | 6 --
4
Validate the function pointer for get_clockgating_state
before making a function call.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/nv.c| 9 ++---
drivers/gpu/drm/amd/amdgpu/soc21.c | 6 --
drivers/gpu/drm/amd/amdgpu/soc24.c | 6 --
3 files changed, 14 insertions(+)
Remove the dummy sw_init functions for all
IP blocks.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c | 6 --
drivers/gpu/drm/amd/amdgpu/cik.c| 6 --
drivers/gpu/drm/amd/amdgpu/si.c | 6 --
3 files changed, 18 deletions(-)
diff --git a/drivers
Before making a function call to sw_fini, validate
the function pointer like we do in sw_init.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
Remove the unused and commented out code for vcn/uvd IP's.
Also remove the functions which are set to NULL as these are
global structures and all members are set to 0 or NULL during
initialization.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c | 4 -
drivers/gpu
Remove the dummy wait_for_idle functions for all
ip blocks.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c | 6 --
drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c | 5 -
drivers/gpu/drm/amd/amdgpu/amdgpu_vkms.c | 6 --
drivers/gpu/drm/amd/am
Before making a function call to suspend, validate
the function pointer like we do in sw_init.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/aldebaran.c | 15 ++--
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 26 -
drivers/gpu/drm/amd/amdgpu/amdgpu_
Remove the dummy sw_fini functions for all
ip blocks.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c | 6 --
drivers/gpu/drm/amd/amdgpu/cik.c| 6 --
drivers/gpu/drm/amd/amdgpu/nv.c | 6 --
drivers/gpu/drm/amd/amdgpu/si.c | 7 ---
d
Remove the dummy hw_fini functions for all
ip blocks.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vkms.c | 6 --
drivers/gpu/drm/amd/amdgpu/cik.c | 6 --
drivers/gpu/drm/amd/amdgpu/mes_v12_0.c | 11 +--
drivers/gpu/drm/amd/amdgpu/si.c |
Before making a function call to resume, validate
the function pointer like we do in sw_init.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/aldebaran.c | 23 ++
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 47 -
drivers/gpu/drm/amd/amdgpu/sienna_cic
v4: hw_init/hw_fini functions are mandatory and raise error message if
these functions are not defined.
v3: Added 2 new patches to clean get_clocking_state and hw_init
validation.
Sunil Khatri (15):
drm/amdgpu: validate sw_init before function call
drm/amdgpu: clean the dummy sw_init
Remove the dummy resume functions for all
ip blocks.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c
index 9b98b40ac4db..1383fd1644d
Before making a function call to wait_for_idle,
validate the function pointer like we do in sw_init.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
Remove the dummy soft_reset functions for all
ip blocks.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c | 6 --
drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c | 5 -
drivers/gpu/drm/amd/amdgpu/amdgpu_vkms.c | 6 --
drivers/gpu/drm/amd/amdgp
Before making a function call to sw_init, validate
the function pointer like we do in late_init.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
b
Before making a function call to hw_fini, validate
the function pointer like we do in sw_init.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 33 ++
1 file changed, 22 insertions(+), 11 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu
Am 17.10.24 um 12:06 schrieb Sunil Khatri:
Before making a function call to hw_fini, validate
the function pointer like we do in sw_init.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 33 ++
1 file changed, 22 insertions(+), 11 deletions(-)
On 10/17/2024 5:50 PM, Christian König wrote:
Am 17.10.24 um 12:06 schrieb Sunil Khatri:
Validate the function pointer for get_clockgating_state
before making a function call.
Oh, I'm not sure if that is necessary or not. The NBIO, HDP and SMUIO
functions are not IP specific.
For many socs
On 10/17/2024 5:22 PM, Christian König wrote:
Am 17.10.24 um 12:06 schrieb Sunil Khatri:
Before making a function call to hw_fini, validate
the function pointer like we do in sw_init.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 33 ++
1
On 10/17/2024 5:24 PM, Christian König wrote:
Am 17.10.24 um 12:06 schrieb Sunil Khatri:
Before making a function call to suspend, validate
the function pointer like we do in sw_init.
Same comment as on patch #5, please add an amdgpu_ip_block_suspend()
function and just call it multiple tim
On 10/17/2024 5:25 PM, Christian König wrote:
Same comment as patch #5, please add an amdgpu_ip_block_resume()
function.
Sure
Regards,
Christian.
Am 17.10.24 um 12:06 schrieb Sunil Khatri:
Before making a function call to resume, validate
the function pointer like we do in sw_init.
Signed
On 10/17/2024 5:28 PM, Christian König wrote:
Am 17.10.24 um 12:06 schrieb Sunil Khatri:
Remove the dummy hw_fini functions for all
ip blocks.
As discussed on the call hw_init() and hw_fini() are the only two
functions I consider mandatory.
So please drop this patch here and as a follow u
Am 15.10.24 um 09:43 schrieb Arunpravin Paneer Selvam:
Add user fence wait IOCTL timeline syncobj support.
v2:(Christian)
- handle dma_fence_wait() return value.
- shorten the variable name syncobj_timeline_points a bit.
- move num_points up to avoid padding issues.
v3:(Christian)
-
On 10/17/2024 5:48 PM, Christian König wrote:
Am 17.10.24 um 12:06 schrieb Sunil Khatri:
Remove the unused and commented out code for vcn/uvd IP's.
Also remove the functions which are set to NULL as these are
global structures and all members are set to 0 or NULL during
initialization.
Pleas
On 10/17/2024 5:53 PM, Christian König wrote:
Patches #1-#4, #8-#10,#13 are Reviewed-by: Christian König
Since those patches should be independent maybe push them to
amd-staging-drm-next before continuing working on the set.
Sure if the order isnt needed i will push those out of what you
Am 17.10.24 um 12:06 schrieb Sunil Khatri:
Remove the dummy hw_fini functions for all
ip blocks.
As discussed on the call hw_init() and hw_fini() are the only two
functions I consider mandatory.
So please drop this patch here and as a follow up task maybe look into
why vkms and mes_v12 don'
[AMD Official Use Only - AMD Internal Distribution Only]
> -Original Message-
> From: Lazar, Lijo
> Sent: October 16, 2024 11:18 PM
> To: Liu, Leo ; Koenig, Christian
> ; amd-gfx@lists.freedesktop.org
> Cc: Zhang, Hawking ; Deucher, Alexander
> ; Sundararaju, Sathishkumar
> ; Jiang, Sonny
Am 17.10.24 um 12:06 schrieb Sunil Khatri:
Remove the unused and commented out code for vcn/uvd IP's.
Also remove the functions which are set to NULL as these are
global structures and all members are set to 0 or NULL during
initialization.
Please separate that into two patches, one for the vcn
Am 17.10.24 um 12:06 schrieb Sunil Khatri:
Validate the function pointer for get_clockgating_state
before making a function call.
Oh, I'm not sure if that is necessary or not. The NBIO, HDP and SMUIO
functions are not IP specific.
Christian.
Signed-off-by: Sunil Khatri
---
drivers/gpu/
Patches #1-#4, #8-#10,#13 are Reviewed-by: Christian König
Since those patches should be independent maybe push them to
amd-staging-drm-next before continuing working on the set.
Regards,
Christian.
Am 17.10.24 um 12:06 schrieb Sunil Khatri:
v4: hw_init/hw_fini functions are mandatory and
Am 17.10.24 um 12:06 schrieb Sunil Khatri:
Before making a function call to suspend, validate
the function pointer like we do in sw_init.
Same comment as on patch #5, please add an amdgpu_ip_block_suspend()
function and just call it multiple times.
Regards,
Christian.
Signed-off-by: Sunil
Same comment as patch #5, please add an amdgpu_ip_block_resume() function.
Regards,
Christian.
Am 17.10.24 um 12:06 schrieb Sunil Khatri:
Before making a function call to resume, validate
the function pointer like we do in sw_init.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/
From: Boyuan Zhang
Perform dump_ip_state only for the instance of the current vcn IP block,
instead of perform it for all vcn instances.
Signed-off-by: Boyuan Zhang
---
drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c | 27 +--
drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c | 27 +-
From: Boyuan Zhang
Pass instance parameter to amdgpu_vcn_hw_init(), and perform
hw init ONLY for the given vcn instance, instead of for all
vcn instances. Modify each vcn generation accordingly.
Signed-off-by: Boyuan Zhang
---
drivers/gpu/drm/amd/amdgpu/vcn_v2_5.c | 39 +++--
drivers
From: Boyuan Zhang
Pass instance parameter to amdgpu_vcn_resume(), and perform
resume ONLY for the given vcn instance, instead of for all
vcn instances. Modify each vcn generation accordingly.
Signed-off-by: Boyuan Zhang
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c | 60
From: Boyuan Zhang
Add a new function to count the number of instance of the same IP block
in the current ip_block list, then use the returned count value to set
the newly defined instance variable in ip_block, to track the instance
number of each ip_block.
Signed-off-by: Boyuan Zhang
Signed-of
From: Boyuan Zhang
Pass instance parameter to set_dec_ring_funcs(), set_enc_ring_funcs(),
and set_irq_funcs(), and perform function setup ONLY for the given vcn
instance, instead of for all vcn instances. Modify each vcn generation
accordingly.
Signed-off-by: Boyuan Zhang
---
drivers/gpu/drm/a
From: Boyuan Zhang
Set powergating state by vcn instance in idle_work_handler() and
ring_begin_use() functions for vcn with multiple instances.
v2: Add instance parameter to amdgpu_device_ip_set_powergating_state(),
instead of creating new function.
Signed-off-by: Boyuan Zhang
---
drivers/gpu
From: Boyuan Zhang
Pass ip_block instead of adev in set_clockgating_state() and is_idle()
callback functions. Modify set_clockgating_state() and is_idle() ip
functions for all correspoding ip blocks.
Signed-off-by: Boyuan Zhang
---
drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c | 4 ++--
drive
From: Boyuan Zhang
Perform is_idle only for the instance of the current vcn IP block,
instead of perform it for all vcn instances.
Signed-off-by: Boyuan Zhang
---
drivers/gpu/drm/amd/amdgpu/vcn_v2_5.c | 11 +--
drivers/gpu/drm/amd/amdgpu/vcn_v3_0.c | 11 +--
drivers/gpu/drm
From: Boyuan Zhang
Add an instance parameter to amdgpu_dpm_set_powergating_by_smu() function,
and use the instance to call set_powergating_by_smu().
v2: remove duplicated functions.
remove for-loop in amdgpu_dpm_set_powergating_by_smu(), and temporarily
move it to amdgpu_dpm_enable_vcn(), in or
From: Boyuan Zhang
For vcn 2_5, add ip_block for each vcn instance during discovery stage.
And only powering on/off one of the vcn instance using the
instance value stored in ip_block, instead of powering on/off all
vcn instances. Modify the existing functions to use the instance value
in ip_blo
From: Boyuan Zhang
Pass ip_block instead of adev in set_powergating_state callback function.
Modify set_powergating_state ip functions for all correspoding ip blocks.
v2: fix a ip block index error.
Signed-off-by: Boyuan Zhang
Suggested-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdg
From: Boyuan Zhang
Move all per instance variables from amdgpu_vcn to amdgpu_vcn_inst.
Move adev->vcn.fw[i] from amdgpu_vcn to amdgpu_vcn_inst.
Move adev->vcn.vcn_config[i] from amdgpu_vcn to amdgpu_vcn_inst.
Move adev->vcn.vcn_codec_disable_mask[i] from amdgpu_vcn to amdgpu_vcn_inst.
Signed-of
From: Boyuan Zhang
Add an instance parameter to set_powergating_by_smu() function, and
re-write all amd_pm functions accordingly. Then use the instance to
call smu_dpm_set_vcn_enable().
v2: remove duplicated functions.
remove for-loop in smu_dpm_set_power_gate(), and temporarily move it to
to a
From: Boyuan Zhang
Pass instance parameter to amdgpu_vcn_early_init(), and perform
early init ONLY for the given vcn instance, instead of for all
vcn instances. Modify each vcn generation accordingly.
Signed-off-by: Boyuan Zhang
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c | 23
From: Boyuan Zhang
Previously idle working handling is for all VCN instances. As a result, when one
of the instance finishes its job, the idle work can't be triggered if the other
instance is still busy.
Now, move the idle_work from amdgpu_vcn to amdgpu_vcn_inst, in order to
track work by vcn in
From: Boyuan Zhang
Add an instance parameter to amdgpu_dpm_enable_vcn() function, and change
all calls from vcn ip functions to add instance argument. vcn generations
with only one instance (v1.0, v2.0) always use 0 as instance number. vcn
generations with multiple instances (v2.5, v3.0, v4.0, v4
From: Boyuan Zhang
For vcn 4_0, add ip_block for each vcn instance during discovery stage.
And only powering on/off one of the vcn instance using the
instance value stored in ip_block, instead of powering on/off all
vcn instances. Modify the existing functions to use the instance value
in ip_blo
From: Boyuan Zhang
Pass instance parameter to amdgpu_vcn_sw_fini(), and perform
sw fini ONLY for the given vcn instance, instead of for all
vcn instances. Modify each vcn generation accordingly.
Signed-off-by: Boyuan Zhang
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c | 36 --
From: Boyuan Zhang
Pass instance parameter to amdgpu_vcn_sw_init(), and perform
sw init ONLY for the given vcn instance, instead of for all
vcn instances. Modify each vcn generation accordingly.
Signed-off-by: Boyuan Zhang
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c | 83 --
From: Boyuan Zhang
For vcn 4_0_5, add ip_block for each vcn instance during discovery stage.
And only powering on/off one of the vcn instance using the
instance value stored in ip_block, instead of powering on/off all
vcn instances. Modify the existing functions to use the instance value
in ip_b
From: Boyuan Zhang
vcn gating state should now be based on instance. For example, instance 0
can be gated while instance 1 is ungated, or vice versa.
Therefore, change the cur_state to be an array, so that it can track the
gating status for each vcn instance now.
v2: remove redundant codes in v
From: Boyuan Zhang
For vcn 3_0, add ip_block for each vcn instance during discovery stage.
And only powering on/off one of the vcn instance using the
instance value stored in ip_block, instead of powering on/off all
vcn instances. Modify the existing functions to use the instance value
in ip_blo
From: Boyuan Zhang
Pass instance parameter to amdgpu_vcn_setup_ucode(), and perform
setup ucode ONLY for the given vcn instance, instead of for all
vcn instances. Modify each vcn generation accordingly.
Signed-off-by: Boyuan Zhang
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c | 37 --
From: Boyuan Zhang
Add an instance parameter to the existing function dpm_set_vcn_enable()
for future implementation. Re-write all pptable functions accordingly.
v2: Remove duplicated dpm_set_vcn_enable() functions in v1. Instead,
adding instance parameter to existing functions.
Signed-off-by:
From: Boyuan Zhang
For smu ip with multiple vcn instances (smu 11/13/14), remove all the
for loop in dpm_set_vcn_enable() functions. And use the instance
argument to power up/down vcn for the given instance only, instead
of powering up/down for all vcn instances.
v2: remove all duplicated functi
From: Boyuan Zhang
First, add an instance parameter to smu_dpm_set_vcn_enable() function,
and calling dpm_set_vcn_enable() with this given instance.
Second, modify vcn_gated to be an array, to track the gating status
for each vcn instance separately.
With these 2 changes, smu_dpm_set_vcn_enable
From: Boyuan Zhang
Previously, all vcn instance will be powered on/off at the same time
even only one of the instance requests power status change. This patch set
enables vcn to ONLY power on/off the instance that requires power status
change. Other vcn instances will remain the original power st
On 10/17/2024 6:37 PM, Liu, Leo wrote:
> [AMD Official Use Only - AMD Internal Distribution Only]
>
>> -Original Message-
>> From: Lazar, Lijo
>> Sent: October 16, 2024 11:18 PM
>> To: Liu, Leo ; Koenig, Christian
>> ; amd-gfx@lists.freedesktop.org
>> Cc: Zhang, Hawking ; Deucher, Alex
Am 17.10.24 um 14:55 schrieb Khatri, Sunil:
On 10/17/2024 5:50 PM, Christian König wrote:
Am 17.10.24 um 12:06 schrieb Sunil Khatri:
Validate the function pointer for get_clockgating_state
before making a function call.
Oh, I'm not sure if that is necessary or not. The NBIO, HDP and SMUIO
From: Boyuan Zhang
Pass instance parameter to amdgpu_vcn_suspend(), and perform
suspend ONLY for the given vcn instance, instead of for all
vcn instances. Modify each vcn generation accordingly.
Signed-off-by: Boyuan Zhang
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c | 35 --
[AMD Official Use Only - AMD Internal Distribution Only]
> -Original Message-
> From: Lazar, Lijo
> Sent: October 17, 2024 9:17 AM
> To: Liu, Leo ; Koenig, Christian
> ; amd-gfx@lists.freedesktop.org
> Cc: Zhang, Hawking ; Deucher, Alexander
> ; Sundararaju, Sathishkumar
> ; Jiang, Sonny
From: Boyuan Zhang
Perform set_powergating_state only for the instance of the current vcn
IP block, instead of perform it for all vcn instances.
Signed-off-by: Boyuan Zhang
---
drivers/gpu/drm/amd/amdgpu/vcn_v2_5.c | 313
drivers/gpu/drm/amd/amdgpu/vcn_v3_0.c | 20
From: Boyuan Zhang
For vcn 5_0_0, add ip_block for each vcn instance during discovery stage.
And only powering on/off one of the vcn instance using the
instance value stored in ip_block, instead of powering on/off all
vcn instances. Modify the existing functions to use the instance value
in ip_b
From: Boyuan Zhang
For vcn 4_0_3, add ip_block for each vcn instance during discovery stage.
And only powering on/off one of the vcn instance using the
instance value stored in ip_block, instead of powering on/off all
vcn instances. Modify the existing functions to use the instance value
in ip_b
From: Boyuan Zhang
Perform wait_for_idle only for the instance of the current vcn IP block,
instead of perform it for all vcn instances.
Signed-off-by: Boyuan Zhang
---
drivers/gpu/drm/amd/amdgpu/vcn_v2_5.c | 18 +-
drivers/gpu/drm/amd/amdgpu/vcn_v3_0.c | 17 ---
From: Boyuan Zhang
Perform print_ip_state only for the instance of the current vcn IP block,
instead of perform it for all vcn instances.
Signed-off-by: Boyuan Zhang
---
drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c | 33 -
drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c | 33 +++
On Thu, Oct 17, 2024 at 3:23 AM Prike Liang wrote:
>
> Need to dereference the atcs acpi buffer after
> the method is executed, otherwise it will result in
> a memory leak.
>
> Signed-off-by: Prike Liang
Reviewed-by: Alex Deucher
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c | 1 +
> 1 fil
[Public]
Hi all,
This week this patchset was tested on 4 systems, two dGPU and two APU based,
and tested across multiple display and connection types.
APU
* Single Display eDP -> 1080p 60hz, 2560x1600 120hz, 1920x1200 165hz
* Single Display DP -> 4k144hz, 4k240hz
* Multi
[Public]
The series
Reviewed-by: Roman Li
> -Original Message-
> From: SHANMUGAM, SRINIVASAN
> Sent: Thursday, October 17, 2024 2:56 AM
> To: Siqueira, Rodrigo ; Pillai, Aurabindo
>
> Cc: amd-gfx@lists.freedesktop.org; SHANMUGAM, SRINIVASAN
> ; Chung, ChiaHsuan (Tom)
> ; Li, Roman ; Hu
This commit adds the cleaner shader microcode for GFX9.4.2 GPUs. The
cleaner shader is a piece of GPU code that is used to clear or
initialize certain GPU resources, such as Local Data Share (LDS), Vector
General Purpose Registers (VGPRs), and Scalar General Purpose Registers
(SGPRs).
Clearing the
On 2024-10-17 12:12, Shaoyun Liu wrote:
From: shaoyunl
Add back kfd queues in start scheduling that originally been
removed on stop scheduling.
Signed-off-by: Shaoyun Liu
Reviewed-by: Felix Kuehling
---
.../drm/amd/amdkfd/kfd_device_queue_manager.c | 40 +--
1 file ch
On 10/16/2024 22:34, Rodrigo Siqueira wrote:
The amdgpu driver is composed of multiple components, each of which can
be a source of some specific problem that the user/developer can see.
This commit introduces steps to narrow down and collect display
information.
Cc: Leo Li
Cc: Aurabindo Pillai
On 2024-10-15 17:21, Xiaogang.Chen wrote:
From: Xiaogang Chen
The purpose of this patch is having kfd driver function as expected during AMD
gpu device plug/unplug.
When an AMD gpu device got unplug kfd driver stops all queues from this device.
If there are user processes still ref the rende
On 2024-10-17 12:12, Shaoyun Liu wrote:
From: shaoyunl
Add back kfd queues in start scheduling that originally been
removed on stop scheduling.
Signed-off-by: Shaoyun Liu
---
.../drm/amd/amdkfd/kfd_device_queue_manager.c | 40 +--
1 file change
compute/gfx may have multiple rings on some hardware.
In some cases, userspace wants to run jobs on a specific ring for validation
purposes.
This debugfs entry helps to disable or enable submitting jobs to a specific
ring.
This entry is populated only if there are at least two or more cores in th
Userspace wants to run jobs on a specific sdma ring for verification purposes.
This debugfs entry helps to disable or enable submitting jobs to a specific
ring.
This entry is populated only if there are at least two or more cores in the
sdma ip.
Signed-off-by: Jesse Zhang
Suggested-by:Alex Deuc
On 10/18/2024 1:10 AM, Dan Carpenter wrote:
> This was supposed to be an unlock instead of a lock. The original
> code will lead to a deadlock.
>
> Fixes: ee52489d1210 ("drm/amdgpu: Place NPS mode request on unload")
> Signed-off-by: Dan Carpenter
Thanks, this is being taken care with a foll
On a hive, NPS request is placed by the first one for all devices in the
hive. If the request fails, mark the mode as UNKNOWN so that subsequent
devices on unload don't request it. Also, fix the mutex double lock
issue in error condition, should have been mutex_unlock.
Signed-off-by: Lijo Lazar
R
VCN shared memory is in framebuffer and there are some flags initialized
during sw_init. Ideally, such programming should be during hw_init.
Make sure the flags are saved during reset on initialization since that
reset will affect frame buffer region. For clarity, separate it out to
another functi
[AMD Official Use Only - AMD Internal Distribution Only]
Ping. Please help review.
Thanks,
Victor
-Original Message-
From: Victor Zhao
Sent: Thursday, October 17, 2024 4:35 PM
To: amd-gfx@lists.freedesktop.org
Cc: Zhao, Victor
Subject: [PATCH] drm/amdkfd: fix the hang caused by the wri
From: shaoyunl
Add back kfd queues in start scheduling that originally been
removed on stop scheduling.
Signed-off-by: Shaoyun Liu
---
.../drm/amd/amdkfd/kfd_device_queue_manager.c | 40 +--
1 file changed, 37 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdk
On Thu, Oct 17, 2024 at 2:49 AM Srinivasan Shanmugam
wrote:
>
> This patch adds support for the cleaner shader feature onto GFX9.4.2
> hardware, as part of the leftover locals feature. The cleaner shader is
> important for ensuring process isolation and maintaining KGD/KFD
> serialization by clear
Before making a function call to wait_for_idle,
validate the function pointer like we do in sw_init.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
Remove the dummy soft_reset functions for all
ip blocks.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c | 6 --
drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c | 6 --
drivers/gpu/drm/amd/amdgpu/amdgpu_vkms.c | 6 --
drivers/gpu/drm/amd/amdg
Use the helper function amdgpu_ip_block_resume where
same checks and calls are repeated.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu.h| 2 ++
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 17 +
2 files changed, 19 insertions(+)
diff --git a/drivers/
We dont need to set the functions to NULL which arent
needed as global structure members are by default
set to zero or NULL for pointers.
Cc: Leo Liu
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c | 4
drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c |
Before making a function call to resume, validate
the function pointer like we do in sw_init.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/aldebaran.c | 11 --
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 24 +++--
drivers/gpu/drm/amd/amdgpu/sienna_cic
Before making a function call to suspend, validate
the function pointer like we do in sw_init.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/aldebaran.c | 9 ++---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 15 +++
drivers/gpu/drm/amd/amdgpu/amdgpu_reset.c
Remove the dummy resume functions for all
ip blocks.
Signed-off-by: Sunil Khatri
---
drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c
index 9b98b40ac4db..1383fd1644d
1 - 100 of 112 matches
Mail list logo
