[PATCH RFC 2/2] drm/amdgpu: Set/clear CPU_ACCESS flag on page fault and move to VRAM

Christian König deathsimple at vodafone.de
Mon Jul 3 11:47:23 UTC 2017


Am 03.07.2017 um 11:49 schrieb Michel Dänzer:
>> Instead of messing with all this I suggest that we just add a jiffies
>> based timeout to the BO when we can clear the flag. For kernel BOs this
>> timeout is just infinity.
>>
>> Then we check in amdgpu_cs_bo_validate() before generating the
>> placements if we could clear the flag and do so based on the timeout.
> The idea for this patch was to save the memory and CPU cycles needed for
> that approach.
But when we clear the flag on the end of the move we already moved the 
BO to visible VRAM again.

Only on after the next swapout/swapin cycle we see an effect of that change.

Is that the intended approach?

Regards,
Christian.


More information about the amd-gfx mailing list