Bug 91023 - [HSW SNA] unwanted "animation" on VT-leave
Summary: [HSW SNA] unwanted "animation" on VT-leave
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: XOrg git
Hardware: Other All
: medium normal
Assignee: Intel GFX Bugs mailing list
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-06-18 21:27 UTC by David Herrmann
Modified: 2017-02-23 21:46 UTC (History)
2 users (show)

See Also:
i915 platform: ALL
i915 features: display/Other


Attachments

Description David Herrmann 2015-06-18 21:27:17 UTC
Hey

If I switch from Xorg to fbcon, I get an unwanted "animation". It takes exactly 4 frames until the new fbcon content is properly displayed. Each frame moves the content by 25% from the right (and sometimes left) side into the screen. It really looks like an intended animation that "moves" in the new screen from the right side.

This only happens if I use the intel DDX with SNA enabled. I neither happens with UXA nor modesetting+glamour. In fact, I can only reproduce it with SNA.

I've been experiencing this for quite a while (~1 year?), but it looked really intentional, so I wasn't aware this is not a feature. Until today, as I tried asking how to disable it and no-one knew what I was talking about..

I don't mind much, I mean it really looks rather nice. But maybe you guys care.
It's an HSW machine (Lenovo X240), and the only noticeable log message is this:

Jun 18 20:12:15 david-t2 kernel: [drm:intel_set_cpu_fifo_underrun_reporting [i915]] *ERROR* uncleared fifo underrun on pipe A
Jun 18 20:12:15 david-t2 kernel: [drm:intel_cpu_fifo_underrun_irq_handler [i915]] *ERROR* CPU pipe A FIFO underrun

(which happens exactly once at boot).

I can run this with drm.debug=foobar if you want. Just let me know what kind of information you need.

Thanks!



gen: 7
pch: 3
is_mobile: yes
is_i85x: no
is_i915g: no
is_i945gm: no
is_g33: no
need_gfx_hws: yes
is_g4x: no
is_pineview: no
is_broadwater: no
is_crestline: no
is_ivybridge: no
is_valleyview: no
is_haswell: yes
is_skylake: no
is_preliminary: no
has_fbc: yes
has_pipe_cxsr: no
has_hotplug: yes
cursor_needs_physical: no
has_overlay: no
overlay_needs_physical: no
supports_tv: no
has_llc: yes
has_ddi: yes
has_fpga_dbg: yes
Comment 1 Chris Wilson 2015-06-19 08:27:54 UTC
I presume the difference is that sna disables the primary plane on the CRTCs prior to VT switch (so that we do not leak information to the next client).

If you did

diff --git a/src/sna/sna_display.c b/src/sna/sna_display.c
index 53eb9ca..af44dbc 100644
--- a/src/sna/sna_display.c
+++ b/src/sna/sna_display.c
@@ -7347,6 +7347,7 @@ void sna_mode_reset(struct sna *sna)
        xf86CrtcConfigPtr config = XF86_CRTC_CONFIG_PTR(sna->scrn);
        int i;
 
+       return;
        if (sna->flags & SNA_IS_HOSTED)
                return;
 

then you will get the same behaviour in sna as uxa on VT switch.
Comment 2 David Herrmann 2015-06-19 13:30:42 UTC
Yes, this fixes it.
Comment 3 Jani Nikula 2015-08-18 15:38:17 UTC
Chris, planning on doing something about this?
Comment 4 Chris Wilson 2015-08-18 15:51:35 UTC
(In reply to Jani Nikula from comment #3)
> Chris, planning on doing something about this?

What? It's a kernel bug.
Comment 5 Jani Nikula 2015-08-18 15:58:10 UTC
(In reply to Chris Wilson from comment #4)
> (In reply to Jani Nikula from comment #3)
> > Chris, planning on doing something about this?
> 
> What? It's a kernel bug.

Sorry, was browsing the bugs a tad too fast. I take that as a "no". ;)
Comment 6 Jani Nikula 2016-04-25 09:10:39 UTC
David, can you, uh, check if we've maybe done something about this, intentionally or otherwise...?
Comment 7 Jani Saarinen 2016-12-09 11:19:01 UTC
Reporter or Chris, is this still valid issue?
Comment 8 Ricardo 2017-02-23 21:46:26 UTC
Looks like it was a kernel bug, but fixed with Chris suggestion, also pretty old with no recent activity


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.