Re: [PATCH] remove cacheline alignment from inode slabs

[Andrew Morton]

Denis Vlasenko <vda@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> wrote:
>
> +ifneq ($(CONFIG_CC_ALIGN_FUNCTIONS),0)
>  +CFLAGS		+= -falign-functions=$(CONFIG_CC_ALIGN_FUNCTIONS)
>  +endif
>  +ifneq ($(CONFIG_CC_ALIGN_LABELS),0)
>  +CFLAGS		+= -falign-labels=$(CONFIG_CC_ALIGN_LABELS)
>  +endif
>  +ifneq ($(CONFIG_CC_ALIGN_LOOPS),0)
>  +CFLAGS		+= -falign-loops=$(CONFIG_CC_ALIGN_LOOPS)
>  +endif
>  +ifneq ($(CONFIG_CC_ALIGN_JUMPS),0)
>  +CFLAGS		+= -falign-jumps=$(CONFIG_CC_ALIGN_JUMPS)
>  +endif

When sending optimisation patches, please include information on how
effective the optimisation is.  Before and after code sizes would be
appropriate here.

gcc-2.95.x barfs over all four of the above options.  Some testing of
$(GCC_VERSION) is needed.  That's only set up by arch/i386/Makefile at
present.

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