Re: [PATCH v5 2/5] read-cache: load cache extensions on a worker thread
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Sep 17, 2018, 17:19 UTC
- Message-ID
- <xmqqwork6nyn.fsf@gitster-ct.c.googlers.com>
- In-Reply-To
- <CACsJy8DtYrB99_7GyZaHsLTG8Ff7Mt_hENwhLE6_x6b5zzQ9Hg@mail.gmail.com>
Duy Nguyen <pclouds@gmail.com> writes:
Show 7 quoted lines
> On Sat, Sep 15, 2018 at 12:22 PM Duy Nguyen <pclouds@gmail.com> wrote: >> Wait there's no way to disable this parallel reading? Does not sound >> right. And if ordinary numbers mean the number of threads then 0 >> should mean no threading. Auto detection could have a new keyword, >> like 'auto'. > > My bad. Disabling threading means _1_ thread. What was I thinking...
I did the same during my earlier review. It seems that it somehow is unintuitive to us that we do not specify how many _extra_ threads of control we dedicate to ;-).