[PATCH] drm/amdgpu: check for allocation failure in amdgpu_vkms_sw_init()

Alex Deucher alexdeucher at gmail.com
Fri Aug 6 19:57:17 UTC 2021


Applied.  Thanks!

Alex

On Fri, Aug 6, 2021 at 11:09 AM Christian König
<christian.koenig at amd.com> wrote:
>
> Am 06.08.21 um 17:05 schrieb Dan Carpenter:
> > Check whether the kcalloc() fails and return -ENOMEM if it does.
> >
> > Fixes: eeba0b9046fc ("drm/amdgpu: create amdgpu_vkms (v4)")
> > Signed-off-by: Dan Carpenter <dan.carpenter at oracle.com>
>
> Reviewed-by: Christian König <christian.koenig at amd.com>
>
> > ---
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_vkms.c | 2 ++
> >   1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vkms.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vkms.c
> > index 50bdc39733aa..ce982afeff91 100644
> > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vkms.c
> > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vkms.c
> > @@ -482,6 +482,8 @@ static int amdgpu_vkms_sw_init(void *handle)
> >               return r;
> >
> >       adev->amdgpu_vkms_output = kcalloc(adev->mode_info.num_crtc, sizeof(struct amdgpu_vkms_output), GFP_KERNEL);
>
> Is the line above not a bit long?
>
> > +     if (!adev->amdgpu_vkms_output)
> > +             return -ENOMEM;
> >
> >       /* allocate crtcs, encoders, connectors */
> >       for (i = 0; i < adev->mode_info.num_crtc; i++) {
>


More information about the amd-gfx mailing list