diff options
| author | Jingwen Chen <Jingwen.Chen2@amd.com> | 2021-01-19 16:54:50 +0800 | 
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2021-01-25 17:46:48 -0500 | 
| commit | 64dcf2f01d59cf9fad19b1a387bd39736a8f4d69 (patch) | |
| tree | 840a300244954cba88eead9f62f54e2bb2b66100 /drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c | |
| parent | 9024fb08159ae8f3a707c9f1facb878c84aae735 (diff) | |
drm/amd/amdgpu: add error handling to amdgpu_virt_read_pf2vf_data
[Why]
when vram lost happened in guest, try to write vram can lead to
kernel stuck.
[How]
When the readback data is invalid, don't do write work, directly
reschedule a new work.
Signed-off-by: Jingwen Chen <Jingwen.Chen2@amd.com>
Reviewed-by: Monk Liu<monk.liu@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c')
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c | 6 | 
1 files changed, 5 insertions, 1 deletions
| diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c index abe538bcea90..5da04d45b637 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c @@ -560,10 +560,14 @@ static int amdgpu_virt_write_vf2pf_data(struct amdgpu_device *adev)  static void amdgpu_virt_update_vf2pf_work_item(struct work_struct *work)  {  	struct amdgpu_device *adev = container_of(work, struct amdgpu_device, virt.vf2pf_work.work); +	int ret; -	amdgpu_virt_read_pf2vf_data(adev); +	ret = amdgpu_virt_read_pf2vf_data(adev); +	if (ret) +		goto out;  	amdgpu_virt_write_vf2pf_data(adev); +out:  	schedule_delayed_work(&(adev->virt.vf2pf_work), adev->virt.vf2pf_update_interval_ms);  } | 
