[PATCH 05/11] drm/radeon: fence PT updates manually
Alex Deucher
alexdeucher at gmail.com
Wed Sep 17 13:52:09 PDT 2014
On Fri, Sep 12, 2014 at 7:52 AM, Christian König
<deathsimple at vodafone.de> wrote:
> From: Christian König <christian.koenig at amd.com>
>
> This allows us to add the real execution fence as shared.
>
> Signed-off-by: Christian König <christian.koenig at amd.com>
Minor typo noted below.
> ---
> drivers/gpu/drm/radeon/radeon_object.c | 19 ++++++++++
> drivers/gpu/drm/radeon/radeon_object.h | 2 ++
> drivers/gpu/drm/radeon/radeon_vm.c | 65 +++++++++++++++++++++-------------
> 3 files changed, 62 insertions(+), 24 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_object.c b/drivers/gpu/drm/radeon/radeon_object.c
> index 8abee5f..8714baf 100644
> --- a/drivers/gpu/drm/radeon/radeon_object.c
> +++ b/drivers/gpu/drm/radeon/radeon_object.c
> @@ -790,3 +790,22 @@ int radeon_bo_wait(struct radeon_bo *bo, u32 *mem_type, bool no_wait)
> ttm_bo_unreserve(&bo->tbo);
> return r;
> }
> +
> +/**
> + * radeon_bo_fence - add fence to buffer object
> + *
> + * @bo: buffer object in question
> + * @fence: fence to add
> + * @shared: true if fence should be added shared
> + *
> + */
> +void radeon_bo_fence(struct radeon_bo *bo, struct radeon_fence *fence,
> + bool shared)
> +{
> + struct reservation_object *resv = bo->tbo.resv;
> +
> + if (shared)
> + reservation_object_add_shared_fence(resv, &fence->base);
> + else
> + reservation_object_add_excl_fence(resv, &fence->base);
> +}
> diff --git a/drivers/gpu/drm/radeon/radeon_object.h b/drivers/gpu/drm/radeon/radeon_object.h
> index 98a47fd..ec80b28 100644
> --- a/drivers/gpu/drm/radeon/radeon_object.h
> +++ b/drivers/gpu/drm/radeon/radeon_object.h
> @@ -154,6 +154,8 @@ extern void radeon_bo_move_notify(struct ttm_buffer_object *bo,
> struct ttm_mem_reg *new_mem);
> extern int radeon_bo_fault_reserve_notify(struct ttm_buffer_object *bo);
> extern int radeon_bo_get_surface_reg(struct radeon_bo *bo);
> +extern void radeon_bo_fence(struct radeon_bo *bo, struct radeon_fence *fence,
> + bool shared);
>
> /*
> * sub allocation
> diff --git a/drivers/gpu/drm/radeon/radeon_vm.c b/drivers/gpu/drm/radeon/radeon_vm.c
> index 411146a..84b2735 100644
> --- a/drivers/gpu/drm/radeon/radeon_vm.c
> +++ b/drivers/gpu/drm/radeon/radeon_vm.c
> @@ -143,7 +143,7 @@ struct radeon_cs_reloc *radeon_vm_get_bos(struct radeon_device *rdev,
> list[0].prefered_domains = RADEON_GEM_DOMAIN_VRAM;
> list[0].allowed_domains = RADEON_GEM_DOMAIN_VRAM;
> list[0].tv.bo = &vm->page_directory->tbo;
> - list[0].tv.shared = false;
> + list[0].tv.shared = true;
> list[0].tiling_flags = 0;
> list[0].handle = 0;
> list_add(&list[0].tv.head, head);
> @@ -157,7 +157,7 @@ struct radeon_cs_reloc *radeon_vm_get_bos(struct radeon_device *rdev,
> list[idx].prefered_domains = RADEON_GEM_DOMAIN_VRAM;
> list[idx].allowed_domains = RADEON_GEM_DOMAIN_VRAM;
> list[idx].tv.bo = &list[idx].robj->tbo;
> - list[idx].tv.shared = false;
> + list[idx].tv.shared = true;
> list[idx].tiling_flags = 0;
> list[idx].handle = 0;
> list_add(&list[idx++].tv.head, head);
> @@ -388,35 +388,25 @@ static void radeon_vm_set_pages(struct radeon_device *rdev,
> static int radeon_vm_clear_bo(struct radeon_device *rdev,
> struct radeon_bo *bo)
> {
> - struct ttm_validate_buffer tv;
> - struct ww_acquire_ctx ticket;
> - struct list_head head;
> struct radeon_ib ib;
> unsigned entries;
> uint64_t addr;
> int r;
>
> - memset(&tv, 0, sizeof(tv));
> - tv.bo = &bo->tbo;
> - tv.shared = false;
> -
> - INIT_LIST_HEAD(&head);
> - list_add(&tv.head, &head);
> -
> - r = ttm_eu_reserve_buffers(&ticket, &head, true);
> - if (r)
> + r = radeon_bo_reserve(bo, false);
> + if (r)
> return r;
>
> - r = ttm_bo_validate(&bo->tbo, &bo->placement, true, false);
> - if (r)
> - goto error;
> + r = ttm_bo_validate(&bo->tbo, &bo->placement, true, false);
> + if (r)
> + goto error_unreserve;
>
> addr = radeon_bo_gpu_offset(bo);
> entries = radeon_bo_size(bo) / 8;
>
> r = radeon_ib_get(rdev, R600_RING_TYPE_DMA_INDEX, &ib, NULL, 256);
> if (r)
> - goto error;
> + goto error_unreserve;
>
> ib.length_dw = 0;
>
> @@ -426,15 +416,15 @@ static int radeon_vm_clear_bo(struct radeon_device *rdev,
>
> r = radeon_ib_schedule(rdev, &ib, NULL, false);
> if (r)
> - goto error;
> + goto error_free;
>
> - ttm_eu_fence_buffer_objects(&ticket, &head, &ib.fence->base);
> - radeon_ib_free(rdev, &ib);
> + radeon_bo_fence(bo, ib.fence, false);
>
> - return 0;
> +error_free:
> + radeon_ib_free(rdev, &ib);
>
> -error:
> - ttm_eu_backoff_reservation(&ticket, &head);
> +error_unreserve:
> + radeon_bo_unreserve(bo);
> return r;
> }
>
> @@ -706,6 +696,7 @@ int radeon_vm_update_page_directory(struct radeon_device *rdev,
> radeon_ib_free(rdev, &ib);
> return r;
> }
> + radeon_bo_fence(pd, ib.fence, false);
> radeon_fence_unref(&vm->fence);
> vm->fence = radeon_fence_ref(ib.fence);
> radeon_fence_unref(&vm->last_flush);
> @@ -862,6 +853,31 @@ static void radeon_vm_update_ptes(struct radeon_device *rdev,
> }
>
> /**
> + * radeon_vm_fence_pts - fence page tables after an update
> + *
> + * @vm: requested vm
> + * @start: start of GPU address range
> + * @end: end of GPU address range
> + * @fence: fence to use
> + *
> + * Fence the page tables in the range @start - @end (cayman+).
> + *
> + * Global and local mutex must be locked!
> + */
> +static void radeon_vm_fence_pts(struct radeon_vm *vm,
> + int64_t start, uint64_t end,
typo: uint64_t start
> + struct radeon_fence *fence)
> +{
> + unsigned i;
> +
> + start >>= radeon_vm_block_size;
> + end >>= radeon_vm_block_size;
> +
> + for (i = start; i <= end; ++i)
> + radeon_bo_fence(vm->page_tables[i].bo, fence, false);
> +}
> +
> +/**
> * radeon_vm_bo_update - map a bo into the vm page table
> *
> * @rdev: radeon_device pointer
> @@ -972,6 +988,7 @@ int radeon_vm_bo_update(struct radeon_device *rdev,
> radeon_ib_free(rdev, &ib);
> return r;
> }
> + radeon_vm_fence_pts(vm, bo_va->it.start, bo_va->it.last + 1, ib.fence);
> radeon_fence_unref(&vm->fence);
> vm->fence = radeon_fence_ref(ib.fence);
> radeon_ib_free(rdev, &ib);
> --
> 1.9.1
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
More information about the dri-devel
mailing list