[PATCH] drm/amdgpu: check rlc_g firmware pointer is valid before using it
shaoyunl
shaoyun.liu at amd.com
Mon Jan 13 16:20:08 UTC 2020
ping.
On 2020-01-10 1:33 p.m., shaoyunl wrote:
> In SRIOV, rlc_g firmware is loaded by host, guest driver won't load it which will
> cause the rlc_fw pointer is null
>
> Change-Id: Id16f65171dd427d623af4c5bc75f674019e63dec
> Signed-off-by: shaoyunl <shaoyun.liu at amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> index 379e46c..07e22f2 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> @@ -808,10 +808,11 @@ static int gfx_v10_0_init_microcode(struct amdgpu_device *adev)
> info = &adev->firmware.ucode[AMDGPU_UCODE_ID_RLC_G];
> info->ucode_id = AMDGPU_UCODE_ID_RLC_G;
> info->fw = adev->gfx.rlc_fw;
> - header = (const struct common_firmware_header *)info->fw->data;
> - adev->firmware.fw_size +=
> - ALIGN(le32_to_cpu(header->ucode_size_bytes), PAGE_SIZE);
> -
> + if (info->fw) {
> + header = (const struct common_firmware_header *)info->fw->data;
> + adev->firmware.fw_size +=
> + ALIGN(le32_to_cpu(header->ucode_size_bytes), PAGE_SIZE);
> + }
> if (adev->gfx.rlc.is_rlc_v2_1 &&
> adev->gfx.rlc.save_restore_list_cntl_size_bytes &&
> adev->gfx.rlc.save_restore_list_gpm_size_bytes &&
More information about the amd-gfx
mailing list