> On Friday 23 April 2010 15:32:57 Owain Ainsworth wrote: > > CVSROOT: /cvs > > Module name: src > > Changes by: o...@cvs.openbsd.org 2010/04/23 13:32:57 > > > > Modified files: > > sys/ntfs : ntfs_ihash.c > > > > Log message: > > It is about time that we stopped pretending simple_locks are locks. > > > > replace ntfs_nthash_slock usage with comments prefixed XXXLOCKING (for > > grepability). > > > > This lock looks to be correct, but it could well be the bad way to do it > > (having a rwlock for inserts to avoid races inserting the same inode but > > then simple locking on list accesses). > > > > approach discussed with deraadt@ > > First, thanks for the bit of love on the ntfs stuff. I use my laptop > routinely now, to rescue files from diseased Windows machines. > > For this change (and the cd?), is there anything special I should be > looking out for, when testing this? Anything that might push the > envelope some?
Read the commit again. It changes nothing.
