> On Oct 13, 2015, at 04:04, Sam Ravnborg <s...@ravnborg.org> 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










Reply via email to