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