[PATCH 2/2] drm/amdgpu: Optimize mutex usage (v2)

Michel Dänzer michel at daenzer.net
Fri Jun 16 05:45:50 UTC 2017


On 16/06/17 02:03 PM, Alex Xie wrote:
> Use rw_semaphore instead of mutex for bo_lists.
> 
> In original function amdgpu_bo_list_get, the waiting
> for result->lock can be quite long while mutex
> bo_list_lock was holding. It can make other tasks
> waiting for bo_list_lock for long period too.
> Change bo_list_lock to rw_semaphore can avoid most of
> such long waiting.
> 
> Secondly, this patch allows several tasks(readers of idr)
> to proceed at the same time.
> 
> v2: use rcu and kref (Dave Airlie and Christian König)

At least the first two paragraphs above are inaccurate and confusing in
v2. Please modify the commit log to reflect what the patch actually does
now.


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer


More information about the amd-gfx mailing list