reduce roomid_mutex_federation

Signed-off-by: Jason Volk <jason@zemos.net>

src/api/server/send.rs

@@ -1,4 +1,4 @@
-use std::{collections::BTreeMap, sync::Arc, time::Instant};
+use std::{collections::BTreeMap, time::Instant};
 
 use axum_client_ip::InsecureClientIp;
 use conduit::debug_warn;
@@ -107,16 +107,11 @@ pub(crate) async fn send_transaction_message_route(
 	let mut resolved_map = BTreeMap::new();
 	for (event_id, value, room_id) in parsed_pdus {
 		let pdu_start_time = Instant::now();
-		let mutex = Arc::clone(
-			services()
+		let mutex_lock = services()
 			.globals
 			.roomid_mutex_federation
-				.write()
-				.await
-				.entry(room_id.clone())
-				.or_default(),
-		);
-		let mutex_lock = mutex.lock().await;
+			.lock(&room_id)
+			.await;
 		resolved_map.insert(
 			event_id.clone(),
 			services()

src/api/server/send_join.rs

@@ -1,6 +1,6 @@
 #![allow(deprecated)]
 
-use std::{collections::BTreeMap, sync::Arc};
+use std::collections::BTreeMap;
 
 use ruma::{
 	api::{client::error::ErrorKind, federation::membership::create_join_event},
@@ -148,16 +148,11 @@ async fn create_join_event(
 		.fetch_required_signing_keys([&value], &pub_key_map)
 		.await?;
 
-	let mutex = Arc::clone(
-		services()
+	let mutex_lock = services()
 		.globals
 		.roomid_mutex_federation
-			.write()
-			.await
-			.entry(room_id.to_owned())
-			.or_default(),
-	);
-	let mutex_lock = mutex.lock().await;
+		.lock(room_id)
+		.await;
 	let pdu_id: Vec<u8> = services()
 		.rooms
 		.event_handler

src/api/server/send_leave.rs

@@ -1,6 +1,6 @@
 #![allow(deprecated)]
 
-use std::{collections::BTreeMap, sync::Arc};
+use std::collections::BTreeMap;
 
 use ruma::{
 	api::{client::error::ErrorKind, federation::membership::create_leave_event},
@@ -154,16 +154,11 @@ async fn create_leave_event(origin: &ServerName, room_id: &RoomId, pdu: &RawJson
 		.fetch_required_signing_keys([&value], &pub_key_map)
 		.await?;
 
-	let mutex = Arc::clone(
-		services()
+	let mutex_lock = services()
 		.globals
 		.roomid_mutex_federation
-			.write()
-			.await
-			.entry(room_id.to_owned())
-			.or_default(),
-	);
-	let mutex_lock = mutex.lock().await;
+		.lock(room_id)
+		.await;
 	let pdu_id: Vec<u8> = services()
 		.rooms
 		.event_handler

src/service/globals/mod.rs

@@ -56,7 +56,7 @@ pub struct Service {
 	pub bad_query_ratelimiter: Arc<RwLock<HashMap<OwnedServerName, RateLimitState>>>,
 	pub roomid_mutex_insert: MutexMap<OwnedRoomId, ()>,
 	pub roomid_mutex_state: RwLock<HashMap<OwnedRoomId, Arc<Mutex<()>>>>,
-	pub roomid_mutex_federation: RwLock<HashMap<OwnedRoomId, Arc<Mutex<()>>>>, // this lock will be held longer
+	pub roomid_mutex_federation: MutexMap<OwnedRoomId, ()>,
 	pub roomid_federationhandletime: RwLock<HashMap<OwnedRoomId, (OwnedEventId, Instant)>>,
 	pub updates_handle: Mutex<Option<JoinHandle<()>>>,
 	pub stateres_mutex: Arc<Mutex<()>>,
@@ -118,7 +118,7 @@ impl Service {
 			bad_query_ratelimiter: Arc::new(RwLock::new(HashMap::new())),
 			roomid_mutex_state: RwLock::new(HashMap::new()),
 			roomid_mutex_insert: MutexMap::<OwnedRoomId, ()>::new(),
-			roomid_mutex_federation: RwLock::new(HashMap::new()),
+			roomid_mutex_federation: MutexMap::<OwnedRoomId, ()>::new(),
 			roomid_federationhandletime: RwLock::new(HashMap::new()),
 			updates_handle: Mutex::new(None),
 			stateres_mutex: Arc::new(Mutex::new(())),

src/service/rooms/timeline/mod.rs

@@ -1122,16 +1122,11 @@ impl Service {
 		let (event_id, value, room_id) = parse_incoming_pdu(&pdu)?;
 
 		// Lock so we cannot backfill the same pdu twice at the same time
-		let mutex = Arc::clone(
-			services()
+		let mutex_lock = services()
 			.globals
 			.roomid_mutex_federation
-				.write()
-				.await
-				.entry(room_id.clone())
-				.or_default(),
-		);
-		let mutex_lock = mutex.lock().await;
+			.lock(&room_id)
+			.await;
 
 		// Skip the PDU if we already have it as a timeline event
 		if let Some(pdu_id) = self.get_pdu_id(&event_id)? {