I'm not 100% sure that patch was correct.
When is amdgpu_ctx_create() called? The VMID is reserved for the whole
process, not just a context.
Regards,
Christian.
Am 31.10.2017 um 16:57 schrieb Marek Olšák:
I addressed the feedback and pushed the patch.
Marek
On Tue, Oct 31, 2017 at 4:50 PM, Michel Dänzer <mic...@daenzer.net> wrote:
On 31/10/17 04:40 PM, Andrey Grodzovsky wrote:
Signed-off-by: Andrey Grodzovsky <andrey.grodzov...@amd.com>
[...]
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
index 8f43e93..1155492 100644
--- a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
+++ b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
@@ -256,6 +256,14 @@ static struct radeon_winsys_ctx *amdgpu_ctx_create(struct
radeon_winsys *ws)
goto error_create;
}
+ if (ctx->ws->reserve_vmid) {
+ r = amdgpu_vm_reserve_vmid(ctx->ctx, 0);
+ if (r) {
+ fprintf(stderr, "amdgpu: amdgpu_cs_ctx_create failed. (%i)\n", r);
This should say "amdgpu: amdgpu_vm_reserve_vmid failed. (%i)\n".
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X developer
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
