mirror of
https://github.com/nxp-imx/linux-imx.git
synced 2025-07-17 22:59:37 +02:00
drm/amdgpu/vcn: not pause dpg for unified queue
commit 7d75ef3736
upstream.
For unified queue, DPG pause for encoding is done inside VCN firmware,
so there is no need to pause dpg based on ring type in kernel.
For VCN3 and below, pausing DPG for encoding in kernel is still needed.
v2: add more comments
v3: update commit message
Signed-off-by: Boyuan Zhang <boyuan.zhang@amd.com>
Acked-by: Alex Deucher <alexander.deucher@amd.com>
Reviewed-by: Ruijing Dong <ruijing.dong@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
44bb8f18a6
commit
c6372cbd91
|
@ -372,7 +372,9 @@ static void amdgpu_vcn_idle_work_handler(struct work_struct *work)
|
||||||
for (i = 0; i < adev->vcn.num_enc_rings; ++i)
|
for (i = 0; i < adev->vcn.num_enc_rings; ++i)
|
||||||
fence[j] += amdgpu_fence_count_emitted(&adev->vcn.inst[j].ring_enc[i]);
|
fence[j] += amdgpu_fence_count_emitted(&adev->vcn.inst[j].ring_enc[i]);
|
||||||
|
|
||||||
if (adev->pg_flags & AMD_PG_SUPPORT_VCN_DPG) {
|
/* Only set DPG pause for VCN3 or below, VCN4 and above will be handled by FW */
|
||||||
|
if (adev->pg_flags & AMD_PG_SUPPORT_VCN_DPG &&
|
||||||
|
!adev->vcn.using_unified_queue) {
|
||||||
struct dpg_pause_state new_state;
|
struct dpg_pause_state new_state;
|
||||||
|
|
||||||
if (fence[j] ||
|
if (fence[j] ||
|
||||||
|
@ -418,7 +420,9 @@ void amdgpu_vcn_ring_begin_use(struct amdgpu_ring *ring)
|
||||||
amdgpu_device_ip_set_powergating_state(adev, AMD_IP_BLOCK_TYPE_VCN,
|
amdgpu_device_ip_set_powergating_state(adev, AMD_IP_BLOCK_TYPE_VCN,
|
||||||
AMD_PG_STATE_UNGATE);
|
AMD_PG_STATE_UNGATE);
|
||||||
|
|
||||||
if (adev->pg_flags & AMD_PG_SUPPORT_VCN_DPG) {
|
/* Only set DPG pause for VCN3 or below, VCN4 and above will be handled by FW */
|
||||||
|
if (adev->pg_flags & AMD_PG_SUPPORT_VCN_DPG &&
|
||||||
|
!adev->vcn.using_unified_queue) {
|
||||||
struct dpg_pause_state new_state;
|
struct dpg_pause_state new_state;
|
||||||
|
|
||||||
if (ring->funcs->type == AMDGPU_RING_TYPE_VCN_ENC) {
|
if (ring->funcs->type == AMDGPU_RING_TYPE_VCN_ENC) {
|
||||||
|
@ -444,8 +448,12 @@ void amdgpu_vcn_ring_begin_use(struct amdgpu_ring *ring)
|
||||||
|
|
||||||
void amdgpu_vcn_ring_end_use(struct amdgpu_ring *ring)
|
void amdgpu_vcn_ring_end_use(struct amdgpu_ring *ring)
|
||||||
{
|
{
|
||||||
|
struct amdgpu_device *adev = ring->adev;
|
||||||
|
|
||||||
|
/* Only set DPG pause for VCN3 or below, VCN4 and above will be handled by FW */
|
||||||
if (ring->adev->pg_flags & AMD_PG_SUPPORT_VCN_DPG &&
|
if (ring->adev->pg_flags & AMD_PG_SUPPORT_VCN_DPG &&
|
||||||
ring->funcs->type == AMDGPU_RING_TYPE_VCN_ENC)
|
ring->funcs->type == AMDGPU_RING_TYPE_VCN_ENC &&
|
||||||
|
!adev->vcn.using_unified_queue)
|
||||||
atomic_dec(&ring->adev->vcn.inst[ring->me].dpg_enc_submission_cnt);
|
atomic_dec(&ring->adev->vcn.inst[ring->me].dpg_enc_submission_cnt);
|
||||||
|
|
||||||
atomic_dec(&ring->adev->vcn.total_submission_cnt);
|
atomic_dec(&ring->adev->vcn.total_submission_cnt);
|
||||||
|
|
Loading…
Reference in New Issue
Block a user