Re: [PATCH v2 0/5] Add reservation_object to drm_gem_object

From: Daniel Vetter <daniel-/w4YWyX8dFk@public.gmane.org>
To: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Cc: Sean Paul <sean-p7yTbzM4H96eqtR555YLDQ@public.gmane.org>,
	Rob Clark <robdclark-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Maxime Ripard
	<maxime.ripard-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org>,
	David Airlie <airlied-cv59FeDIM0c@public.gmane.org>,
	linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Maarten Lankhorst
	<maarten.lankhorst-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>,
	etnaviv-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
	Eric Anholt <eric-WhKQ6XTQaPysTnJN9+BGXg@public.gmane.org>,
	Christian Gmeiner
	<christian.gmeiner-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Daniel Vetter <daniel-/w4YWyX8dFk@public.gmane.org>,
	Russell King
	<linux+etnaviv-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org>,
	freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
	Lucas Stach <l.stach-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
Subject: Re: [PATCH v2 0/5] Add reservation_object to drm_gem_object
Date: Tue, 12 Feb 2019 15:22:01 +0100	[thread overview]
Message-ID: <20190212142201.GD23159@phenom.ffwll.local> (raw)
In-Reply-To: <20190202154158.10443-1-robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>

On Sat, Feb 02, 2019 at 09:41:53AM -0600, Rob Herring wrote:
> This series implements the todo to add reservation_object to 
> drm_gem_object. I converted the easy drivers, but not Intel or AMD. The 
> series is build tested only.
> 
> Rob
> 
> v2:
>  - Fix missing kerneldoc
>  - Reword todo with what is left todo.
>  - Fix timeout error handling (added to drm_gem_reservation_object_wait)

I like. On the series:

Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>

for msm just ping Sean Paul on irc, he's usually quick with taking a lock.
And I'd merge the entire series through drm-misc, least amount of trouble.
-Daniel

> 
> Rob Herring (5):
>   drm: Add reservation_object to drm_gem_object
>   drm: etnaviv: Switch to use drm_gem_object reservation_object
>   drm: msm: Switch to use drm_gem_object reservation_object
>   drm: v3d: Switch to use drm_gem_object reservation_object
>   drm: vc4: Switch to use drm_gem_object reservation_object
> 
>  Documentation/gpu/todo.rst                   |  8 ++--
>  drivers/gpu/drm/drm_gem.c                    | 43 ++++++++++++++++++++
>  drivers/gpu/drm/drm_prime.c                  |  1 +
>  drivers/gpu/drm/etnaviv/etnaviv_drv.c        |  1 -
>  drivers/gpu/drm/etnaviv/etnaviv_drv.h        |  1 -
>  drivers/gpu/drm/etnaviv/etnaviv_gem.c        | 16 +++-----
>  drivers/gpu/drm/etnaviv/etnaviv_gem.h        |  4 --
>  drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c  |  7 ----
>  drivers/gpu/drm/etnaviv/etnaviv_gem_submit.c | 22 +++++-----
>  drivers/gpu/drm/msm/msm_drv.c                |  1 -
>  drivers/gpu/drm/msm/msm_drv.h                |  1 -
>  drivers/gpu/drm/msm/msm_gem.c                | 27 ++++--------
>  drivers/gpu/drm/msm/msm_gem_prime.c          |  7 ----
>  drivers/gpu/drm/msm/msm_gem_submit.c         |  8 ++--
>  drivers/gpu/drm/v3d/v3d_bo.c                 | 14 +------
>  drivers/gpu/drm/v3d/v3d_drv.c                |  1 -
>  drivers/gpu/drm/v3d/v3d_drv.h                |  6 ---
>  drivers/gpu/drm/v3d/v3d_gem.c                | 35 ++++------------
>  drivers/gpu/drm/vc4/vc4_bo.c                 | 15 +------
>  drivers/gpu/drm/vc4/vc4_drv.c                |  1 -
>  drivers/gpu/drm/vc4/vc4_drv.h                |  6 ---
>  drivers/gpu/drm/vc4/vc4_gem.c                | 18 ++++----
>  drivers/gpu/drm/vc4/vc4_plane.c              |  2 +-
>  include/drm/drm_gem.h                        | 21 ++++++++++
>  24 files changed, 119 insertions(+), 147 deletions(-)
> 
> -- 
> 2.19.1
> 

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno