[PATCH] drm/etnaviv: clean up etnaviv_gem_new_handle

Christian Gmeiner christian.gmeiner at gmail.com
Mon Apr 15 14:20:56 UTC 2019


Am Mo., 15. Apr. 2019 um 15:12 Uhr schrieb Lucas Stach <l.stach at pengutronix.de>:
>
> Setting the GFP flags does not need a new code block if moved to
> the right location, which makes this function a bit easier to read.
>
> Signed-off-by: Lucas Stach <l.stach at pengutronix.de>

Reviewed-by: Christian Gmeiner <christian.gmeiner at gmail.com>

> ---
>  drivers/gpu/drm/etnaviv/etnaviv_gem.c | 24 +++++++++---------------
>  1 file changed, 9 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/gpu/drm/etnaviv/etnaviv_gem.c b/drivers/gpu/drm/etnaviv/etnaviv_gem.c
> index 1fa74226db91..9ac9da1a769d 100644
> --- a/drivers/gpu/drm/etnaviv/etnaviv_gem.c
> +++ b/drivers/gpu/drm/etnaviv/etnaviv_gem.c
> @@ -628,24 +628,18 @@ int etnaviv_gem_new_handle(struct drm_device *dev, struct drm_file *file,
>         lockdep_set_class(&to_etnaviv_bo(obj)->lock, &etnaviv_shm_lock_class);
>
>         ret = drm_gem_object_init(dev, obj, size);
> -       if (ret == 0) {
> -               struct address_space *mapping;
> -
> -               /*
> -                * Our buffers are kept pinned, so allocating them
> -                * from the MOVABLE zone is a really bad idea, and
> -                * conflicts with CMA. See comments above new_inode()
> -                * why this is required _and_ expected if you're
> -                * going to pin these pages.
> -                */
> -               mapping = obj->filp->f_mapping;
> -               mapping_set_gfp_mask(mapping, GFP_HIGHUSER |
> -                                    __GFP_RETRY_MAYFAIL | __GFP_NOWARN);
> -       }
> -
>         if (ret)
>                 goto fail;
>
> +       /*
> +        * Our buffers are kept pinned, so allocating them from the MOVABLE
> +        * zone is a really bad idea, and conflicts with CMA. See comments
> +        * above new_inode() why this is required _and_ expected if you're
> +        * going to pin these pages.
> +        */
> +       mapping_set_gfp_mask(obj->filp->f_mapping, GFP_HIGHUSER |
> +                            __GFP_RETRY_MAYFAIL | __GFP_NOWARN);
> +
>         etnaviv_gem_obj_add(dev, obj);
>
>         ret = drm_gem_handle_create(file, obj, handle);
> --
> 2.20.1
>


-- 
greets
--
Christian Gmeiner, MSc

https://christian-gmeiner.info


More information about the etnaviv mailing list