Bug 4024

Summary: start qemuarm with "core-image-sato", use "dmesg | grep -i error" shows 2 errors
Product: [Runtime] General Runtime Reporter: Ning Zhang <ning.zhang>
Component: General RuntimeAssignee: Lucian Musat <georgex.l.musat>
Status: RESOLVED WONTFIX QA Contact:
Severity: normal    
Priority: Low CC: alexandru.c.georgescu, bruce.ashfield, corneliux.stoicescu, dvhart, jessica.zhang, ke.zou, richard.purdie, yi.zhao
Version: 1.4   
Target Milestone: 1.8 M2   
Hardware: x86   
OS: arm   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: No (bug/feature does not impact docs)

Description Ning Zhang 2013-03-15 07:55:12 UTC
Image location: http://autobuilder.yoctoproject.org/pub/nightly/20130313-3

Branch: poky/master
Git rev: 4da406cd50cf02a63ed76810a670aba1eb561c79

test step:
1. bootup qemuarm with "core-image-sato-qemuarm-20130313194304.rootfs.ext3"
2. open terminal from qemuarm
3. execute "dmesg | grep -i error", there should no error list. But it contains two errors like below:
mmci-pl18x: probe of fpga:05 failed with error -38
mmci-pl18x: probe of fpga:0b failed with error -38
4. when use "core-image-sato-sdk-qemuarm-20130313194304.rootfs.ext3", same errors:
[    4.430820] mmci-pl18x: probe of fpga:05 failed with error -38
[    4.443642] mmci-pl18x: probe of fpga:0b failed with error -38

comments:
do the same test on qemuppc, no this issue.
Comment 1 Darren Hart 2013-03-18 21:16:24 UTC
This was reported by the AB and fixed with:

commit d5267afa634a4da33f5d5a88c678387a9b7ec1b1
Author: Richard Purdie <richard.purdie@linuxfoundation.org>
Date:   Sat Mar 2 22:31:19 2013 +0000
Comment 2 Ning Zhang 2013-03-25 08:19:00 UTC
(In reply to comment #1)
> This was reported by the AB and fixed with:
> 
> commit d5267afa634a4da33f5d5a88c678387a9b7ec1b1
> Author: Richard Purdie <richard.purdie@linuxfoundation.org>
> Date:   Sat Mar 2 22:31:19 2013 +0000

Hi,
I saw this fix only modify the test code, to skip mmci-pl18x dmesg. Does this mean we do not care about the mmci-pl18x error info anymore?

Since only arm qemu has this issue, ppc no this error. This modification will skip all qemus mmci-pl18x test. Does it mean we do not need test it on other qemus, such as ppc?
Thanks.
Comment 3 Darren Hart 2013-03-25 15:49:35 UTC
No, this is really a workaround until a proper fix can be made.
Comment 4 Ning Zhang 2013-03-26 01:43:03 UTC
(In reply to comment #3)
> No, this is really a workaround until a proper fix can be made.

Got it. 
Then should we reopen this bug to continue tracking this issue until it be really fixed?
Thanks.
Comment 5 Stoicescu Cornel 2013-10-15 10:30:27 UTC
Issue found in poky master:cb2836fee444bf5f7bd74d0612a4cd8f1b5d2e89. Reopening.
Comment 6 Saul Wold 2013-10-15 15:37:28 UTC
Just want to confirm this is the same issue or different?  Did the same exception (grepping for the pl18x) get moved from the qemuimage-tests to the new test framework?

Is this something we can really fix in the kernel?
 (Bruce & Darren Cc'ed)
Comment 7 Darren Hart 2013-10-15 16:01:55 UTC
If I remember correctly, we didn't feel it was appropriate to fix in the kernel as it was really an issue with qemu's implementation of the given hardware. Bruce, was that about right?

I believe the proper fix would be to qemu. However, I don't think it will be considered high enough priority to time for and we should probably just ensure the new tests are still ignoring it.
Comment 8 Stoicescu Cornel 2013-10-16 08:05:45 UTC
This was discovered in a manual test of the qemu image. The AB tests still ignore it; see the dmesg grep below:

'dmesg | grep -v mmci-pl18x | grep -v "error changing net interface name" | grep -i error'

Even though it's ignored in automated tests, the issue is still present.
Comment 9 Bruce Ashfield 2013-10-17 18:58:09 UTC
(In reply to comment #7)
> If I remember correctly, we didn't feel it was appropriate to fix in the
> kernel as it was really an issue with qemu's implementation of the given
> hardware. Bruce, was that about right?
> 
> I believe the proper fix would be to qemu. However, I don't think it will be
> considered high enough priority to time for and we should probably just
> ensure the new tests are still ignoring it.

Correct. As with the interrupt mappings, we want qemu to represent the h/w. So to
silence this, we want a qemu fix, instead of hacking around it in the kernel. 

This is still a low priority, but I suppose we can keep it around :)
Comment 10 Stephen K Jolley 2014-07-31 15:09:32 UTC
Can you re-test this problem.
Comment 11 Ke Zou 2014-08-01 06:17:02 UTC
This issue still remains in 1.7M2 rc1 build
http://autobuilder.yoctoproject.org/pub/releases/yocto-1.7_M2.rc1/

sh-4.3# dmesg | grep -i error

[    6.035639] mmci-pl18x: probe of fpga:05 failed with error -22
[    6.039104] mmci-pl18x: probe of fpga:0b failed with error -22
Comment 12 Richard Purdie 2014-12-18 15:50:28 UTC
At this point we're not going to enhance qemu to address this, we're also not going to hack the kernel to disable it, so marking as WONTFIX. The automated version of the QA test already ignore this 'error' message and the manual QA versions of the test should be dropped.