LBRYCommunity/lbrycrd
1
0
Fork 0
Code Issues 29 Pull requests 1 Projects Releases 32 Packages Wiki Activity

Merge #12923: util: Pass pthread_self() to pthread_setschedparam instead of 0

Browse source 
b86730a util: Remove designator initializer from ScheduleBatchPriority (Wladimir J. van der Laan)
cff66e6 util: Pass pthread_self() to pthread_setschedparam instead of 0 (Wladimir J. van der Laan)

Pull request description:

  Nowhere in the man page of `pthread_setschedparam` it is mentioned that `0` is a valid value. The example uses `pthread_self()`, so should we.

  (noticed by Anthony Towns)
  Fixes #12915.

Tree-SHA512: 249e93b1ae7e3ba28de6ee6288400b91d21ca1b4ca41d82211f6c9609b62deb5ac87182c7bf08471d3a3e0c1af314c9ecd41f8ae864febe963b1de8a816dd82f
This commit is contained in:
Wladimir J. van der Laan 2018-04-09 17:32:55 +02:00
commit a7cbe38ae2
No known key found for this signature in database
GPG key ID: 1E4AED62986CD25D
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/util.cpp
View file

@ -1064,8 +1064,8 @@ fs::path AbsPathForConfigVal(const fs::path& path, bool net_specific)
int ScheduleBatchPriority(void) int ScheduleBatchPriority(void)
{ {
#ifdef SCHED_BATCH #ifdef SCHED_BATCH
const static sched_param param{.sched_priority = 0}; const static sched_param param{0};
if (int ret = pthread_setschedparam(0, SCHED_BATCH, &param)) { if (int ret = pthread_setschedparam(pthread_self(), SCHED_BATCH, &param)) {
LogPrintf("Failed to pthread_setschedparam: %s\n", strerror(errno)); LogPrintf("Failed to pthread_setschedparam: %s\n", strerror(errno));
return ret; return ret;
} }