On Wednesday, September 17, 2008 5:48 am Thomas Hellström wrote: > Jesse Barnes wrote: > > Ok, here's an updated patch (note: only compile tested!) which I think > > addresses all of the comments I received. I switched over to using > > dev_mapping for unmapping and tried to cleanup the mm mmap & private > > fields a bit. > > > > How does it look? > > > > Thanks, > > Jesse > > Oops, sorry for the previous spam, I hit "send" too soon. > > This looks better, only the > unmap_mapping_range() <-> fault() > race remains. Should be easily fixable with a mutex, though.
Right I'll fix that up and see if Eric can roll this into his GEM tree (after testing of course). Thanks a lot for your review. Jesse ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ -- _______________________________________________ Dri-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/dri-devel
