]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
drm/amdgpu/sriov add amdgpu_amdkfd_pre_reset in gpu reset
authorJack Zhang <Jack.Zhang1@amd.com>
Thu, 2 Apr 2020 07:10:24 +0000 (15:10 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 1 Oct 2020 11:17:54 +0000 (13:17 +0200)
[ Upstream commit 04bef61e5da18c2b301c629a209ccdba4d4c6fbb ]

kfd_pre_reset will free mem_objs allocated by kfd_gtt_sa_allocate

Without this change, sriov tdr code path will never free those allocated
memories and get memory leak.

v2:add a bugfix for kiq ring test fail

Signed-off-by: Jack Zhang <Jack.Zhang1@amd.com>
Reviewed-by: Monk Liu <monk.liu@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c
drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v9.c
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c

index d10f483f5e2735a88fd77d84874a786933620e25..ce30d4e8bf25ff0749c648ef5e7bf6252a320c26 100644 (file)
@@ -644,6 +644,9 @@ static int kgd_hqd_destroy(struct kgd_dev *kgd, void *mqd,
        uint32_t temp;
        struct v10_compute_mqd *m = get_mqd(mqd);
 
+       if (amdgpu_sriov_vf(adev) && adev->in_gpu_reset)
+               return 0;
+
 #if 0
        unsigned long flags;
        int retry;
index e262f2ac07a3590e6360960e60248d932c71ad00..92754cfb980867288810a71197c52fe30c1249b3 100644 (file)
@@ -540,6 +540,9 @@ int kgd_gfx_v9_hqd_destroy(struct kgd_dev *kgd, void *mqd,
        uint32_t temp;
        struct v9_mqd *m = get_mqd(mqd);
 
+       if (amdgpu_sriov_vf(adev) && adev->in_gpu_reset)
+               return 0;
+
        if (adev->in_gpu_reset)
                return -EIO;
 
index 5e1dce424154718aee08e5ad34f90ba08b138f9d..4105fbf571674a533727330e694958ef7bb4b2c6 100644 (file)
@@ -3466,6 +3466,8 @@ static int amdgpu_device_reset_sriov(struct amdgpu_device *adev,
        if (r)
                return r;
 
+       amdgpu_amdkfd_pre_reset(adev);
+
        /* Resume IP prior to SMC */
        r = amdgpu_device_ip_reinit_early_sriov(adev);
        if (r)