From 954c0e6fbc8cb20e2dd9cb6118bf1d60f16d9fa8 Mon Sep 17 00:00:00 2001 From: Jan Berdajs Date: Mon, 9 Jun 2014 15:56:39 +0200 Subject: [PATCH] fix opt_kernel_path writing over the allocated memory! also some small changes for more robust hard restart --- driver-opencl.c | 5 +++-- sgminer.c | 23 +++++++++++++++++------ 2 files changed, 20 insertions(+), 8 deletions(-) diff --git a/driver-opencl.c b/driver-opencl.c index d81e9fd2..12b0b938 100644 --- a/driver-opencl.c +++ b/driver-opencl.c @@ -1226,8 +1226,6 @@ static bool opencl_thread_prepare(struct thr_info *thr) strcpy(name, ""); applog(LOG_INFO, "Init GPU thread %i GPU %i virtual GPU %i", i, gpu, virtual_gpu); - if (thrdata) - thrdata->queue_kernel_parameters = cgpu->algorithm.queue_kernel; clStates[i] = initCl(virtual_gpu, name, sizeof(name), &cgpu->algorithm); if (!clStates[i]) { @@ -1440,6 +1438,9 @@ static void opencl_thread_shutdown(struct thr_info *thr) free(clState->extra_kernels); free(clState); } + free(((struct opencl_thread_data *)thr->cgpu_data)->res); + free(thr->cgpu_data); + thr->cgpu_data = NULL; } struct device_drv opencl_drv = { diff --git a/sgminer.c b/sgminer.c index 24e43ea3..9f7f73e1 100644 --- a/sgminer.c +++ b/sgminer.c @@ -6231,8 +6231,10 @@ static void get_work_prepare_thread(struct thr_info *mythr, struct work *work) for (i = 0; i < mining_threads; i++) { struct thr_info *thr = mining_thr[i]; thr->cgpu->algorithm = work->pool->algorithm; - if (soft_restart) + if (soft_restart) { thr->cgpu->drv->thread_prepare(thr); + thr->cgpu->drv->thread_init(thr); + } // Necessary because algorithms can have dramatically different diffs thr->cgpu->drv->working_diff = 1; @@ -7815,6 +7817,8 @@ static void restart_mining_threads(unsigned int new_n_threads) static void *restart_mining_threads_thread(void *userdata) { + pthread_detach(pthread_self()); + restart_mining_threads((unsigned int) (intptr_t) userdata); return NULL; @@ -7993,11 +7997,18 @@ int main(int argc, char *argv[]) free(cnfbuf); cnfbuf = NULL; } -#ifdef _MSC_VER - strcat(opt_kernel_path, "\\"); -#else - strcat(opt_kernel_path, "/"); -#endif + + if (opt_kernel_path) { + char *old_path = opt_kernel_path; + opt_kernel_path = (char *)alloca(PATH_MAX); + strcpy(opt_kernel_path, old_path); + #ifdef _MSC_VER + strcat(opt_kernel_path, "\\"); + #else + strcat(opt_kernel_path, "/"); + #endif + } + if (want_per_device_stats) opt_log_output = true;