Yeah, but Vega10 has UVD7 and in uvd_v7_0.c we have:
> static void uvd_v7_0_ring_emit_hdp_invalidate(struct amdgpu_ring *ring)
> {
> amdgpu_ring_write(ring, PACKET0(SOC15_REG_OFFSET(HDP, 0,
> mmHDP_DEBUG0), 0));
> amdgpu_ring_write(ring, 1);
> }
That should probably be fixed as well.
Regards,
Christian.
Am 18.09.2017 um 13:03 schrieb Liu, Monk:
> Only vega10 has this register
>
> -----Original Message-----
> From: Christian König [mailto:ckoenig.leichtzumerken at gmail.com]
> Sent: 2017å¹´9æ??18æ?¥ 17:20
> To: Liu, Monk <Monk.Liu at amd.com>; amd-gfx at lists.freedesktop.org
> Subject: Re: [PATCH 12/18] drm/amdgpu:use formal register to trigger hdp invalidate
>
> Am 18.09.2017 um 08:11 schrieb Monk Liu:
>> Change-Id: I61dc02ea6a450f9acfa3bae07aa20244261f5369
>> Signed-off-by: Monk Liu <Monk.Liu at amd.com>
> Reviewed-by: Christian König <christian.koenig at amd.com>
>
> Please scan the code once more, we most likely have used mmHDP_DEBUG0 for this at even more places.
>
> Christian.
>
>> ---
>> drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 2 +-
>> drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c | 2 +-
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
>> b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
>> index f201510..44960b3 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
>> @@ -3549,7 +3549,7 @@ static void gfx_v9_0_ring_emit_hdp_flush(struct amdgpu_ring *ring)
>> static void gfx_v9_0_ring_emit_hdp_invalidate(struct amdgpu_ring *ring)
>> {
>> gfx_v9_0_write_data_to_reg(ring, 0, true,
>> - SOC15_REG_OFFSET(HDP, 0, mmHDP_DEBUG0), 1);
>> + SOC15_REG_OFFSET(HDP, 0, mmHDP_READ_CACHE_INVALIDATE), 1);
>> }
>>
>> static void gfx_v9_0_ring_emit_ib_gfx(struct amdgpu_ring *ring, diff
>> --git a/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
>> b/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
>> index fd7c72a..d5f3848 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
>> @@ -398,7 +398,7 @@ static void sdma_v4_0_ring_emit_hdp_invalidate(struct amdgpu_ring *ring)
>> {
>> amdgpu_ring_write(ring, SDMA_PKT_HEADER_OP(SDMA_OP_SRBM_WRITE) |
>> SDMA_PKT_SRBM_WRITE_HEADER_BYTE_EN(0xf));
>> - amdgpu_ring_write(ring, SOC15_REG_OFFSET(HDP, 0, mmHDP_DEBUG0));
>> + amdgpu_ring_write(ring, SOC15_REG_OFFSET(HDP, 0,
>> +mmHDP_READ_CACHE_INVALIDATE));
>> amdgpu_ring_write(ring, 1);
>> }
>>
>
