MirrorHub/construct
0
0
Fork 0
mirror of https://github.com/matrix-construct/construct synced 2024-11-18 16:00:57 +01:00
Code Issues Releases Wiki Activity

ircd:Ⓜ️:vm: Partial revert of a35328f38; remove mutex; use sequencing.

Browse source
This commit is contained in:
Jason Volk 2019-09-10 14:41:39 -07:00
parent ee3dcf5075
commit ec4c6c46ac
3 changed files with 6 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
include/ircd/m/vm.h
View file

@ -37,7 +37,6 @@ namespace ircd::m::vm
namespace ircd::m::vm::sequence
{
extern ctx::dock dock;
extern ctx::mutex mutex;
extern uint64_t retired; // already written; always monotonic
extern uint64_t committed; // pending write; usually monotonic
extern uint64_t uncommitted; // evaluating; not monotonic

3
ircd/m.cc
View file

@ -1536,9 +1536,6 @@ ircd::m::vm::eval::operator()(const event &event)
// sequence
//
decltype(ircd::m::vm::sequence::mutex)
ircd::m::vm::sequence::mutex;
decltype(ircd::m::vm::sequence::dock)
ircd::m::vm::sequence::dock;

17
modules/m_vm.cc
View file

@ -949,6 +949,12 @@ ircd::m::vm::execute_pdu(eval &eval,
assert(sequence::committed < sequence::get(eval));
assert(sequence::retired < sequence::get(eval));
sequence::committed = sequence::get(eval);
if(m::exists(event_id) && !opts.replays)
throw error
{
fault::EXISTS, "Write commit rejected :Event has already been evaluated."
};
if(opts.write)
write_prepare(eval, event);
@ -963,17 +969,6 @@ ircd::m::vm::execute_pdu(eval &eval,
// Commit the transaction to database iff this eval is at the stack base.
if(opts.write && !eval.sequence_shared[0])
{
const std::lock_guard lock
{
sequence::mutex
};
if(m::exists(event_id) && !opts.replays)
throw error
{
fault::EXISTS, "Write commit rejected :Event has already been evaluated."
};
write_commit(eval);
}