On Sun 01-01-17 22:59:40, Michal Nazarewicz wrote: [...] > Actually, Linux style is more like: > > #ifdef CONFIG_CMA_DEBUG > static void cma_debug_show_areas() > { > … > } > #else > static inline void cma_debug_show_areas() { } > #endif yes, we usually do that when the function is externally visible. Inline ifdef for static functions saves few lines. Not that it would matter much though. -- Michal Hocko SUSE Labs -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>