Bug 73642 - [HSW Bisected]Many pm_pc8 subcases fail with headless
Summary: [HSW Bisected]Many pm_pc8 subcases fail with headless
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: unspecified
Hardware: All Linux (All)
: high major
Assignee: Paulo Zanoni
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-01-15 03:28 UTC by lu hua
Modified: 2017-10-06 14:40 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
dmesg (94.12 KB, text/plain)
2014-01-15 03:28 UTC, lu hua
no flags Details

Description lu hua 2014-01-15 03:28:10 UTC
Created attachment 92108 [details]
dmesg

System Environment:
--------------------------
Platform: Haswell(nv)
Kernel: (drm-intel-nightly)07db3e0b8568d81226e2e17dbb1b340668cb2287

Bug detailed description:
-------------------------
Many pm_pc8 subcases fail on haswell with NVIDIA card GT440.
They fail on -nightly and -queued kernel, work well on -fixes kernel. 

The latest known good commit:d299cce76e001406df40c4ff712b33fccbce1222
The latest known bad commit: 534e3730732ee01e43cefa02240b0552a62781e4

run subcase i2c
output:
Runtime PM support: 1
PC8 residency support: 0
Test assertion failure function enable_one_screen, file pm_pc8.c:354:
Last errno: 5, Input/output error
Failed assertion: enable_one_screen_with_type(data, SCREEN_TYPE_ANY)
Subtest i2c: FAIL

Reproduce steps:
----------------
1. ./pm_pc8 --run-subtest i2c
Comment 1 Paulo Zanoni 2014-01-27 15:27:24 UTC
Do you have any outputs attached to the Intel card? It doesn't make sense to not have any, but I could try to SKIP the whole suite if that's the case.
Comment 2 Paulo Zanoni 2014-01-28 13:37:45 UTC
Also, could you please finish bisecting?

Thanks,
Paulo
Comment 3 lu hua 2014-01-29 08:43:25 UTC
Test on latest -queued kernel.
The fail goes away.
output:
IGT-Version: 1.5-gd1b5823 (x86_64) (Linux: 3.13.0-rc8_drm-intel-next-queued_369a13_20140129_debug+ x86_64)
Runtime PM support: 1
PC8 residency support: 0
Subtest i2c: SUCCESS
Comment 4 Paulo Zanoni 2014-02-06 12:39:24 UTC
(In reply to comment #3)
> Test on latest -queued kernel.
> The fail goes away.
> output:
> IGT-Version: 1.5-gd1b5823 (x86_64) (Linux:
> 3.13.0-rc8_drm-intel-next-queued_369a13_20140129_debug+ x86_64)
> Runtime PM support: 1
> PC8 residency support: 0
> Subtest i2c: SUCCESS

I'm a little confused, but I'm going to assume it works now because either a monitor was connected to the Intel graphics card or whatever was just in -fixes got merged to -nightly and -queued. So closing bug.

If you think the bug is still not closed, please reopen it and explain the remaining problem.

Thanks,
Paulo
Comment 5 lu hua 2014-02-08 06:22:20 UTC
> I'm a little confused, but I'm going to assume it works now because either a
> monitor was connected to the Intel graphics card or whatever was just in
> -fixes got merged to -nightly and -queued. So closing bug.
> 
> If you think the bug is still not closed, please reopen it and explain the
> remaining problem.
> 
> Thanks,
> Paulo

Paulo, pm_pc8 should skip except ULT?  This pass happens on desktop(haswell with nv), If it's expected, I will close it.
Comment 6 lu hua 2014-02-11 02:46:01 UTC
This fail happens with headless.
Comment 3's pass due to connect HDMI.
Reopen it.
Comment 7 lu hua 2014-02-25 03:21:29 UTC
It doesn't cause by NV.
It fails with headless.

Bisect shows: df4547d82589714f1b0cecd93569130a452cbf46 is the first bad commit
commit df4547d82589714f1b0cecd93569130a452cbf46
Author: Paulo Zanoni <paulo.r.zanoni@intel.com>
Date:   Fri Dec 13 15:22:32 2013 -0200

    drm/i915: add runtime PM support on Haswell

    Now that all the infrastructure is in place and all the tests from
    pm_pc8 pass, we can finally enable the feature.

    Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
    Reviewed-by: Rodrigo Vivi <rodrigo.vivi@gmail.com>
    Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Comment 8 Daniel Vetter 2014-03-03 09:46:58 UTC
I guess we need to skip some (or even all) of the pc8 tests if there's no output. And add a big fixme that we need to revisit this once we have working output config injection support for igt kms tests. Paulo, can you pls take a look at this again?
Comment 9 Paulo Zanoni 2014-04-04 16:49:35 UTC
Hi

I believe this is fixed by commit "tests/pm_pc8: skip tests when no screens are connected", which was committed to the intel-gpu-tools repository today.

If the bug still persists, please reopen.

Thanks,
Paulo
Comment 10 lu hua 2014-04-08 06:12:49 UTC
Verified.Fixed.
Comment 11 Elizabeth 2017-10-06 14:40:26 UTC
Closing old verified.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.