From 2e3f7a4c9fd09ebf0e980b80c67b5d40c7f3b08e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Joakim=20Frosteg=C3=A5rd?= Date: Sat, 20 Nov 2021 15:14:02 +0100 Subject: [PATCH] Update TODO --- TODO.md | 24 ++++-------------------- 1 file changed, 4 insertions(+), 20 deletions(-) diff --git a/TODO.md b/TODO.md index 5927ae5..98be979 100644 --- a/TODO.md +++ b/TODO.md @@ -2,13 +2,13 @@ * readme * document privilege dropping and cpu pinning - * document access list log output levels and that it exits program - on start if parsing fails * config * add flag to print parsed config when starting * fail on unrecognized keys +* quit whole program if any thread panics + * implement socket_recv_size and ipv6_only in glommio implementations * CI @@ -44,7 +44,7 @@ * glommio * fix memory leak / huge growth -## Less important +# Less important * extract response peers: extract "one extra" to compensate for removal, of sender if present in selection? maybe make criterion benchmark, @@ -65,30 +65,14 @@ * very small amount of connections means very small number of peers per torrents, so tracker handling of large number is not really assessed -## aquatic_udp -* use key from request as part of peer map key like in aquatic_http? need to - check protocol specification - # Not important -## General -* have main thread wait for any of the threads returning, quit - if that is the since since it means a panic occured - ## aquatic_ws * write new version of extract_response_peers which checks for equality with peer sending request? It could return an arrayvec or smallvec by the way (but then the size needs to be adjusted together with the corresponding config var, or the config var needs to be removed) -## aquatic_udp -* Does it really make sense to include peer address in peer map key? I have - to think about why I included it in the first place. - -## aquatic_udp_protocol -* Don't do endian conversion where unnecessary, such as for connection id and - transaction id? - ## aquatic_cli_helpers * Include config field comments in exported toml (likely quite a bit of work) @@ -108,4 +92,4 @@ ## aquatic_udp_protocol * Use `bytes` crate: seems to worsen performance somewhat * Zerocopy (https://docs.rs/zerocopy/0.3.0/zerocopy/index.html) for requests - and responses? Doesn't work on Vec etc + and responses. Doesn't improve performance