> On Oct 13, 2015, at 04:04, Sam Ravnborg <sam@xxxxxxxxxxxx> wrote: > >> --- a/include/asm-generic/vmlinux.lds.h >> +++ b/include/asm-generic/vmlinux.lds.h >> >> -#define BSS_SECTION(sbss_align, bss_align, stop_align) \ >> +#define BSS_SECTION(sbss_align, bss_align, initbss_align, stop_align) \ > > A few comments: > > 1) - please align the backslash at the end of the > line with the backslash above it. > 2) - you need to fix all the remaining users of BSS_SECTION. > 3) - do we really need the flexibility to specify an alignment (stop_align)? > If not - drop the extra argument. > > Sam i change lots of __initdata to __initbss to test it on ARM arch,
Attachment:
initbss_change.diff
Description: Binary data
build result : yalin@ubuntu:~/linux-next$ ll ../kernel_out_arm/arch/arm/boot/*Image* -rwxrwxr-x 1 yalin yalin 14487552 Oct 13 15:08 ../kernel_out_arm/arch/arm/boot/Image* # apply the patch -rwxrwxr-x 1 yalin yalin 14512128 Oct 12 11:48 ../kernel_out_arm/arch/arm/boot/Image_old* -rwxrwxr-x 1 yalin yalin 6479568 Oct 13 15:08 ../kernel_out_arm/arch/arm/boot/zImage* # apply the patch -rwxrwxr-x 1 yalin yalin 6479664 Oct 12 17:31 ../kernel_out_arm/arch/arm/boot/zImage_old* Image size shrink about 24576 bytes zImage seems not change much . it will be more useful for platform which don’t use compress Image like ARM64 / x86 platform i think. Thanks