On Fri, Nov 05, 2021 at 08:32:16PM +0800, lance.lmw...@gmail.com wrote: > From: Limin Wang <lance.lmw...@gmail.com> > > Signed-off-by: Limin Wang <lance.lmw...@gmail.com> > --- > libavutil/slicethread.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/slicethread.c b/libavutil/slicethread.c > index 867ce32..7b3a106 100644 > --- a/libavutil/slicethread.c > +++ b/libavutil/slicethread.c > @@ -104,7 +104,7 @@ int avpriv_slicethread_create(AVSliceThread **pctx, void > *priv, > if (!nb_threads) { > int nb_cpus = av_cpu_count(); > if (nb_cpus > 1) > - nb_threads = nb_cpus + 1; > + nb_threads = FFMIN(nb_cpus + 1, 16);
why should the threads be limited ? why should they be limited at 16 for everyone ? one mighht be a 2 core cpu one might have 200 cores i think the commit message should be more verbose describing what the problem is that this is fixing thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Homeopathy is like voting while filling the ballot out with transparent ink. Sometimes the outcome one wanted occurs. Rarely its worse than filling out a ballot properly.
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".