sgminer

Commit Graph

Author	SHA1	Message	Date
Noel Maersk	bfedb403fb	config: move more functions from miner.c to configuration.c	11 years ago
Noel Maersk	5491d10bf3	config: remove `static` from declarations in configuration.h	11 years ago
Noel Maersk	c581fb3332	config: move configuration options/functions to configuration.{c,h}	11 years ago
Noel Maersk	a7d2356cef	core: remove benchmark mode. It cannot be used as-is. It should probably be implemented as a standalone "fake pool" application anyway, to properly gauge the effect of queue/scantime/expiry and network latency. If someone really needs this, they can revert this commit.	11 years ago
Noel Maersk	474c783316	core: rename sgminer.c to miner.c. Remember forking from cgminer? Changing cgminer->sgminer was a drag. Help future others have less of this.	11 years ago
Noel Maersk	92c3b5eb48	api: set algorithm properly on addpool.	11 years ago
Noel Maersk	3c3a58475e	ui+api: ask for pool name/description/algorithm when adding pool from NCurses/API. Closes #116. This is trivial and shouldn't be so hard: it required modifying functions in both sgminer.c (for NCurses stuff) and api.c. There is much code repetition, since the NCurses interface is hard-coded in. Removing it would simplify things greatly.	11 years ago
troky	01e96606f0	Fixed (pool) algo selection.	11 years ago
Noel Maersk	da667d63e0	config: write pool name and description to config.	11 years ago
Noel Maersk	b3d483a035	core: revert two of the changes in Bllacky's pull request #187 . `ndigits` is actually an int, used as a field width specifier in snprintf(). s is an extensively-used pointer.	11 years ago
Bllacky	fe64ac99d6	Update sgminer.c One small change to work well with previous ones. Number of warnings now is 151, down from 157. Still a long way to go. Most of the serious warnings are related to variables in miner.h, which is so intertwined with everything else that I fear of breaking something or making performance worse if I touch miner.h too much.	11 years ago
Bllacky	1dc8aabda9	Update sgminer.c Minor changes to variable declaration. Should reduce the number of warnings by about 5.	11 years ago
Noel Maersk	aa5f4c713b	config: add `const` qualifier to set_pool_{algorithm,nfactor}(). MSVS doesn't like implicit casting.	11 years ago
Noel Maersk	e7078a5e16	misc: clarify currentpool() comment.	11 years ago
Noel Maersk	f6616aa63b	config: allow specifying per-pool algorithm with `--algorithm` instead of `--pool-algorithm`. This will probably only work with config file, though, since it relies on json_array_index.	11 years ago
Noel Maersk	53a199a21d	config: use get_pool_name() in set_pool_algorithm() and set_pool_nfactor().	11 years ago
Noel Maersk	af879b1c77	core/misc: rename set_poolname() to set_pool_name().	11 years ago
Noel Maersk	3ddf6baad9	core: rename default_algorithm to opt_algorithm. ... So it's easier to find when refactoring the configuration system later on.	11 years ago
Noel Maersk	b6218e20c1	config: rename --pool-priority to --priority, --coin to --description. There are now get_current_pool() and current_pool(), which is confusing. Moved them closer together, so it's at least obvious.	11 years ago
Jan Berdajs	cdb817bcbd	Change algorithm more robustly and faster, previous way was buggy	11 years ago
Jan Berdajs	c2131df6aa	add support for setting algorithm/nfactor for each pool separately, and fix bugged OpenCL thread cleanup (thread shutdown)	11 years ago
Noel Maersk	a57f841570	Revert "fix for mingw-w64" This reverts commit `32b13ccb46`. Reported by Wolf0 (aka wolf9466) to not break MinGW 64-bit compilation with GCC 4.8.2 (latest stable).	11 years ago
Noel Maersk	db0f7cf6f0	core: silence gcc warning about sprintf() use in add_pool() - this time for sure.	11 years ago
Noel Maersk	5896da18b1	core: remove total_users, total_passes, total_userpasses (no longer used). Silence gcc warnings. Need for these accounting vars removed a few commits ago by troky.	11 years ago
Noel Maersk	08ff1574df	config: write `algorithm` to config if not default. Closes https://github.com/veox/sgminer/issues/175	11 years ago
Noel Maersk	8203fea539	config: move `--coin` in config table (alphabetically).	11 years ago
Noel Maersk	10ec7769c8	pool: don't pass unused argument to sprintf() in add_pool(). Silences gcc warning.	11 years ago
Noel Maersk	ac3d13880c	Revert "core: correctly parse boolean configuration values." This reverts commit `8dd1c2d4af`. Introduced a security bug: since CCAN/opt is used to parse the JSON config (something it is not meant to do), it was possible to specify the default of "api-listen":false in .conf and expect the API to be disabled. However, since CCAN/opt does not check for parameters to OPT_WITHOUT_ARG, this got set to true anyway, and enabled the API on a (possibly unfirewalled) network. For this reason, configuration options are not named uniformly. This should have been reverted as soon as it became known: `4c4b909be6` (Fri Feb 28) Sorry for that.	11 years ago
troky	bcb78ed91b	Added --coin pool parameter.	11 years ago
troky	17f5e296a9	Fixed configuration/command line parsing.	11 years ago
troky	8db53026cb	Added --pool-priority	11 years ago
Martin Danielsen	01b3f70b63	A new --no-client-reconnect command that disables the 'client.reconnect' function. It looks like there's an exploit that abuses said command, but it is still not clear exactly how. There's also an additional message when the reconnect happens: "WARNING: POTENTIAL CLIENT.EXPLOIT!", but it requires you to be actively monitoring your log to catch it, and in which case you already get a "Reconnect requested from Pool 0 to 127.0.0.1" message. Note that disabling 'client.reconnect' might affect some pools that rely on the feature, like pools that you lease your rig to. Oh and this is dry-coded. :)	11 years ago
Noel Maersk	106dbee1f9	core: check if GPU thread is disabled before checking if it's waiting on getwork. Otherwise the watchdog thread segfaults on checking an uninitialised thr->getwork value.	11 years ago
Noel Maersk	87801c2578	opencl: rename cgpu_info->kname to cgpu_info->kernelname.	11 years ago
Noel Maersk	1333ed576d	core: simplify kernel selection code. WIP! Use a string instead of a state-machine-ey kernel selection mechanism where kernel names have to be predefined. This should allow just dropping new kernels into dir `kernel` without bloating the code in three other places. Is in dire need of a cleanup, function parameter check, edge case check - all the usual testing. In particular, checking these definitions/keywords: * enum cl_kernels * kname * [c]gpu[s]->kernel (and similar) * memory cleanup after strdup()?.. * chosen_kernel * queue_scrypt_kernel * strbuf * initCl	11 years ago
Noel Maersk	fc902f0f41	pool: add back --poolname as hidden option for backward compatibility. + Udate NEWS.	11 years ago
troky	49aef7100f	Reverting "poolname" to "name".	11 years ago
unknown	958764fecc	MSVS build: Fixed debug build. MSVS doesn't like uninitialized variables.	11 years ago
unknown	0e37fc24b0	Added algorrithm name to "coin" API command. Changed default algo name to "scrypt".	11 years ago
Noel Maersk	a3f9b24c69	core: set default algorithm for when neither --algorithm nor --nfactor are provided.	11 years ago
Noel Maersk	c6a27709f8	core: use global algorithm structure instead of opt_nfactor. Also squashed: config: add log messages to set_algo() and set_nfactor(). algorithm: use set_algorithm_nfactor() when setting default nfactor in set_algorithm(). Otherwise algorithm->n defaults to 0. P.S. Did I already mention how this could have been C++?..	11 years ago
Noel Maersk	a0c52bf67c	config: introduce set_nfactor() and use it to call set_algorithm_nfactor(). Had two bugs: 1. Will not compile due to unknown algorithm_t size. 2. nfactor is set to 0 (bad calling, fix later). So squashed two commits: 1. algorithm: move algorithm_t definition to header. 2. config: if --nfactor is specified, properly set algorithm->nfactor.	11 years ago
Noel Maersk	92b7770212	config: add `--algorithm` option and documentation. Doc in `doc/configuration.md` (has to be started sometime, no?). Configuration function has to be lamely-named set_algo(), because set_algorithm() is already declared in algorithm.h (prevent namespace conflict). algorithm has to be added as global variable due to the way the callback is done (by CCAN/opt, which in itself is nice). This can be cleaned up significantly by (at least) introducing a global configuration struct, but there is no reason to do it now just for this - better a wholesale manana.	11 years ago
Noel Maersk	ded4452316	ui: check for opt_incognito is Display settings menu.	11 years ago
Noel Maersk	9c0f912434	core: remove ifdef from opt_compact and opt_incognito declaration.	11 years ago
Noel Maersk	428061e416	pool: display URL instead of "Pool N" if no poolname provided. This works since get_pool_name() in pool.c checks if an empty string is provided, and uses the URL instead if so.	11 years ago
Noel Maersk	d58de45c0e	pool: use get_pool_name() where appropriate.	11 years ago
Noel Maersk	0a4f71882c	pool: use get_pool_user().	11 years ago
Noel Maersk	8e4fb3c554	pool: abstract get_pool_name() The introduction of get_pool_name() warrants a separate function so opt_incognito does not litter everywhere.	11 years ago
Noel Maersk	0d562ffbf5	core: use opt_incognito in a few more applog() places.	11 years ago

5 Commits (82173bafa6b39cadf55589b3df5b002fcd9767c8)