diff options
author | Alex Auvolat <alex@adnab.me> | 2022-08-31 16:10:14 +0200 |
---|---|---|
committer | Alex Auvolat <alex@adnab.me> | 2022-08-31 16:10:14 +0200 |
commit | 984ba65e658a920f2d8e67ce808e9acd257c4d86 (patch) | |
tree | 396eea229649725627dd687dfb7d4e4e392d8d03 /src/proto.rs | |
parent | 01db3c43193d9a24ef4cd1a1b1993f4d44a9aa9b (diff) | |
download | netapp-984ba65e658a920f2d8e67ce808e9acd257c4d86.tar.gz netapp-984ba65e658a920f2d8e67ce808e9acd257c4d86.zip |
Better messages in proto.rs
Diffstat (limited to 'src/proto.rs')
-rw-r--r-- | src/proto.rs | 43 |
1 files changed, 31 insertions, 12 deletions
diff --git a/src/proto.rs b/src/proto.rs index 41a2e47..8f7e70f 100644 --- a/src/proto.rs +++ b/src/proto.rs @@ -1,6 +1,6 @@ use std::collections::{HashMap, VecDeque}; -use std::sync::Arc; use std::fmt::Write; +use std::sync::Arc; use log::trace; @@ -99,7 +99,14 @@ impl SendQueue { let mut ret = String::new(); for (prio, q) in self.items.iter() { for item in q.iter() { - write!(&mut ret, " [{} {} ({})]", prio, item.data.len() - item.cursor, item.id).unwrap(); + write!( + &mut ret, + " [{} {} ({})]", + prio, + item.data.len() - item.cursor, + item.id + ) + .unwrap(); } } ret @@ -130,7 +137,13 @@ pub(crate) trait SendLoop: Sync { while !should_exit || !sending.is_empty() { trace!("send_loop({}): queue = {}", debug_name, sending.dump()); if let Ok((id, prio, data)) = msg_recv.try_recv() { - trace!("send_loop({}): got {}, {} bytes", debug_name, id, data.len()); + trace!( + "send_loop({}): new message to send, id = {}, prio = {}, {} bytes", + debug_name, + id, + prio, + data.len() + ); sending.push(SendQueueItem { id, prio, @@ -170,7 +183,13 @@ pub(crate) trait SendLoop: Sync { } else { let sth = msg_recv.recv().await; if let Some((id, prio, data)) = sth { - trace!("send_loop({}): got {}, {} bytes", debug_name, id, data.len()); + trace!( + "send_loop({}): new message to send, id = {}, prio = {}, {} bytes", + debug_name, + id, + prio, + data.len() + ); sending.push(SendQueueItem { id, prio, @@ -199,17 +218,12 @@ pub(crate) trait SendLoop: Sync { pub(crate) trait RecvLoop: Sync + 'static { fn recv_handler(self: &Arc<Self>, id: RequestID, msg: Vec<u8>); - async fn recv_loop<R>( - self: Arc<Self>, - mut read: R, - debug_name: String, - ) -> Result<(), Error> + async fn recv_loop<R>(self: Arc<Self>, mut read: R, debug_name: String) -> Result<(), Error> where R: AsyncReadExt + Unpin + Send + Sync, { let mut receiving = HashMap::new(); loop { - trace!("recv_loop({}): reading packet", debug_name); let mut header_id = [0u8; RequestID::BITS as usize / 8]; match read.read_exact(&mut header_id[..]).await { Ok(_) => (), @@ -217,12 +231,17 @@ pub(crate) trait RecvLoop: Sync + 'static { Err(e) => return Err(e.into()), }; let id = RequestID::from_be_bytes(header_id); - trace!("recv_loop({}): got header id: {:04x}", debug_name, id); let mut header_size = [0u8; ChunkLength::BITS as usize / 8]; read.read_exact(&mut header_size[..]).await?; let size = ChunkLength::from_be_bytes(header_size); - trace!("recv_loop({}): got header size: {:04x}", debug_name, size); + trace!( + "recv_loop({}): got header id = {}, size = 0x{:04x} ({} bytes)", + debug_name, + id, + size, + size & !CHUNK_HAS_CONTINUATION + ); let has_cont = (size & CHUNK_HAS_CONTINUATION) != 0; let size = size & !CHUNK_HAS_CONTINUATION; |