Re: [PATCH] Revert "drm/radeon: Try evicting from CPU accessible to inaccessible VRAM first"

2017-03-23 Thread Zachary Michaels
> > No, I've requested reverting the patch for now because it causes an > obviously and rather severe problem. If you guys can quickly find how to > fix it feel free to use that instead. > > My mistake! That makes sense. Thanks again. ___ dri-devel mailin

Re: [PATCH] Revert "drm/radeon: Try evicting from CPU accessible to inaccessible VRAM first"

2017-03-23 Thread Zachary Michaels
> > Was userspace maybe performing concurrent CPU access to the BOs in > question? As far as I know Julien has demonstrated that this is not the case. > I hope we can find a better solution. Understood -- I thought you might not want to take this patch, but I went ahead and sent it out becaus

[PATCH] Revert "drm/radeon: Try evicting from CPU accessible to inaccessible VRAM first"

2017-03-22 Thread Zachary Michaels
We were experiencing an infinite loop due to VRAM bos getting added back to the VRAM lru on eviction via ttm_bo_mem_force_space, and reverting this commit solves the problem. Signed-off-by: Zachary Michaels Signed-off-by: Julien Isorce --- drivers/gpu/drm/radeon/radeon_ttm.c | 25