[SSSD] [PATCH 2/4] Move DEBUG macro body to debug_fn

Simo Sorce simo at redhat.com
Thu Oct 10 20:29:17 UTC 2013


On Thu, 2013-10-10 at 16:25 -0400, Simo Sorce wrote:
> The code size reduction looks appealing, however please check the
> level
> before calling the function, directly in the DEBUG() macro.
> This way we completely avoid the function call when we are not going
> to
> debug anything.
> 
We could even go as far as wrapping the level test into unlikely() so
that the compiler can help branch prediction by not calling the function
by default.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York




More information about the sssd-devel mailing list