Bug 1391 - qemuppc: linux-yocto-rt_3.0 boot hangs with "hda: lost interrupt" messages
Summary: qemuppc: linux-yocto-rt_3.0 boot hangs with "hda: lost interrupt" messages
Status: VERIFIED FIXED
Alias: None
Product: BSPs
Classification: Build System, Metadata & Runtime
Component: bsps-runtime (show other bugs)
Version: unspecified
Hardware: x86 ppc
: Medium normal
Target Milestone: 1.1
Assignee: Bruce Ashfield
QA Contact:
URL:
Whiteboard: Patch submitted to list (24-Aug-2011)
Depends on: 1516
Blocks:
  Show dependency tree
 
Reported: 2011-08-22 10:18 UTC by Darren Hart
Modified: 2011-09-22 14:25 UTC (History)
3 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Darren Hart 2011-08-22 10:18:06 UTC
Summary says it all really. Trying to boot qemuppc with the linux-yocto-rt_3.0 kernel never reaches init. It runs to serial init and hangs for a while, then continues and spews "hda: lost interrupt" messages for a while, then goes silent. It never reaches init.
Comment 1 Bruce Ashfield 2011-08-22 11:07:37 UTC
We'll work to have one ppc target as a reference for -rt, but it may have to
be the fsl_mpc hardware target. Our interrupt changes for qemuppc are
clearly at play here as they are bolted onto the generic IRQ changes and 
look to be missing the right handlers for -rt support.
Comment 2 Liming Wang 2011-08-23 23:45:36 UTC
Hi, bruce
I ever provided a patch to fix interrupt bug of qemuppc, I saw that you haven't merged it into qemuppc branch. I use that patch to test, qemuppc can boot successfully:
#uname -a
Linux qemuppc 3.0.1-rt8-yocto-preempt-rt+ #2 PREEMPT RT Wed Aug 24 14:08:13 CST 2011 ppc GNU/Linux

The mail is here:
> for qemuppc issue, please use attached patch.
> The root cause is that irq_state_clr_disabled is to enable irq, not
> disable irq. So
> We should modify function "irq_state_set_disabled".
>
> With this patch, qemuppc has booted up successfully.

Ah yes! Patch fuzz'ing at its best. I haven't had a chance to
debug this *at all*, so I'm glad you got to it before I did! :)

I'll add your Sign-off to the patches.

Cheers,

Bruce
Comment 3 Liming Wang 2011-08-24 00:39:00 UTC
Hi bruce,

Another issue, when I build linux-yocto-rt for qemuppc,  then

$git log origin/yocto/standard/base
I can't find the following commit:

commit 4aa5ca50274dc527d473cdf512196018811054cf
Author: Jason Wessel <jason.wessel@windriver.com>
Date:   Wed Jan 7 00:59:33 2009 -0500

    NFS: allow nfs root mount to use alternate rpc ports
    
    Allow an nfs root mount to use alternate RPC ports for mountd and nfsd.
    
    Signed-off-by: Jason Wessel <jason.wessel@windriver.com>
    [forward port to 2.6.33+]
    Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>

If I build a linux-yocto, I can find that commit. What's the difference for that two git trees? Why some commits don't exist in the linux-yocto-rt tree? Without that commit, unfs can't boot successfully on qemu.
Comment 4 Bruce Ashfield 2011-08-24 05:56:37 UTC
Liming: that interrupt patch is definitely merged. I now know what
the problem is. We are incorrectly mapping and building the wrong 
branch for qemuppc-rt, I'll make an update an all those changes should
be visible.
Comment 5 Bruce Ashfield 2011-08-24 11:32:06 UTC
I've fixed the tree to have all the yocto/standard/base patches restored (it wasn't the problem here, but they were missing temporarily due to the 3.0.3 merge) and I've created a yocto/standard/preempt-rt/qemu-ppc32 branch that has all the relevant qemu changes for preempt-rt. 

This will go out for merge shortly.
Comment 6 Darren Hart 2011-08-24 13:49:19 UTC
Bruce, have you tested qemuppc boot for linux-yocto-rt_3.0?
Comment 7 Bruce Ashfield 2011-08-24 13:58:39 UTC
see Liming's update .. he did. I confirmed that this builds and has all the right changes.
Comment 9 Liming Wang 2011-09-12 22:59:29 UTC
Hi Darren,

This bug has been fixed, please help us to verify this bug and mark it "verified fixed", thanks a lot!
Comment 10 Darren Hart 2011-09-22 14:25:45 UTC
Managed to test with a pending fix for Bug 1516.