[PATCH 1/1] drm: ttm: Don't bail from ttm_global_init if debugfs_create_dir fails

Jason Ekstrand jason at jlekstrand.net
Mon Aug 16 14:28:39 UTC 2021


Makes sense

Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>

On Mon, Aug 16, 2021 at 2:40 AM Christian König
<christian.koenig at amd.com> wrote:
>
> Am 10.08.21 um 21:59 schrieb Dan Moulding:
> > In 69de4421bb4c ("drm/ttm: Initialize debugfs from
> > ttm_global_init()"), ttm_global_init was changed so that if creation
> > of the debugfs global root directory fails, ttm_global_init will bail
> > out early and return an error, leading to initialization failure of
> > DRM drivers. However, not every system will be using debugfs. On such
> > a system, debugfs directory creation can be expected to fail, but DRM
> > drivers must still be usable. This changes it so that if creation of
> > TTM's debugfs root directory fails, then no biggie: keep calm and
> > carry on.
> >
> > Fixes: 69de4421bb4c ("drm/ttm: Initialize debugfs from ttm_global_init()")
> > Signed-off-by: Dan Moulding <dmoulding at me.com>
>
> Good point, patch is Reviewed-by: Christian König
> <christian.koenig at amd.com>.
>
> Going to pick that up later today.
>
> Regards,
> Christian.
>
> > ---
> >   drivers/gpu/drm/ttm/ttm_device.c | 2 --
> >   1 file changed, 2 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/ttm/ttm_device.c b/drivers/gpu/drm/ttm/ttm_device.c
> > index 74e3b460132b..2df59b3c2ea1 100644
> > --- a/drivers/gpu/drm/ttm/ttm_device.c
> > +++ b/drivers/gpu/drm/ttm/ttm_device.c
> > @@ -78,9 +78,7 @@ static int ttm_global_init(void)
> >
> >       ttm_debugfs_root = debugfs_create_dir("ttm", NULL);
> >       if (IS_ERR(ttm_debugfs_root)) {
> > -             ret = PTR_ERR(ttm_debugfs_root);
> >               ttm_debugfs_root = NULL;
> > -             goto out;
> >       }
> >
> >       /* Limit the number of pages in the pool to about 50% of the total
>


More information about the dri-devel mailing list