Re: [PATCH] Some love to default profiler

From: Andi Kleen
Date: Wed Jul 04 2007 - 17:44:19 EST


Alexey Dobriyan <adobriyan@xxxxxxxxx> writes:

> 2) Put profile.c under CONFIG_PROFILING. You enabled profiling in config, you
> will get it. Removes conditional branch from schedule(). Code savings on my
> usual config:
>
> text data bss dec hex filename
> 2921871 179895 180224 3281990 321446 vmlinux before
> 2920141 179847 180224 3280212 320d54 vmlinux after
> --------------------------------------------------------------
> -1730 -48 -1778

I think it's better to keep it by default. 1.7k is not really all that much.

-Andi
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/