Re: [PATCH] Deinline a few functions in mmap.c

From: Christoph Hellwig
Date: Sat Jul 05 2008 - 13:10:36 EST


On Sat, Jul 05, 2008 at 06:37:30PM +0200, Denys Vlasenko wrote:
> #if defined(CONFIG_STACK_GROWSUP) || defined(CONFIG_IA64)
> /*
> * PA-RISC uses this for its stack; IA64 for its Register Backing Store.
> * vma is the last one with address > vma->vm_end. Have to extend vma.
> */
> #ifndef CONFIG_IA64
> -static inline
> +static
> #endif

Unrelated note, but I think this ifdef should go. We've always prefered
slightly less possible optimizations over horribly ugly ifdefs.

--
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/