Hi, Christian,
On 12/14/2017 09:40 AM, Christian König wrote:
Hi Thomas,
sorry for that. Noted on the rest of that series as well that we need
to improve the commit messages. But this one somehow slipped through
because I discussed this change previously internally with Roger.
That made the change completely logical for me, but without this
context everybody else just thinks "Hui what?". Going to keep that in
mind the next time.
But back to topic: This series allows BOs which share the same
reservation object as the BO currently allocated/validated to be
evicted even when they are reserved.
This is useful because amdgpu wants to use a single reservation object
for almost all BOs of a process.
Yes, that indeed makes the whole thing more clear, and makes sense.
Out of interest, is the shared reservation object usage a speed
optimization (avoiding the ww_mutex_locks at reservation time?)
or something else?
I guess that even if LRU lists might get crowded with unevictable BOs,
iterating through those lists isn't really part of the fast path.
/Thomas
Regards,
Christian.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
