* [PATCH] drm/amdgpu/vcn: fix vcn ring test failure in igt reload test
@ 2022-03-15 2:33 Tianci Yin
2022-03-15 9:25 ` Paul Menzel
2022-03-15 12:01 ` James Zhu
0 siblings, 2 replies; 4+ messages in thread
From: Tianci Yin @ 2022-03-15 2:33 UTC (permalink / raw)
To: amd-gfx; +Cc: Wang Yu, Zhu James, Tianci Yin, Guchun Chen
From: "Tianci.Yin" <tianci.yin@amd.com>
[why]
On Renoir, vcn ring test failed on the second time insmod in the reload
test. After invetigation, it proves that vcn only can disable dpg under
dpg unpause mode (dpg unpause mode is default for dec only, dpg pause
mode is for dec/enc).
[how]
unpause dpg in dpg stopping procedure.
Change-Id: If6ec3af694e1d6b63ebce386a563f03ca6d291c1
Signed-off-by: Tianci.Yin <tianci.yin@amd.com>
---
drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c b/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c
index 319ac8ea434b..6e0972cd1f2f 100644
--- a/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c
@@ -1098,8 +1098,10 @@ static int vcn_v2_0_start(struct amdgpu_device *adev)
static int vcn_v2_0_stop_dpg_mode(struct amdgpu_device *adev)
{
+ struct dpg_pause_state state = {.fw_based = VCN_DPG_STATE__UNPAUSE};
uint32_t tmp;
+ vcn_v2_0_pause_dpg_mode(adev, 0, &state);
/* Wait for power status to be 1 */
SOC15_WAIT_ON_RREG(UVD, 0, mmUVD_POWER_STATUS, 1,
UVD_POWER_STATUS__UVD_POWER_STATUS_MASK);
--
2.25.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] drm/amdgpu/vcn: fix vcn ring test failure in igt reload test
2022-03-15 2:33 [PATCH] drm/amdgpu/vcn: fix vcn ring test failure in igt reload test Tianci Yin
@ 2022-03-15 9:25 ` Paul Menzel
2022-03-15 12:01 ` James Zhu
1 sibling, 0 replies; 4+ messages in thread
From: Paul Menzel @ 2022-03-15 9:25 UTC (permalink / raw)
To: Tianci Yin; +Cc: Wang Yu, Zhu James, Guchun Chen, amd-gfx
Dear Tianci,
Am 15.03.22 um 03:33 schrieb Tianci Yin:
> From: "Tianci.Yin" <tianci.yin@amd.com>
Please remove the dot/period in the name. (`git config --global
user.name "Tianci Yin"`.
> [why]
> On Renoir, vcn ring test failed on the second time insmod in the reload
> test.
Does Linux log a warning/error? Please paste it.
> After invetigation, it proves that vcn only can disable dpg under
investigation
> dpg unpause mode (dpg unpause mode is default for dec only, dpg pause
> mode is for dec/enc).
Is that documented in a datasheet? Please mention name, revision, and
section.
Why is this only needed for VCN v2.0 and not the other implementations?
> [how]
> unpause dpg in dpg stopping procedure.
>
> Change-Id: If6ec3af694e1d6b63ebce386a563f03ca6d291c1
Remove?
> Signed-off-by: Tianci.Yin <tianci.yin@amd.com>
Ditto.
How did you test this exactly? Did you turn on DRM debug messages to
verify that the function was called?
> ---
> drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c b/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c
> index 319ac8ea434b..6e0972cd1f2f 100644
> --- a/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c
> @@ -1098,8 +1098,10 @@ static int vcn_v2_0_start(struct amdgpu_device *adev)
>
> static int vcn_v2_0_stop_dpg_mode(struct amdgpu_device *adev)
> {
> + struct dpg_pause_state state = {.fw_based = VCN_DPG_STATE__UNPAUSE};
> uint32_t tmp;
>
> + vcn_v2_0_pause_dpg_mode(adev, 0, &state);
> /* Wait for power status to be 1 */
> SOC15_WAIT_ON_RREG(UVD, 0, mmUVD_POWER_STATUS, 1,
> UVD_POWER_STATUS__UVD_POWER_STATUS_MASK);
Kind regards,
Paul
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] drm/amdgpu/vcn: fix vcn ring test failure in igt reload test
2022-03-15 2:33 [PATCH] drm/amdgpu/vcn: fix vcn ring test failure in igt reload test Tianci Yin
2022-03-15 9:25 ` Paul Menzel
@ 2022-03-15 12:01 ` James Zhu
2022-03-15 14:11 ` Yin, Tianci (Rico)
1 sibling, 1 reply; 4+ messages in thread
From: James Zhu @ 2022-03-15 12:01 UTC (permalink / raw)
To: Tianci Yin, amd-gfx; +Cc: Wang Yu, Zhu James, Guchun Chen
This patch is Reviewed-by: James Zhu <James.Zhu@amd.com>
On 2022-03-14 10:33 p.m., Tianci Yin wrote:
> From: "Tianci.Yin" <tianci.yin@amd.com>
>
> [why]
> On Renoir, vcn ring test failed on the second time insmod in the reload
> test. After invetigation, it proves that vcn only can disable dpg under
> dpg unpause mode (dpg unpause mode is default for dec only, dpg pause
> mode is for dec/enc).
>
> [how]
> unpause dpg in dpg stopping procedure.
>
> Change-Id: If6ec3af694e1d6b63ebce386a563f03ca6d291c1
> Signed-off-by: Tianci.Yin <tianci.yin@amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c b/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c
> index 319ac8ea434b..6e0972cd1f2f 100644
> --- a/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c
> @@ -1098,8 +1098,10 @@ static int vcn_v2_0_start(struct amdgpu_device *adev)
>
> static int vcn_v2_0_stop_dpg_mode(struct amdgpu_device *adev)
> {
> + struct dpg_pause_state state = {.fw_based = VCN_DPG_STATE__UNPAUSE};
> uint32_t tmp;
>
> + vcn_v2_0_pause_dpg_mode(adev, 0, &state);
> /* Wait for power status to be 1 */
> SOC15_WAIT_ON_RREG(UVD, 0, mmUVD_POWER_STATUS, 1,
> UVD_POWER_STATUS__UVD_POWER_STATUS_MASK);
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] drm/amdgpu/vcn: fix vcn ring test failure in igt reload test
2022-03-15 12:01 ` James Zhu
@ 2022-03-15 14:11 ` Yin, Tianci (Rico)
0 siblings, 0 replies; 4+ messages in thread
From: Yin, Tianci (Rico) @ 2022-03-15 14:11 UTC (permalink / raw)
To: Zhu, James, amd-gfx@lists.freedesktop.org
Cc: Wang, Yu (Charlie), Chen, Guchun
[-- Attachment #1: Type: text/plain, Size: 1868 bytes --]
[AMD Official Use Only]
Thank you for reviewing James!
Regards,
Rico
________________________________
From: Zhu, James <James.Zhu@amd.com>
Sent: Tuesday, March 15, 2022 20:01
To: Yin, Tianci (Rico) <Tianci.Yin@amd.com>; amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org>
Cc: Chen, Guchun <Guchun.Chen@amd.com>; Zhu, James <James.Zhu@amd.com>; Wang, Yu (Charlie) <Yu.Wang4@amd.com>
Subject: Re: [PATCH] drm/amdgpu/vcn: fix vcn ring test failure in igt reload test
This patch is Reviewed-by: James Zhu <James.Zhu@amd.com>
On 2022-03-14 10:33 p.m., Tianci Yin wrote:
> From: "Tianci.Yin" <tianci.yin@amd.com>
>
> [why]
> On Renoir, vcn ring test failed on the second time insmod in the reload
> test. After invetigation, it proves that vcn only can disable dpg under
> dpg unpause mode (dpg unpause mode is default for dec only, dpg pause
> mode is for dec/enc).
>
> [how]
> unpause dpg in dpg stopping procedure.
>
> Change-Id: If6ec3af694e1d6b63ebce386a563f03ca6d291c1
> Signed-off-by: Tianci.Yin <tianci.yin@amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c b/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c
> index 319ac8ea434b..6e0972cd1f2f 100644
> --- a/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c
> @@ -1098,8 +1098,10 @@ static int vcn_v2_0_start(struct amdgpu_device *adev)
>
> static int vcn_v2_0_stop_dpg_mode(struct amdgpu_device *adev)
> {
> + struct dpg_pause_state state = {.fw_based = VCN_DPG_STATE__UNPAUSE};
> uint32_t tmp;
>
> + vcn_v2_0_pause_dpg_mode(adev, 0, &state);
> /* Wait for power status to be 1 */
> SOC15_WAIT_ON_RREG(UVD, 0, mmUVD_POWER_STATUS, 1,
> UVD_POWER_STATUS__UVD_POWER_STATUS_MASK);
[-- Attachment #2: Type: text/html, Size: 3616 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-03-15 14:11 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-03-15 2:33 [PATCH] drm/amdgpu/vcn: fix vcn ring test failure in igt reload test Tianci Yin
2022-03-15 9:25 ` Paul Menzel
2022-03-15 12:01 ` James Zhu
2022-03-15 14:11 ` Yin, Tianci (Rico)
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.