Op 19-11-12 14:26, Thomas Hellstrom schreef: > Hi, > > On 11/12/2012 03:00 PM, Maarten Lankhorst wrote: >> move to release_list instead > > Can you describe why this change is made? cleanup? reorder locks in a later > patch? > Also please describe why you need move_notify and ttm unbind / destroy to be > outside of > reservation, because that's the main change in this patch and it's not even > mentioned in the > commit message. Ok is a reword enough? In that case I'll resend.
I moved all the destruction to happen when release_list refcount drops to 0. This removes the special handling of ttm_bo_cleanup_memtype_use, and makes it part of the normal bo destruction instead. It also meant that move_notify and unbind/destroy was without reservation, simply because it was done during normal destruction instead. At that point you may no longer hold a reservation, but you can already be sure you're the only one touching it. It is optional and I can drop this patch if the behavior change is unwanted. ~Maarten
