Convert remaining [f]print to applog().

Also, remove a few superfluous printouts.
This commit is contained in:
Jeff Garzik 2011-03-21 04:02:13 -04:00 committed by Jeff Garzik
parent 144cf62d7c
commit 2fd9d54443
4 changed files with 36 additions and 41 deletions

View file

@ -203,12 +203,12 @@ static bool jobj_binary(const json_t *obj, const char *key,
tmp = json_object_get(obj, key); tmp = json_object_get(obj, key);
if (!tmp) { if (!tmp) {
fprintf(stderr, "JSON key '%s' not found\n", key); applog(LOG_ERR, "JSON key '%s' not found", key);
return false; return false;
} }
hexstr = json_string_value(tmp); hexstr = json_string_value(tmp);
if (!hexstr) { if (!hexstr) {
fprintf(stderr, "JSON key '%s' is not a string\n", key); applog(LOG_ERR, "JSON key '%s' is not a string", key);
return false; return false;
} }
if (!hex2bin(buf, hexstr, buflen)) if (!hex2bin(buf, hexstr, buflen))
@ -221,22 +221,22 @@ static bool work_decode(const json_t *val, struct work *work)
{ {
if (!jobj_binary(val, "midstate", if (!jobj_binary(val, "midstate",
work->midstate, sizeof(work->midstate))) { work->midstate, sizeof(work->midstate))) {
fprintf(stderr, "JSON inval midstate\n"); applog(LOG_ERR, "JSON inval midstate");
goto err_out; goto err_out;
} }
if (!jobj_binary(val, "data", work->data, sizeof(work->data))) { if (!jobj_binary(val, "data", work->data, sizeof(work->data))) {
fprintf(stderr, "JSON inval data\n"); applog(LOG_ERR, "JSON inval data");
goto err_out; goto err_out;
} }
if (!jobj_binary(val, "hash1", work->hash1, sizeof(work->hash1))) { if (!jobj_binary(val, "hash1", work->hash1, sizeof(work->hash1))) {
fprintf(stderr, "JSON inval hash1\n"); applog(LOG_ERR, "JSON inval hash1");
goto err_out; goto err_out;
} }
if (!jobj_binary(val, "target", work->target, sizeof(work->target))) { if (!jobj_binary(val, "target", work->target, sizeof(work->target))) {
fprintf(stderr, "JSON inval target\n"); applog(LOG_ERR, "JSON inval target");
goto err_out; goto err_out;
} }
@ -258,7 +258,7 @@ static bool submit_upstream_work(CURL *curl, const struct work *work)
/* build hex string */ /* build hex string */
hexstr = bin2hex(work->data, sizeof(work->data)); hexstr = bin2hex(work->data, sizeof(work->data));
if (!hexstr) { if (!hexstr) {
fprintf(stderr, "submit_upstream_work OOM\n"); applog(LOG_ERR, "submit_upstream_work OOM");
goto out; goto out;
} }
@ -268,12 +268,12 @@ static bool submit_upstream_work(CURL *curl, const struct work *work)
hexstr); hexstr);
if (opt_debug) if (opt_debug)
fprintf(stderr, "DBG: sending RPC call:\n%s", s); applog(LOG_DEBUG, "DBG: sending RPC call: %s", s);
/* issue JSON-RPC request */ /* issue JSON-RPC request */
val = json_rpc_call(curl, rpc_url, userpass, s, false, false); val = json_rpc_call(curl, rpc_url, userpass, s, false, false);
if (!val) { if (!val) {
fprintf(stderr, "submit_upstream_work json_rpc_call failed\n"); applog(LOG_ERR, "submit_upstream_work json_rpc_call failed");
goto out; goto out;
} }
@ -339,16 +339,14 @@ static bool workio_get_work(struct workio_cmd *wc, CURL *curl)
/* obtain new work from bitcoin via JSON-RPC */ /* obtain new work from bitcoin via JSON-RPC */
while (!get_upstream_work(curl, ret_work)) { while (!get_upstream_work(curl, ret_work)) {
fprintf(stderr, "json_rpc_call failed, ");
if ((opt_retries >= 0) && (++failures > opt_retries)) { if ((opt_retries >= 0) && (++failures > opt_retries)) {
fprintf(stderr, "terminating workio thread\n"); applog(LOG_ERR, "json_rpc_call failed, terminating workio thread");
free(ret_work); free(ret_work);
return false; return false;
} }
/* pause, then restart work-request loop */ /* pause, then restart work-request loop */
fprintf(stderr, "retry after %d seconds\n", applog(LOG_ERR, "json_rpc_call failed, retry after %d seconds",
opt_fail_pause); opt_fail_pause);
sleep(opt_fail_pause); sleep(opt_fail_pause);
} }
@ -367,12 +365,12 @@ static bool workio_submit_work(struct workio_cmd *wc, CURL *curl)
/* submit solution to bitcoin via JSON-RPC */ /* submit solution to bitcoin via JSON-RPC */
while (!submit_upstream_work(curl, wc->u.work)) { while (!submit_upstream_work(curl, wc->u.work)) {
if ((opt_retries >= 0) && (++failures > opt_retries)) { if ((opt_retries >= 0) && (++failures > opt_retries)) {
fprintf(stderr, "...terminating workio thread\n"); applog(LOG_ERR, "...terminating workio thread");
return false; return false;
} }
/* pause, then restart work-request loop */ /* pause, then restart work-request loop */
fprintf(stderr, "...retry after %d seconds\n", applog(LOG_ERR, "...retry after %d seconds",
opt_fail_pause); opt_fail_pause);
sleep(opt_fail_pause); sleep(opt_fail_pause);
} }
@ -388,7 +386,7 @@ static void *workio_thread(void *userdata)
curl = curl_easy_init(); curl = curl_easy_init();
if (!curl) { if (!curl) {
fprintf(stderr, "CURL initialization failed\n"); applog(LOG_ERR, "CURL initialization failed");
return NULL; return NULL;
} }
@ -513,8 +511,8 @@ static void *miner_thread(void *userdata)
/* obtain new work from internal workio thread */ /* obtain new work from internal workio thread */
if (!get_work(mythr, &work)) { if (!get_work(mythr, &work)) {
fprintf(stderr, "work retrieval failed, exiting " applog(LOG_ERR, "work retrieval failed, exiting "
"mining thread %d\n", mythr->id); "mining thread %d", mythr->id);
goto out; goto out;
} }
@ -639,7 +637,7 @@ static void *longpoll_thread(void *userdata)
curl = curl_easy_init(); curl = curl_easy_init();
if (!curl) { if (!curl) {
fprintf(stderr, "CURL initialization failed\n"); applog(LOG_ERR, "CURL initialization failed");
goto out; goto out;
} }
@ -657,10 +655,10 @@ static void *longpoll_thread(void *userdata)
} else { } else {
if (failures++ < 10) { if (failures++ < 10) {
sleep(30); sleep(30);
fprintf(stderr, applog(LOG_ERR,
"longpoll failed, sleeping for 30s\n"); "longpoll failed, sleeping for 30s\n");
} else { } else {
fprintf(stderr, applog(LOG_ERR,
"longpoll failed, ending thread\n"); "longpoll failed, ending thread\n");
goto out; goto out;
} }
@ -715,7 +713,7 @@ static void parse_arg (int key, char *arg)
json_decref(opt_config); json_decref(opt_config);
opt_config = json_load_file(arg, &err); opt_config = json_load_file(arg, &err);
if (!json_is_object(opt_config)) { if (!json_is_object(opt_config)) {
fprintf(stderr, "JSON decode of %s failed\n", arg); applog(LOG_ERR, "JSON decode of %s failed", arg);
show_usage(); show_usage();
} }
break; break;
@ -810,7 +808,7 @@ static void parse_config(void)
} else if (!options[i].has_arg && json_is_true(val)) } else if (!options[i].has_arg && json_is_true(val))
parse_arg(options[i].val, ""); parse_arg(options[i].val, "");
else else
fprintf(stderr, "JSON option %s invalid\n", applog(LOG_ERR, "JSON option %s invalid",
options[i].name); options[i].name);
} }
} }
@ -870,7 +868,7 @@ int main (int argc, char *argv[])
/* start work I/O thread */ /* start work I/O thread */
if (pthread_create(&thr->pth, NULL, workio_thread, thr)) { if (pthread_create(&thr->pth, NULL, workio_thread, thr)) {
fprintf(stderr, "workio thread create failed\n"); applog(LOG_ERR, "workio thread create failed");
return 1; return 1;
} }
@ -885,7 +883,7 @@ int main (int argc, char *argv[])
/* start longpoll thread */ /* start longpoll thread */
if (pthread_create(&thr->pth, NULL, longpoll_thread, thr)) { if (pthread_create(&thr->pth, NULL, longpoll_thread, thr)) {
fprintf(stderr, "longpoll thread create failed\n"); applog(LOG_ERR, "longpoll thread create failed");
return 1; return 1;
} }
} else } else
@ -901,7 +899,7 @@ int main (int argc, char *argv[])
return 1; return 1;
if (pthread_create(&thr->pth, NULL, miner_thread, thr)) { if (pthread_create(&thr->pth, NULL, miner_thread, thr)) {
fprintf(stderr, "thread %d create failed\n", i); applog(LOG_ERR, "thread %d create failed", i);
return 1; return 1;
} }

View file

@ -59,7 +59,10 @@ void *alloca (size_t);
#include <syslog.h> #include <syslog.h>
#else #else
enum { enum {
LOG_ERR,
LOG_WARNING,
LOG_INFO, LOG_INFO,
LOG_DEBUG,
}; };
#endif #endif

View file

@ -608,8 +608,6 @@ bool scanhash_asm32(int thr_id, const unsigned char *midstate,
} }
if ((n >= max_nonce) || work_restart[thr_id].restart) { if ((n >= max_nonce) || work_restart[thr_id].restart) {
if (opt_debug)
fprintf(stderr, "DBG: end of nonce range\n");
*hashes_done = stat_ctr; *hashes_done = stat_ctr;
return false; return false;
} }

22
util.c
View file

@ -150,9 +150,6 @@ static size_t resp_hdr_cb(void *ptr, size_t size, size_t nmemb, void *user_data)
char *rem, *val = NULL, *key = NULL; char *rem, *val = NULL, *key = NULL;
void *tmp; void *tmp;
if (opt_protocol)
printf("In resp_hdr_cb\n");
val = calloc(1, ptrlen); val = calloc(1, ptrlen);
key = calloc(1, ptrlen); key = calloc(1, ptrlen);
if (!key || !val) if (!key || !val)
@ -183,7 +180,7 @@ static size_t resp_hdr_cb(void *ptr, size_t size, size_t nmemb, void *user_data)
goto out; goto out;
if (opt_protocol) if (opt_protocol)
printf("HTTP hdr(%s): %s\n", key, val); applog(LOG_DEBUG, "HTTP hdr(%s): %s", key, val);
if (!strcasecmp("X-Long-Polling", key)) { if (!strcasecmp("X-Long-Polling", key)) {
hi->lp_path = val; /* steal memory reference */ hi->lp_path = val; /* steal memory reference */
@ -241,7 +238,7 @@ json_t *json_rpc_call(CURL *curl, const char *url,
curl_easy_setopt(curl, CURLOPT_POST, 1); curl_easy_setopt(curl, CURLOPT_POST, 1);
if (opt_protocol) if (opt_protocol)
printf("JSON protocol request:\n%s\n", rpc_req); applog(LOG_DEBUG, "JSON protocol request:\n%s\n", rpc_req);
upload_data.buf = rpc_req; upload_data.buf = rpc_req;
upload_data.len = strlen(rpc_req); upload_data.len = strlen(rpc_req);
@ -257,7 +254,7 @@ json_t *json_rpc_call(CURL *curl, const char *url,
rc = curl_easy_perform(curl); rc = curl_easy_perform(curl);
if (rc) { if (rc) {
fprintf(stderr, "HTTP request failed: %s\n", curl_err_str); applog(LOG_ERR, "HTTP request failed: %s", curl_err_str);
goto err_out; goto err_out;
} }
@ -272,13 +269,13 @@ json_t *json_rpc_call(CURL *curl, const char *url,
val = json_loads(all_data.buf, &err); val = json_loads(all_data.buf, &err);
if (!val) { if (!val) {
fprintf(stderr, "JSON decode failed(%d): %s\n", err.line, err.text); applog(LOG_ERR, "JSON decode failed(%d): %s", err.line, err.text);
goto err_out; goto err_out;
} }
if (opt_protocol) { if (opt_protocol) {
char *s = json_dumps(val, JSON_INDENT(3)); char *s = json_dumps(val, JSON_INDENT(3));
printf("JSON protocol response:\n%s\n", s); applog(LOG_DEBUG, "JSON protocol response:\n%s", s);
free(s); free(s);
} }
@ -297,7 +294,7 @@ json_t *json_rpc_call(CURL *curl, const char *url,
else else
s = strdup("(unknown reason)"); s = strdup("(unknown reason)");
fprintf(stderr, "JSON-RPC call failed: %s\n", s); applog(LOG_ERR, "JSON-RPC call failed: %s", s);
free(s); free(s);
@ -336,7 +333,7 @@ bool hex2bin(unsigned char *p, const char *hexstr, size_t len)
unsigned int v; unsigned int v;
if (!hexstr[1]) { if (!hexstr[1]) {
fprintf(stderr, "hex2bin str truncated\n"); applog(LOG_ERR, "hex2bin str truncated");
return false; return false;
} }
@ -345,8 +342,7 @@ bool hex2bin(unsigned char *p, const char *hexstr, size_t len)
hex_byte[2] = 0; hex_byte[2] = 0;
if (sscanf(hex_byte, "%x", &v) != 1) { if (sscanf(hex_byte, "%x", &v) != 1) {
fprintf(stderr, "hex2bin sscanf '%s' failed\n", applog(LOG_ERR, "hex2bin sscanf '%s' failed", hex_byte);
hex_byte);
return false; return false;
} }
@ -421,7 +417,7 @@ bool fulltest(const unsigned char *hash, const unsigned char *target)
hash_str = bin2hex(hash_swap, 32); hash_str = bin2hex(hash_swap, 32);
target_str = bin2hex(target_swap, 32); target_str = bin2hex(target_swap, 32);
fprintf(stderr, " Proof: %s\nTarget: %s\nTrgVal? %s\n", applog(LOG_DEBUG, " Proof: %s\nTarget: %s\nTrgVal? %s",
hash_str, hash_str,
target_str, target_str,
rc ? "YES (hash < target)" : rc ? "YES (hash < target)" :