Commit a0a5ab3e authored by Lucas Stach's avatar Lucas Stach

drm/etnaviv: call correct function when trying to vmap a DMABUF

When trying to get the vmap address of an imported buffer, we must
call into the appropriate helper function, to allow the exporter to
establish the vmap, instead of trying to vmap the buffer on our own.

Add an indirection through etnaviv_gem_ops to allow the correct
implementation to be called.
Signed-off-by: default avatarLucas Stach <l.stach@pengutronix.de>
parent ce3088fd
...@@ -357,23 +357,35 @@ void *etnaviv_gem_vmap(struct drm_gem_object *obj) ...@@ -357,23 +357,35 @@ void *etnaviv_gem_vmap(struct drm_gem_object *obj)
{ {
struct etnaviv_gem_object *etnaviv_obj = to_etnaviv_bo(obj); struct etnaviv_gem_object *etnaviv_obj = to_etnaviv_bo(obj);
mutex_lock(&etnaviv_obj->lock); if (etnaviv_obj->vaddr)
if (!etnaviv_obj->vaddr) { return etnaviv_obj->vaddr;
struct page **pages = etnaviv_gem_get_pages(etnaviv_obj);
if (IS_ERR(pages)) {
mutex_unlock(&etnaviv_obj->lock);
return NULL;
}
etnaviv_obj->vaddr = vmap(pages, obj->size >> PAGE_SHIFT, mutex_lock(&etnaviv_obj->lock);
VM_MAP, pgprot_writecombine(PAGE_KERNEL)); /*
} * Need to check again, as we might have raced with another thread
* while waiting for the mutex.
*/
if (!etnaviv_obj->vaddr)
etnaviv_obj->vaddr = etnaviv_obj->ops->vmap(etnaviv_obj);
mutex_unlock(&etnaviv_obj->lock); mutex_unlock(&etnaviv_obj->lock);
return etnaviv_obj->vaddr; return etnaviv_obj->vaddr;
} }
static void *etnaviv_gem_vmap_impl(struct etnaviv_gem_object *obj)
{
struct page **pages;
lockdep_assert_held(&obj->lock);
pages = etnaviv_gem_get_pages(obj);
if (IS_ERR(pages))
return NULL;
return vmap(pages, obj->base.size >> PAGE_SHIFT,
VM_MAP, pgprot_writecombine(PAGE_KERNEL));
}
static inline enum dma_data_direction etnaviv_op_to_dma_dir(u32 op) static inline enum dma_data_direction etnaviv_op_to_dma_dir(u32 op)
{ {
if (op & ETNA_PREP_READ) if (op & ETNA_PREP_READ)
...@@ -524,6 +536,7 @@ static void etnaviv_gem_shmem_release(struct etnaviv_gem_object *etnaviv_obj) ...@@ -524,6 +536,7 @@ static void etnaviv_gem_shmem_release(struct etnaviv_gem_object *etnaviv_obj)
static const struct etnaviv_gem_ops etnaviv_gem_shmem_ops = { static const struct etnaviv_gem_ops etnaviv_gem_shmem_ops = {
.get_pages = etnaviv_gem_shmem_get_pages, .get_pages = etnaviv_gem_shmem_get_pages,
.release = etnaviv_gem_shmem_release, .release = etnaviv_gem_shmem_release,
.vmap = etnaviv_gem_vmap_impl,
}; };
void etnaviv_gem_free_object(struct drm_gem_object *obj) void etnaviv_gem_free_object(struct drm_gem_object *obj)
...@@ -868,6 +881,7 @@ static void etnaviv_gem_userptr_release(struct etnaviv_gem_object *etnaviv_obj) ...@@ -868,6 +881,7 @@ static void etnaviv_gem_userptr_release(struct etnaviv_gem_object *etnaviv_obj)
static const struct etnaviv_gem_ops etnaviv_gem_userptr_ops = { static const struct etnaviv_gem_ops etnaviv_gem_userptr_ops = {
.get_pages = etnaviv_gem_userptr_get_pages, .get_pages = etnaviv_gem_userptr_get_pages,
.release = etnaviv_gem_userptr_release, .release = etnaviv_gem_userptr_release,
.vmap = etnaviv_gem_vmap_impl,
}; };
int etnaviv_gem_new_userptr(struct drm_device *dev, struct drm_file *file, int etnaviv_gem_new_userptr(struct drm_device *dev, struct drm_file *file,
......
...@@ -78,6 +78,7 @@ struct etnaviv_gem_object *to_etnaviv_bo(struct drm_gem_object *obj) ...@@ -78,6 +78,7 @@ struct etnaviv_gem_object *to_etnaviv_bo(struct drm_gem_object *obj)
struct etnaviv_gem_ops { struct etnaviv_gem_ops {
int (*get_pages)(struct etnaviv_gem_object *); int (*get_pages)(struct etnaviv_gem_object *);
void (*release)(struct etnaviv_gem_object *); void (*release)(struct etnaviv_gem_object *);
void *(*vmap)(struct etnaviv_gem_object *);
}; };
static inline bool is_active(struct etnaviv_gem_object *etnaviv_obj) static inline bool is_active(struct etnaviv_gem_object *etnaviv_obj)
......
...@@ -77,9 +77,17 @@ static void etnaviv_gem_prime_release(struct etnaviv_gem_object *etnaviv_obj) ...@@ -77,9 +77,17 @@ static void etnaviv_gem_prime_release(struct etnaviv_gem_object *etnaviv_obj)
drm_prime_gem_destroy(&etnaviv_obj->base, etnaviv_obj->sgt); drm_prime_gem_destroy(&etnaviv_obj->base, etnaviv_obj->sgt);
} }
static void *etnaviv_gem_prime_vmap_impl(struct etnaviv_gem_object *etnaviv_obj)
{
lockdep_assert_held(&etnaviv_obj->lock);
return dma_buf_vmap(etnaviv_obj->base.import_attach->dmabuf);
}
static const struct etnaviv_gem_ops etnaviv_gem_prime_ops = { static const struct etnaviv_gem_ops etnaviv_gem_prime_ops = {
/* .get_pages should never be called */ /* .get_pages should never be called */
.release = etnaviv_gem_prime_release, .release = etnaviv_gem_prime_release,
.vmap = etnaviv_gem_prime_vmap_impl,
}; };
struct drm_gem_object *etnaviv_gem_prime_import_sg_table(struct drm_device *dev, struct drm_gem_object *etnaviv_gem_prime_import_sg_table(struct drm_device *dev,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment