cvs commit: src/sys/ufs/ufs dirhash.h ufs_dirhash.c

2008-09-16 Thread John Baldwin
jhb 2008-09-16 16:23:56 UTC FreeBSD src repository Modified files: sys/ufs/ufs dirhash.h ufs_dirhash.c Log: SVN rev 183080 on 2008-09-16 16:23:56Z by jhb Fix a race with shared lookups on UFS. If the the dirhash code reached the cap on memory usage, then shar

Re: cvs commit: src/sys/ufs/ufs dirhash.h ufs_dirhash.c

2008-04-11 Thread David Malone
> Now instead of freeing a recycled dirhash pointer in ip->i_dirhash I lock > it and re-create it. This makes various races simpler. The dirhash has > to be minimally constructed for dirhash_free_locked to function correctly > if we bail out early. Ah - OK - I think that probably partially ex

Re: cvs commit: src/sys/ufs/ufs dirhash.h ufs_dirhash.c

2008-04-11 Thread Jeff Roberson
On Fri, 11 Apr 2008, David Malone wrote: On Fri, Apr 11, 2008 at 09:48:12AM +, Jeff Roberson wrote: - Use a lockmgr lock rather than a mtx to protect dirhash. This lock may be held for the duration of the various dirhash operations which avoids many complex unlock/lock/revalid

Re: cvs commit: src/sys/ufs/ufs dirhash.h ufs_dirhash.c

2008-04-11 Thread David Malone
On Fri, Apr 11, 2008 at 09:48:12AM +, Jeff Roberson wrote: >- Use a lockmgr lock rather than a mtx to protect dirhash. This lock > may be held for the duration of the various dirhash operations which > avoids many complex unlock/lock/revalidate sequences. >- Permit shared loc

cvs commit: src/sys/ufs/ufs dirhash.h ufs_dirhash.c

2008-04-11 Thread Jeff Roberson
jeff2008-04-11 09:48:12 UTC FreeBSD src repository Modified files: sys/ufs/ufs dirhash.h ufs_dirhash.c Log: - Use a lockmgr lock rather than a mtx to protect dirhash. This lock may be held for the duration of the various dirhash operations which avoids ma