On Tue, May 22, 2012 at 8:50 AM, Dave Airlie <airlied at gmail.com> wrote:
> From: Alex Deucher <alexander.deucher at amd.com>
>
> This adds prime->fd and fd->prime support to radeon.
> It passes the sg object to ttm and then populates
> the gart entries using it.
>
> Compile tested only.
>
> v2: stub kmap + use new helpers + add reimporting
>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
> Signed-off-by: Dave Airlie <airlied at redhat.com>
Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
> ---
> ?drivers/gpu/drm/radeon/Makefile ? ? ? ? ? ? | ? ?2 +-
> ?drivers/gpu/drm/radeon/evergreen_blit_kms.c | ? ?2 +-
> ?drivers/gpu/drm/radeon/r600.c ? ? ? ? ? ? ? | ? ?4 +-
> ?drivers/gpu/drm/radeon/r600_blit_kms.c ? ? ?| ? ?2 +-
> ?drivers/gpu/drm/radeon/radeon_benchmark.c ? | ? ?4 +-
> ?drivers/gpu/drm/radeon/radeon_device.c ? ? ?| ? ?2 +-
> ?drivers/gpu/drm/radeon/radeon_drv.c ? ? ? ? | ? 14 ++-
> ?drivers/gpu/drm/radeon/radeon_gart.c ? ? ? ?| ? ?2 +-
> ?drivers/gpu/drm/radeon/radeon_gem.c ? ? ? ? | ? ?4 +-
> ?drivers/gpu/drm/radeon/radeon_object.c ? ? ?| ? ?6 +-
> ?drivers/gpu/drm/radeon/radeon_object.h ? ? ?| ? ?7 +-
> ?drivers/gpu/drm/radeon/radeon_prime.c ? ? ? | ?176
> +++++++++++++++++++++++++++
> ?drivers/gpu/drm/radeon/radeon_ring.c ? ? ? ?| ? ?4 +-
> ?drivers/gpu/drm/radeon/radeon_sa.c ? ? ? ? ?| ? ?2 +-
> ?drivers/gpu/drm/radeon/radeon_test.c ? ? ? ?| ? ?4 +-
> ?drivers/gpu/drm/radeon/radeon_ttm.c ? ? ? ? | ? 16 ++-
> ?drivers/gpu/drm/radeon/si.c ? ? ? ? ? ? ? ? | ? ?6 +-
> ?17 files changed, 232 insertions(+), 25 deletions(-)
> ?create mode 100644 drivers/gpu/drm/radeon/radeon_prime.c
>
> diff --git a/drivers/gpu/drm/radeon/Makefile b/drivers/gpu/drm/radeon/Makefile
> index 1efb6eb..a6598fd 100644
> --- a/drivers/gpu/drm/radeon/Makefile
> +++ b/drivers/gpu/drm/radeon/Makefile
> @@ -72,7 +72,7 @@ radeon-y += radeon_device.o radeon_asic.o radeon_kms.o \
> ? ? ? ?evergreen.o evergreen_cs.o evergreen_blit_shaders.o
> evergreen_blit_kms.o \
> ? ? ? ?evergreen_hdmi.o radeon_trace_points.o ni.o cayman_blit_shaders.o \
> ? ? ? ?atombios_encoders.o radeon_semaphore.o radeon_sa.o atombios_i2c.o si.o
> \
> - ? ? ? si_blit_shaders.o
> + ? ? ? si_blit_shaders.o radeon_prime.o
>
> ?radeon-$(CONFIG_COMPAT) += radeon_ioc32.o
> ?radeon-$(CONFIG_VGA_SWITCHEROO) += radeon_atpx_handler.o
> diff --git a/drivers/gpu/drm/radeon/evergreen_blit_kms.c
> b/drivers/gpu/drm/radeon/evergreen_blit_kms.c
> index 30f0480..1e96bd4 100644
> --- a/drivers/gpu/drm/radeon/evergreen_blit_kms.c
> +++ b/drivers/gpu/drm/radeon/evergreen_blit_kms.c
> @@ -668,7 +668,7 @@ int evergreen_blit_init(struct radeon_device *rdev)
> ? ? ? ?obj_size = ALIGN(obj_size, 256);
>
> ? ? ? ?r = radeon_bo_create(rdev, obj_size, PAGE_SIZE, true,
> RADEON_GEM_DOMAIN_VRAM,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? &rdev->r600_blit.shader_obj);
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ?NULL, &rdev->r600_blit.shader_obj);
> ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ?DRM_ERROR("evergreen failed to allocate shader\n");
> ? ? ? ? ? ? ? ?return r;
> diff --git a/drivers/gpu/drm/radeon/r600.c b/drivers/gpu/drm/radeon/r600.c
> index ab5d6f2..f388a1d 100644
> --- a/drivers/gpu/drm/radeon/r600.c
> +++ b/drivers/gpu/drm/radeon/r600.c
> @@ -1231,7 +1231,7 @@ int r600_vram_scratch_init(struct radeon_device *rdev)
> ? ? ? ?if (rdev->vram_scratch.robj == NULL) {
> ? ? ? ? ? ? ? ?r = radeon_bo_create(rdev, RADEON_GPU_PAGE_SIZE,
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? PAGE_SIZE, true, RADEON_GEM_DOMAIN_VRAM,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?&rdev->vram_scratch.robj);
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?NULL, &rdev->vram_scratch.robj);
> ? ? ? ? ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ? ? ? ? ?return r;
> ? ? ? ? ? ? ? ?}
> @@ -2769,7 +2769,7 @@ int r600_ih_ring_alloc(struct radeon_device *rdev)
> ? ? ? ? ? ? ? ?r = radeon_bo_create(rdev, rdev->ih.ring_size,
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? PAGE_SIZE, true,
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? RADEON_GEM_DOMAIN_GTT,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?&rdev->ih.ring_obj);
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?NULL, &rdev->ih.ring_obj);
> ? ? ? ? ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ? ? ? ? ?DRM_ERROR("radeon: failed to create ih ring buffer
> (%d).\n", r);
> ? ? ? ? ? ? ? ? ? ? ? ?return r;
> diff --git a/drivers/gpu/drm/radeon/r600_blit_kms.c
> b/drivers/gpu/drm/radeon/r600_blit_kms.c
> index ef20822..03b6e0d 100644
> --- a/drivers/gpu/drm/radeon/r600_blit_kms.c
> +++ b/drivers/gpu/drm/radeon/r600_blit_kms.c
> @@ -552,7 +552,7 @@ int r600_blit_init(struct radeon_device *rdev)
> ? ? ? ?obj_size = ALIGN(obj_size, 256);
>
> ? ? ? ?r = radeon_bo_create(rdev, obj_size, PAGE_SIZE, true,
> RADEON_GEM_DOMAIN_VRAM,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? &rdev->r600_blit.shader_obj);
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ?NULL, &rdev->r600_blit.shader_obj);
> ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ?DRM_ERROR("r600 failed to allocate shader\n");
> ? ? ? ? ? ? ? ?return r;
> diff --git a/drivers/gpu/drm/radeon/radeon_benchmark.c
> b/drivers/gpu/drm/radeon/radeon_benchmark.c
> index fef7b72..364f5b1 100644
> --- a/drivers/gpu/drm/radeon/radeon_benchmark.c
> +++ b/drivers/gpu/drm/radeon/radeon_benchmark.c
> @@ -103,7 +103,7 @@ static void radeon_benchmark_move(struct radeon_device
> *rdev, unsigned size,
> ? ? ? ?int time;
>
> ? ? ? ?n = RADEON_BENCHMARK_ITERATIONS;
> - ? ? ? r = radeon_bo_create(rdev, size, PAGE_SIZE, true, sdomain, &sobj);
> + ? ? ? r = radeon_bo_create(rdev, size, PAGE_SIZE, true, sdomain, NULL,
> &sobj);
> ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ?goto out_cleanup;
> ? ? ? ?}
> @@ -115,7 +115,7 @@ static void radeon_benchmark_move(struct radeon_device
> *rdev, unsigned size,
> ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ?goto out_cleanup;
> ? ? ? ?}
> - ? ? ? r = radeon_bo_create(rdev, size, PAGE_SIZE, true, ddomain, &dobj);
> + ? ? ? r = radeon_bo_create(rdev, size, PAGE_SIZE, true, ddomain, NULL,
> &dobj);
> ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ?goto out_cleanup;
> ? ? ? ?}
> diff --git a/drivers/gpu/drm/radeon/radeon_device.c
> b/drivers/gpu/drm/radeon/radeon_device.c
> index 5763701..066c98b 100644
> --- a/drivers/gpu/drm/radeon/radeon_device.c
> +++ b/drivers/gpu/drm/radeon/radeon_device.c
> @@ -193,7 +193,7 @@ int radeon_wb_init(struct radeon_device *rdev)
>
> ? ? ? ?if (rdev->wb.wb_obj == NULL) {
> ? ? ? ? ? ? ? ?r = radeon_bo_create(rdev, RADEON_GPU_PAGE_SIZE, PAGE_SIZE,
> true,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? RADEON_GEM_DOMAIN_GTT, &rdev->wb.wb_obj);
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?RADEON_GEM_DOMAIN_GTT, NULL,
> &rdev->wb.wb_obj);
> ? ? ? ? ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ? ? ? ? ?dev_warn(rdev->dev, "(%d) create WB bo failed\n", r);
> ? ? ? ? ? ? ? ? ? ? ? ?return r;
> diff --git a/drivers/gpu/drm/radeon/radeon_drv.c
> b/drivers/gpu/drm/radeon/radeon_drv.c
> index e62e56a..f0bb2b5 100644
> --- a/drivers/gpu/drm/radeon/radeon_drv.c
> +++ b/drivers/gpu/drm/radeon/radeon_drv.c
> @@ -105,6 +105,11 @@ int radeon_mode_dumb_create(struct drm_file *file_priv,
> ?int radeon_mode_dumb_destroy(struct drm_file *file_priv,
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? struct drm_device *dev,
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? uint32_t handle);
> +struct dma_buf *radeon_gem_prime_export(struct drm_device *dev,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? struct drm_gem_object *obj,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? int flags);
> +struct drm_gem_object *radeon_gem_prime_import(struct drm_device *dev,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?struct dma_buf *dma_buf);
>
> ?#if defined(CONFIG_DEBUG_FS)
> ?int radeon_debugfs_init(struct drm_minor *minor);
> @@ -333,7 +338,8 @@ static const struct file_operations
> radeon_driver_kms_fops = {
> ?static struct drm_driver kms_driver = {
> ? ? ? ?.driver_features =
> ? ? ? ? ? ?DRIVER_USE_AGP | DRIVER_USE_MTRR | DRIVER_PCI_DMA | DRIVER_SG |
> - ? ? ? ? ? DRIVER_HAVE_IRQ | DRIVER_HAVE_DMA | DRIVER_IRQ_SHARED |
> DRIVER_GEM,
> + ? ? ? ? ? DRIVER_HAVE_IRQ | DRIVER_HAVE_DMA | DRIVER_IRQ_SHARED |
> DRIVER_GEM |
> + ? ? ? ? ? DRIVER_PRIME,
> ? ? ? ?.dev_priv_size = 0,
> ? ? ? ?.load = radeon_driver_load_kms,
> ? ? ? ?.firstopen = radeon_driver_firstopen_kms,
> @@ -368,6 +374,12 @@ static struct drm_driver kms_driver = {
> ? ? ? ?.dumb_map_offset = radeon_mode_dumb_mmap,
> ? ? ? ?.dumb_destroy = radeon_mode_dumb_destroy,
> ? ? ? ?.fops = &radeon_driver_kms_fops,
> +
> + ? ? ? .prime_handle_to_fd = drm_gem_prime_handle_to_fd,
> + ? ? ? .prime_fd_to_handle = drm_gem_prime_fd_to_handle,
> + ? ? ? .gem_prime_export = radeon_gem_prime_export,
> + ? ? ? .gem_prime_import = radeon_gem_prime_import,
> +
> ? ? ? ?.name = DRIVER_NAME,
> ? ? ? ?.desc = DRIVER_DESC,
> ? ? ? ?.date = DRIVER_DATE,
> diff --git a/drivers/gpu/drm/radeon/radeon_gart.c
> b/drivers/gpu/drm/radeon/radeon_gart.c
> index 8e9ef34..4fa5402 100644
> --- a/drivers/gpu/drm/radeon/radeon_gart.c
> +++ b/drivers/gpu/drm/radeon/radeon_gart.c
> @@ -80,7 +80,7 @@ int radeon_gart_table_vram_alloc(struct radeon_device *rdev)
> ? ? ? ?if (rdev->gart.robj == NULL) {
> ? ? ? ? ? ? ? ?r = radeon_bo_create(rdev, rdev->gart.table_size,
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? PAGE_SIZE, true, RADEON_GEM_DOMAIN_VRAM,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?&rdev->gart.robj);
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?NULL, &rdev->gart.robj);
> ? ? ? ? ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ? ? ? ? ?return r;
> ? ? ? ? ? ? ? ?}
> diff --git a/drivers/gpu/drm/radeon/radeon_gem.c
> b/drivers/gpu/drm/radeon/radeon_gem.c
> index e15cb1f..c0130b0 100644
> --- a/drivers/gpu/drm/radeon/radeon_gem.c
> +++ b/drivers/gpu/drm/radeon/radeon_gem.c
> @@ -42,6 +42,8 @@ void radeon_gem_object_free(struct drm_gem_object *gobj)
> ? ? ? ?struct radeon_bo *robj = gem_to_radeon_bo(gobj);
>
> ? ? ? ?if (robj) {
> + ? ? ? ? ? ? ? if (robj->gem_base.import_attach)
> + ? ? ? ? ? ? ? ? ? ? ? drm_prime_gem_destroy(&robj->gem_base, robj->tbo.sg);
> ? ? ? ? ? ? ? ?radeon_bo_unref(&robj);
> ? ? ? ?}
> ?}
> @@ -59,7 +61,7 @@ int radeon_gem_object_create(struct radeon_device *rdev,
> int size,
> ? ? ? ?if (alignment < PAGE_SIZE) {
> ? ? ? ? ? ? ? ?alignment = PAGE_SIZE;
> ? ? ? ?}
> - ? ? ? r = radeon_bo_create(rdev, size, alignment, kernel, initial_domain,
> &robj);
> + ? ? ? r = radeon_bo_create(rdev, size, alignment, kernel, initial_domain,
> NULL, &robj);
> ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ?if (r != -ERESTARTSYS)
> ? ? ? ? ? ? ? ? ? ? ? ?DRM_ERROR("Failed to allocate GEM object (%d, %d, %u,
> %d)\n",
> diff --git a/drivers/gpu/drm/radeon/radeon_object.c
> b/drivers/gpu/drm/radeon/radeon_object.c
> index 1affbc9..830f1a7 100644
> --- a/drivers/gpu/drm/radeon/radeon_object.c
> +++ b/drivers/gpu/drm/radeon/radeon_object.c
> @@ -104,7 +104,7 @@ void radeon_ttm_placement_from_domain(struct radeon_bo
> *rbo, u32 domain)
>
> ?int radeon_bo_create(struct radeon_device *rdev,
> ? ? ? ? ? ? ? ? ? ? unsigned long size, int byte_align, bool kernel, u32
> domain,
> - ? ? ? ? ? ? ? ? ? ?struct radeon_bo **bo_ptr)
> + ? ? ? ? ? ? ? ? ? ?struct sg_table *sg, struct radeon_bo **bo_ptr)
> ?{
> ? ? ? ?struct radeon_bo *bo;
> ? ? ? ?enum ttm_bo_type type;
> @@ -120,6 +120,8 @@ int radeon_bo_create(struct radeon_device *rdev,
> ? ? ? ?}
> ? ? ? ?if (kernel) {
> ? ? ? ? ? ? ? ?type = ttm_bo_type_kernel;
> + ? ? ? } else if (sg) {
> + ? ? ? ? ? ? ? type = ttm_bo_type_sg;
> ? ? ? ?} else {
> ? ? ? ? ? ? ? ?type = ttm_bo_type_device;
> ? ? ? ?}
> @@ -155,7 +157,7 @@ retry:
> ? ? ? ?mutex_lock(&rdev->vram_mutex);
> ? ? ? ?r = ttm_bo_init(&rdev->mman.bdev, &bo->tbo, size, type,
> ? ? ? ? ? ? ? ? ? ? ? ?&bo->placement, page_align, 0, !kernel, NULL,
> - ? ? ? ? ? ? ? ? ? ? ? acc_size, NULL, &radeon_ttm_bo_destroy);
> + ? ? ? ? ? ? ? ? ? ? ? acc_size, sg, &radeon_ttm_bo_destroy);
> ? ? ? ?mutex_unlock(&rdev->vram_mutex);
> ? ? ? ?if (unlikely(r != 0)) {
> ? ? ? ? ? ? ? ?if (r != -ERESTARTSYS) {
> diff --git a/drivers/gpu/drm/radeon/radeon_object.h
> b/drivers/gpu/drm/radeon/radeon_object.h
> index befec7d..17fb99f 100644
> --- a/drivers/gpu/drm/radeon/radeon_object.h
> +++ b/drivers/gpu/drm/radeon/radeon_object.h
> @@ -111,9 +111,10 @@ extern int radeon_bo_wait(struct radeon_bo *bo, u32
> *mem_type,
> ? ? ? ? ? ? ? ? ? ? ? ? ?bool no_wait);
>
> ?extern int radeon_bo_create(struct radeon_device *rdev,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? unsigned long size, int byte_align,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? bool kernel, u32 domain,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? struct radeon_bo **bo_ptr);
> + ? ? ? ? ? ? ? ? ? ? ? ? ? unsigned long size, int byte_align,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? bool kernel, u32 domain,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? struct sg_table *sg,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? struct radeon_bo **bo_ptr);
> ?extern int radeon_bo_kmap(struct radeon_bo *bo, void **ptr);
> ?extern void radeon_bo_kunmap(struct radeon_bo *bo);
> ?extern void radeon_bo_unref(struct radeon_bo **bo);
> diff --git a/drivers/gpu/drm/radeon/radeon_prime.c
> b/drivers/gpu/drm/radeon/radeon_prime.c
> new file mode 100644
> index 0000000..24f5175
> --- /dev/null
> +++ b/drivers/gpu/drm/radeon/radeon_prime.c
> @@ -0,0 +1,176 @@
> +/*
> + * Copyright 2012 Advanced Micro Devices, Inc.
> + *
> + * 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 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 COPYRIGHT HOLDER(S) OR AUTHOR(S) 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.
> + *
> + * based on nouveau_prime.c
> + *
> + * Authors: Alex Deucher
> + */
> +#include "drmP.h"
> +#include "drm.h"
> +
> +#include "radeon.h"
> +#include "radeon_drm.h"
> +
> +#include <linux/dma-buf.h>
> +
> +static struct sg_table *radeon_gem_map_dma_buf(struct dma_buf_attachment
> *attachment,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?enum dma_data_direction dir)
> +{
> + ? ? ? struct radeon_bo *bo = attachment->dmabuf->priv;
> + ? ? ? struct drm_device *dev = bo->rdev->ddev;
> + ? ? ? int npages = bo->tbo.num_pages;
> + ? ? ? struct sg_table *sg;
> + ? ? ? int nents;
> +
> + ? ? ? mutex_lock(&dev->struct_mutex);
> + ? ? ? sg = drm_prime_pages_to_sg(bo->tbo.ttm->pages, npages);
> + ? ? ? nents = dma_map_sg(attachment->dev, sg->sgl, sg->nents, dir);
> + ? ? ? mutex_unlock(&dev->struct_mutex);
> + ? ? ? return sg;
> +}
> +
> +static void radeon_gem_unmap_dma_buf(struct dma_buf_attachment *attachment,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?struct sg_table *sg, enum
> dma_data_direction dir)
> +{
> + ? ? ? dma_unmap_sg(attachment->dev, sg->sgl, sg->nents, dir);
> + ? ? ? sg_free_table(sg);
> + ? ? ? kfree(sg);
> +}
> +
> +static void radeon_gem_dmabuf_release(struct dma_buf *dma_buf)
> +{
> + ? ? ? struct radeon_bo *bo = dma_buf->priv;
> +
> + ? ? ? if (bo->gem_base.export_dma_buf == dma_buf) {
> + ? ? ? ? ? ? ? DRM_ERROR("unreference dmabuf %p\n", &bo->gem_base);
> + ? ? ? ? ? ? ? bo->gem_base.export_dma_buf = NULL;
> + ? ? ? ? ? ? ? drm_gem_object_unreference_unlocked(&bo->gem_base);
> + ? ? ? }
> +}
> +
> +static void *radeon_gem_kmap_atomic(struct dma_buf *dma_buf, unsigned long
> page_num)
> +{
> + ? ? ? return NULL;
> +}
> +
> +static void radeon_gem_kunmap_atomic(struct dma_buf *dma_buf, unsigned long
> page_num, void *addr)
> +{
> +
> +}
> +static void *radeon_gem_kmap(struct dma_buf *dma_buf, unsigned long page_num)
> +{
> + ? ? ? return NULL;
> +}
> +
> +static void radeon_gem_kunmap(struct dma_buf *dma_buf, unsigned long
> page_num, void *addr)
> +{
> +
> +}
> +
> +struct dma_buf_ops radeon_dmabuf_ops = ?{
> + ? ? ? .map_dma_buf = radeon_gem_map_dma_buf,
> + ? ? ? .unmap_dma_buf = radeon_gem_unmap_dma_buf,
> + ? ? ? .release = radeon_gem_dmabuf_release,
> + ? ? ? .kmap = radeon_gem_kmap,
> + ? ? ? .kmap_atomic = radeon_gem_kmap_atomic,
> + ? ? ? .kunmap = radeon_gem_kunmap,
> + ? ? ? .kunmap_atomic = radeon_gem_kunmap_atomic,
> +};
> +
> +static int radeon_prime_create(struct drm_device *dev,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?size_t size,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?struct sg_table *sg,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?struct radeon_bo **pbo)
> +{
> + ? ? ? struct radeon_device *rdev = dev->dev_private;
> + ? ? ? struct radeon_bo *bo;
> + ? ? ? int ret;
> +
> + ? ? ? ret = radeon_bo_create(rdev, size, PAGE_SIZE, false,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?RADEON_GEM_DOMAIN_GTT, sg, pbo);
> + ? ? ? if (ret)
> + ? ? ? ? ? ? ? return ret;
> + ? ? ? bo = *pbo;
> + ? ? ? bo->gem_base.driver_private = bo;
> +
> + ? ? ? mutex_lock(&rdev->gem.mutex);
> + ? ? ? list_add_tail(&bo->list, &rdev->gem.objects);
> + ? ? ? mutex_unlock(&rdev->gem.mutex);
> +
> + ? ? ? return 0;
> +}
> +
> +struct dma_buf *radeon_gem_prime_export(struct drm_device *dev,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? struct drm_gem_object *obj,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? int flags)
> +{
> + ? ? ? struct radeon_bo *bo = gem_to_radeon_bo(obj);
> + ? ? ? int ret = 0;
> +
> + ? ? ? /* pin buffer into GTT */
> + ? ? ? ret = radeon_bo_pin(bo, RADEON_GEM_DOMAIN_GTT, NULL);
> + ? ? ? if (ret)
> + ? ? ? ? ? ? ? return ERR_PTR(ret);
> +
> + ? ? ? return dma_buf_export(bo, &radeon_dmabuf_ops, obj->size, flags);
> +}
> +
> +struct drm_gem_object *radeon_gem_prime_import(struct drm_device *dev,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?struct dma_buf *dma_buf)
> +{
> + ? ? ? struct dma_buf_attachment *attach;
> + ? ? ? struct sg_table *sg;
> + ? ? ? struct radeon_bo *bo;
> + ? ? ? int ret;
> +
> + ? ? ? if (dma_buf->ops == &radeon_dmabuf_ops) {
> + ? ? ? ? ? ? ? bo = dma_buf->priv;
> + ? ? ? ? ? ? ? if (bo->gem_base.dev == dev) {
> + ? ? ? ? ? ? ? ? ? ? ? drm_gem_object_reference(&bo->gem_base);
> + ? ? ? ? ? ? ? ? ? ? ? return &bo->gem_base;
> + ? ? ? ? ? ? ? }
> + ? ? ? }
> +
> + ? ? ? /* need to attach */
> + ? ? ? attach = dma_buf_attach(dma_buf, dev->dev);
> + ? ? ? if (IS_ERR(attach))
> + ? ? ? ? ? ? ? return ERR_CAST(attach);
> +
> + ? ? ? sg = dma_buf_map_attachment(attach, DMA_BIDIRECTIONAL);
> + ? ? ? if (IS_ERR(sg)) {
> + ? ? ? ? ? ? ? ret = PTR_ERR(sg);
> + ? ? ? ? ? ? ? goto fail_detach;
> + ? ? ? }
> +
> + ? ? ? ret = radeon_prime_create(dev, dma_buf->size, sg, &bo);
> + ? ? ? if (ret)
> + ? ? ? ? ? ? ? goto fail_unmap;
> +
> + ? ? ? bo->gem_base.import_attach = attach;
> +
> + ? ? ? return &bo->gem_base;
> +
> +fail_unmap:
> + ? ? ? dma_buf_unmap_attachment(attach, sg, DMA_BIDIRECTIONAL);
> +fail_detach:
> + ? ? ? dma_buf_detach(dma_buf, attach);
> + ? ? ? return ERR_PTR(ret);
> +}
> diff --git a/drivers/gpu/drm/radeon/radeon_ring.c
> b/drivers/gpu/drm/radeon/radeon_ring.c
> index a5dee76..493a7be 100644
> --- a/drivers/gpu/drm/radeon/radeon_ring.c
> +++ b/drivers/gpu/drm/radeon/radeon_ring.c
> @@ -390,8 +390,8 @@ int radeon_ring_init(struct radeon_device *rdev, struct
> radeon_ring *ring, unsig
> ? ? ? ?/* Allocate ring buffer */
> ? ? ? ?if (ring->ring_obj == NULL) {
> ? ? ? ? ? ? ? ?r = radeon_bo_create(rdev, ring->ring_size, PAGE_SIZE, true,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? RADEON_GEM_DOMAIN_GTT,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? &ring->ring_obj);
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?RADEON_GEM_DOMAIN_GTT,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?NULL, &ring->ring_obj);
> ? ? ? ? ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ? ? ? ? ?dev_err(rdev->dev, "(%d) ring create failed\n", r);
> ? ? ? ? ? ? ? ? ? ? ? ?return r;
> diff --git a/drivers/gpu/drm/radeon/radeon_sa.c
> b/drivers/gpu/drm/radeon/radeon_sa.c
> index c3ac7f4..32059b7 100644
> --- a/drivers/gpu/drm/radeon/radeon_sa.c
> +++ b/drivers/gpu/drm/radeon/radeon_sa.c
> @@ -65,7 +65,7 @@ int radeon_sa_bo_manager_init(struct radeon_device *rdev,
> ? ? ? ?}
>
> ? ? ? ?r = radeon_bo_create(rdev, size, RADEON_GPU_PAGE_SIZE, true,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ?RADEON_GEM_DOMAIN_CPU, &sa_manager->bo);
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ?RADEON_GEM_DOMAIN_CPU, NULL, &sa_manager->bo);
> ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ?dev_err(rdev->dev, "(%d) failed to allocate bo for manager\n",
> r);
> ? ? ? ? ? ? ? ?return r;
> diff --git a/drivers/gpu/drm/radeon/radeon_test.c
> b/drivers/gpu/drm/radeon/radeon_test.c
> index b057387..efff929 100644
> --- a/drivers/gpu/drm/radeon/radeon_test.c
> +++ b/drivers/gpu/drm/radeon/radeon_test.c
> @@ -59,7 +59,7 @@ void radeon_test_moves(struct radeon_device *rdev)
> ? ? ? ?}
>
> ? ? ? ?r = radeon_bo_create(rdev, size, PAGE_SIZE, true,
> RADEON_GEM_DOMAIN_VRAM,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? &vram_obj);
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ?NULL, &vram_obj);
> ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ?DRM_ERROR("Failed to create VRAM object\n");
> ? ? ? ? ? ? ? ?goto out_cleanup;
> @@ -78,7 +78,7 @@ void radeon_test_moves(struct radeon_device *rdev)
> ? ? ? ? ? ? ? ?void **vram_start, **vram_end;
>
> ? ? ? ? ? ? ? ?r = radeon_bo_create(rdev, size, PAGE_SIZE, true,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?RADEON_GEM_DOMAIN_GTT, gtt_obj + i);
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?RADEON_GEM_DOMAIN_GTT, NULL, gtt_obj +
> i);
> ? ? ? ? ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ? ? ? ? ?DRM_ERROR("Failed to create GTT object %d\n", i);
> ? ? ? ? ? ? ? ? ? ? ? ?goto out_cleanup;
> diff --git a/drivers/gpu/drm/radeon/radeon_ttm.c
> b/drivers/gpu/drm/radeon/radeon_ttm.c
> index 0f6aee8..c94a225 100644
> --- a/drivers/gpu/drm/radeon/radeon_ttm.c
> +++ b/drivers/gpu/drm/radeon/radeon_ttm.c
> @@ -611,10 +611,18 @@ static int radeon_ttm_tt_populate(struct ttm_tt *ttm)
> ? ? ? ?struct radeon_ttm_tt *gtt = (void *)ttm;
> ? ? ? ?unsigned i;
> ? ? ? ?int r;
> + ? ? ? bool slave = !!(ttm->page_flags & TTM_PAGE_FLAG_SG);
>
> ? ? ? ?if (ttm->state != tt_unpopulated)
> ? ? ? ? ? ? ? ?return 0;
>
> + ? ? ? if (slave && ttm->sg) {
> + ? ? ? ? ? ? ? drm_prime_sg_to_page_addr_arrays(ttm->sg, ttm->pages,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?gtt->ttm.dma_address,
> ttm->num_pages);
> + ? ? ? ? ? ? ? ttm->state = tt_unbound;
> + ? ? ? ? ? ? ? return 0;
> + ? ? ? }
> +
> ? ? ? ?rdev = radeon_get_rdev(ttm->bdev);
> ?#if __OS_HAS_AGP
> ? ? ? ?if (rdev->flags & RADEON_IS_AGP) {
> @@ -655,6 +663,10 @@ static void radeon_ttm_tt_unpopulate(struct ttm_tt *ttm)
> ? ? ? ?struct radeon_device *rdev;
> ? ? ? ?struct radeon_ttm_tt *gtt = (void *)ttm;
> ? ? ? ?unsigned i;
> + ? ? ? bool slave = !!(ttm->page_flags & TTM_PAGE_FLAG_SG);
> +
> + ? ? ? if (slave)
> + ? ? ? ? ? ? ? return;
>
> ? ? ? ?rdev = radeon_get_rdev(ttm->bdev);
> ?#if __OS_HAS_AGP
> @@ -726,8 +738,8 @@ int radeon_ttm_init(struct radeon_device *rdev)
> ? ? ? ? ? ? ? ?return r;
> ? ? ? ?}
> ? ? ? ?r = radeon_bo_create(rdev, 256 * 1024, PAGE_SIZE, true,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? RADEON_GEM_DOMAIN_VRAM,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? &rdev->stollen_vga_memory);
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ?RADEON_GEM_DOMAIN_VRAM,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ?NULL, &rdev->stollen_vga_memory);
> ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ?return r;
> ? ? ? ?}
> diff --git a/drivers/gpu/drm/radeon/si.c b/drivers/gpu/drm/radeon/si.c
> index 63ae41d..549732e 100644
> --- a/drivers/gpu/drm/radeon/si.c
> +++ b/drivers/gpu/drm/radeon/si.c
> @@ -2976,7 +2976,8 @@ int si_rlc_init(struct radeon_device *rdev)
> ? ? ? ?/* save restore block */
> ? ? ? ?if (rdev->rlc.save_restore_obj == NULL) {
> ? ? ? ? ? ? ? ?r = radeon_bo_create(rdev, RADEON_GPU_PAGE_SIZE, PAGE_SIZE,
> true,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? RADEON_GEM_DOMAIN_VRAM,
> &rdev->rlc.save_restore_obj);
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?RADEON_GEM_DOMAIN_VRAM, NULL,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?&rdev->rlc.save_restore_obj);
> ? ? ? ? ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ? ? ? ? ?dev_warn(rdev->dev, "(%d) create RLC sr bo failed\n",
> r);
> ? ? ? ? ? ? ? ? ? ? ? ?return r;
> @@ -3000,7 +3001,8 @@ int si_rlc_init(struct radeon_device *rdev)
> ? ? ? ?/* clear state block */
> ? ? ? ?if (rdev->rlc.clear_state_obj == NULL) {
> ? ? ? ? ? ? ? ?r = radeon_bo_create(rdev, RADEON_GPU_PAGE_SIZE, PAGE_SIZE,
> true,
> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? RADEON_GEM_DOMAIN_VRAM,
> &rdev->rlc.clear_state_obj);
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?RADEON_GEM_DOMAIN_VRAM, NULL,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?&rdev->rlc.clear_state_obj);
> ? ? ? ? ? ? ? ?if (r) {
> ? ? ? ? ? ? ? ? ? ? ? ?dev_warn(rdev->dev, "(%d) create RLC c bo failed\n",
> r);
> ? ? ? ? ? ? ? ? ? ? ? ?si_rlc_fini(rdev);
> --
> 1.7.6
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
