[OpenAFS] Re: [OpenAFS-devel] 1.6 and post-1.6 OpenAFS branch management and schedule

[Rainer Toebbicke]

Tom Keiser schrieb:

> 
> I'll note that bitmap-later is also dangerous--it has several known
> race conditions (e.g. VFreeBitmapEntry_r is just plain wrong;
> GetBitmap() relies upon microarchitectural store ordering rules that
> no modern processor guarantees, ...).  These can result in various
> classes of corruption from vnodes that fail to be freed until salvage,
> to multiple allocations of the same vnode.
> 
> 

Thanks for the hint - could you elaborate on that?

I frowned first time I saw VOL_UNLOCK prior to calling GetBitmap but trusted 
in the code being water-proof. Perhaps this deserves a fix until the dust 
around DAFS settles.



-- 
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Rainer Toebbicke
European Laboratory for Particle Physics(CERN) - Geneva, Switzerland
Phone: +41 22 767 8985       Fax: +41 22 767 7155