Fixes & improvements

Faster DAG copying on CUDA

Author: nicehashdev

ethminer/MinerAux.h

@@ -1018,12 +1018,13 @@ class MinerCLI
 
 		try
 		{
-			boost::array<char, 1> recv_buf;
+			boost::array<char, 16> recv_buf;
 			boost::system::error_code error;
 			size_t s = api_socket->receive_from(boost::asio::buffer(recv_buf), remote_endpoint, 0, error);
 			if (error && error != boost::asio::error::would_block)
 				throw boost::system::system_error(error);
-			if (s != 1) return 0;
+			if (s < 1) return 0;
+			//std::cout << "API received " << s << std::endl;
 			return (int)recv_buf.at(0);
 		}
 		catch (std::exception& e)
@@ -1058,6 +1059,7 @@ class MinerCLI
 				api_socket = new boost::asio::ip::udp::socket(m_io_service, endpoint);
 				boost::asio::socket_base::non_blocking_io nb(true);
 				api_socket->io_control(nb);
+				minelog << "API port " << m_apiPort << " bound";
 			}
 			catch (std::exception& e)
 			{
@@ -1166,6 +1168,7 @@ class MinerCLI
 							double spd = 0;
 							if (mp.ms > 0) spd = (double)mp.hashes / (mp.ms * 1000);
 							api_socket->send_to(boost::asio::buffer((void*)&spd, sizeof(spd)), remote_endpoint);
+							//std::cout << "API replied "<< std::endl;
 						}
 					}
 					if (std::chrono::duration_cast<std::chrono::milliseconds>(std::chrono::steady_clock::now() - tp).count() < m_farmRecheckPeriod)

libethash-cuda/ethash_cuda_miner.cpp

@@ -271,7 +271,8 @@ bool ethash_cuda_miner::init(ethash_light_t _light, uint8_t const* _lightData, u
 
 			if (_cpyToHost)
 			{
-				uint8_t* memoryDAG = new uint8_t[dagSize];
+				uint8_t* memoryDAG = nullptr; // = new uint8_t[dagSize];
+				CUDA_SAFE_CALL(cudaHostAlloc(&memoryDAG, dagSize, 0));
 				if (!memoryDAG) throw std::runtime_error("Failed to init host memory for DAG, not enough memory?");
 
 				cout << "Copying DAG from GPU #" << m_device << " to host" << endl;

libethcore/EthashCUDAMiner.cpp

@@ -180,7 +180,8 @@ void EthashCUDAMiner::workLoop()
 					// free DAG if already existing
 					if (s_dagInHostMemory && *s_dagSeed != w.seedHash)
 					{
-						delete[] s_dagInHostMemory;
+						//delete[] s_dagInHostMemory;
+						CUDA_SAFE_CALL(cudaFreeHost((void*)s_dagInHostMemory));
 						s_dagInHostMemory = nullptr;
 
 						cout << "Freeing DAG from host" << endl;
@@ -212,7 +213,8 @@ void EthashCUDAMiner::workLoop()
 				if (s_dagLoadIndex >= s_numInstances && s_dagInHostMemory)
 				{
 					// all devices have loaded DAG, we can free now
-					delete[] s_dagInHostMemory;
+					//delete[] s_dagInHostMemory;
+					CUDA_SAFE_CALL(cudaFreeHost((void*)s_dagInHostMemory));
 					s_dagInHostMemory = nullptr;
 
 					cout << "Freeing DAG from host" << endl;