[PATCH] drm/amdgpu: fix amdgpu_vm_handle_moved as well

Christian König deathsimple at vodafone.de
Tue Sep 12 08:31:19 UTC 2017


>>   +        if (bo_va->base.bo->tbo.resv != vm->root.base.bo->tbo.resv) {
> When we expand mapping fence, we will sync all moved update and clear 
> here, instead of moved update in amdgpu_vm_bo_update in previous patch.

Yeah, turned out this patch actually didn't worked as expected because 
the bo_va->last_pt_update fence could be stale and old and replaces the 
new and fresh fence in vm->last_update resulting in VM faults.

I've send a V2 of that patch yesterday which fixes this (and is quite a 
bit cleaner in general).

Please take a look at that one instead.

Thanks,
Christian.


More information about the amd-gfx mailing list