Commit b12d2173 authored by Janne Grunau's avatar Janne Grunau

threads: limit the number of automatic threads to MAX_AUTO_THREADS

The extra thread added in {frame_}*thread_init was not taken into
account. Explicitly sets thread_count to 1 if only one CPU core was
detected. Also fixes two typos in comments.
parent da7c65f0
...@@ -186,7 +186,7 @@ static int get_logical_cpus(AVCodecContext *avctx) ...@@ -186,7 +186,7 @@ static int get_logical_cpus(AVCodecContext *avctx)
nb_cpus = sysconf(_SC_NPROCESSORS_ONLN); nb_cpus = sysconf(_SC_NPROCESSORS_ONLN);
#endif #endif
av_log(avctx, AV_LOG_DEBUG, "detected %d logical cores\n", nb_cpus); av_log(avctx, AV_LOG_DEBUG, "detected %d logical cores\n", nb_cpus);
return FFMIN(nb_cpus, MAX_AUTO_THREADS); return nb_cpus;
} }
...@@ -296,9 +296,11 @@ static int thread_init(AVCodecContext *avctx) ...@@ -296,9 +296,11 @@ static int thread_init(AVCodecContext *avctx)
if (!thread_count) { if (!thread_count) {
int nb_cpus = get_logical_cpus(avctx); int nb_cpus = get_logical_cpus(avctx);
// use number of cores + 1 as thread count if there is motre than one // use number of cores + 1 as thread count if there is more than one
if (nb_cpus > 1) if (nb_cpus > 1)
thread_count = avctx->thread_count = nb_cpus + 1; thread_count = avctx->thread_count = FFMIN(nb_cpus + 1, MAX_AUTO_THREADS);
else
thread_count = avctx->thread_count = 1;
} }
if (thread_count <= 1) { if (thread_count <= 1) {
...@@ -767,9 +769,11 @@ static int frame_thread_init(AVCodecContext *avctx) ...@@ -767,9 +769,11 @@ static int frame_thread_init(AVCodecContext *avctx)
if (!thread_count) { if (!thread_count) {
int nb_cpus = get_logical_cpus(avctx); int nb_cpus = get_logical_cpus(avctx);
// use number of cores + 1 as thread count if there is motre than one // use number of cores + 1 as thread count if there is more than one
if (nb_cpus > 1) if (nb_cpus > 1)
thread_count = avctx->thread_count = nb_cpus + 1; thread_count = avctx->thread_count = FFMIN(nb_cpus + 1, MAX_AUTO_THREADS);
else
thread_count = avctx->thread_count = 1;
} }
if (thread_count <= 1) { if (thread_count <= 1) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment