highbank: smc change in kernel source breaks qemu booting

Reported by dann frazier on 2012-09-09
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Linaro QEMU
Undecided
Unassigned
qemu-linaro (Ubuntu)
Undecided
Unassigned

Bug Description

qemu-linaro is no longer able to boot Ubuntu's highbank kernels. I bisected this down to the following changeset in git://kernel.ubuntu.com/ubuntu/ubuntu-precise.git :

commit 4627bd9d888bb1986597525841a1ef304dc2cb70
Author: Rob Herring <email address hidden>
Date: Thu Apr 26 11:40:26 2012 -0500

    UBUNTU: SAUCE: ARM: highbank: Add smc calls to enable/disable the L2

    BugLink: http://bugs.launchpad.net/bugs/1000831

    Linux runs in non-secure mode on highbank, so we need secure monitor calls
    to enable and disable the PL310.

    Signed-off-by: Rob Herring <email address hidden>
    Signed-off-by: Ike Panhc <email address hidden>
    Acked-by: Bryan Wu <email address hidden>
    Signed-off-by: Tim Gardner <email address hidden>

This changeset is in both the current precise and quantal kernels, so neither boot in qemu-linaro at this time. I'm filing this as a qemu-linaro bug instead of a kernel bug as these kernels boot fine on real hardware.

This is reproducible with both qemu-linaro in precise/quantal (w/ the additional highbank fixes in lp:~dannf/qemu-linaro/highbank), as well as latest qemu-linaro from git (top of tree currently at 54ec369).

Rob Herring (r-herring) wrote :

The attached patch will fix the problem, but this is probably not an acceptable fix for upstream. It assumes that implementing the smc call as a nop would work for all cases. On the the other hand, it does allow the common case of enabling the L2 cache on highbank and omap to work.

dann frazier (dannf) wrote :

Thanks Rob. I've ported this patch to the qemu-linaro tree and verified that I'm again able to boot Ubuntu kernels with it. I've also applied this to ppa:dannf/qemu-highbank, where I'm collecting all the necessary fixes to boot highbank kernels in precise.

The attachment "0001-arm-make-smc-calls-behave-as-nops.patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags: added: patch
Peter Maydell (pmaydell) wrote :

http://lists.gnu.org/archive/html/qemu-devel/2012-05/msg03012.html is my current proposed long-term upstream-acceptable approach to TrustZone handling.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers