Re: [Qemu-devel] [RFC V6 07/11] quorum: Add quorum mechanism.

[Benoît Canet]

> Hmm, adler32() is basically a weak hashing mechanism; are you sure you
> won't have any false collisions?  Furthermore, how does this compare
> with the series for adding deduplication, which uses much
> stronger/longer hashes, but where those take more time to compute?  Is
> there any way you can share efforts between the two series?

I agree that using sha256 would be a better option as it will not be called
frequently.
However the call to gnutls is a one liner so it's likely not shareable.
The configure would need to be modified to require gnutls and It likely not
factorisable either.

I'll change the code to use sha256.

Regards

Benoît


> 
> > +    /* get the index of the first successfull read */
> 
> s/successfull/successful/ (several times in this patch)
> 
> -- 
> Eric Blake   eblake redhat com    +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
>