add boot command support to Snowball

Bug #1098097 reported by Fathi Boudra
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
LAVA Dispatcher
Fix Released
High
Andy Doan

Bug Description

Please, add boot command support to Snowball as added to OE build on panda.
As discussed, a new boot_cmds_with_dtb or boot_cmds_fdt entry.

Related branches

Fathi Boudra (fboudra)
summary: - add boot command support to supported boards
+ add boot command support to Snowball
Changed in lava-dispatcher:
status: New → Triaged
importance: Undecided → High
assignee: nobody → Andy Doan (doanac)
milestone: none → 2013.01
Revision history for this message
Andy Doan (doanac) wrote :

I have an MP in place to help silence some dumb warnings this config change will cause.

Fathi - the main thing I need to know is what you want the actual boot commands for the FDT scenario to be?

Changed in lava-dispatcher:
status: Triaged → In Progress
Fathi Boudra (fboudra)
Changed in lava-dispatcher:
status: In Progress → Incomplete
Revision history for this message
Fathi Boudra (fboudra) wrote :

Latest snowball images have uinitrd, the boot commands should look like below:
boot_cmds = mmc init,
mmc rescan 1,
setenv bootcmd "'fat load mmc 1:3 0x00100000 /uImage; fat load mmc 1:3 0x01000000 /uInitrd; bootm 0x00100000 0x01000000'"
setenv bootargs "'console=ttyAMA2,115200n8 root=/dev/mmcblk1p5 rootwait ro fixrtc nocompcache vmalloc=300M mem=128M@0 mali.mali_mem=32M@128M hwmem=168M@160M mem=48M@328M mem_issw=1M@383M mem=640M@384M'",
boot

Andrey is looking into enabling DT, I let him follow up with the right boot commands.

Revision history for this message
Fathi Boudra (fboudra) wrote :

boot commands with uInitrd and dtb:
boot_cmds = mmc init,
mmc rescan 1,
setenv bootcmd "'fat load mmc 1:3 0x00100000 /uImage; fat load mmc 1:3 0x01000000 /uInitrd; fat load mmc 1:3 0x00f00000 /board.dtb; bootm 0x00100000 0x01000000 0x00f00000'"
setenv bootargs "'console=ttyAMA2,115200n8 root=/dev/mmcblk1p5 rootwait ro fixrtc nocompcache vmalloc=300M mem=128M@0 mali.mali_mem=32M@128M hwmem=168M@160M mem=48M@328M mem_issw=1M@383M mem=640M@384M'",
boot

Changed in lava-dispatcher:
status: Incomplete → In Progress
Revision history for this message
Andy Doan (doanac) wrote :

I've checked this into our salt repo first. So we can test it some before adding to lava-dispatcher permanently.

Should be able to use this with a boot action like:

{
          "command": "boot_linaro_image",
   "parameters": {
    "options": ["boot_cmds=boot_cmds_fdt"]
   }
    },

Revision history for this message
Fathi Boudra (fboudra) wrote :

http://validation.linaro.org/lava-server/scheduler/job/46183/log_file#entry16

<LAVA_DISPATCHER>2013-01-26 09:24:48 PM INFO: [ACTION-B] boot_linaro_image is started with {u'options': [u'boot_cmds=boot_cmds_fdt']}
<LAVA_DISPATCHER>2013-01-26 09:24:48 PM INFO: Boot the test image
<LAVA_DISPATCHER>2013-01-26 09:24:48 PM WARNING: Invalid boot option: boot_cmds
<LAVA_DISPATCHER>2013-01-26 09:24:48 PM INFO: Perform soft reboot the system

Revision history for this message
Andy Doan (doanac) wrote :

good news - I'm probably just dumb and created the wrong device-config. I'm testing something in staging now, and can probably roll out a fix very soon.

Revision history for this message
Andy Doan (doanac) wrote :

a "fix" has been deployed. I've re-submitted the job here:

  http://validation.linaro.org/lava-server/scheduler/job/46374

Revision history for this message
Andy Doan (doanac) wrote :

ugh - didn't work. let me actually do some debugging

Revision history for this message
Andy Doan (doanac) wrote :

this reuired a code change so we didn't break android:

https://code.launchpad.net/~doanac/lava-dispatcher/boot-cmds/+merge/145282

Andy Doan (doanac)
Changed in lava-dispatcher:
status: In Progress → Fix Committed
Revision history for this message
Andrey Konovalov (andrey-konovalov) wrote :

I've tried this job which sets "options": ["boot_cmds=boot_cmds_fdt"] :
http://validation.linaro.org/lava-server/scheduler/job/46482/definition

According to the log, LAVA doesn't seem to handle this boot_cmds setting:
http://validation.linaro.org/lava-server/scheduler/job/46482/log_file#entry14
"<LAVA_DISPATCHER>2013-01-29 04:19:10 PM WARNING: Invalid boot option: boot_cmds"

Also the U-Boot 2009.11 is not good for booting with DT. Only U-Boot 2013.01 (it seems to be the latest one) is able to pass the dtb to the kernel.

Revision history for this message
Andy Doan (doanac) wrote : Re: [Bug 1098097] Re: add boot command support to Snowball

On 01/29, Andrey Konovalov wrote:
> I've tried this job which sets "options": ["boot_cmds=boot_cmds_fdt"] :
> http://validation.linaro.org/lava-server/scheduler/job/46482/definition
>
> According to the log, LAVA doesn't seem to handle this boot_cmds setting:
> http://validation.linaro.org/lava-server/scheduler/job/46482/log_file#entry14
> "<LAVA_DISPATCHER>2013-01-29 04:19:10 PM WARNING: Invalid boot option: boot_cmds"

The fix has been "committed", not "released". I'm hoping to deploy in a
few hours. When its done, I'll personally verify this is working before
asking you guys to try this again.

> Also the U-Boot 2009.11 is not good for booting with DT. Only U-Boot
> 2013.01 (it seems to be the latest one) is able to pass the dtb to the
> kernel.

This is unfortunate. When I first asked, I was told this u-boot was
sufficient. Changing u-boot for snowballs will be a non-trivial amount
of work. We'll need to identify a release with this, create a master
image in the lab, and run a large number of tests against this to ensure
it will serve as a stable master image.

Fathi - probably should create a BP for this if its the case.

Revision history for this message
Fathi Boudra (fboudra) wrote :

The u-boot issue isn't new, Andrey raised it each time we discussed the topic...
In addition, it has been raised mid-december with http://bazaar.launchpad.net/~fboudra/lava-dispatcher/load-uinitrd-ubuntu/revision/506

the whole master image story and hardcoded boot commands is a mess :(

Changed in lava-dispatcher:
status: Fix Committed → Fix Released
Changed in lava-dispatcher:
status: Fix Released → Fix Committed
Andy Doan (doanac)
Changed in lava-dispatcher:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.