On Wed, Dec 5, 2012 at 8:15 PM, Jerome Glisse <j.glisse at gmail.com> wrote: > On Fri, Nov 30, 2012 at 7:11 AM, Maarten Lankhorst > <maarten.lankhorst at canonical.com> wrote: >> With all the previous patches there shouldn't be anything lying on the >> reservations being atomic >> with removal of the bo's from the lru lists any more. >> >> As such we can finally fix the locking primitives and make it act like >> normal mutex calls. >> >> Patch 1 is the actual removal of the guarantee in ttm_bo_reserve >> patch 2 is a cleanup of ttm_eu_backoff_reservation from removing that >> guarantee >> patch 3...6 are introducing ttm_bo_reserve_slowpath. >> >> After this series, this should directly map to my proposed extensions to >> mutex. > > For the series : > Reviewed-by: Jerome Glisse <jglisse at redhat.com> > > Few comments in some patches but nothing important.
Actually while about to send my comments i thought of some bad scenario see patch 4. Cheers, Jerome