* [PATCH 1/2] drm/amdgpu: wait on page directory changes.
@ 2015-08-14 18:08 Bas Nieuwenhuizen
2015-08-14 18:08 ` [PATCH 2/2] drm/amdgpu: do not redundantly update page tables Bas Nieuwenhuizen
2015-08-14 18:56 ` [PATCH 1/2] drm/amdgpu: wait on page directory changes Christian König
0 siblings, 2 replies; 3+ messages in thread
From: Bas Nieuwenhuizen @ 2015-08-14 18:08 UTC (permalink / raw)
To: dri-devel
Pagetables can be moved and therefore the page directory update can be necessary
for the current cs even if none of the the bo's are moved. In that scenario
there is no fence between the sdma0 and gfx ring, so we add one.
Signed-off-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
---
drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 +
drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 4 ++++
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 6 ++++++
3 files changed, 11 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
index 6934f52..c2383bb 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
@@ -981,6 +981,7 @@ struct amdgpu_vm {
/* contains the page directory */
struct amdgpu_bo *page_directory;
unsigned max_pde_used;
+ struct fence *page_directory_fence;
/* array of page tables, one for each page directory entry */
struct amdgpu_vm_pt *page_tables;
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
index f428288..01b4a67 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
@@ -552,6 +552,10 @@ static int amdgpu_bo_vm_update_pte(struct amdgpu_cs_parser *p,
if (r)
return r;
+ r = amdgpu_sync_fence(adev, &p->ibs[0].sync, vm->page_directory_fence);
+ if (r)
+ return r;
+
r = amdgpu_vm_clear_freed(adev, vm);
if (r)
return r;
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index b3f5d04..e02e353 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -495,6 +495,9 @@ int amdgpu_vm_update_page_directory(struct amdgpu_device *adev,
if (r)
goto error_free;
amdgpu_bo_fence(pd, fence, true);
+
+ fence_put(vm->page_directory_fence);
+ vm->page_directory_fence = fence_get(fence);
}
if (!amdgpu_enable_scheduler || ib->length_dw == 0) {
@@ -1287,6 +1290,8 @@ int amdgpu_vm_init(struct amdgpu_device *adev, struct amdgpu_vm *vm)
return -ENOMEM;
}
+ vm->page_directory_fence = NULL;
+
r = amdgpu_bo_create(adev, pd_size, align, true,
AMDGPU_GEM_DOMAIN_VRAM, 0,
NULL, &vm->page_directory);
@@ -1335,6 +1340,7 @@ void amdgpu_vm_fini(struct amdgpu_device *adev, struct amdgpu_vm *vm)
kfree(vm->page_tables);
amdgpu_bo_unref(&vm->page_directory);
+ fence_put(vm->page_directory_fence);
for (i = 0; i < AMDGPU_MAX_RINGS; ++i) {
amdgpu_fence_unref(&vm->ids[i].flushed_updates);
--
2.5.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 3+ messages in thread* [PATCH 2/2] drm/amdgpu: do not redundantly update page tables.
2015-08-14 18:08 [PATCH 1/2] drm/amdgpu: wait on page directory changes Bas Nieuwenhuizen
@ 2015-08-14 18:08 ` Bas Nieuwenhuizen
2015-08-14 18:56 ` [PATCH 1/2] drm/amdgpu: wait on page directory changes Christian König
1 sibling, 0 replies; 3+ messages in thread
From: Bas Nieuwenhuizen @ 2015-08-14 18:08 UTC (permalink / raw)
To: dri-devel
Signed-off-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index e02e353..9a1da86 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -844,7 +844,7 @@ int amdgpu_vm_bo_update(struct amdgpu_device *adev,
struct ttm_mem_reg *mem)
{
struct amdgpu_vm *vm = bo_va->vm;
- struct amdgpu_bo_va_mapping *mapping;
+ struct amdgpu_bo_va_mapping *mapping, *next;
uint32_t flags;
uint64_t addr;
int r;
@@ -864,11 +864,14 @@ int amdgpu_vm_bo_update(struct amdgpu_device *adev,
list_splice_init(&bo_va->valids, &bo_va->invalids);
spin_unlock(&vm->status_lock);
- list_for_each_entry(mapping, &bo_va->invalids, list) {
+ list_for_each_entry_safe(mapping, next, &bo_va->invalids, list) {
r = amdgpu_vm_bo_update_mapping(adev, vm, mapping, addr,
flags, &bo_va->last_pt_update);
if (r)
return r;
+
+ list_del_init(&mapping->list);
+ list_add(&mapping->list, &bo_va->valids);
}
spin_lock(&vm->status_lock);
--
2.5.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH 1/2] drm/amdgpu: wait on page directory changes.
2015-08-14 18:08 [PATCH 1/2] drm/amdgpu: wait on page directory changes Bas Nieuwenhuizen
2015-08-14 18:08 ` [PATCH 2/2] drm/amdgpu: do not redundantly update page tables Bas Nieuwenhuizen
@ 2015-08-14 18:56 ` Christian König
1 sibling, 0 replies; 3+ messages in thread
From: Christian König @ 2015-08-14 18:56 UTC (permalink / raw)
To: Bas Nieuwenhuizen, dri-devel
On 14.08.2015 20:08, Bas Nieuwenhuizen wrote:
> Pagetables can be moved and therefore the page directory update can be necessary
> for the current cs even if none of the the bo's are moved. In that scenario
> there is no fence between the sdma0 and gfx ring, so we add one.
>
> Signed-off-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
Nice catch, this patch is Reviewed-by: Christian König
<christian.koenig@amd.com>
The other one we found internally already as well.
Regards,
Christian.
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 +
> drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 4 ++++
> drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 6 ++++++
> 3 files changed, 11 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> index 6934f52..c2383bb 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> @@ -981,6 +981,7 @@ struct amdgpu_vm {
> /* contains the page directory */
> struct amdgpu_bo *page_directory;
> unsigned max_pde_used;
> + struct fence *page_directory_fence;
>
> /* array of page tables, one for each page directory entry */
> struct amdgpu_vm_pt *page_tables;
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> index f428288..01b4a67 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> @@ -552,6 +552,10 @@ static int amdgpu_bo_vm_update_pte(struct amdgpu_cs_parser *p,
> if (r)
> return r;
>
> + r = amdgpu_sync_fence(adev, &p->ibs[0].sync, vm->page_directory_fence);
> + if (r)
> + return r;
> +
> r = amdgpu_vm_clear_freed(adev, vm);
> if (r)
> return r;
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> index b3f5d04..e02e353 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> @@ -495,6 +495,9 @@ int amdgpu_vm_update_page_directory(struct amdgpu_device *adev,
> if (r)
> goto error_free;
> amdgpu_bo_fence(pd, fence, true);
> +
> + fence_put(vm->page_directory_fence);
> + vm->page_directory_fence = fence_get(fence);
> }
>
> if (!amdgpu_enable_scheduler || ib->length_dw == 0) {
> @@ -1287,6 +1290,8 @@ int amdgpu_vm_init(struct amdgpu_device *adev, struct amdgpu_vm *vm)
> return -ENOMEM;
> }
>
> + vm->page_directory_fence = NULL;
> +
> r = amdgpu_bo_create(adev, pd_size, align, true,
> AMDGPU_GEM_DOMAIN_VRAM, 0,
> NULL, &vm->page_directory);
> @@ -1335,6 +1340,7 @@ void amdgpu_vm_fini(struct amdgpu_device *adev, struct amdgpu_vm *vm)
> kfree(vm->page_tables);
>
> amdgpu_bo_unref(&vm->page_directory);
> + fence_put(vm->page_directory_fence);
>
> for (i = 0; i < AMDGPU_MAX_RINGS; ++i) {
> amdgpu_fence_unref(&vm->ids[i].flushed_updates);
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-08-14 18:56 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-14 18:08 [PATCH 1/2] drm/amdgpu: wait on page directory changes Bas Nieuwenhuizen
2015-08-14 18:08 ` [PATCH 2/2] drm/amdgpu: do not redundantly update page tables Bas Nieuwenhuizen
2015-08-14 18:56 ` [PATCH 1/2] drm/amdgpu: wait on page directory changes Christian König
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.