Re: [PATCH 17/24] drm/vgem: switch to postclose

Sean Paul
2017-03-13T19:11:07Z

On Wed, Mar 08, 2017 at 03:12:50PM +0100, Daniel Vetter wrote:
> I didn't spot anything that would require ordering here (well not
> anywhere else either), and I'm trying to unify at least modern drivers
> on one close hook.
>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

Reviewed-by: Sean Paul <seanpaul@chromium.org>

> ---
> drivers/gpu/drm/vgem/vgem_drv.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/vgem/vgem_drv.c b/drivers/gpu/drm/vgem/vgem_drv.c
> index a1f42d125e6e..9fee38a942c4 100644
> --- a/drivers/gpu/drm/vgem/vgem_drv.c
> +++ b/drivers/gpu/drm/vgem/vgem_drv.c
> @@ -104,7 +104,7 @@ static int vgem_open(struct drm_device *dev, struct drm_file *file)
> return 0;
> }
>
> -static void vgem_preclose(struct drm_device *dev, struct drm_file *file)
> +static void vgem_postclose(struct drm_device *dev, struct drm_file *file)
> {
> struct vgem_file *vfile = file->driver_priv;
>
> @@ -303,7 +303,7 @@ static int vgem_prime_mmap(struct drm_gem_object *obj,
> static struct drm_driver vgem_driver = {
> .driver_features = DRIVER_GEM | DRIVER_PRIME,
> .open = vgem_open,
> - .preclose = vgem_preclose,
> + .postclose = vgem_postclose,
> .gem_free_object_unlocked = vgem_gem_free_object,
> .gem_vm_ops = &vgem_gem_vm_ops,
> .ioctls = vgem_ioctls,
> --
> 2.11.0
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org