[ofa-general] Re: [PATCH/RFC] IB/ehca: Make symbols used only in a single source file static
Hoang-Nam Nguyen
HNGUYEN at de.ibm.com
Wed Mar 19 08:35:04 PDT 2008
> Allow the compiler to optimize better and generate smaller code:
>
> add/remove: 0/6 grow/shrink: 2/0 up/down: 1528/-1864 (-336)
> function old new delta
> .ehca_set_pagebuf 1344 2172 +828
> .ehca_probe 2312 3012 +700
> ehca_set_pagebuf_phys 24 - -24
> ehca_set_pagebuf_fmr 24 - -24
> ehca_init_device 24 - -24
> .ehca_set_pagebuf_fmr 480 - -480
> .ehca_set_pagebuf_phys 512 - -512
> .ehca_init_device 800 - -800
>
> Also this fixes warnings like:
>
> drivers/infiniband/hw/ehca/ehca_mrmw.c:2015:5: warning: symbol
> 'ehca_set_pagebuf_fmr' was not declared. Should it be static?
>
> Signed-off-by: Roland Dreier <rolandd at cisco.com>
> ---
Looks good to me. Much thanks!
Nam
More information about the general
mailing list