Regression in booting HelenOS/ppc under Qemu

Bug #942299 reported by Jakub Jermar on 2012-02-27
This bug affects 1 person
Affects Status Importance Assigned to Milestone
HelenOS branches

Bug Description

Mark Cave-Ayland identified Qemu commit 41557447d30eeb944e42069513df13585f5e6c7f as the first bad commit which prevents HelenOS/ppc from booting with OpenBIOS taken from Qemu 0.11.1. Note that we deliberately use the OpenBIOS from Qemu 0.11.1 because there is another suspected, unrelated bug in newer versions of OpenBIOS.

For reproducing purposes, the following HelenOS image can be used (latest HelenOS exhibits the same behavior):

Commit 41557447d30eeb944e42069513df13585f5e6c7f will result in HelenOS not making any further progress after it prints "init: Spawning /srv/devfs".

The previous commit, f844c817d726cd2bdb431aa41c8217891ede2eaf, works fine - HelenOS will spawn a couple of shell instances and will be accepting user input.

Jakub Jermar (jakub) wrote :

How to reproduce:

1. make sure openbios-ppc from Qemu 0.11.1 is installed
2. $ qemu-system-ppc -cdrom HelenOS-0.4.2-ppc32.iso -boot d

Changed in helenos:
status: Unknown → New
Artyom Tarasenko (atar4qemu) wrote :

Just for the reference,

commit 41557447d30eeb944e42069513df13585f5e6c7f
Author: Alexander Graf <email address hidden>
Date: Fri Sep 10 15:08:34 2010 +0000

    PPC: Redesign interrupt trigger path

    According to the Book3S spec, the interrupt context starts with an MSR
    value that is rather simple. If we leave out the HV case, it's almost
    always 0.

    To reflect this, let's redesign the way that MSR value gets calculated.
    Using this, we also squash the bug where MSR_POW can slip through into
    the interrupt handler MSR.

    Reported-by: Thomas Monjalon <email address hidden>
    Signed-off-by: Alexander Graf <email address hidden>
    Signed-off-by: Edgar E. Iglesias <email address hidden>

Just for an update: the TLB patch has been applied to ppc-next by Andreas and so should be in git master shortly:

I've also reworked the MSR flags patch based upon feedback from David/Scott and posted a revised version here, for which I'm still awaiting feedback (I suspect we need to wait for Alex to get back from holiday before we can get a final decision on this):

Changed in helenos:
status: New → Unknown
agraf (agraf) wrote :

Mark's TLB patch was enough to fix HelenOS :).

Changed in qemu:
status: New → Fix Committed
Stewart Smith (stewart-ibm) wrote :

Works in current qemu trunk, setting Fix Released.

Changed in qemu:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

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