Merge pull request #2877

43f5269f Wallets now do not depend on the daemon rpc lib (moneromooo-monero)
bb89ae8b move connection_basic and network_throttle from src/p2p to epee (moneromooo-monero)
4abf25f3 cryptonote_core does not depend on p2p anymore (moneromooo-monero)

10 files changed, 90 insertions, 1261 deletions

diff --git a/src/p2p/connection_basic.cpp b/src/p2p/connection_basic.cpp
deleted file mode 100644
index 06baa7893..000000000
--- a/src/p2p/connection_basic.cpp
+++ /dev/null
@@ -1,290 +0,0 @@
-/// @file
-/// @author rfree (current maintainer in monero.cc project)
-/// @brief base for connection, contains e.g. the ratelimit hooks
-
-// Copyright (c) 2014-2017, The Monero Project
-// 
-// All rights reserved.
-// 
-// Redistribution and use in source and binary forms, with or without modification, are
-// permitted provided that the following conditions are met:
-// 
-// 1. Redistributions of source code must retain the above copyright notice, this list of
-//    conditions and the following disclaimer.
-// 
-// 2. Redistributions in binary form must reproduce the above copyright notice, this list
-//    of conditions and the following disclaimer in the documentation and/or other
-//    materials provided with the distribution.
-// 
-// 3. Neither the name of the copyright holder nor the names of its contributors may be
-//    used to endorse or promote products derived from this software without specific
-//    prior written permission.
-// 
-// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY
-// EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
-// MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL
-// THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
-// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
-// PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
-// INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
-// STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF
-// THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-
-/* rfree: implementation for the non-template base, can be used by connection<> template class in abstract_tcp_server2 file  */
-
-#include "connection_basic.hpp"
-
-#include <boost/asio.hpp>
-#include <string>
-#include <vector>
-#include <boost/noncopyable.hpp>
-#include <boost/shared_ptr.hpp>
-#include <atomic>
-
-#include <boost/asio.hpp>
-#include <boost/array.hpp>
-#include <boost/noncopyable.hpp>
-#include <boost/shared_ptr.hpp>
-#include <boost/enable_shared_from_this.hpp>
-#include <boost/interprocess/detail/atomic.hpp>
-#include <boost/thread/thread.hpp>
-
-#include <memory>
-
-#include "syncobj.h"
-
-#include "net/net_utils_base.h" 
-#include "misc_log_ex.h" 
-#include <boost/lambda/bind.hpp>
-#include <boost/lambda/lambda.hpp>
-#include <boost/uuid/random_generator.hpp>
-#include <boost/chrono.hpp>
-#include <boost/utility/value_init.hpp>
-#include <boost/asio/deadline_timer.hpp>
-#include <boost/date_time/posix_time/posix_time.hpp>
-#include <boost/thread/thread.hpp>
-#include <boost/filesystem.hpp>
-#include "misc_language.h"
-#include "pragma_comp_defs.h"
-#include <fstream>
-#include <sstream>
-#include <iomanip>
-#include <algorithm>
-#include <mutex>
-
-#include <boost/asio/basic_socket.hpp>
-#include <boost/asio/ip/unicast.hpp>
-#include "net/abstract_tcp_server2.h"
-
-// TODO:
-#include "network_throttle-detail.hpp"
-#include "cryptonote_core/cryptonote_core.h"
-
-#undef MONERO_DEFAULT_LOG_CATEGORY
-#define MONERO_DEFAULT_LOG_CATEGORY "net.p2p"
-
-// ################################################################################################
-// local (TU local) headers
-// ################################################################################################
-
-namespace epee
-{
-namespace net_utils
-{
-
-  std::string to_string(t_connection_type type)
-  {
-	  if (type == e_connection_type_NET)
-		return std::string("NET");
-	  else if (type == e_connection_type_RPC)
-	    return std::string("RPC");
-	  else if (type == e_connection_type_P2P)
-	    return std::string("P2P");
-	  
-	  return std::string("UNKNOWN");
-  }
-
-
-/* ============================================================================ */
-
-class connection_basic_pimpl {
-	public:
-		connection_basic_pimpl(const std::string &name);
-
-		static int m_default_tos;
-
-		network_throttle_bw m_throttle; // per-perr
-    critical_section m_throttle_lock;
-
-		int m_peer_number; // e.g. for debug/stats
-};
-
-
-} // namespace
-} // namespace
-
-// ################################################################################################
-// The implementation part
-// ################################################################################################
-
-namespace epee
-{
-namespace net_utils
-{
-
-// ================================================================================================
-// connection_basic_pimpl
-// ================================================================================================
-	
-connection_basic_pimpl::connection_basic_pimpl(const std::string &name) : m_throttle(name) { }
-
-// ================================================================================================
-// connection_basic
-// ================================================================================================
-
-// static variables:
-int connection_basic_pimpl::m_default_tos;
-
-// methods:
-connection_basic::connection_basic(boost::asio::io_service& io_service, std::atomic<long> &ref_sock_count, std::atomic<long> &sock_number)
-	: 
-	mI( new connection_basic_pimpl("peer") ),
-	strand_(io_service),
-	socket_(io_service),
-	m_want_close_connection(false), 
-	m_was_shutdown(false),
-	m_ref_sock_count(ref_sock_count)
-{ 
-	++ref_sock_count; // increase the global counter
-	mI->m_peer_number = sock_number.fetch_add(1); // use, and increase the generated number
-
-	std::string remote_addr_str = "?";
-	try { boost::system::error_code e; remote_addr_str = socket_.remote_endpoint(e).address().to_string(); } catch(...){} ;
-
-	_note("Spawned connection p2p#"<<mI->m_peer_number<<" to " << remote_addr_str << " currently we have sockets count:" << m_ref_sock_count);
-	//boost::filesystem::create_directories("log/dr-monero/net/");
-}
-
-connection_basic::~connection_basic() noexcept(false) {
-	std::string remote_addr_str = "?";
-	m_ref_sock_count--;
-	try { boost::system::error_code e; remote_addr_str = socket_.remote_endpoint(e).address().to_string(); } catch(...){} ;
-	_note("Destructing connection p2p#"<<mI->m_peer_number << " to " << remote_addr_str);
-}
-
-void connection_basic::set_rate_up_limit(uint64_t limit) {
-	{
-		CRITICAL_REGION_LOCAL(	network_throttle_manager::m_lock_get_global_throttle_out );
-		network_throttle_manager::get_global_throttle_out().set_target_speed(limit);
-	}
-	save_limit_to_file(limit);
-}
-
-void connection_basic::set_rate_down_limit(uint64_t limit) {
-	{
-	  CRITICAL_REGION_LOCAL(	network_throttle_manager::m_lock_get_global_throttle_in );
-		network_throttle_manager::get_global_throttle_in().set_target_speed(limit);
-	}
-
-	{
-	  CRITICAL_REGION_LOCAL(	network_throttle_manager::m_lock_get_global_throttle_inreq );
-		network_throttle_manager::get_global_throttle_inreq().set_target_speed(limit);
-	}
-    save_limit_to_file(limit);
-}
-
-uint64_t connection_basic::get_rate_up_limit() {
-    uint64_t limit;
-    {
-         CRITICAL_REGION_LOCAL( network_throttle_manager::m_lock_get_global_throttle_out );
-         limit = network_throttle_manager::get_global_throttle_out().get_target_speed();
-	}
-    return limit;
-}
-
-uint64_t connection_basic::get_rate_down_limit() {
-    uint64_t limit;
-    {
-         CRITICAL_REGION_LOCAL( network_throttle_manager::m_lock_get_global_throttle_in );
-         limit = network_throttle_manager::get_global_throttle_in().get_target_speed();
-	}
-    return limit;
-}
-
-void connection_basic::save_limit_to_file(int limit) {
-}
- 
-void connection_basic::set_tos_flag(int tos) {
-	connection_basic_pimpl::m_default_tos = tos;
-}
-
-int connection_basic::get_tos_flag() {
-	return connection_basic_pimpl::m_default_tos;
-}
-
-void connection_basic::sleep_before_packet(size_t packet_size, int phase,  int q_len) {
-	double delay=0; // will be calculated
-	do
-	{ // rate limiting
-		if (m_was_shutdown) { 
-			_dbg2("m_was_shutdown - so abort sleep");
-			return;
-		}
-
-		{ 
-			CRITICAL_REGION_LOCAL(	network_throttle_manager::m_lock_get_global_throttle_out );
-			delay = network_throttle_manager::get_global_throttle_out().get_sleep_time_after_tick( packet_size );
-		}
-
-		delay *= 0.50;
-		if (delay > 0) {
-            long int ms = (long int)(delay * 1000);
-			MTRACE("Sleeping in " << __FUNCTION__ << " for " << ms << " ms before packet_size="<<packet_size); // debug sleep
-			boost::this_thread::sleep(boost::posix_time::milliseconds( ms ) );
-		}
-	} while(delay > 0);
-
-// XXX LATER XXX
-	{
-	  CRITICAL_REGION_LOCAL(	network_throttle_manager::m_lock_get_global_throttle_out );
-		network_throttle_manager::get_global_throttle_out().handle_trafic_exact( packet_size ); // increase counter - global
-	}
-
-}
-void connection_basic::set_start_time() {
-	CRITICAL_REGION_LOCAL(	network_throttle_manager::m_lock_get_global_throttle_out );
-	m_start_time = network_throttle_manager::get_global_throttle_out().get_time_seconds();
-}
-
-void connection_basic::do_send_handler_write(const void* ptr , size_t cb ) {
-        // No sleeping here; sleeping is done once and for all in connection<t_protocol_handler>::handle_write
-	MTRACE("handler_write (direct) - before ASIO write, for packet="<<cb<<" B (after sleep)");
-	set_start_time();
-}
-
-void connection_basic::do_send_handler_write_from_queue( const boost::system::error_code& e, size_t cb, int q_len ) {
-        // No sleeping here; sleeping is done once and for all in connection<t_protocol_handler>::handle_write
-	MTRACE("handler_write (after write, from queue="<<q_len<<") - before ASIO write, for packet="<<cb<<" B (after sleep)");
-
-	set_start_time();
-}
-
-void connection_basic::logger_handle_net_read(size_t size) { // network data read
-}
-
-void connection_basic::logger_handle_net_write(size_t size) {
-}
-
-double connection_basic::get_sleep_time(size_t cb) {
-	CRITICAL_REGION_LOCAL(epee::net_utils::network_throttle_manager::network_throttle_manager::m_lock_get_global_throttle_out);
-    auto t = network_throttle_manager::get_global_throttle_out().get_sleep_time(cb);
-    return t;
-}
-
-void connection_basic::set_save_graph(bool save_graph) {
-}
-
-
-} // namespace
-} // namespace
-
diff --git a/src/p2p/connection_basic.hpp b/src/p2p/connection_basic.hpp
deleted file mode 100644
index 16de469a7..000000000
--- a/src/p2p/connection_basic.hpp
+++ /dev/null
@@ -1,141 +0,0 @@
-/// @file
-/// @author rfree (current maintainer in monero.cc project)
-/// @brief base for connection, contains e.g. the ratelimit hooks
-
-// ! This file might contain variable names same as in template class connection<> 
-// ! from files contrib/epee/include/net/abstract_tcp_server2.*
-// ! I am not a lawyer; afaik APIs, var names etc are not copyrightable ;)
-// ! (how ever if in some wonderful juristdictions that is not the case, then why not make another sub-class withat that members and licence it as epee part)
-// ! Working on above premise, IF this is valid in your juristdictions, then consider this code as released as:
-
-// Copyright (c) 2014-2017, The Monero Project
-// 
-// All rights reserved.
-// 
-// Redistribution and use in source and binary forms, with or without modification, are
-// permitted provided that the following conditions are met:
-// 
-// 1. Redistributions of source code must retain the above copyright notice, this list of
-//    conditions and the following disclaimer.
-// 
-// 2. Redistributions in binary form must reproduce the above copyright notice, this list
-//    of conditions and the following disclaimer in the documentation and/or other
-//    materials provided with the distribution.
-// 
-// 3. Neither the name of the copyright holder nor the names of its contributors may be
-//    used to endorse or promote products derived from this software without specific
-//    prior written permission.
-// 
-// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY
-// EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
-// MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL
-// THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
-// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
-// PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
-// INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
-// STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF
-// THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-
-/* rfree: place for hanlers for the non-template base, can be used by connection<> template class in abstract_tcp_server2 file  */
-
-#ifndef INCLUDED_p2p_connection_basic_hpp
-#define INCLUDED_p2p_connection_basic_hpp
-
-
-#include <boost/asio.hpp>
-#include <string>
-#include <vector>
-#include <boost/noncopyable.hpp>
-#include <boost/shared_ptr.hpp>
-#include <atomic>
-
-#include <boost/asio.hpp>
-#include <boost/array.hpp>
-#include <boost/noncopyable.hpp>
-#include <boost/shared_ptr.hpp>
-#include <boost/enable_shared_from_this.hpp>
-#include <boost/interprocess/detail/atomic.hpp>
-#include <boost/thread/thread.hpp>
-
-#include <memory>
-
-#include "net/net_utils_base.h"
-#include "syncobj.h"
-
-namespace epee
-{
-namespace net_utils
-{
-
-  /************************************************************************/
-  /*                                                                      */
-  /************************************************************************/
-  /// Represents a single connection from a client.
-
-class connection_basic_pimpl; // PIMPL for this class
-
-  enum t_connection_type { // type of the connection (of this server), e.g. so that we will know how to limit it
-	  e_connection_type_NET = 0, // default (not used?)
-	  e_connection_type_RPC = 1, // the rpc commands  (probably not rate limited, not chunked, etc)
-	  e_connection_type_P2P = 2  // to other p2p node (probably limited)
-  };
-  
-  std::string to_string(t_connection_type type);
-
-class connection_basic { // not-templated base class for rapid developmet of some code parts
-	public:
-		std::unique_ptr< connection_basic_pimpl > mI; // my Implementation
-
-		// moved here from orginal connecton<> - common member variables that do not depend on template in connection<>
-    volatile uint32_t m_want_close_connection;
-    std::atomic<bool> m_was_shutdown;
-    critical_section m_send_que_lock;
-    std::list<std::string> m_send_que;
-    volatile bool m_is_multithreaded;
-    double m_start_time;
-    /// Strand to ensure the connection's handlers are not called concurrently.
-    boost::asio::io_service::strand strand_;
-    /// Socket for the connection.
-    boost::asio::ip::tcp::socket socket_;
-
-		std::atomic<long> &m_ref_sock_count; // reference to external counter of existing sockets that we will ++/--
-	public:
-		// first counter is the ++/-- count of current sockets, the other socket_number is only-increasing ++ number generator
-		connection_basic(boost::asio::io_service& io_service, std::atomic<long> &ref_sock_count, std::atomic<long> &sock_number);
-
-		virtual ~connection_basic() noexcept(false);
-
-		// various handlers to be called from connection class:
-		void do_send_handler_write(const void * ptr , size_t cb);
-		void do_send_handler_write_from_queue(const boost::system::error_code& e, size_t cb , int q_len); // from handle_write, sending next part
-
-		void logger_handle_net_write(size_t size); // network data written
-		void logger_handle_net_read(size_t size); // network data read
-
-		void set_start_time();
-
-		// config for rate limit
-		
-		static void set_rate_up_limit(uint64_t limit);
-		static void set_rate_down_limit(uint64_t limit);
-		static uint64_t get_rate_up_limit();
-		static uint64_t get_rate_down_limit();
-
-		// config misc
-		static void set_tos_flag(int tos); // ToS / QoS flag
-		static int get_tos_flag();
-
-		// handlers and sleep
-		void sleep_before_packet(size_t packet_size, int phase, int q_len); // execute a sleep ; phase is not really used now(?)
-		static void save_limit_to_file(int limit); ///< for dr-monero
-		static double get_sleep_time(size_t cb);
-		
-		static void set_save_graph(bool save_graph);
-};
-
-} // nameserver
-} // nameserver
-
-#endif
-
-
diff --git a/src/p2p/net_node.cpp b/src/p2p/net_node.cpp
new file mode 100644
index 000000000..4ea08a1f8
--- /dev/null
+++ b/src/p2p/net_node.cpp
@@ -0,0 +1,65 @@
+// Copyright (c) 2014-2017, The Monero Project
+//
+// All rights reserved.
+//
+// Redistribution and use in source and binary forms, with or without modification, are
+// permitted provided that the following conditions are met:
+//
+// 1. Redistributions of source code must retain the above copyright notice, this list of
+//    conditions and the following disclaimer.
+//
+// 2. Redistributions in binary form must reproduce the above copyright notice, this list
+//    of conditions and the following disclaimer in the documentation and/or other
+//    materials provided with the distribution.
+//
+// 3. Neither the name of the copyright holder nor the names of its contributors may be
+//    used to endorse or promote products derived from this software without specific
+//    prior written permission.
+//
+// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY
+// EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
+// MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL
+// THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+// PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+// INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
+// STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF
+// THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+//
+// Parts of this file are originally copyright (c) 2012-2013 The Cryptonote developers
+
+#include "common/command_line.h"
+#include "net_node.h"
+
+namespace nodetool
+{
+    const command_line::arg_descriptor<std::string> arg_p2p_bind_ip        = {"p2p-bind-ip", "Interface for p2p network protocol", "0.0.0.0"};
+    const command_line::arg_descriptor<std::string> arg_p2p_bind_port = {
+        "p2p-bind-port"
+      , "Port for p2p network protocol"
+      , std::to_string(config::P2P_DEFAULT_PORT)
+      };
+    const command_line::arg_descriptor<std::string> arg_testnet_p2p_bind_port = {
+        "testnet-p2p-bind-port"
+      , "Port for testnet p2p network protocol"
+      , std::to_string(config::testnet::P2P_DEFAULT_PORT)
+      };
+    const command_line::arg_descriptor<uint32_t>    arg_p2p_external_port  = {"p2p-external-port", "External port for p2p network protocol (if port forwarding used with NAT)", 0};
+    const command_line::arg_descriptor<bool>        arg_p2p_allow_local_ip = {"allow-local-ip", "Allow local ip add to peer list, mostly in debug purposes"};
+    const command_line::arg_descriptor<std::vector<std::string> > arg_p2p_add_peer   = {"add-peer", "Manually add peer to local peerlist"};
+    const command_line::arg_descriptor<std::vector<std::string> > arg_p2p_add_priority_node   = {"add-priority-node", "Specify list of peers to connect to and attempt to keep the connection open"};
+    const command_line::arg_descriptor<std::vector<std::string> > arg_p2p_add_exclusive_node   = {"add-exclusive-node", "Specify list of peers to connect to only."
+                                                                                                  " If this option is given the options add-priority-node and seed-node are ignored"};
+    const command_line::arg_descriptor<std::vector<std::string> > arg_p2p_seed_node   = {"seed-node", "Connect to a node to retrieve peer addresses, and disconnect"};
+    const command_line::arg_descriptor<bool> arg_p2p_hide_my_port   =    {"hide-my-port", "Do not announce yourself as peerlist candidate", false, true};
+
+    const command_line::arg_descriptor<bool>        arg_no_igd  = {"no-igd", "Disable UPnP port mapping"};
+    const command_line::arg_descriptor<int64_t>     arg_out_peers = {"out-peers", "set max number of out peers", -1};
+    const command_line::arg_descriptor<int> arg_tos_flag = {"tos-flag", "set TOS flag", -1};
+
+    const command_line::arg_descriptor<int64_t> arg_limit_rate_up = {"limit-rate-up", "set limit-rate-up [kB/s]", -1};
+    const command_line::arg_descriptor<int64_t> arg_limit_rate_down = {"limit-rate-down", "set limit-rate-down [kB/s]", -1};
+    const command_line::arg_descriptor<int64_t> arg_limit_rate = {"limit-rate", "set limit-rate [kB/s]", -1};
+
+    const command_line::arg_descriptor<bool> arg_save_graph = {"save-graph", "Save data for dr monero", false};
+}
diff --git a/src/p2p/net_node.h b/src/p2p/net_node.h
index 8bbaa9138..294ccde9e 100644
--- a/src/p2p/net_node.h
+++ b/src/p2p/net_node.h
@@ -30,13 +30,6 @@
 
 #pragma once
 #include <boost/thread.hpp>
-#include <boost/bind.hpp>
-#include <boost/bimap.hpp>
-#include <boost/multi_index_container.hpp>
-#include <boost/multi_index/ordered_index.hpp>
-#include <boost/multi_index/identity.hpp>
-#include <boost/multi_index/member.hpp>
-#include <boost/multi_index/global_fun.hpp>
 #include <boost/program_options/options_description.hpp>
 #include <boost/program_options/variables_map.hpp>
 #include <boost/serialization/version.hpp>
@@ -335,8 +328,30 @@ namespace nodetool
 
     bool m_testnet;
   };
-}
 
-#include "net_node.inl"
+    const int64_t default_limit_up = 2048;
+    const int64_t default_limit_down = 8192;
+    extern const command_line::arg_descriptor<std::string> arg_p2p_bind_ip;
+    extern const command_line::arg_descriptor<std::string> arg_p2p_bind_port;
+    extern const command_line::arg_descriptor<std::string> arg_testnet_p2p_bind_port;
+    extern const command_line::arg_descriptor<uint32_t>    arg_p2p_external_port;
+    extern const command_line::arg_descriptor<bool>        arg_p2p_allow_local_ip;
+    extern const command_line::arg_descriptor<std::vector<std::string> > arg_p2p_add_peer;
+    extern const command_line::arg_descriptor<std::vector<std::string> > arg_p2p_add_priority_node;
+    extern const command_line::arg_descriptor<std::vector<std::string> > arg_p2p_add_exclusive_node;
+    extern const command_line::arg_descriptor<std::vector<std::string> > arg_p2p_seed_node;
+    extern const command_line::arg_descriptor<bool> arg_p2p_hide_my_port;
+
+    extern const command_line::arg_descriptor<bool>        arg_no_igd;
+    extern const command_line::arg_descriptor<bool>        arg_offline;
+    extern const command_line::arg_descriptor<int64_t>     arg_out_peers;
+    extern const command_line::arg_descriptor<int> arg_tos_flag;
+
+    extern const command_line::arg_descriptor<int64_t> arg_limit_rate_up;
+    extern const command_line::arg_descriptor<int64_t> arg_limit_rate_down;
+    extern const command_line::arg_descriptor<int64_t> arg_limit_rate;
+
+    extern const command_line::arg_descriptor<bool> arg_save_graph;
+}
 
 POP_WARNINGS
diff --git a/src/p2p/net_node.inl b/src/p2p/net_node.inl
index c6792f138..55be7c2cb 100644
--- a/src/p2p/net_node.inl
+++ b/src/p2p/net_node.inl
@@ -30,11 +30,10 @@
 
 // IP blocking adapted from Boolberry
 
-#pragma once
-
 #include <algorithm>
 #include <boost/date_time/posix_time/posix_time.hpp>
 #include <boost/thread/thread.hpp>
+#include <boost/bind.hpp>
 #include <atomic>
 
 #include "version.h"
@@ -70,40 +69,6 @@
 
 namespace nodetool
 {
-  namespace
-  {
-    const int64_t default_limit_up = 2048;   // kB/s
-    const int64_t default_limit_down = 8192; // kB/s
-    const command_line::arg_descriptor<std::string> arg_p2p_bind_ip        = {"p2p-bind-ip", "Interface for p2p network protocol", "0.0.0.0"};
-    const command_line::arg_descriptor<std::string> arg_p2p_bind_port = {
-        "p2p-bind-port"
-      , "Port for p2p network protocol"
-      , std::to_string(config::P2P_DEFAULT_PORT)
-      };
-    const command_line::arg_descriptor<std::string> arg_testnet_p2p_bind_port = {
-        "testnet-p2p-bind-port"
-      , "Port for testnet p2p network protocol"
-      , std::to_string(config::testnet::P2P_DEFAULT_PORT)
-      };
-    const command_line::arg_descriptor<uint32_t>    arg_p2p_external_port  = {"p2p-external-port", "External port for p2p network protocol (if port forwarding used with NAT)", 0};
-    const command_line::arg_descriptor<bool>        arg_p2p_allow_local_ip = {"allow-local-ip", "Allow local ip add to peer list, mostly in debug purposes"};
-    const command_line::arg_descriptor<std::vector<std::string> > arg_p2p_add_peer   = {"add-peer", "Manually add peer to local peerlist"};
-    const command_line::arg_descriptor<std::vector<std::string> > arg_p2p_add_priority_node   = {"add-priority-node", "Specify list of peers to connect to and attempt to keep the connection open"};
-    const command_line::arg_descriptor<std::vector<std::string> > arg_p2p_add_exclusive_node   = {"add-exclusive-node", "Specify list of peers to connect to only."
-                                                                                                  " If this option is given the options add-priority-node and seed-node are ignored"};
-    const command_line::arg_descriptor<std::vector<std::string> > arg_p2p_seed_node   = {"seed-node", "Connect to a node to retrieve peer addresses, and disconnect"};
-    const command_line::arg_descriptor<bool> arg_p2p_hide_my_port   =    {"hide-my-port", "Do not announce yourself as peerlist candidate", false, true};
-
-    const command_line::arg_descriptor<bool>        arg_no_igd  = {"no-igd", "Disable UPnP port mapping"};
-    const command_line::arg_descriptor<int64_t>     arg_out_peers = {"out-peers", "set max number of out peers", -1};
-    const command_line::arg_descriptor<int> arg_tos_flag = {"tos-flag", "set TOS flag", -1};
-
-    const command_line::arg_descriptor<int64_t> arg_limit_rate_up = {"limit-rate-up", "set limit-rate-up [kB/s]", -1};
-    const command_line::arg_descriptor<int64_t> arg_limit_rate_down = {"limit-rate-down", "set limit-rate-down [kB/s]", -1};
-    const command_line::arg_descriptor<int64_t> arg_limit_rate = {"limit-rate", "set limit-rate [kB/s]", -1};
-
-    const command_line::arg_descriptor<bool> arg_save_graph = {"save-graph", "Save data for dr monero", false};
-  }
   //-----------------------------------------------------------------------------------
   template<class t_payload_net_handler>
   void node_server<t_payload_net_handler>::init_options(boost::program_options::options_description& desc)
diff --git a/src/p2p/net_peerlist.h b/src/p2p/net_peerlist.h
index 8372445aa..8216e9be6 100644
--- a/src/p2p/net_peerlist.h
+++ b/src/p2p/net_peerlist.h
@@ -33,8 +33,6 @@
 #include <list>
 #include <set>
 #include <map>
-//#include <boost/bimap.hpp>
-//#include <boost/bimap/multiset_of.hpp>
 #include <boost/archive/binary_iarchive.hpp>
 #include <boost/archive/portable_binary_oarchive.hpp>
 #include <boost/archive/portable_binary_iarchive.hpp>
diff --git a/src/p2p/network_throttle-detail.cpp b/src/p2p/network_throttle-detail.cpp
deleted file mode 100644
index 651e01e6b..000000000
--- a/src/p2p/network_throttle-detail.cpp
+++ /dev/null
@@ -1,369 +0,0 @@
-/// @file
-/// @author rfree (current maintainer in monero.cc project)
-/// @brief implementaion for throttling of connection (count and rate-limit speed etc)
-
-// Copyright (c) 2014-2017, The Monero Project
-// 
-// All rights reserved.
-// 
-// Redistribution and use in source and binary forms, with or without modification, are
-// permitted provided that the following conditions are met:
-// 
-// 1. Redistributions of source code must retain the above copyright notice, this list of
-//    conditions and the following disclaimer.
-// 
-// 2. Redistributions in binary form must reproduce the above copyright notice, this list
-//    of conditions and the following disclaimer in the documentation and/or other
-//    materials provided with the distribution.
-// 
-// 3. Neither the name of the copyright holder nor the names of its contributors may be
-//    used to endorse or promote products derived from this software without specific
-//    prior written permission.
-// 
-// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY
-// EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
-// MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL
-// THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
-// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
-// PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
-// INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
-// STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF
-// THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-
-/* rfree: implementation for throttle details */
-
-#include <boost/asio.hpp>
-#include <string>
-#include <vector>
-#include <boost/noncopyable.hpp>
-#include <boost/shared_ptr.hpp>
-#include <atomic>
-
-#include <boost/asio.hpp>
-#include <boost/array.hpp>
-#include <boost/noncopyable.hpp>
-#include <boost/shared_ptr.hpp>
-#include <boost/enable_shared_from_this.hpp>
-#include <boost/interprocess/detail/atomic.hpp>
-#include <boost/thread/thread.hpp>
-
-#include <memory>
-
-#include "syncobj.h"
-
-#include "net/net_utils_base.h" 
-#include "misc_log_ex.h" 
-#include <boost/lambda/bind.hpp>
-#include <boost/lambda/lambda.hpp>
-#include <boost/uuid/random_generator.hpp>
-#include <boost/chrono.hpp>
-#include <boost/utility/value_init.hpp>
-#include <boost/asio/deadline_timer.hpp>
-#include <boost/date_time/posix_time/posix_time.hpp>
-#include <boost/thread/thread.hpp> 
-#include "misc_language.h"
-#include "pragma_comp_defs.h"
-#include <sstream>
-#include <iomanip>
-#include <algorithm>
-
-
-
-#include <boost/asio/basic_socket.hpp>
-#include <boost/asio/ip/unicast.hpp>
-#include "net/abstract_tcp_server2.h"
-
-// TODO:
-#include "network_throttle-detail.hpp"
-
-#undef MONERO_DEFAULT_LOG_CATEGORY
-#define MONERO_DEFAULT_LOG_CATEGORY "net.throttle"
-
-// ################################################################################################
-// ################################################################################################
-// the "header part". Not separeted out for .hpp because point of this modification is 
-// to rebuild just 1 translation unit while working on this code.
-// (But maybe common parts will be separated out later though - if needed)
-// ################################################################################################
-// ################################################################################################
-
-namespace epee
-{
-namespace net_utils
-{
-
-
-/* ============================================================================ */
-
-class connection_basic_pimpl {
-	public:
-		connection_basic_pimpl(const std::string &name);
-
-		static int m_default_tos;
-
-		network_throttle_bw m_throttle; // per-perr
-    critical_section m_throttle_lock;
-
-		void _packet(size_t packet_size, int phase, int q_len); // execute a sleep ; phase is not really used now(?) could be used for different kinds of sleep e.g. direct/queue write
-};
-
-
-} // namespace
-} // namespace
-
-
-
-
-
-
-// ################################################################################################
-// ################################################################################################
-// The implementation part
-// ################################################################################################
-// ################################################################################################
-
-namespace epee
-{
-namespace net_utils
-{
-
-// ================================================================================================
-// network_throttle
-// ================================================================================================
-
-network_throttle::~network_throttle() { }
-
-network_throttle::packet_info::packet_info() 
-	: m_size(0)
-{ 
-}
-
-network_throttle::network_throttle(const std::string &nameshort, const std::string &name, int window_size)
-    : m_window_size( (window_size==-1) ? 10 : window_size  ),
-	  m_history( m_window_size ), m_nameshort(nameshort)
-{
-	set_name(name);
-	m_network_add_cost = 128;
-	m_network_minimal_segment = 256;
-	m_network_max_segment = 1024*1024;
-	m_any_packet_yet = false;
-	m_slot_size = 1.0; // hard coded in few places
-	m_target_speed = 16 * 1024; // other defaults are probably defined in the command-line parsing code when this class is used e.g. as main global throttle
-}
-
-void network_throttle::set_name(const std::string &name) 
-{
-	m_name = name;
-}
-
-void network_throttle::set_target_speed( network_speed_kbps target ) 
-{
-    m_target_speed = target * 1024;
-	MINFO("Setting LIMIT: " << target << " kbps");
-}
-
-network_speed_kbps network_throttle::get_target_speed()
-{
-	return m_target_speed / 1024;
-}
-			
-void network_throttle::tick()
-{
-	double time_now = get_time_seconds();
-	if (!m_any_packet_yet) m_start_time = time_now; // starting now
-
-	network_time_seconds current_sample_time_slot = time_to_slot( time_now ); // T=13.7 --> 13  (for 1-second smallwindow)
-	network_time_seconds last_sample_time_slot = time_to_slot( m_last_sample_time );
-
-	// moving to next position, and filling gaps
-	// !! during this loop the m_last_sample_time and last_sample_time_slot mean the variable moved in +1
-	// TODO optimize when moving few slots at once
-	while ( (!m_any_packet_yet) || (last_sample_time_slot < current_sample_time_slot))
-	{
-		_dbg3("Moving counter buffer by 1 second " << last_sample_time_slot << " < " << current_sample_time_slot << " (last time " << m_last_sample_time<<")");
-		// rotate buffer 
-		for (size_t i=m_history.size()-1; i>=1; --i) m_history[i] = m_history[i-1];
-		m_history[0] = packet_info();
-		if (! m_any_packet_yet) 
-		{
-			m_last_sample_time = time_now;	
-		}
-		m_last_sample_time += 1;	last_sample_time_slot = time_to_slot( m_last_sample_time ); // increase and recalculate time, time slot
-		m_any_packet_yet=true;
-	}
-	m_last_sample_time = time_now; // the real exact last time
-}
-
-void network_throttle::handle_trafic_exact(size_t packet_size) 
-{
-	_handle_trafic_exact(packet_size, packet_size);
-}
-
-void network_throttle::_handle_trafic_exact(size_t packet_size, size_t orginal_size)
-{
-	tick();
-
-	calculate_times_struct cts ;  calculate_times(packet_size, cts , false, -1);
-	calculate_times_struct cts2;  calculate_times(packet_size, cts2, false, 5);
-	m_history[0].m_size += packet_size;
-
-	std::ostringstream oss; oss << "["; 	for (auto sample: m_history) oss << sample.m_size << " ";	 oss << "]" << std::ends;
-	std::string history_str = oss.str();
-
-	MTRACE("Throttle " << m_name << ": packet of ~"<<packet_size<<"b " << " (from "<<orginal_size<<" b)"
-        << " Speed AVG=" << std::setw(4) <<  ((long int)(cts .average/1024)) <<"[w="<<cts .window<<"]"
-        <<           " " << std::setw(4) <<  ((long int)(cts2.average/1024)) <<"[w="<<cts2.window<<"]"
-				<<" / " << " Limit="<< ((long int)(m_target_speed/1024)) <<" KiB/sec "
-				<< " " << history_str
-		);
-}
-
-void network_throttle::handle_trafic_tcp(size_t packet_size)
-{
-	size_t all_size = packet_size + m_network_add_cost;
-	all_size = std::max( m_network_minimal_segment , all_size);
-	_handle_trafic_exact( all_size , packet_size );
-}
-
-network_time_seconds network_throttle::get_sleep_time_after_tick(size_t packet_size) {
-	tick();
-	return get_sleep_time(packet_size);
-}
-
-void network_throttle::logger_handle_net(const std::string &filename, double time, size_t size) {
-    boost::mutex mutex;
-    mutex.lock(); {
-        std::fstream file;
-        file.open(filename.c_str(), std::ios::app | std::ios::out );
-        file.precision(6);
-        if(!file.is_open())
-            _warn("Can't open file " << filename);
-        file << static_cast<int>(time) << " " << static_cast<double>(size/1024) << "\n";
-        file.close();
-    }  mutex.unlock();
-}
-
-// fine tune this to decide about sending speed:
-network_time_seconds network_throttle::get_sleep_time(size_t packet_size) const 
-{
-	double D2=0;
-	calculate_times_struct cts = { 0, 0, 0, 0};
-	calculate_times(packet_size, cts, true, m_window_size); D2=cts.delay;
-	return D2;
-}
-
-// MAIN LOGIC:
-void network_throttle::calculate_times(size_t packet_size, calculate_times_struct &cts, bool dbg, double force_window) const 
-{
-    const double the_window_size = std::max( (double)m_window_size ,
-		((force_window>0) ? force_window : m_window_size) 
-	);
-
-	if (!m_any_packet_yet) {
-		cts.window=0; cts.average=0; cts.delay=0; 
-		cts.recomendetDataSize = m_network_minimal_segment; // should be overrided by caller anyway
-		return ; // no packet yet, I can not decide about sleep time
-	}
-
-	network_time_seconds window_len = (the_window_size-1) * m_slot_size ; // -1 since current slot is not finished
-	window_len += (m_last_sample_time - time_to_slot(m_last_sample_time));  // add the time for current slot e.g. 13.7-13 = 0.7
-
-	auto time_passed = get_time_seconds() - m_start_time;
-	cts.window = std::max( std::min( window_len , time_passed ) , m_slot_size )  ; // window length resulting from size of history but limited by how long ago history was started,
-	// also at least slot size (e.g. 1 second) to not be ridiculous
-	// window_len e.g. 5.7 because takes into account current slot time
-
-	size_t Epast = 0; // summ of traffic till now
-	for (auto sample : m_history) Epast += sample.m_size; 
-
-	const size_t E = Epast;
-	const size_t Enow = Epast + packet_size ; // including the data we're about to send now
-
-	const double M = m_target_speed; // max
-	const double D1 = (Epast - M*cts.window) / M; // delay - how long to sleep to get back to target speed
-	const double D2 = (Enow  - M*cts.window) / M; // delay - how long to sleep to get back to target speed (including current packet)
-
-    cts.delay = (D1*0.80 + D2*0.20); // finall sleep depends on both with/without current packet
-	//				update_overheat();
-	cts.average = Epast/cts.window; // current avg. speed (for info)
-
-	if (Epast <= 0) {
-		if (cts.delay>=0) cts.delay = 0; // no traffic in history so we will not wait
-	}
-
-    double Wgood=-1;
-    {	// how much data we recommend now to download
-        Wgood = the_window_size + 1;
-        cts.recomendetDataSize = M*cts.window - E;
-    }
-
-	if (dbg) {
-		std::ostringstream oss; oss << "["; 	for (auto sample: m_history) oss << sample.m_size << " ";	 oss << "]" << std::ends;
-		std::string history_str = oss.str();
-		MTRACE((cts.delay > 0 ? "SLEEP" : "")
-			<< "dbg " << m_name << ": " 
-			<< "speed is A=" << std::setw(8) <<cts.average<<" vs "
-			<< "Max=" << std::setw(8) <<M<<" "
-			<< " so sleep: "
-			<< "D=" << std::setw(8) <<cts.delay<<" sec "
-			<< "E="<< std::setw(8) << E << " (Enow="<<std::setw(8)<<Enow<<") "
-            << "M=" << std::setw(8) << M <<" W="<< std::setw(8) << cts.window << " "
-            << "R=" << std::setw(8) << cts.recomendetDataSize << " Wgood" << std::setw(8) << Wgood << " "
-			<< "History: " << std::setw(8) << history_str << " "
-			<< "m_last_sample_time=" << std::setw(8) << m_last_sample_time
-		);
-
-	}
-}
-
-double network_throttle::get_time_seconds() const {
-	#if defined(__APPLE__)
-	auto point = std::chrono::system_clock::now();
-	#else
-	auto point = std::chrono::steady_clock::now();
-	#endif
-	auto time_from_epoh = point.time_since_epoch();
-	auto ms = std::chrono::duration_cast< std::chrono::milliseconds >( time_from_epoh ).count();
-	double ms_f = ms;
-	return ms_f / 1000.;
-}
-
-size_t network_throttle::get_recommended_size_of_planned_transport_window(double force_window) const {
-	calculate_times_struct cts = { 0, 0, 0, 0};
-	network_throttle::calculate_times(0, cts, true, force_window);
-	cts.recomendetDataSize += m_network_add_cost;
-	if (cts.recomendetDataSize<0) cts.recomendetDataSize=0;
-	if (cts.recomendetDataSize>m_network_max_segment) cts.recomendetDataSize=m_network_max_segment;
-	size_t RI = (long int)cts.recomendetDataSize;
-	return RI;
-}
-
-size_t network_throttle::get_recommended_size_of_planned_transport() const {
-	size_t R1=0,R2=0,R3=0;
-	R1 = get_recommended_size_of_planned_transport_window( -1 );
-	R2 = get_recommended_size_of_planned_transport_window(m_window_size / 2);
-	R3 = get_recommended_size_of_planned_transport_window( 5              );
-	auto RM = std::min(R1, std::min(R2,R3));
-
-	const double a1=20, a2=10, a3=10, am=10; // weight of the various windows in decisssion // TODO 70 => 20
-	return (R1*a1 + R2*a2 + R3*a3 + RM*am) / (a1+a2+a3+am);
-}
-
-double network_throttle::get_current_speed() const {
-	unsigned int bytes_transferred = 0;
-	if (m_history.size() == 0 || m_slot_size == 0)
-		return 0;
-		
-	auto it = m_history.begin();
-	while (it < m_history.end() - 1)
-	{
-		bytes_transferred += it->m_size;
-		it ++;
-	}
-	
-	return bytes_transferred / ((m_history.size() - 1) * m_slot_size);
-}
-
-} // namespace
-} // namespace
-
diff --git a/src/p2p/network_throttle-detail.hpp b/src/p2p/network_throttle-detail.hpp
deleted file mode 100644
index 676d4341a..000000000
--- a/src/p2p/network_throttle-detail.hpp
+++ /dev/null
@@ -1,125 +0,0 @@
-/// @file
-/// @author rfree (current maintainer in monero.cc project)
-/// @brief implementaion for throttling of connection (count and rate-limit speed etc)
-
-// Copyright (c) 2014-2017, The Monero Project
-// 
-// All rights reserved.
-// 
-// Redistribution and use in source and binary forms, with or without modification, are
-// permitted provided that the following conditions are met:
-// 
-// 1. Redistributions of source code must retain the above copyright notice, this list of
-//    conditions and the following disclaimer.
-// 
-// 2. Redistributions in binary form must reproduce the above copyright notice, this list
-//    of conditions and the following disclaimer in the documentation and/or other
-//    materials provided with the distribution.
-// 
-// 3. Neither the name of the copyright holder nor the names of its contributors may be
-//    used to endorse or promote products derived from this software without specific
-//    prior written permission.
-// 
-// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY
-// EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
-// MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL
-// THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
-// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
-// PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
-// INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
-// STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF
-// THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-
-/* rfree: throttle details, implementing rate limiting */
-
-
-#ifndef INCLUDED_src_p2p_throttle_detail_hpp
-#define INCLUDED_src_p2p_throttle_detail_hpp
-
-#include "network_throttle.hpp"
-
-namespace epee
-{
-namespace net_utils
-{
-
-
-class network_throttle : public i_network_throttle {
-	private:
-		struct packet_info {
-			size_t m_size; // octets sent. Summary for given small-window (e.g. for all packaged in 1 second)
-			packet_info();
-		};
-
-
-		network_speed_bps m_target_speed;
-		size_t m_network_add_cost; // estimated add cost of headers 
-		size_t m_network_minimal_segment; // estimated minimal cost of sending 1 byte to round up to
-		size_t m_network_max_segment; // recommended max size of 1 TCP transmission
-
-		const size_t m_window_size; // the number of samples to average over
-		network_time_seconds m_slot_size; // the size of one slot. TODO: now hardcoded for 1 second e.g. in time_to_slot()
-		// TODO for big window size, for performance better the substract on change of m_last_sample_time instead of recalculating average of eg >100 elements
-
-		std::vector< packet_info > m_history; // the history of bw usage
-		network_time_seconds m_last_sample_time; // time of last history[0] - so we know when to rotate the buffer
-		network_time_seconds m_start_time; // when we were created
-		bool m_any_packet_yet; // did we yet got any packet to count
-
-		std::string m_name; // my name for debug and logs
-		std::string m_nameshort; // my name for debug and logs (used in log file name)
-
-	// each sample is now 1 second
-	public:
-		network_throttle(const std::string &nameshort, const std::string &name, int window_size=-1);
-		virtual ~network_throttle();
-		virtual void set_name(const std::string &name);
-		virtual void set_target_speed( network_speed_kbps target );
-		virtual network_speed_kbps get_target_speed();
-
-		// add information about events:
-		virtual void handle_trafic_exact(size_t packet_size); ///< count the new traffic/packet; the size is exact considering all network costs
-		virtual void handle_trafic_tcp(size_t packet_size); ///< count the new traffic/packet; the size is as TCP, we will consider MTU etc
-
-		virtual void tick(); ///< poke and update timers/history (recalculates, moves the history if needed, checks the real clock etc)
-
-		virtual double get_time_seconds() const ; ///< timer that we use, time in seconds, monotionic
-
-		// time calculations:
-		virtual void calculate_times(size_t packet_size, calculate_times_struct &cts, bool dbg, double force_window) const; ///< MAIN LOGIC (see base class for info)
-
-		virtual network_time_seconds get_sleep_time_after_tick(size_t packet_size); ///< increase the timer if needed, and get the package size
-		virtual network_time_seconds get_sleep_time(size_t packet_size) const; ///< gets the Delay (recommended Delay time) from calc. (not safe: only if time didnt change?) TODO
-
-		virtual size_t get_recommended_size_of_planned_transport() const; ///< what should be the size (bytes) of next data block to be transported
-		virtual size_t get_recommended_size_of_planned_transport_window(double force_window) const;  ///< ditto, but for given windows time frame
-		virtual double get_current_speed() const;
-
-	private:
-		virtual network_time_seconds time_to_slot(network_time_seconds t) const { return std::floor( t ); } // convert exact time eg 13.7 to rounded time for slot number in history 13
-        virtual void _handle_trafic_exact(size_t packet_size, size_t orginal_size);
-        virtual void logger_handle_net(const std::string &filename, double time, size_t size);
-};
-
-/***
- * The complete set of traffic throttle for one typical connection
-*/
-struct network_throttle_bw {
-	public:
-		network_throttle m_in; ///< for incomming traffic (this we can not controll directly as it depends of what others send to us - usually)
-		network_throttle m_inreq; ///< for requesting incomming traffic (this is exact usually)
-		network_throttle m_out; ///< for outgoing traffic that we just sent (this is exact usually)
-
-	public:
-		network_throttle_bw(const std::string &name1);
-};
-
-
-
-} // namespace net_utils
-} // namespace epee
-
-
-#endif
-
-
diff --git a/src/p2p/network_throttle.cpp b/src/p2p/network_throttle.cpp
deleted file mode 100644
index 74b20376d..000000000
--- a/src/p2p/network_throttle.cpp
+++ /dev/null
@@ -1,113 +0,0 @@
-/**
-@file
-@author rfree (current maintainer in monero.cc project)
-@brief interface for throttling of connection (count and rate-limit speed etc) 
-@details <PRE>
-
-Throttling work by:
-1) taking note of all traffic (hooks added e.g. to connection class) and measuring speed
-2) depending on that information we sleep before sending out data (or send smaller portions of data)
-3) depending on the information we can also sleep before sending requests or ask for smaller sets of data to download
-
-</PRE> 
-
-@image html images/net/rate1-down-1k.png
-@image html images/net/rate1-down-full.png
-@image html images/net/rate1-up-10k.png
-@image html images/net/rate1-up-full.png
-@image html images/net/rate2-down-100k.png
-@image html images/net/rate2-down-10k.png
-@image html images/net/rate2-down-50k.png
-@image html images/net/rate2-down-full.png
-@image html images/net/rate2-up-100k.png
-@image html images/net/rate2-up-10k.png
-@image html images/net/rate3-up-10k.png
-
-
-*/
-
-// Copyright (c) 2014-2017, The Monero Project
-// 
-// All rights reserved.
-// 
-// Redistribution and use in source and binary forms, with or without modification, are
-// permitted provided that the following conditions are met:
-// 
-// 1. Redistributions of source code must retain the above copyright notice, this list of
-//    conditions and the following disclaimer.
-// 
-// 2. Redistributions in binary form must reproduce the above copyright notice, this list
-//    of conditions and the following disclaimer in the documentation and/or other
-//    materials provided with the distribution.
-// 
-// 3. Neither the name of the copyright holder nor the names of its contributors may be
-//    used to endorse or promote products derived from this software without specific
-//    prior written permission.
-// 
-// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY
-// EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
-// MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL
-// THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
-// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
-// PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
-// INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
-// STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF
-// THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-
-#include "network_throttle-detail.hpp"
-
-namespace epee
-{
-namespace net_utils
-{
-
-// ================================================================================================
-// network_throttle_manager
-// ================================================================================================
-
-// ================================================================================================
-// static:
-boost::mutex network_throttle_manager::m_lock_get_global_throttle_in;
-boost::mutex network_throttle_manager::m_lock_get_global_throttle_inreq;
-boost::mutex network_throttle_manager::m_lock_get_global_throttle_out;
-
-int network_throttle_manager::xxx;
-
-
-// ================================================================================================
-// methods:
-i_network_throttle & network_throttle_manager::get_global_throttle_in() { 
-	static network_throttle obj_get_global_throttle_in("in/all","<<< global-IN",10);
-	return obj_get_global_throttle_in;
-}
-
-
-
-i_network_throttle & network_throttle_manager::get_global_throttle_inreq() { 
-	static network_throttle obj_get_global_throttle_inreq("inreq/all", "<== global-IN-REQ",10);
-	return obj_get_global_throttle_inreq;
-}
-
-
-i_network_throttle & network_throttle_manager::get_global_throttle_out() { 
-	static network_throttle obj_get_global_throttle_out("out/all", ">>> global-OUT",10);
-	return obj_get_global_throttle_out;
-}
-
-
-
-
-network_throttle_bw::network_throttle_bw(const std::string &name1) 
-	: m_in("in/"+name1, name1+"-DOWNLOAD"), m_inreq("inreq/"+name1, name1+"-DOWNLOAD-REQUESTS"), m_out("out/"+name1, name1+"-UPLOAD")
-{ }
-
-
-
-
-} // namespace 
-} // namespace 
-
-
-
-
-
diff --git a/src/p2p/network_throttle.hpp b/src/p2p/network_throttle.hpp
deleted file mode 100644
index bf1f93859..000000000
--- a/src/p2p/network_throttle.hpp
+++ /dev/null
@@ -1,176 +0,0 @@
-/// @file
-/// @author rfree (current maintainer in monero.cc project)
-/// @brief interface for throttling of connection (count and rate-limit speed etc)
-
-// Copyright (c) 2014-2017, The Monero Project
-// 
-// All rights reserved.
-// 
-// Redistribution and use in source and binary forms, with or without modification, are
-// permitted provided that the following conditions are met:
-// 
-// 1. Redistributions of source code must retain the above copyright notice, this list of
-//    conditions and the following disclaimer.
-// 
-// 2. Redistributions in binary form must reproduce the above copyright notice, this list
-//    of conditions and the following disclaimer in the documentation and/or other
-//    materials provided with the distribution.
-// 
-// 3. Neither the name of the copyright holder nor the names of its contributors may be
-//    used to endorse or promote products derived from this software without specific
-//    prior written permission.
-// 
-// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY
-// EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
-// MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL
-// THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
-// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
-// PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
-// INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
-// STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF
-// THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-
-/* rfree: throttle basic interface */
-/* rfree: also includes the manager for singeton/global such objects */
-
-
-#ifndef INCLUDED_p2p_network_throttle_hpp
-#define INCLUDED_p2p_network_throttle_hpp
-
-#include <boost/asio.hpp>
-#include <string>
-#include <vector>
-#include <boost/noncopyable.hpp>
-#include <boost/shared_ptr.hpp>
-#include <atomic>
-
-#include <boost/asio.hpp>
-#include <boost/array.hpp>
-#include <boost/noncopyable.hpp>
-#include <boost/shared_ptr.hpp>
-#include <boost/enable_shared_from_this.hpp>
-#include <boost/interprocess/detail/atomic.hpp>
-#include <boost/thread/thread.hpp>
-
-#include "syncobj.h"
-
-#include "net/net_utils_base.h" 
-#include "misc_log_ex.h" 
-#include <boost/lambda/bind.hpp>
-#include <boost/lambda/lambda.hpp>
-#include <boost/uuid/random_generator.hpp>
-#include <boost/chrono.hpp>
-#include <boost/utility/value_init.hpp>
-#include <boost/asio/deadline_timer.hpp>
-#include <boost/date_time/posix_time/posix_time.hpp>
-#include <boost/thread/thread.hpp> 
-#include "misc_language.h"
-#include "pragma_comp_defs.h"
-#include <sstream>
-#include <iomanip>
-#include <algorithm>
-
-#include <memory>
-#include <mutex>
-#include <fstream>
-
-namespace epee
-{
-namespace net_utils
-{
-
-// just typedefs to in code define the units used. TODO later it will be enforced that casts to other numericals are only explicit to avoid mistakes? use boost::chrono?
-typedef double network_speed_kbps;   // externally, for parameters and return values, all defined in kilobytes per second
-typedef double network_speed_bps;    // throttle-internally, bytes per second
-typedef double network_time_seconds;
-typedef double network_MB;
-
-class i_network_throttle;
-
-/***
-@brief All information about given throttle - speed calculations
-*/
-struct calculate_times_struct {
-		double average;
-		double window;
-		double delay;
-		double recomendetDataSize;
-};
-typedef calculate_times_struct calculate_times_struct;
-
-
-namespace cryptonote { class cryptonote_protocol_handler_base; } // a friend class // TODO friend not working
-
-/*** 
-@brief Access to simple throttles, with singlton to access global network limits
-*/
-class network_throttle_manager {
-	// provides global (singleton) in/inreq/out throttle access
-
-	// [[note1]] see also http://www.nuonsoft.com/blog/2012/10/21/implementing-a-thread-safe-singleton-with-c11/
-	// [[note2]] _inreq is the requested in traffic - we anticipate we will get in-bound traffic soon as result of what we do (e.g. that we sent network downloads requests)
-	
-	//protected:
-	public: // XXX
-
-    static boost::mutex m_lock_get_global_throttle_in;
-    static boost::mutex m_lock_get_global_throttle_inreq;
-    static boost::mutex m_lock_get_global_throttle_out;
-
-		friend class cryptonote::cryptonote_protocol_handler_base; // FRIEND - to directly access global throttle-s. !! REMEMBER TO USE LOCKS!
-		friend class connection_basic; // FRIEND - to directly access global throttle-s. !! REMEMBER TO USE LOCKS!
-		friend class connection_basic_pimpl; // ditto
-
-		static int xxx;
-
-	public:
-		static i_network_throttle & get_global_throttle_in(); ///< singleton ; for friend class ; caller MUST use proper locks! like m_lock_get_global_throttle_in
-		static i_network_throttle & get_global_throttle_inreq(); ///< ditto ; use lock ... use m_lock_get_global_throttle_inreq obviously
-		static i_network_throttle & get_global_throttle_out(); ///< ditto ; use lock ... use m_lock_get_global_throttle_out obviously
-};
-
-
-
-/***
-@brief interface for the throttle, see the derivated class
-*/
-class i_network_throttle {
-	public:
-		virtual void set_name(const std::string &name)=0;
-		virtual void set_target_speed( network_speed_kbps target )=0;
-		virtual network_speed_kbps get_target_speed()=0;
-
-		virtual void handle_trafic_exact(size_t packet_size) =0; // count the new traffic/packet; the size is exact considering all network costs
-		virtual void handle_trafic_tcp(size_t packet_size) =0; // count the new traffic/packet; the size is as TCP, we will consider MTU etc
-		virtual void tick() =0; // poke and update timers/history
-		
-		// time calculations:
-		
-		virtual void calculate_times(size_t packet_size, calculate_times_struct &cts, bool dbg, double force_window) const =0; // assuming sending new package (or 0), calculate:
-		// Average, Window, Delay, Recommended data size ; also gets dbg=debug flag, and forced widnow size if >0 or -1 for not forcing window size 
-
-		// Average speed, Window size, recommended Delay to sleep now, Recommended size of data to send now
-
-		virtual network_time_seconds get_sleep_time(size_t packet_size) const =0; // gets the D (recommended Delay time) from calc
-		virtual network_time_seconds get_sleep_time_after_tick(size_t packet_size) =0; // ditto, but first tick the timer
-
-		virtual size_t get_recommended_size_of_planned_transport() const =0; // what should be the recommended limit of data size that we can transport over current network_throttle in near future
-
-		virtual double get_time_seconds() const =0; // a timer
-        virtual void logger_handle_net(const std::string &filename, double time, size_t size)=0;
-
-
-};
-
-
-// ... more in the -advanced.h file
-
-
-} // namespace net_utils
-} // namespace epee
-
-
-#endif
-
-
-