drm/amdgpu: change how we update mmRLC_SPM_MC_CNTL
In pp_one_vf mode avoid the extra overhead and read/write the registers without the KIQ. Signed-off-by: Christian König <christian.koenig@amd.com> Reviewed-by: Monk Liu <monk.liu@amd.com> Acked-by: Yintian Tao <yintian.tao@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
This commit is contained in:
committed by
Alex Deucher
parent
a891d239f9
commit
e09d40bdba
@@ -7030,13 +7030,20 @@ static int gfx_v10_0_update_gfx_clock_gating(struct amdgpu_device *adev,
|
|||||||
|
|
||||||
static void gfx_v10_0_update_spm_vmid(struct amdgpu_device *adev, unsigned vmid)
|
static void gfx_v10_0_update_spm_vmid(struct amdgpu_device *adev, unsigned vmid)
|
||||||
{
|
{
|
||||||
u32 data;
|
u32 reg, data;
|
||||||
|
|
||||||
data = RREG32_SOC15(GC, 0, mmRLC_SPM_MC_CNTL);
|
reg = SOC15_REG_OFFSET(GC, 0, mmRLC_SPM_MC_CNTL);
|
||||||
|
if (amdgpu_sriov_is_pp_one_vf(adev))
|
||||||
|
data = RREG32_NO_KIQ(reg);
|
||||||
|
else
|
||||||
|
data = RREG32(reg);
|
||||||
|
|
||||||
data &= ~RLC_SPM_MC_CNTL__RLC_SPM_VMID_MASK;
|
data &= ~RLC_SPM_MC_CNTL__RLC_SPM_VMID_MASK;
|
||||||
data |= (vmid & RLC_SPM_MC_CNTL__RLC_SPM_VMID_MASK) << RLC_SPM_MC_CNTL__RLC_SPM_VMID__SHIFT;
|
data |= (vmid & RLC_SPM_MC_CNTL__RLC_SPM_VMID_MASK) << RLC_SPM_MC_CNTL__RLC_SPM_VMID__SHIFT;
|
||||||
|
|
||||||
|
if (amdgpu_sriov_is_pp_one_vf(adev))
|
||||||
|
WREG32_SOC15_NO_KIQ(GC, 0, mmRLC_SPM_MC_CNTL, data);
|
||||||
|
else
|
||||||
WREG32_SOC15(GC, 0, mmRLC_SPM_MC_CNTL, data);
|
WREG32_SOC15(GC, 0, mmRLC_SPM_MC_CNTL, data);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -5615,11 +5615,17 @@ static void gfx_v8_0_update_spm_vmid(struct amdgpu_device *adev, unsigned vmid)
|
|||||||
{
|
{
|
||||||
u32 data;
|
u32 data;
|
||||||
|
|
||||||
|
if (amdgpu_sriov_is_pp_one_vf(adev))
|
||||||
|
data = RREG32_NO_KIQ(mmRLC_SPM_VMID);
|
||||||
|
else
|
||||||
data = RREG32(mmRLC_SPM_VMID);
|
data = RREG32(mmRLC_SPM_VMID);
|
||||||
|
|
||||||
data &= ~RLC_SPM_VMID__RLC_SPM_VMID_MASK;
|
data &= ~RLC_SPM_VMID__RLC_SPM_VMID_MASK;
|
||||||
data |= (vmid & RLC_SPM_VMID__RLC_SPM_VMID_MASK) << RLC_SPM_VMID__RLC_SPM_VMID__SHIFT;
|
data |= (vmid & RLC_SPM_VMID__RLC_SPM_VMID_MASK) << RLC_SPM_VMID__RLC_SPM_VMID__SHIFT;
|
||||||
|
|
||||||
|
if (amdgpu_sriov_is_pp_one_vf(adev))
|
||||||
|
WREG32_NO_KIQ(mmRLC_SPM_VMID, data);
|
||||||
|
else
|
||||||
WREG32(mmRLC_SPM_VMID, data);
|
WREG32(mmRLC_SPM_VMID, data);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -4950,13 +4950,20 @@ static int gfx_v9_0_update_gfx_clock_gating(struct amdgpu_device *adev,
|
|||||||
|
|
||||||
static void gfx_v9_0_update_spm_vmid(struct amdgpu_device *adev, unsigned vmid)
|
static void gfx_v9_0_update_spm_vmid(struct amdgpu_device *adev, unsigned vmid)
|
||||||
{
|
{
|
||||||
u32 data;
|
u32 reg, data;
|
||||||
|
|
||||||
data = RREG32_SOC15(GC, 0, mmRLC_SPM_MC_CNTL);
|
reg = SOC15_REG_OFFSET(GC, 0, mmRLC_SPM_MC_CNTL);
|
||||||
|
if (amdgpu_sriov_is_pp_one_vf(adev))
|
||||||
|
data = RREG32_NO_KIQ(reg);
|
||||||
|
else
|
||||||
|
data = RREG32(reg);
|
||||||
|
|
||||||
data &= ~RLC_SPM_MC_CNTL__RLC_SPM_VMID_MASK;
|
data &= ~RLC_SPM_MC_CNTL__RLC_SPM_VMID_MASK;
|
||||||
data |= (vmid & RLC_SPM_MC_CNTL__RLC_SPM_VMID_MASK) << RLC_SPM_MC_CNTL__RLC_SPM_VMID__SHIFT;
|
data |= (vmid & RLC_SPM_MC_CNTL__RLC_SPM_VMID_MASK) << RLC_SPM_MC_CNTL__RLC_SPM_VMID__SHIFT;
|
||||||
|
|
||||||
|
if (amdgpu_sriov_is_pp_one_vf(adev))
|
||||||
|
WREG32_SOC15_NO_KIQ(GC, 0, mmRLC_SPM_MC_CNTL, data);
|
||||||
|
else
|
||||||
WREG32_SOC15(GC, 0, mmRLC_SPM_MC_CNTL, data);
|
WREG32_SOC15(GC, 0, mmRLC_SPM_MC_CNTL, data);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user