Re: [PATCH] kernel 3.8: routerstationpro: use KERNEL_IMAGE_MAXSIZE to test the image

[Michel Thebeau <michel.thebeau@windriver.com>]

On 13-04-25 05:21 PM, Andrea Adami wrote:
> On Tue, Apr 23, 2013 at 2:25 PM, Michel Thebeau
> <michel.thebeau@windriver.com> wrote:
>>
>>
>> On 13-04-23 04:16 AM, Andrea Adami wrote:
>>> On Mon, Apr 22, 2013 at 11:20 PM,  <michel.thebeau@windriver.com> wrote:
>>>> From: Michel Thebeau <michel.thebeau@windriver.com>
>>>>
>>>> routerstationpro is limited to 16mb flash size.  Exceeding that size may
>>>> not be noticed immediately, which we don't want.
>>>>
>>>> "The current latest kernel for the routerstationpro machine is far
>>>> beyond the 16MB of it's flash size [0]. I think it would be good to
>>>> [test if the size of the image is above that size]."
>>>>
>>>> Setting this variable will enable the do_sizecheck task and produce an
>>>> error when the image is too large:
>>>>
>>>> "ERROR: This kernel (size=90230265 > 16777216) is too big for your
>>>> device. Please reduce the size of the kernel by making more of it
>>>> modular."
>>>>
>>>> Taken from commit 83716e40edacfd20dec8ef41f415ad5828d4643b.
>>>>
>>>> [YOCTO #3514]
>>>>
>>>> Signed-off-by: Michel Thebeau <michel.thebeau@windriver.com>
>>>> ---
>>>>  .../recipes-kernel/linux/linux-yocto_3.8.bbappend  |    3 +++
>>>>  1 file changed, 3 insertions(+)
>>>>
>>>> diff --git a/meta-yocto-bsp/recipes-kernel/linux/linux-yocto_3.8.bbappend b/meta-yocto-bsp/recipes-kernel/linux/linux-yocto_3.8.bbappend
>>>> index 479a386..769dcc0 100644
>>>> --- a/meta-yocto-bsp/recipes-kernel/linux/linux-yocto_3.8.bbappend
>>>> +++ b/meta-yocto-bsp/recipes-kernel/linux/linux-yocto_3.8.bbappend
>>>> @@ -12,3 +12,6 @@ COMPATIBLE_MACHINE_mpc8315e-rdb = "mpc8315e-rdb"
>>>>  COMPATIBLE_MACHINE_routerstationpro = "routerstationpro"
>>>>  COMPATIBLE_MACHINE_beagleboard = "beagleboard"
>>>>  COMPATIBLE_MACHINE_atom-pc = "atom-pc"
>>>> +
>>>> +# routerstationpro has a flash size of 16mb
>>>> +KERNEL_IMAGE_MAXSIZE_append_routerstationpro = "16777216"
>>>
>>> Why would it need an _append_ ?
>>> KERNEL_IMAGE_MAXSIZE_routerstationpro = "16777216" should be enough.
>>>
>>> And by the way such kernel sizes look really exaggerated.
>>
>> Thanks Andrea,
>>
>> I'll drop _append_.
>>
>> I'm not sure what you mean mean by exaggerated.  I multiplied 16mb with
>> 2^20 == 16777216.
>>
>> M
> 
> My comment was about the kernel sizes listed in the error msg:
> 
>  "ERROR: This kernel (size=90230265 > 16777216) is too big for your..."
> 
> Not knowing about the specific machine I can only guess this kernel is
> embedding an initramfs for some specific purpose...but in my
> experience I've never ever seen such huge sizes for a kernel.
> 
> Perplexed...
> 

Hi Andrea,

poky commits cd2f003b512adf545dd3e5edd9aad68e09e9cda7 and
95b71d2b03a3a6dc5d32a7d6a84ec3420f39aa61 strip these sections from the
vmlinux elf image:
    .debug_aranges
    .debug_info
    .debug_abbrev
    .debug_line
    .debug_frame
    .debug_str
    .debug_loc
    .debug_ranges
    .symtab
    .strtab

The largest section remaining is .text
${CROSS_COMPILE}readelf -S vmlinux
...
  [ 1] .text             PROGBITS...	5823f0...
...

So the 90mb you are seeing is mostly .debug sections, which we removed
for bug 3515 - "The size of the kernel image is too big [to fit in flash]"

Thanks for getting back to me.  I'll resend the patches tomorrow.

M

> Andrea
> 
> 
>>
>>>
>>> Regards
>>>
>>> Andrea
>>>
>>>
>>>> --
>>>> 1.7.9.7
>>>>
>>>> _______________________________________________
>>>> poky mailing list
>>>> poky@yoctoproject.org
>>>> https://lists.yoctoproject.org/listinfo/poky
>>> _______________________________________________
>>> poky mailing list
>>> poky@yoctoproject.org
>>> https://lists.yoctoproject.org/listinfo/poky
>>>
> _______________________________________________
> poky mailing list
> poky@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/poky
>