[PATCH v2 2/2] drm/amd/amdgpu: SRIOV full reset issue with VCN

[Deng, Emily]

[AMD Official Use Only - General]

Series Reviewed-by: Emily Deng <Emily.Deng at amd.com>

>-----Original Message-----
>From: amd-gfx <amd-gfx-bounces at lists.freedesktop.org> On Behalf Of Bokun
>Zhang
>Sent: Thursday, November 30, 2023 8:21 AM
>To: amd-gfx at lists.freedesktop.org
>Cc: Zhang, Bokun <Bokun.Zhang at amd.com>
>Subject: [PATCH v2 2/2] drm/amd/amdgpu: SRIOV full reset issue with VCN
>
>- After a full reset, VF's FB will be cleaned. This
>  includes the VCN's fw_shared memory.
>
>  However, there is no suspend-resume routine for
>  SRIOV VF. Therefore, the data in the fw_shared
>  memory will be lost forever and it causes engine
>  hang later on.
>
>  We must repopulate the data in fw_shared during
>  SRIOV hw_init
>
>Signed-off-by: Bokun Zhang <Bokun.Zhang at amd.com>
>---
> drivers/gpu/drm/amd/amdgpu/vcn_v4_0.c | 3 +++
> 1 file changed, 3 insertions(+)
>
>diff --git a/drivers/gpu/drm/amd/amdgpu/vcn_v4_0.c
>b/drivers/gpu/drm/amd/amdgpu/vcn_v4_0.c
>index 54b03df63a51..b71590b67e20 100644
>--- a/drivers/gpu/drm/amd/amdgpu/vcn_v4_0.c
>+++ b/drivers/gpu/drm/amd/amdgpu/vcn_v4_0.c
>@@ -1280,6 +1280,9 @@ static int vcn_v4_0_start_sriov(struct
>amdgpu_device *adev)
>               if (adev->vcn.harvest_config & (1 << i))
>                       continue;
>
>+              // Must re/init fw_shared at beginning
>+              vcn_v4_0_fw_shared_init(adev, i);
>+
>               table_size = 0;
>
>
>       MMSCH_V4_0_INSERT_DIRECT_RD_MOD_WT(SOC15_REG_OFFSET(VC
>N, i,
>--
>2.34.1