Re: [PATCH] md5: accepts a new --threads option

[Jim Meyering]

Pádraig Brady wrote:
> Jim Meyering wrote:
>> Pádraig Brady wrote:
>>
>>> p.s. I'll look at bypassing stdio on input to see
>>> if I can get at least the 2% back
>>
>> IMHO, even if it did, it would not be worth it.
>
> Right, a quick test here shows only a 0.8% gain from
> bypassing stdio. However I also noticed that the
> digest routines in gnulib do fread(4096).
> Bumping that up to 32KiB gives a 3% boost.

Nice.

> That would be increased even more if more efficient
> checksumming routines are used (which is on the horizon).
> Note the coreutils min was bumped to 32KiB recently:
> http://git.savannah.gnu.org/gitweb/?p=coreutils.git;a=commit;h=55efc5f
>
> Does anyone have any objections to increasing
> the stack requirement by 28672 bytes?

That sounds reasonable to me, too.
No objection here.