diff options
author | Cory Fields <cory-nospam-@coryfields.com> | 2018-09-27 18:08:39 -0400 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2018-12-09 21:08:48 +0800 |
commit | 89282379baa503156d9b85f116ae5672f8588b39 (patch) | |
tree | 9e2c4f7a4ea4a4c74817e7e9e58ff7b19b967759 | |
parent | 2753285be72dd051cf2987ca5c80d0b149914020 (diff) |
threads: fix unitialized members in sched_param
Building with gcc 8.2 against musl libc, which apparently has more attributes
available in its sched_param. The following warnings were produced:
warning: missing initializer for member 'sched_param::sched_ss_low_priority' [-Wmissing-field-initializers]
warning: missing initializer for member 'sched_param::sched_ss_repl_period' [-Wmissing-field-initializers]
warning: missing initializer for member 'sched_param::sched_ss_init_budget' [-Wmissing-field-initializers]
warning: missing initializer for member 'sched_param::sched_ss_max_repl' [-Wmissing-field-initializers]
Since the current thread may have interesting non-zero values for these fields,
we want to be sure to only change the intended one. Query and modify the
current sched_param rather than starting from a zeroed one.
-rw-r--r-- | src/util/system.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/util/system.cpp b/src/util/system.cpp index 8e201ec590..4309da3b54 100644 --- a/src/util/system.cpp +++ b/src/util/system.cpp @@ -1293,7 +1293,7 @@ fs::path AbsPathForConfigVal(const fs::path& path, bool net_specific) int ScheduleBatchPriority() { #ifdef SCHED_BATCH - const static sched_param param{0}; + const static sched_param param{}; if (int ret = pthread_setschedparam(pthread_self(), SCHED_BATCH, ¶m)) { LogPrintf("Failed to pthread_setschedparam: %s\n", strerror(errno)); return ret; |