> -----Original Message----- > From: Kangjie Lu [mailto:[email protected]] > Sent: Wednesday, December 26, 2018 2:24 PM > To: [email protected] > Cc: [email protected]; Deucher, Alexander > <[email protected]>; Koenig, Christian > <[email protected]>; Zhou, David(ChunMing) > <[email protected]>; David Airlie <[email protected]>; Daniel Vetter > <[email protected]>; Zhu, Rex <[email protected]>; Huang, Ray > <[email protected]>; Zhang, Hawking <[email protected]>; Xu, > Feifei <[email protected]>; Gao, Likun <[email protected]>; Francis, > David <[email protected]>; [email protected]; dri- > [email protected]; [email protected] > Subject: [PATCH] gpu: drm: fix an improper check of > amdgpu_bo_create_kernel > > adev->firmware.fw_buf being not NULL may not indicate kernel buffer is > created successful. A better way is to check the status (return value) > of it. The fix does so.
Actually, it is the same. If bo is created successfully, the amdgpu_bo object will be created. But using "ret" to align with other function should be better as the return status. Thanks. Reviewed-by: Huang Rui <[email protected]> > > Signed-off-by: Kangjie Lu <[email protected]> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c | 18 ++++++++++++------ > 1 file changed, 12 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c > index 7b33867036e7..ba3c1cfb2c35 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c > @@ -422,13 +422,19 @@ static int amdgpu_ucode_patch_jt(struct > amdgpu_firmware_info *ucode, > > int amdgpu_ucode_create_bo(struct amdgpu_device *adev) > { > + int ret; > + > if (adev->firmware.load_type != AMDGPU_FW_LOAD_DIRECT) { > - amdgpu_bo_create_kernel(adev, adev->firmware.fw_size, > PAGE_SIZE, > - amdgpu_sriov_vf(adev) ? > AMDGPU_GEM_DOMAIN_VRAM : AMDGPU_GEM_DOMAIN_GTT, > - &adev->firmware.fw_buf, > - &adev->firmware.fw_buf_mc, > - &adev->firmware.fw_buf_ptr); > - if (!adev->firmware.fw_buf) { > + ret = > + amdgpu_bo_create_kernel(adev, > + adev->firmware.fw_size, > + PAGE_SIZE, > + amdgpu_sriov_vf(adev) ? > AMDGPU_GEM_DOMAIN_VRAM : > + AMDGPU_GEM_DOMAIN_GTT, > + &adev->firmware.fw_buf, > + &adev->firmware.fw_buf_mc, > + &adev->firmware.fw_buf_ptr); > + if (ret) { > dev_err(adev->dev, "failed to create kernel buffer > for firmware.fw_buf\n"); > return -ENOMEM; > } else if (amdgpu_sriov_vf(adev)) { > -- > 2.17.2 (Apple Git-113) _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
