[PATCH] Improve talloc performance
Andrew Bartlett
abartlet at samba.org
Sat Jun 18 05:47:55 UTC 2016
On Fri, 2016-06-17 at 17:12 -0700, Jeremy Allison wrote:
> I'm excited enough about the speed increase to finish
> rewriting this, as it looks great work !
Thanks for looking over it. One of the big things that struck me in
talloc is that it is a core lib, used in every hot codepath, yet it has
a lot of extra features.
What I mean is that adding unlikely() or likely() to a branch doesn't
make that call free. Now, perhaps I'm not building with the right
magic, but I see a lot of CPU cost for these features.
What I mean is that we have the pool code (and I see how that can be
quite fast in the right circumstances), but also the talloc_memlimit
code, which is never used in Samba. Both of these add overhead in the
talloc chunk, and add branches to the core routines. We even have a
getenv() call in talloc_free()! (This belongs in the library
constructor, now that we have one).
Thanks,
Andrew Bartlett
--
Andrew Bartlett http://samba.org/~abartlet/
Authentication Developer, Samba Team http://samba.org
Samba Developer, Catalyst IT http://catalyst.net.nz/services/samba
More information about the samba-technical
mailing list