fastmodel kernel command line apparently missing

Bug #1167862 reported by Riku Voipio
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
LAVA Validation Lab
Fix Released
Critical
Tyler Baker

Bug Description

The semihosting command line should look something like:

 -C cluster.cpu0.semihosting-cmd_line="--kernel vmlinuz.... --dtb rtsm_ve-cortex_a15x1.dtb --initrd initrd--console=ttyAMA0,115200n8 root=/dev/mmcblk0p2 rootwait ro mem=1024M "

(for fast models with big.LITTLE use coretile.cluster0.cpu0.semihosting-cmd_line instead)

Looking at lava submissions for fast models. there is no semihosting-cmd_line included.

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

-snip-
<LAVA_DISPATCHER>2013-04-11 09:55:34 AM INFO: launching fastmodel with command u'sudo -u www-data ARMLMD_LICENSE_FILE="8224@control" /opt/arm/RTSM_A15-A7x14_VE/Linux64_RTSM_VE_Cortex-A15x1-A7x1/RTSM_VE_Cortex-A15x1-A7x1 -a coretile.cluster0.*=/srv/lava/instances/production/var/www/lava-server/images/tmpg4hsOD/img.axf -C motherboard.mmc.p_mmc_file=/srv/lava/instances/production/var/www/lava-server/images/tmpg4hsOD/sd.img -C motherboard.hostbridge.userNetPorts=\'5555=5555\' -C coretile.cluster0.cpu0.semihosting-enable=1 -C motherboard.hostbridge.userNetworking=1 -C coretile.cache_state_modelled=0 -C motherboard.smsc_91c111.enabled=1'

<LAVA_DISPATCHER>2013-04-11 09:55:35 AM INFO: simulator is started connecting to serial port
[bootwrapper] ERROR: Expected -Trying 127.0.0.1...
Connected to localhost.
Escape character is '^]'.
-kernel
[bootwrapper] Usage: [--kernel <kernel filename>] [--no-initrd|--initrd <initrd filename>] [--dtb <dtb filename>] [-- <kernel boot arguments>]
[bootwrapper] ERROR: Incorrect bootwrapper command-line.
[bootwrapper] BOOT FAILED
Simulation is terminating. Reason: Simulation stopped
Connection closed by foreign host.
-snip-

Related branches

Fathi Boudra (fboudra)
Changed in lava-lab:
milestone: none → 2013.04
Revision history for this message
Alexander Sack (asac) wrote :

isn't there a way to tweak the model command line options through lava job parameters?

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

we can pass options:

 {
        "command": "boot_linaro_image",
        "parameters": {
                "options": [
                        "coretile.cluster0.cpu0.semihosting-cmd_line=\"--kernel uImage --dtb $RTSM_DTB --initrd $RTSM_INITRD -- $RTSM_CMDLINE\""
                ]
 }

See http://bazaar.launchpad.net/~linaro-validation/lava-dispatcher/trunk/view/head:/lava_dispatcher/default-config/lava-dispatcher/device-types/fastmodel_A15x4-A7x4.conf for reference.

Revision history for this message
Senthil Kumaran S (stylesen) wrote :
Changed in lava-lab:
importance: Undecided → Critical
Changed in lava-lab:
assignee: nobody → Antonio Terceiro (terceiro)
Revision history for this message
Riku Voipio (riku-voipio) wrote :

I did some more digging. Passing the options as suggested by Fathi doesn't work:

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

We see that lava-dispatcher does not copy from inside the image the needed uImage, uInitrd and dxb file - just the axf file.

Revision history for this message
Antonio Terceiro (terceiro) wrote :

This is weird. I can reproduce this at validation.linaro.org, but not locally yet with an old image I had on my disk.

I am downloading the same that was used in the job report mentioned Riku's original report to test locally

Revision history for this message
Antonio Terceiro (terceiro) wrote :

OK - so i reproduced the problem with the linux-vexpress-kvm image. Maybe the problem is that something in the new images changed and lava was not updated to deal with that.

Fathi Boudra (fboudra)
Changed in lava-lab:
status: New → Confirmed
Fathi Boudra (fboudra)
Changed in lava-lab:
milestone: 2013.04 → 2013.05
Revision history for this message
Antonio Terceiro (terceiro) wrote :

hey guys, could you please give me some feedback on the branch that tries to solve this?

https://code.launchpad.net/~terceiro/lava-dispatcher/fastmodels-semihosting-fix/+merge/161258

it's not complete, but I would like some feedback before going on with this.

Alan Bennett (akbennett)
Changed in lava-lab:
assignee: Antonio Terceiro (terceiro) → Tyler Baker (tyler-baker)
Changed in lava-lab:
status: Confirmed → 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.