[OpenAFS-devel] CheckHost patch version 2

Jim Rees rees@umich.edu
Mon, 21 Nov 2005 16:27:40 -0500


  well, you can but you need to call h_Lock(host) and not h_Lock_r().

Yes, but no one does.

I think my next step is to rationalize the locks so that H_LOCK is only (and
always) used to protect the hash table, and h_Lock is only used to protect
the host entry.  I think the order should be the other way around, too.
With the current code the host could disappear when you drop H_LOCK so you
can get h_Lock.  Unless it's always held (refcounted), which I'm not sure is
the case.