[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] (x)memcoll: performance improvement when input is known to b
From: |
Pádraig Brady |
Subject: |
Re: [PATCH] (x)memcoll: performance improvement when input is known to be NUL delimited. |
Date: |
Fri, 12 Mar 2010 00:45:54 +0000 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.8) Gecko/20100227 Thunderbird/3.0.3 |
On 11/03/10 17:10, Chen Guo wrote:
> Hi Padraig,
> Thanks for the _unlikely insight, that explains why it's not slower
> but the speed up is still quite mysterious. When I ran with the line
> lengths you wanted, with check it is no longer faster, but something
> else I don't quite understand pops up.
>
> The file I usually test on is 1M lines, 98 bytes each not counting
> newline. Average of5 runs:
Thanks for your tenacity on performance testing, it's most welcome.
> bytes with(s) without(s)
> -----------------------
> 31 4.02 4.04
> 32 4.37 4.36
> 64 4.44 4.44
> 66 4.81 4.79
> 95 4.97 4.96
> 97 5.15 5.18
I think you may be running into some measurement bias here:
http://www-plan.cs.colorado.edu/diwan/asplos09.pdf
I would guess that the different size strings are causing
alignment differences and thus change the results
which ultimately depend on many variables.
Varying the sizes disproves that the check is causing
it to run faster and it's probably just insignificant
(because of __unlikely) in the presence of the other variables.
cheers,
Pádraig.
- Re: [PATCH] (x)memcoll: performance improvement when input is known to be NUL delimited., (continued)
- Re: [PATCH] (x)memcoll: performance improvement when input is known to be NUL delimited., Bruno Haible, 2010/03/14
- Re: [PATCH] (x)memcoll: performance improvement when input is known to be NUL delimited., Paolo Bonzini, 2010/03/15
- Re: [PATCH] (x)memcoll: performance improvement when input is known to be NUL delimited., Jim Meyering, 2010/03/15
- Re: abort vs. assert, Bruno Haible, 2010/03/15
- Re: abort vs. assert, Jim Meyering, 2010/03/15
- Re: abort vs. assert, Bruno Haible, 2010/03/15
- Re: abort vs. assert, Paolo Bonzini, 2010/03/16
Re: [PATCH] (x)memcoll: performance improvement when input is known to be NUL delimited., Chen Guo, 2010/03/10
Re: [PATCH] (x)memcoll: performance improvement when input is known to be NUL delimited., Bruno Haible, 2010/03/14