From a9c1c1ac642e127ffb6793c2551c2a438b815fdd Mon Sep 17 00:00:00 2001 From: XMRig Date: Sun, 11 Aug 2019 00:01:48 +0700 Subject: [PATCH 1/8] v3.0.0-dev --- src/version.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/version.h b/src/version.h index a0018a79..81adcd8b 100644 --- a/src/version.h +++ b/src/version.h @@ -28,15 +28,15 @@ #define APP_ID "xmrig" #define APP_NAME "XMRig" #define APP_DESC "XMRig CPU miner" -#define APP_VERSION "2.99.6-beta" +#define APP_VERSION "3.0.0-dev" #define APP_DOMAIN "xmrig.com" #define APP_SITE "www.xmrig.com" #define APP_COPYRIGHT "Copyright (C) 2016-2019 xmrig.com" #define APP_KIND "cpu" -#define APP_VER_MAJOR 2 -#define APP_VER_MINOR 99 -#define APP_VER_PATCH 6 +#define APP_VER_MAJOR 3 +#define APP_VER_MINOR 0 +#define APP_VER_PATCH 0 #ifdef _MSC_VER # if (_MSC_VER >= 1920) From 2ec257284f5bffce5277c35458d3893430c2d843 Mon Sep 17 00:00:00 2001 From: XMRig Date: Mon, 12 Aug 2019 16:52:16 +0700 Subject: [PATCH 2/8] Common API code moved to base/api. --- CMakeLists.txt | 19 +------------------ src/App.cpp | 1 - src/{ => base}/api/Api.cpp | 8 ++++---- src/{ => base}/api/Api.h | 0 src/{ => base}/api/Httpd.cpp | 4 ++-- src/{ => base}/api/Httpd.h | 0 src/{ => base}/api/interfaces/IApiListener.h | 0 src/{ => base}/api/interfaces/IApiRequest.h | 0 src/{ => base}/api/requests/ApiRequest.cpp | 2 +- src/{ => base}/api/requests/ApiRequest.h | 2 +- .../api/requests/HttpApiRequest.cpp | 2 +- src/{ => base}/api/requests/HttpApiRequest.h | 2 +- src/base/base.cmake | 10 ++++++++++ src/base/kernel/Base.cpp | 17 ++++++++--------- src/base/kernel/Base.h | 2 +- src/core/Miner.cpp | 4 ++-- src/core/Miner.h | 2 +- src/net/Network.cpp | 5 ++--- src/net/Network.h | 2 +- 19 files changed, 36 insertions(+), 46 deletions(-) rename src/{ => base}/api/Api.cpp (97%) rename src/{ => base}/api/Api.h (100%) rename src/{ => base}/api/Httpd.cpp (98%) rename src/{ => base}/api/Httpd.h (100%) rename src/{ => base}/api/interfaces/IApiListener.h (100%) rename src/{ => base}/api/interfaces/IApiRequest.h (100%) rename src/{ => base}/api/requests/ApiRequest.cpp (97%) rename src/{ => base}/api/requests/ApiRequest.h (98%) rename src/{ => base}/api/requests/HttpApiRequest.cpp (98%) rename src/{ => base}/api/requests/HttpApiRequest.h (98%) diff --git a/CMakeLists.txt b/CMakeLists.txt index 7600d556..15288319 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -32,7 +32,6 @@ set(HEADERS "${HEADERS_BASE}" "${HEADERS_BASE_HTTP}" "${HEADERS_BACKEND}" - src/api/interfaces/IApiListener.h src/App.h src/core/config/Config_default.h src/core/config/Config_platform.h @@ -232,22 +231,6 @@ if (WITH_EMBEDDED_CONFIG) add_definitions(/DXMRIG_FEATURE_EMBEDDED_CONFIG) endif() -if (WITH_HTTP) - set(HTTP_SOURCES - src/api/Api.cpp - src/api/Api.h - src/api/Httpd.cpp - src/api/Httpd.h - src/api/interfaces/IApiRequest.h - src/api/requests/ApiRequest.cpp - src/api/requests/ApiRequest.h - src/api/requests/HttpApiRequest.cpp - src/api/requests/HttpApiRequest.h - ) -else() - set(HTTP_SOURCES "") -endif() - include_directories(src) include_directories(src/3rdparty) include_directories(${UV_INCLUDE_DIR}) @@ -260,5 +243,5 @@ if (WITH_DEBUG_LOG) add_definitions(/DAPP_DEBUG) endif() -add_executable(${CMAKE_PROJECT_NAME} ${HEADERS} ${SOURCES} ${SOURCES_OS} ${SOURCES_CPUID} ${HEADERS_CRYPTO} ${SOURCES_CRYPTO} ${SOURCES_SYSLOG} ${HTTP_SOURCES} ${TLS_SOURCES} ${XMRIG_ASM_SOURCES} ${CN_GPU_SOURCES}) +add_executable(${CMAKE_PROJECT_NAME} ${HEADERS} ${SOURCES} ${SOURCES_OS} ${SOURCES_CPUID} ${HEADERS_CRYPTO} ${SOURCES_CRYPTO} ${SOURCES_SYSLOG} ${TLS_SOURCES} ${XMRIG_ASM_SOURCES} ${CN_GPU_SOURCES}) target_link_libraries(${CMAKE_PROJECT_NAME} ${XMRIG_ASM_LIBRARY} ${OPENSSL_LIBRARIES} ${UV_LIBRARIES} ${EXTRA_LIBS} ${CPUID_LIB}) diff --git a/src/App.cpp b/src/App.cpp index ccbaad4f..de45a499 100644 --- a/src/App.cpp +++ b/src/App.cpp @@ -28,7 +28,6 @@ #include -#include "api/Api.h" #include "App.h" #include "backend/cpu/Cpu.h" #include "base/io/Console.h" diff --git a/src/api/Api.cpp b/src/base/api/Api.cpp similarity index 97% rename from src/api/Api.cpp rename to src/base/api/Api.cpp index 4c8e7323..f358ac4b 100644 --- a/src/api/Api.cpp +++ b/src/base/api/Api.cpp @@ -32,9 +32,9 @@ #include "3rdparty/http-parser/http_parser.h" -#include "api/Api.h" -#include "api/interfaces/IApiListener.h" -#include "api/requests/HttpApiRequest.h" +#include "base/api/Api.h" +#include "base/api/interfaces/IApiListener.h" +#include "base/api/requests/HttpApiRequest.h" #include "base/kernel/Base.h" #include "base/tools/Buffer.h" #include "base/tools/Chrono.h" @@ -45,7 +45,7 @@ #ifdef XMRIG_FEATURE_HTTP -# include "api/Httpd.h" +# include "base/api/Httpd.h" #endif diff --git a/src/api/Api.h b/src/base/api/Api.h similarity index 100% rename from src/api/Api.h rename to src/base/api/Api.h diff --git a/src/api/Httpd.cpp b/src/base/api/Httpd.cpp similarity index 98% rename from src/api/Httpd.cpp rename to src/base/api/Httpd.cpp index 57a112e9..e61e66f1 100644 --- a/src/api/Httpd.cpp +++ b/src/base/api/Httpd.cpp @@ -24,8 +24,8 @@ #include "3rdparty/http-parser/http_parser.h" -#include "api/Api.h" -#include "api/Httpd.h" +#include "base/api/Api.h" +#include "base/api/Httpd.h" #include "base/io/log/Log.h" #include "base/net/http/HttpApiResponse.h" #include "base/net/http/HttpData.h" diff --git a/src/api/Httpd.h b/src/base/api/Httpd.h similarity index 100% rename from src/api/Httpd.h rename to src/base/api/Httpd.h diff --git a/src/api/interfaces/IApiListener.h b/src/base/api/interfaces/IApiListener.h similarity index 100% rename from src/api/interfaces/IApiListener.h rename to src/base/api/interfaces/IApiListener.h diff --git a/src/api/interfaces/IApiRequest.h b/src/base/api/interfaces/IApiRequest.h similarity index 100% rename from src/api/interfaces/IApiRequest.h rename to src/base/api/interfaces/IApiRequest.h diff --git a/src/api/requests/ApiRequest.cpp b/src/base/api/requests/ApiRequest.cpp similarity index 97% rename from src/api/requests/ApiRequest.cpp rename to src/base/api/requests/ApiRequest.cpp index 3812e419..da73adee 100644 --- a/src/api/requests/ApiRequest.cpp +++ b/src/base/api/requests/ApiRequest.cpp @@ -23,7 +23,7 @@ */ -#include "api/requests/ApiRequest.h" +#include "base/api/requests/ApiRequest.h" xmrig::ApiRequest::ApiRequest(Source source, bool restricted) : diff --git a/src/api/requests/ApiRequest.h b/src/base/api/requests/ApiRequest.h similarity index 98% rename from src/api/requests/ApiRequest.h rename to src/base/api/requests/ApiRequest.h index 05716e29..1fd721f1 100644 --- a/src/api/requests/ApiRequest.h +++ b/src/base/api/requests/ApiRequest.h @@ -27,7 +27,7 @@ #define XMRIG_APIREQUEST_H -#include "api/interfaces/IApiRequest.h" +#include "base/api/interfaces/IApiRequest.h" namespace xmrig { diff --git a/src/api/requests/HttpApiRequest.cpp b/src/base/api/requests/HttpApiRequest.cpp similarity index 98% rename from src/api/requests/HttpApiRequest.cpp rename to src/base/api/requests/HttpApiRequest.cpp index b4dc1810..c3c2dac4 100644 --- a/src/api/requests/HttpApiRequest.cpp +++ b/src/base/api/requests/HttpApiRequest.cpp @@ -23,7 +23,7 @@ */ -#include "api/requests/HttpApiRequest.h" +#include "base/api/requests/HttpApiRequest.h" #include "base/net/http/HttpData.h" #include "rapidjson/error/en.h" diff --git a/src/api/requests/HttpApiRequest.h b/src/base/api/requests/HttpApiRequest.h similarity index 98% rename from src/api/requests/HttpApiRequest.h rename to src/base/api/requests/HttpApiRequest.h index f34d4be5..dc3eb037 100644 --- a/src/api/requests/HttpApiRequest.h +++ b/src/base/api/requests/HttpApiRequest.h @@ -27,7 +27,7 @@ #define XMRIG_HTTPAPIREQUEST_H -#include "api/requests/ApiRequest.h" +#include "base/api/requests/ApiRequest.h" #include "base/net/http/HttpApiResponse.h" #include "base/tools/String.h" diff --git a/src/base/base.cmake b/src/base/base.cmake index ef4da131..bc022c70 100644 --- a/src/base/base.cmake +++ b/src/base/base.cmake @@ -1,4 +1,5 @@ set(HEADERS_BASE + src/base/api/interfaces/IApiListener.h src/base/io/Console.h src/base/io/json/Json.h src/base/io/json/JsonChain.h @@ -114,6 +115,11 @@ endif() if (WITH_HTTP) set(HEADERS_BASE_HTTP src/3rdparty/http-parser/http_parser.h + src/base/api/Api.h + src/base/api/Httpd.h + src/base/api/interfaces/IApiRequest.h + src/base/api/requests/ApiRequest.h + src/base/api/requests/HttpApiRequest.h src/base/kernel/interfaces/IHttpListener.h src/base/kernel/interfaces/IJsonReader.h src/base/kernel/interfaces/ITcpServerListener.h @@ -129,6 +135,10 @@ if (WITH_HTTP) set(SOURCES_BASE_HTTP src/3rdparty/http-parser/http_parser.c + src/base/api/Api.cpp + src/base/api/Httpd.cpp + src/base/api/requests/ApiRequest.cpp + src/base/api/requests/HttpApiRequest.cpp src/base/net/http/HttpApiResponse.cpp src/base/net/http/HttpClient.cpp src/base/net/http/HttpContext.cpp diff --git a/src/base/kernel/Base.cpp b/src/base/kernel/Base.cpp index 03feef89..152f6bcc 100644 --- a/src/base/kernel/Base.cpp +++ b/src/base/kernel/Base.cpp @@ -47,15 +47,8 @@ #ifdef XMRIG_FEATURE_API -# include "api/Api.h" -# include "api/interfaces/IApiRequest.h" -#endif - - -#ifdef XMRIG_FEATURE_EMBEDDED_CONFIG -# include "core/config/Config_default.h" -#endif - +# include "base/api/Api.h" +# include "base/api/interfaces/IApiRequest.h" namespace xmrig { @@ -63,6 +56,12 @@ static const char *kConfigPathV1 = "/1/config"; static const char *kConfigPathV2 = "/2/config"; } // namespace xmrig +#endif + + +#ifdef XMRIG_FEATURE_EMBEDDED_CONFIG +# include "core/config/Config_default.h" +#endif class xmrig::BasePrivate diff --git a/src/base/kernel/Base.h b/src/base/kernel/Base.h index 6a33a802..8eb68866 100644 --- a/src/base/kernel/Base.h +++ b/src/base/kernel/Base.h @@ -26,7 +26,7 @@ #define XMRIG_BASE_H -#include "api/interfaces/IApiListener.h" +#include "base/api/interfaces/IApiListener.h" #include "base/kernel/interfaces/IConfigListener.h" #include "base/kernel/interfaces/IWatcherListener.h" #include "rapidjson/fwd.h" diff --git a/src/core/Miner.cpp b/src/core/Miner.cpp index 4406ce52..801b27c7 100644 --- a/src/core/Miner.cpp +++ b/src/core/Miner.cpp @@ -45,8 +45,8 @@ #ifdef XMRIG_FEATURE_API -# include "api/Api.h" -# include "api/interfaces/IApiRequest.h" +# include "base/api/Api.h" +# include "base/api/interfaces/IApiRequest.h" #endif diff --git a/src/core/Miner.h b/src/core/Miner.h index 035c0205..6fc75cd0 100644 --- a/src/core/Miner.h +++ b/src/core/Miner.h @@ -29,7 +29,7 @@ #include -#include "api/interfaces/IApiListener.h" +#include "base/api/interfaces/IApiListener.h" #include "base/kernel/interfaces/IBaseListener.h" #include "base/kernel/interfaces/ITimerListener.h" #include "crypto/common/Algorithm.h" diff --git a/src/net/Network.cpp b/src/net/Network.cpp index 547a8638..3ab8bec0 100644 --- a/src/net/Network.cpp +++ b/src/net/Network.cpp @@ -34,7 +34,6 @@ #include -#include "api/Api.h" #include "base/io/log/Log.h" #include "base/net/stratum/Client.h" #include "base/net/stratum/SubmitResult.h" @@ -51,8 +50,8 @@ #ifdef XMRIG_FEATURE_API -# include "api/Api.h" -# include "api/interfaces/IApiRequest.h" +# include "base/api/Api.h" +# include "base/api/interfaces/IApiRequest.h" #endif diff --git a/src/net/Network.h b/src/net/Network.h index ddf6d6f3..716ce610 100644 --- a/src/net/Network.h +++ b/src/net/Network.h @@ -30,7 +30,7 @@ #include -#include "api/interfaces/IApiListener.h" +#include "base/api/interfaces/IApiListener.h" #include "base/kernel/interfaces/IBaseListener.h" #include "base/kernel/interfaces/IStrategyListener.h" #include "base/kernel/interfaces/ITimerListener.h" From 6955f4a484ed47327c316fa68880b8057ecad4f9 Mon Sep 17 00:00:00 2001 From: XMRig Date: Mon, 12 Aug 2019 21:13:38 +0700 Subject: [PATCH 3/8] Added pause and resume methods via JSON RPC 2.0 API. --- src/base/api/interfaces/IApiRequest.h | 15 ++- src/base/api/requests/ApiRequest.h | 31 +++--- src/base/api/requests/HttpApiRequest.cpp | 129 +++++++++++++++++++---- src/base/api/requests/HttpApiRequest.h | 7 +- src/core/Miner.cpp | 12 +++ 5 files changed, 156 insertions(+), 38 deletions(-) diff --git a/src/base/api/interfaces/IApiRequest.h b/src/base/api/interfaces/IApiRequest.h index 8e65a921..4f74581c 100644 --- a/src/base/api/interfaces/IApiRequest.h +++ b/src/base/api/interfaces/IApiRequest.h @@ -54,16 +54,28 @@ public: enum RequestType { REQ_UNKNOWN, - REQ_SUMMARY + REQ_SUMMARY, + REQ_JSON_RPC + }; + + + enum ErrorCode : int { + RPC_PARSE_ERROR = -32700, + RPC_INVALID_REQUEST = -32600, + RPC_METHOD_NOT_FOUND = -32601, + RPC_INVALID_PARAMS = -32602 }; virtual ~IApiRequest() = default; + virtual bool accept() = 0; + virtual bool hasParseError() const = 0; virtual bool isDone() const = 0; virtual bool isNew() const = 0; virtual bool isRestricted() const = 0; virtual const rapidjson::Value &json() const = 0; + virtual const String &rpcMethod() const = 0; virtual const String &url() const = 0; virtual int version() const = 0; virtual Method method() const = 0; @@ -71,7 +83,6 @@ public: virtual rapidjson::Value &reply() = 0; virtual RequestType type() const = 0; virtual Source source() const = 0; - virtual void accept() = 0; virtual void done(int status) = 0; }; diff --git a/src/base/api/requests/ApiRequest.h b/src/base/api/requests/ApiRequest.h index 1fd721f1..ad4b0c35 100644 --- a/src/base/api/requests/ApiRequest.h +++ b/src/base/api/requests/ApiRequest.h @@ -28,6 +28,7 @@ #include "base/api/interfaces/IApiRequest.h" +#include "base/tools/String.h" namespace xmrig { @@ -40,28 +41,30 @@ public: ~ApiRequest() override; protected: - inline bool isDone() const override { return m_state == STATE_DONE; } - inline bool isNew() const override { return m_state == STATE_NEW; } - inline bool isRestricted() const override { return m_restricted; } - inline int version() const override { return m_version; } - inline RequestType type() const override { return m_type; } - inline Source source() const override { return m_source; } - inline void accept() override { m_state = STATE_ACCEPTED; } - inline void done(int) override { m_state = STATE_DONE; } - - int m_version = 1; - RequestType m_type = REQ_UNKNOWN; - -private: enum State { STATE_NEW, STATE_ACCEPTED, STATE_DONE }; + inline bool accept() override { m_state = STATE_ACCEPTED; return true; } + inline bool isDone() const override { return m_state == STATE_DONE; } + inline bool isNew() const override { return m_state == STATE_NEW; } + inline bool isRestricted() const override { return m_restricted; } + inline const String &rpcMethod() const override { return m_rpcMethod; } + inline int version() const override { return m_version; } + inline RequestType type() const override { return m_type; } + inline Source source() const override { return m_source; } + inline void done(int) override { m_state = STATE_DONE; } + + int m_version = 1; + RequestType m_type = REQ_UNKNOWN; + State m_state = STATE_NEW; + String m_rpcMethod; + +private: bool m_restricted; Source m_source; - State m_state = STATE_NEW; }; diff --git a/src/base/api/requests/HttpApiRequest.cpp b/src/base/api/requests/HttpApiRequest.cpp index c3c2dac4..ed13e47d 100644 --- a/src/base/api/requests/HttpApiRequest.cpp +++ b/src/base/api/requests/HttpApiRequest.cpp @@ -23,14 +23,45 @@ */ +#include "3rdparty/http-parser/http_parser.h" #include "base/api/requests/HttpApiRequest.h" +#include "base/io/json/Json.h" #include "base/net/http/HttpData.h" #include "rapidjson/error/en.h" +namespace xmrig { + + +static const char *kError = "error"; +static const char *kId = "id"; +static const char *kResult = "result"; + + +static inline const char *rpcError(int code) { + switch (code) { + case IApiRequest::RPC_PARSE_ERROR: + return "Parse error"; + + case IApiRequest::RPC_INVALID_REQUEST: + return "Invalid Request"; + + case IApiRequest::RPC_METHOD_NOT_FOUND: + return "Method not found"; + + case IApiRequest::RPC_INVALID_PARAMS: + return "Invalid params"; + } + + return "Internal error"; +} + + +} // namespace xmrig + + xmrig::HttpApiRequest::HttpApiRequest(const HttpData &req, bool restricted) : ApiRequest(SOURCE_HTTP, restricted), - m_parsed(false), m_req(req), m_res(req.id()), m_url(req.url.c_str()) @@ -41,6 +72,28 @@ xmrig::HttpApiRequest::HttpApiRequest(const HttpData &req, bool restricted) : } } + if (method() == METHOD_POST && url() == "/json_rpc") { + m_type = REQ_JSON_RPC; + accept(); + + if (hasParseError()) { + done(RPC_PARSE_ERROR); + + return; + } + + m_rpcMethod = Json::getString(json(), "method"); + if (m_rpcMethod.isEmpty()) { + done(RPC_INVALID_REQUEST); + + return; + } + + m_state = STATE_NEW; + + return; + } + if (url().size() > 4) { if (memcmp(url().data(), "/2/", 3) == 0) { m_version = 2; @@ -49,6 +102,31 @@ xmrig::HttpApiRequest::HttpApiRequest(const HttpData &req, bool restricted) : } +bool xmrig::HttpApiRequest::accept() +{ + using namespace rapidjson; + + ApiRequest::accept(); + + if (m_parsed == 0 && !m_req.body.empty()) { + m_body.Parse(m_req.body.c_str()); + m_parsed = m_body.HasParseError() ? 2 : 1; + + if (!hasParseError()) { + return true; + } + + if (type() != REQ_JSON_RPC) { + reply().AddMember(StringRef(kError), StringRef(GetParseError_En(m_body.GetParseError())), doc().GetAllocator()); + } + + return false; + } + + return hasParseError(); +} + + const rapidjson::Value &xmrig::HttpApiRequest::json() const { return m_body; @@ -61,27 +139,40 @@ xmrig::IApiRequest::Method xmrig::HttpApiRequest::method() const } -void xmrig::HttpApiRequest::accept() -{ - using namespace rapidjson; - - ApiRequest::accept(); - - if (!m_parsed && !m_req.body.empty()) { - m_parsed = true; - m_body.Parse(m_req.body.c_str()); - - if (m_body.HasParseError()) { - reply().AddMember("error", StringRef(GetParseError_En(m_body.GetParseError())), doc().GetAllocator());; - } - } -} - - void xmrig::HttpApiRequest::done(int status) { ApiRequest::done(status); - m_res.setStatus(status); + if (type() == REQ_JSON_RPC) { + using namespace rapidjson; + auto &allocator = doc().GetAllocator(); + + m_res.setStatus(HTTP_STATUS_OK); + + if (status != HTTP_STATUS_OK) { + if (status == HTTP_STATUS_NOT_FOUND) { + status = RPC_METHOD_NOT_FOUND; + } + + Value error(kObjectType); + error.AddMember("code", status, allocator); + error.AddMember("message", StringRef(rpcError(status)), allocator); + + reply().AddMember(StringRef(kError), error, allocator); + } + else if (!reply().HasMember(kResult)) { + Value result(kObjectType); + result.AddMember("status", "OK", allocator); + + reply().AddMember(StringRef(kResult), result, allocator); + } + + reply().AddMember("jsonrpc", "2.0", allocator); + reply().AddMember(StringRef(kId), Value().CopyFrom(Json::getValue(json(), kId), allocator), allocator); + } + else { + m_res.setStatus(status); + } + m_res.end(); } diff --git a/src/base/api/requests/HttpApiRequest.h b/src/base/api/requests/HttpApiRequest.h index dc3eb037..309b5a6b 100644 --- a/src/base/api/requests/HttpApiRequest.h +++ b/src/base/api/requests/HttpApiRequest.h @@ -44,19 +44,20 @@ public: HttpApiRequest(const HttpData &req, bool restricted); protected: + inline bool hasParseError() const override { return m_parsed == 2; } + inline const String &url() const override { return m_url; } inline rapidjson::Document &doc() override { return m_res.doc(); } inline rapidjson::Value &reply() override { return m_res.doc(); } - inline const String &url() const override { return m_url; } + bool accept() override; const rapidjson::Value &json() const override; Method method() const override; - void accept() override; void done(int status) override; private: - bool m_parsed; const HttpData &m_req; HttpApiResponse m_res; + int m_parsed = 0; rapidjson::Document m_body; String m_url; }; diff --git a/src/core/Miner.cpp b/src/core/Miner.cpp index 801b27c7..26a5d7e7 100644 --- a/src/core/Miner.cpp +++ b/src/core/Miner.cpp @@ -477,5 +477,17 @@ void xmrig::Miner::onRequest(IApiRequest &request) d_ptr->getBackends(request.reply(), request.doc()); } } + else if (request.type() == IApiRequest::REQ_JSON_RPC) { + if (request.rpcMethod() == "pause") { + request.accept(); + + setEnabled(false); + } + else if (request.rpcMethod() == "resume") { + request.accept(); + + setEnabled(true); + } + } } #endif From ed04ecd7353c6d33a2f4f0c7602bb89bba043b9a Mon Sep 17 00:00:00 2001 From: xmrig Date: Tue, 13 Aug 2019 00:24:04 +0700 Subject: [PATCH 4/8] Update CHANGELOG.md --- CHANGELOG.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 9b4317f7..50167977 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -11,6 +11,7 @@ - [#1010](https://github.com/xmrig/xmrig/pull/1010#issuecomment-482632107) Added daemon support (solo mining). - [#1066](https://github.com/xmrig/xmrig/issues/1066#issuecomment-518080529) Added error message if pool not ready for RandomX. - [#1105](https://github.com/xmrig/xmrig/issues/1105) Improved auto configuration for `cn-pico` algorithm. +- Added commands `pause` and `resume` via JSON RPC 2.0 API (`POST /json_rpc`). - Added command line option `--export-topology` for export hwloc topology to a XML file. - Breaked backward compatibility with previous configs and command line, `variant` option replaced to `algo`, global option `algo` removed, all CPU related settings moved to `cpu` object. - Options `av`, `safe` and `max-cpu-usage` removed. From df933964e1b53bdc2085925843fec78b87934cdf Mon Sep 17 00:00:00 2001 From: XMRig Date: Tue, 13 Aug 2019 17:56:32 +0700 Subject: [PATCH 5/8] Fixed autoconfig regression. --- src/backend/cpu/platform/HwlocCpuInfo.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/backend/cpu/platform/HwlocCpuInfo.cpp b/src/backend/cpu/platform/HwlocCpuInfo.cpp index 491305ec..7da99787 100644 --- a/src/backend/cpu/platform/HwlocCpuInfo.cpp +++ b/src/backend/cpu/platform/HwlocCpuInfo.cpp @@ -246,13 +246,14 @@ void xmrig::HwlocCpuInfo::processTopLevelCache(hwloc_obj_t cache, const Algorith findByType(cache, HWLOC_OBJ_CORE, [&cores](hwloc_obj_t found) { cores.emplace_back(found); }); size_t L3 = cache->attr->cache.size; + const bool L3_exclusive = isCacheExclusive(cache); size_t L2 = 0; int L2_associativity = 0; size_t extra = 0; const size_t scratchpad = algorithm.l3(); int intensity = algorithm.maxIntensity() == 1 ? -1 : 1; - if (cache->attr->cache.depth == 3 && isCacheExclusive(cache)) { + if (cache->attr->cache.depth == 3) { for (size_t i = 0; i < cache->arity; ++i) { hwloc_obj_t l2 = cache->children[i]; if (!isCacheObject(l2) || l2->attr == nullptr) { @@ -262,7 +263,7 @@ void xmrig::HwlocCpuInfo::processTopLevelCache(hwloc_obj_t cache, const Algorith L2 += l2->attr->cache.size; L2_associativity = l2->attr->cache.associativity; - if (l2->attr->cache.size >= scratchpad) { + if (L3_exclusive && l2->attr->cache.size >= scratchpad) { extra += scratchpad; } } From 9f4428a484f9ac38288eae7ddc7da78b6224a55a Mon Sep 17 00:00:00 2001 From: XMRig Date: Wed, 14 Aug 2019 02:23:48 +0700 Subject: [PATCH 6/8] Fixed user job recovery after donation round. --- src/backend/common/WorkerJob.h | 1 + 1 file changed, 1 insertion(+) diff --git a/src/backend/common/WorkerJob.h b/src/backend/common/WorkerJob.h index c9a3d55c..4b691952 100644 --- a/src/backend/common/WorkerJob.h +++ b/src/backend/common/WorkerJob.h @@ -56,6 +56,7 @@ public: } if (index() == 1 && job.index() == 0 && job == m_jobs[0]) { + m_index = 0; return; } From ca6376bbaaa74e60bd4edaaf16e49b4a4b0e22ab Mon Sep 17 00:00:00 2001 From: XMRig Date: Wed, 14 Aug 2019 02:27:52 +0700 Subject: [PATCH 7/8] Hide AVX2 information from miner summary as not important (only need for cn/gpu). --- src/Summary.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/Summary.cpp b/src/Summary.cpp index ab6b7b1e..227fdcc8 100644 --- a/src/Summary.cpp +++ b/src/Summary.cpp @@ -71,13 +71,12 @@ static void print_cpu(Config *) { const ICpuInfo *info = Cpu::info(); - Log::print(GREEN_BOLD(" * ") WHITE_BOLD("%-13s%s (%zu)") " %sx64 %sAES %sAVX2", + Log::print(GREEN_BOLD(" * ") WHITE_BOLD("%-13s%s (%zu)") " %sx64 %sAES", "CPU", info->brand(), info->packages(), info->isX64() ? GREEN_BOLD_S : RED_BOLD_S "-", - info->hasAES() ? GREEN_BOLD_S : RED_BOLD_S "-", - info->hasAVX2() ? GREEN_BOLD_S : RED_BOLD_S "-" + info->hasAES() ? GREEN_BOLD_S : RED_BOLD_S "-" ); # if defined(XMRIG_FEATURE_LIBCPUID) || defined (XMRIG_FEATURE_HWLOC) Log::print(WHITE_BOLD(" %-13s") BLACK_BOLD("L2:") WHITE_BOLD("%.1f MB") BLACK_BOLD(" L3:") WHITE_BOLD("%.1f MB") From f4db6b16dd6876766b326d13c9395e1dd89b3e10 Mon Sep 17 00:00:00 2001 From: xmrig Date: Wed, 14 Aug 2019 02:45:53 +0700 Subject: [PATCH 8/8] Update CHANGELOG_OLD.md --- doc/CHANGELOG_OLD.md | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/doc/CHANGELOG_OLD.md b/doc/CHANGELOG_OLD.md index 15ae35a9..58be062b 100644 --- a/doc/CHANGELOG_OLD.md +++ b/doc/CHANGELOG_OLD.md @@ -1,3 +1,9 @@ +# v2.99.6-beta +- Added commands `pause` and `resume` via JSON RPC 2.0 API (`POST /json_rpc`). +- Fixed autoconfig regression (since 2.99.5), mostly `rx/wow` was affected by this bug. +- Fixed user job recovery after donation round. +- Information about AVX2 CPU feature how hidden in miner summary. + # v2.99.5-beta - [#1066](https://github.com/xmrig/xmrig/issues/1066#issuecomment-518080529) Fixed crash and added error message if pool not ready for RandomX. - [#1092](https://github.com/xmrig/xmrig/issues/1092) Fixed crash if wrong CPU affinity used.