Hi,

Unless I'm mistaken, ceph_crc32() is currently used in master via the crc32c() 
method of bufferlist to:

* encode_with_checksum/decode_with_checksum a PGLog entry
* Message::decode_message/Message::encode_message a message via calc_*_crc 
* FileJournal::do_read_entry/FileJournal::prepare_single_write a journal entry
* for information in the report monitory command ( Monitor.cc )

Erasure coded chunks ( i.e. files ) will need checksums. Should this be 
implemented as an optional feature in ceph/src/os/FileStore.{h,cc} ? If the 
underlying filesystem does not provide this feature, FileStore would call 
ceph_crc32 each time the object is modified. A verification method would be 
exposed and used when scrubbing erasure coded pools.

Cheers

-- 
Loïc Dachary, Artisan Logiciel Libre
All that is necessary for the triumph of evil is that good people do nothing.

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to