All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/1] routerstationpro: move board off 3.4 and onto the 3.8 kernel
@ 2013-04-22 21:20 michel.thebeau
  2013-04-22 21:20 ` [PATCH] " michel.thebeau
  2013-04-22 21:20 ` [PATCH] kernel 3.8: routerstationpro: use KERNEL_IMAGE_MAXSIZE to test the image michel.thebeau
  0 siblings, 2 replies; 7+ messages in thread
From: michel.thebeau @ 2013-04-22 21:20 UTC (permalink / raw)
  To: poky, richard.purdie; +Cc: dvhart

Hi,

linux-yocto-3.8 kernel is updated for Routerstation Pro, so that the bsp
can moved to that kernel.

https://lists.yoctoproject.org/pipermail/linux-yocto/2013-April/000546.html

"Feel free to update the preferred version of the board in
meta-yocto-bsp." -Bruce

Hope I have the necessary changes here.  I'm cc'ing the people who were
addressed in the previous uprev for rsp.  Apologies if that's too much.

---

I also noticed that the fix for 3514 needs to carry forward.  Sending
along a patch for that as well.  It probably won't happen again since
people tend to use the previous file version as template.

M


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH] routerstationpro: move board off 3.4 and onto the 3.8 kernel
  2013-04-22 21:20 [PATCH 0/1] routerstationpro: move board off 3.4 and onto the 3.8 kernel michel.thebeau
@ 2013-04-22 21:20 ` michel.thebeau
  2013-04-22 21:20 ` [PATCH] kernel 3.8: routerstationpro: use KERNEL_IMAGE_MAXSIZE to test the image michel.thebeau
  1 sibling, 0 replies; 7+ messages in thread
From: michel.thebeau @ 2013-04-22 21:20 UTC (permalink / raw)
  To: poky, richard.purdie; +Cc: dvhart

From: Michel Thebeau <michel.thebeau@windriver.com>

The updated patch series to support this target is in place on the BSP
specific branch in the 3.8 kernel tree now[1], so we can move it ahead
off of the old 3.4 kernel.

[1] https://lists.yoctoproject.org/pipermail/linux-yocto/2013-April/000542.html

Signed-off-by: Michel Thebeau <michel.thebeau@windriver.com>
---
 meta-yocto-bsp/conf/machine/routerstationpro.conf |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-yocto-bsp/conf/machine/routerstationpro.conf b/meta-yocto-bsp/conf/machine/routerstationpro.conf
index 723625b..d9575eb 100644
--- a/meta-yocto-bsp/conf/machine/routerstationpro.conf
+++ b/meta-yocto-bsp/conf/machine/routerstationpro.conf
@@ -11,7 +11,7 @@ KERNEL_ALT_IMAGETYPE = "vmlinux.bin"
 KERNEL_IMAGE_STRIP_EXTRA_SECTIONS  = ".comment"
 
 PREFERRED_PROVIDER_virtual/kernel ?= "linux-yocto"
-PREFERRED_VERSION_linux-yocto ?= "3.4%"
+PREFERRED_VERSION_linux-yocto ?= "3.8%"
 
 PREFERRED_PROVIDER_virtual/xserver ?= "xserver-xorg"
 XSERVER ?= "xserver-xorg \
-- 
1.7.9.7



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH] kernel 3.8: routerstationpro: use KERNEL_IMAGE_MAXSIZE to test the image
  2013-04-22 21:20 [PATCH 0/1] routerstationpro: move board off 3.4 and onto the 3.8 kernel michel.thebeau
  2013-04-22 21:20 ` [PATCH] " michel.thebeau
@ 2013-04-22 21:20 ` michel.thebeau
  2013-04-23  8:16   ` Andrea Adami
  1 sibling, 1 reply; 7+ messages in thread
From: michel.thebeau @ 2013-04-22 21:20 UTC (permalink / raw)
  To: poky, richard.purdie; +Cc: dvhart

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"
-- 
1.7.9.7



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] kernel 3.8: routerstationpro: use KERNEL_IMAGE_MAXSIZE to test the image
  2013-04-22 21:20 ` [PATCH] kernel 3.8: routerstationpro: use KERNEL_IMAGE_MAXSIZE to test the image michel.thebeau
@ 2013-04-23  8:16   ` Andrea Adami
  2013-04-23 12:25     ` Michel Thebeau
  0 siblings, 1 reply; 7+ messages in thread
From: Andrea Adami @ 2013-04-23  8:16 UTC (permalink / raw)
  To: Poky Project

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.

Regards

Andrea


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


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] kernel 3.8: routerstationpro: use KERNEL_IMAGE_MAXSIZE to test the image
  2013-04-23  8:16   ` Andrea Adami
@ 2013-04-23 12:25     ` Michel Thebeau
  2013-04-25 21:21       ` Andrea Adami
  0 siblings, 1 reply; 7+ messages in thread
From: Michel Thebeau @ 2013-04-23 12:25 UTC (permalink / raw)
  To: Andrea Adami; +Cc: Poky Project



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

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


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] kernel 3.8: routerstationpro: use KERNEL_IMAGE_MAXSIZE to test the image
  2013-04-23 12:25     ` Michel Thebeau
@ 2013-04-25 21:21       ` Andrea Adami
  2013-04-25 22:04         ` Michel Thebeau
  0 siblings, 1 reply; 7+ messages in thread
From: Andrea Adami @ 2013-04-25 21:21 UTC (permalink / raw)
  To: Poky Project

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

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


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] kernel 3.8: routerstationpro: use KERNEL_IMAGE_MAXSIZE to test the image
  2013-04-25 21:21       ` Andrea Adami
@ 2013-04-25 22:04         ` Michel Thebeau
  0 siblings, 0 replies; 7+ messages in thread
From: Michel Thebeau @ 2013-04-25 22:04 UTC (permalink / raw)
  To: Andrea Adami; +Cc: Poky Project



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
> 


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2013-04-25 22:04 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-04-22 21:20 [PATCH 0/1] routerstationpro: move board off 3.4 and onto the 3.8 kernel michel.thebeau
2013-04-22 21:20 ` [PATCH] " michel.thebeau
2013-04-22 21:20 ` [PATCH] kernel 3.8: routerstationpro: use KERNEL_IMAGE_MAXSIZE to test the image michel.thebeau
2013-04-23  8:16   ` Andrea Adami
2013-04-23 12:25     ` Michel Thebeau
2013-04-25 21:21       ` Andrea Adami
2013-04-25 22:04         ` Michel Thebeau

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.