Commit cdec4d36 authored by Eric Anholt's avatar Eric Anholt
Browse files

drm/vc4: Expose dma-buf fences for V3D rendering. 



This is needed for proper synchronization with display on another DRM
device (pl111 or tinydrm) with buffers produced by vc4 V3D.  Fixes the
new igt vc4_dmabuf_poll testcase, and rendering of one of the glmark2
desktop tests on pl111+vc4.

This doesn't yet introduce waits on another device's fences before
vc4's rendering/display, because I don't have testcases for them.

v2: Reuse dma_fence_free(), retitle commit message to clarify that
    it's not a full dma-buf fencing implementation yet.

Signed-off-by: default avatarEric Anholt <eric@anholt.net>
Link: http://patchwork.freedesktop.org/patch/msgid/20170412191202.22740-6-eric@anholt.net


Acked-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent ce9971de

drivers/gpu/drm/vc4/Makefile

+1 −0
Original line number Diff line number Diff line
@@ -9,6 +9,7 @@ vc4-y := \ 
	vc4_drv.o \

	vc4_dpi.o \

	vc4_dsi.o \

	vc4_fence.o \

	vc4_kms.o \

	vc4_gem.o \

	vc4_hdmi.o \

drivers/gpu/drm/vc4/vc4_bo.c

+36 −1
Original line number Diff line number Diff line
@@ -19,6 +19,8 @@ 
 * rendering can return quickly.

 */



#include <linux/dma-buf.h>



#include "vc4_drv.h"

#include "uapi/drm/vc4_drm.h"
@@ -88,6 +90,10 @@ static void vc4_bo_destroy(struct vc4_bo *bo) 


	vc4->bo_stats.num_allocated--;

	vc4->bo_stats.size_allocated -= obj->size;



	if (bo->resv == &bo->_resv)

		reservation_object_fini(bo->resv);



	drm_gem_cma_free_object(obj);

}
@@ -244,8 +250,12 @@ struct vc4_bo *vc4_bo_create(struct drm_device *dev, size_t unaligned_size, 
			return ERR_PTR(-ENOMEM);

		}

	}

	bo = to_vc4_bo(&cma_obj->base);



	return to_vc4_bo(&cma_obj->base);

	bo->resv = &bo->_resv;

	reservation_object_init(bo->resv);



	return bo;

}



int vc4_dumb_create(struct drm_file *file_priv,
@@ -369,6 +379,13 @@ static void vc4_bo_cache_time_timer(unsigned long data) 
	schedule_work(&vc4->bo_cache.time_work);

}



struct reservation_object *vc4_prime_res_obj(struct drm_gem_object *obj)

{

	struct vc4_bo *bo = to_vc4_bo(obj);



	return bo->resv;

}



struct dma_buf *

vc4_prime_export(struct drm_device *dev, struct drm_gem_object *obj, int flags)

{
@@ -440,6 +457,24 @@ void *vc4_prime_vmap(struct drm_gem_object *obj) 
	return drm_gem_cma_prime_vmap(obj);

}



struct drm_gem_object *

vc4_prime_import_sg_table(struct drm_device *dev,

			  struct dma_buf_attachment *attach,

			  struct sg_table *sgt)

{

	struct drm_gem_object *obj;

	struct vc4_bo *bo;



	obj = drm_gem_cma_prime_import_sg_table(dev, attach, sgt);

	if (IS_ERR(obj))

		return obj;



	bo = to_vc4_bo(obj);

	bo->resv = attach->dmabuf->resv;



	return obj;

}



int vc4_create_bo_ioctl(struct drm_device *dev, void *data,

			struct drm_file *file_priv)

{

drivers/gpu/drm/vc4/vc4_drv.c

+2 −1
Original line number Diff line number Diff line
@@ -168,8 +168,9 @@ static struct drm_driver vc4_drm_driver = { 
	.prime_fd_to_handle = drm_gem_prime_fd_to_handle,

	.gem_prime_import = drm_gem_prime_import,

	.gem_prime_export = vc4_prime_export,

	.gem_prime_res_obj = vc4_prime_res_obj,

	.gem_prime_get_sg_table	= drm_gem_cma_prime_get_sg_table,

	.gem_prime_import_sg_table = drm_gem_cma_prime_import_sg_table,

	.gem_prime_import_sg_table = vc4_prime_import_sg_table,

	.gem_prime_vmap = vc4_prime_vmap,

	.gem_prime_vunmap = drm_gem_cma_prime_vunmap,

	.gem_prime_mmap = vc4_prime_mmap,

drivers/gpu/drm/vc4/vc4_drv.h

+30 −0
Original line number Diff line number Diff line
@@ -8,7 +8,9 @@ 


#include "drmP.h"

#include "drm_gem_cma_helper.h"

#include "drm_gem_cma_helper.h"



#include <linux/reservation.h>

#include <drm/drm_encoder.h>



struct vc4_dev {
@@ -56,6 +58,8 @@ struct vc4_dev { 
	/* Protects bo_cache and the BO stats. */

	struct mutex bo_lock;



	uint64_t dma_fence_context;



	/* Sequence number for the last job queued in bin_job_list.

	 * Starts at 0 (no jobs emitted).

	 */
@@ -150,6 +154,10 @@ struct vc4_bo { 
	 * DRM_IOCTL_VC4_CREATE_SHADER_BO.

	 */

	struct vc4_validated_shader_info *validated_shader;



	/* normally (resv == &_resv) except for imported bo's */

	struct reservation_object *resv;

	struct reservation_object _resv;

};



static inline struct vc4_bo *
@@ -158,6 +166,19 @@ to_vc4_bo(struct drm_gem_object *bo) 
	return (struct vc4_bo *)bo;

}



struct vc4_fence {

	struct dma_fence base;

	struct drm_device *dev;

	/* vc4 seqno for signaled() test */

	uint64_t seqno;

};



static inline struct vc4_fence *

to_vc4_fence(struct dma_fence *fence)

{

	return (struct vc4_fence *)fence;

}



struct vc4_seqno_cb {

	struct work_struct work;

	uint64_t seqno;
@@ -230,6 +251,8 @@ struct vc4_exec_info { 
	/* Latest write_seqno of any BO that binning depends on. */

	uint64_t bin_dep_seqno;



	struct dma_fence *fence;



	/* Last current addresses the hardware was processing when the

	 * hangcheck timer checked on us.

	 */
@@ -436,7 +459,11 @@ int vc4_mmap_bo_ioctl(struct drm_device *dev, void *data, 
int vc4_get_hang_state_ioctl(struct drm_device *dev, void *data,

			     struct drm_file *file_priv);

int vc4_mmap(struct file *filp, struct vm_area_struct *vma);

struct reservation_object *vc4_prime_res_obj(struct drm_gem_object *obj);

int vc4_prime_mmap(struct drm_gem_object *obj, struct vm_area_struct *vma);

struct drm_gem_object *vc4_prime_import_sg_table(struct drm_device *dev,

						 struct dma_buf_attachment *attach,

						 struct sg_table *sgt);

void *vc4_prime_vmap(struct drm_gem_object *obj);

void vc4_bo_cache_init(struct drm_device *dev);

void vc4_bo_cache_destroy(struct drm_device *dev);
@@ -468,6 +495,9 @@ int vc4_dpi_debugfs_regs(struct seq_file *m, void *unused); 
extern struct platform_driver vc4_dsi_driver;

int vc4_dsi_debugfs_regs(struct seq_file *m, void *unused);



/* vc4_fence.c */

extern const struct dma_fence_ops vc4_fence_ops;



/* vc4_gem.c */

void vc4_gem_init(struct drm_device *dev);

void vc4_gem_destroy(struct drm_device *dev);

drivers/gpu/drm/vc4/vc4_fence.c

0 → 100644
+56 −0
Original line number Diff line number Diff line 
/*

 * Copyright © 2017 Broadcom

 *

 * Permission is hereby granted, free of charge, to any person obtaining a

 * copy of this software and associated documentation files (the "Software"),

 * to deal in the Software without restriction, including without limitation

 * the rights to use, copy, modify, merge, publish, distribute, sublicense,

 * and/or sell copies of the Software, and to permit persons to whom the

 * Software is furnished to do so, subject to the following conditions:

 *

 * The above copyright notice and this permission notice (including the next

 * paragraph) shall be included in all copies or substantial portions of the

 * Software.

 *

 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR

 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,

 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL

 * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER

 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING

 * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS

 * IN THE SOFTWARE.

 */



#include "vc4_drv.h"



static const char *vc4_fence_get_driver_name(struct dma_fence *fence)

{

	return "vc4";

}



static const char *vc4_fence_get_timeline_name(struct dma_fence *fence)

{

	return "vc4-v3d";

}



static bool vc4_fence_enable_signaling(struct dma_fence *fence)

{

	return true;

}



static bool vc4_fence_signaled(struct dma_fence *fence)

{

	struct vc4_fence *f = to_vc4_fence(fence);

	struct vc4_dev *vc4 = to_vc4_dev(f->dev);



	return vc4->finished_seqno >= f->seqno;

}



const struct dma_fence_ops vc4_fence_ops = {

	.get_driver_name = vc4_fence_get_driver_name,

	.get_timeline_name = vc4_fence_get_timeline_name,

	.enable_signaling = vc4_fence_enable_signaling,

	.signaled = vc4_fence_signaled,

	.wait = dma_fence_default_wait,

	.release = dma_fence_free,

};