Removed RSIs about the writeback's mc_transaction_t constructor.
Made snapshots_test.cc use the read-only version of the transaction_t constructor.
Removed a bunch of SAMRSIs in the unit tests about disk_ack_signal, that were unnecessary.
Made writeback_t::sync and writeback_t::sync_patiently not return a boolean, fix shutdown "deadlock" bug.