[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] md5: accepts a new --threads option
From: |
Jim Meyering |
Subject: |
Re: [PATCH] md5: accepts a new --threads option |
Date: |
Thu, 22 Oct 2009 14:05:30 +0200 |
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.
- Re: [PATCH] md5: accepts a new --threads option, (continued)
- Re: [PATCH] md5: accepts a new --threads option, Pádraig Brady, 2009/10/20
- Re: [PATCH] md5: accepts a new --threads option, Jim Meyering, 2009/10/21
- Re: [PATCH] md5: accepts a new --threads option, Pádraig Brady, 2009/10/21
- Re: [PATCH] md5: accepts a new --threads option, Jim Meyering, 2009/10/21
- Re: [PATCH] md5: accepts a new --threads option, Pádraig Brady, 2009/10/21
- Re: [PATCH] md5: accepts a new --threads option, Jim Meyering, 2009/10/21
- Re: [PATCH] md5: accepts a new --threads option, Pádraig Brady, 2009/10/21
- RE: [PATCH] md5: accepts a new --threads option, Voelker, Bernhard, 2009/10/21
- Re: [PATCH] md5: accepts a new --threads option, Pádraig Brady, 2009/10/22
- Re: [PATCH] md5: accepts a new --threads option, Eric Blake, 2009/10/22
- Re: [PATCH] md5: accepts a new --threads option,
Jim Meyering <=
- Re: [PATCH] md5: accepts a new --threads option, Paolo Bonzini, 2009/10/22
- Re: [PATCH] md5: accepts a new --threads option, Pádraig Brady, 2009/10/23
- Re: [PATCH] md5: accepts a new --threads option, Jim Meyering, 2009/10/23
- Re: [PATCH] md5: accepts a new --threads option, Pádraig Brady, 2009/10/23
- Re: [PATCH] md5: accepts a new --threads option, Jim Meyering, 2009/10/23
- Re: [PATCH] md5: accepts a new --threads option, Jim Meyering, 2009/10/23
- Re: [PATCH] md5: accepts a new --threads option, Paolo Bonzini, 2009/10/23
- Re: [PATCH] md5: accepts a new --threads option, Pádraig Brady, 2009/10/23
- Re: [PATCH] md5: accepts a new --threads option, Jim Meyering, 2009/10/23
- Re: [PATCH] md5: accepts a new --threads option, Pádraig Brady, 2009/10/23