diff options
| author | Bernard Zhao <bernard@vivo.com> | 2020-11-17 18:55:01 -0800 | 
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2020-11-24 12:04:39 -0500 | 
| commit | 95666c6cfe9247610efbd68c228f0831d4c18a51 (patch) | |
| tree | 842a47e554b2d8ea0c84aee29737be8d2bcfade1 /drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c | |
| parent | b14a04fee90889b011509fe6ade1ddf93fdd9f49 (diff) | |
amd/amdgpu: use kmalloc_array to replace kmalloc with multiply
Fix check_patch.pl warning:
WARNING: Prefer kmalloc_array over kmalloc with multiply
+bps = kmalloc(align_space * sizeof((*data)->bps), GFP_KERNEL);
WARNING: Prefer kmalloc_array over kmalloc with multiply
+bps_bo = kmalloc(align_space * sizeof((*data)->bps_bo),
GFP_KERNEL);
kmalloc_array has multiply overflow check, which will be safer.
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Bernard Zhao <bernard@vivo.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 | 4 | 
1 files changed, 2 insertions, 2 deletions
| diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c index a36c648bbe91..9cb96e4ff14c 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c @@ -280,8 +280,8 @@ static int amdgpu_virt_init_ras_err_handler_data(struct amdgpu_device *adev)  	if (!*data)  		return -ENOMEM; -	bps = kmalloc(align_space * sizeof((*data)->bps), GFP_KERNEL); -	bps_bo = kmalloc(align_space * sizeof((*data)->bps_bo), GFP_KERNEL); +	bps = kmalloc_array(align_space, sizeof((*data)->bps), GFP_KERNEL); +	bps_bo = kmalloc_array(align_space, sizeof((*data)->bps_bo), GFP_KERNEL);  	if (!bps || !bps_bo) {  		kfree(bps); | 
