svn commit: samba r21278 - in branches/SAMBA_3_0/source/smbd: .
James Peach
jpeach at samba.org
Sat Feb 17 18:19:08 GMT 2007
On 17/02/2007, at 5:02 AM, Volker Lendecke wrote:
> On Sat, Feb 17, 2007 at 01:18:49PM +0100, Volker Lendecke wrote:
>> On Sun, Feb 11, 2007 at 05:59:53PM -0800, Jeremy Allison wrote:
>>> Can you run cachegrind on both methods to see why the
>>> memcpy ends up taking less time ?
>>
>> Ok, without the patch for 10000 runs cachegrind says:
>
> Hmm. I take everything back :-)
>
> I got fooled by -DDEVELOPER and clobber_region. Without that
> both versions just don't show any significant difference:
[snip]
> Then the argument would be that with the patch we gain some
> code simplicity, which was the original goal :-)
My 2c is that these sorts of changes will have variable impact
depending on the machine architecture and system libraries, eg. the
system's memory bandwidth and malloc implementation. You should go
for whichever code is cleanest.
--
James Peach | jpeach at samba.org
More information about the samba-technical
mailing list