b92e54333f
A branch instruction in a branch delay slot confuses the execution pipeline and causes an exception. bootblock.S was written 'by hand', has a branch instruction in branch delay slot and includes '.set noreorder' directive, which causes it to crash when trying to branch to main(). Adding a nop instruction fixes the problem. Also adding a nop after the last branch in the file just in case main() returns and the object linked next starts with a branch. BUG=chrome-os-partner:31438 TEST=Running on the simulator can reach main() now Change-Id: I0882b2eb5ce426f5a311018ffbb6f37a2ca64d98 Signed-off-by: Vadim Bendebury <vbendeb@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/221421 Reviewed-by: Aaron Durbin <adurbin@chromium.org> Reviewed-on: http://review.coreboot.org/9183 Reviewed-by: Ronald G. Minnich <rminnich@gmail.com> Tested-by: build bot (Jenkins)
48 lines
1.4 KiB
ArmAsm
48 lines
1.4 KiB
ArmAsm
/*
|
|
* This file is part of the coreboot project.
|
|
*
|
|
* Copyright (C) 2014 Imagination Technologies
|
|
*
|
|
* This program is free software; you can redistribute it and/or modify
|
|
* it under the terms of the GNU General Public License as published by
|
|
* the Free Software Foundation; version 2 of the License.
|
|
*
|
|
* This program is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
* GNU General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU General Public License
|
|
* along with this program; if not, write to the Free Software
|
|
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
|
|
*/
|
|
|
|
.set noreorder /* Prevent assembler from "optimizing" this code. */
|
|
|
|
.section ".start", "ax", %progbits
|
|
.globl _start
|
|
_start:
|
|
/* Set the stack pointer */
|
|
li $sp, CONFIG_BOOTBLOCK_STACK_TOP
|
|
|
|
/*
|
|
* Initialise the stack to a known value, used later to check for
|
|
* overflow.
|
|
*/
|
|
li $t0, CONFIG_BOOTBLOCK_STACK_BOTTOM
|
|
addi $t1, $sp, -4
|
|
li $t2, 0xdeadbeef
|
|
1: sw $t2, 0($t0)
|
|
bne $t0, $t1, 1b
|
|
addi $t0, $t0, 4
|
|
|
|
/* Run main */
|
|
b main
|
|
|
|
/*
|
|
* Should never return from main. Make sure there is no branch in the
|
|
* branch delay slot.
|
|
*/
|
|
2: nop
|
|
b 2b
|
|
nop /* Make sure there is no branch after this either. */
|