Bug 16961 - 3298249d8267c27235653f17379f8c breaks Xv overlay (display corruption)
Summary: 3298249d8267c27235653f17379f8c breaks Xv overlay (display corruption)
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/nouveau (show other bugs)
Version: unspecified
Hardware: x86 (IA32) Linux (All)
: medium normal
Assignee: Nouveau Project
QA Contact: Xorg Project Team
Depends on:
Reported: 2008-08-02 10:55 UTC by Mathieu Bérard
Modified: 2008-08-05 14:40 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:

screenshot (135.18 KB, image/jpeg)
2008-08-02 10:57 UTC, Mathieu Bérard
no flags Details

Description Mathieu Bérard 2008-08-02 10:55:42 UTC
Xv overlay has a pretty major display corruption bug which
was bisected down to commit 3298249d8267c27235653f17379f8c
(xv: reformat source so mere mortals can understand it)

Visually I see flashing bands of alternatively greed/purple colors
that seems mixed with the normal image
The corruption the not seems to affect every frame,
It more looks like something like 1 frame over 2 are corrupted, which
visually create a flashing effect.

A (bad) screenshot on a CRT monitor is attached.

The hardware is a NV20 GeForce 2 Ti 500
Comment 1 Mathieu Bérard 2008-08-02 10:57:24 UTC
Created attachment 18082 [details]
Comment 2 Pekka Paalanen 2008-08-02 12:43:58 UTC
I have a very similar problem on nv20, but I bisected it to commit
9f53b2a341a81e1bff87fee8bf5c2754cac17849 "xv: remove drawable handling from hw-specific PutImage.. it's common..."

For me it affects only the real overlay. Blitter adapted works fine.
Comment 3 Mathieu Bérard 2008-08-02 13:25:14 UTC
I've checked and I can confirm that:
3298249d8267c27235653f17379f8c~1 is good
3298249d8267c27235653f17379f8c has image corruption

So there might be another bug here

The blitter adapter is also unaffected here, only video overlay
Comment 4 Danny 2008-08-03 22:22:30 UTC
After my statement on irc friday (as moondrake) I also did a bisect. Same results Mathieu Bérard. But I noticed while testing that there may be at least 2 issues, so maybe pq is also right:)

Anyway, the commit is rather big, I'll go through it tonight to see what broke.

Comment 5 Danny 2008-08-04 06:11:59 UTC
A brief update: I spotted one error in the xv rework that darktama has now fixed. However, more issues may remain so it needs some testing (I try to look into it tonight).
Comment 6 Pekka Paalanen 2008-08-05 05:57:44 UTC
After darktama's fix in commit 4b24be811b839f9f836b9f8574fc808d2182dd30, "xv: fix thinko from when source was reformatted", the overlay now works like it used to before this latest total breakage. So it was 3298249d8267c27235653f17379f8c after all.
Comment 7 Mathieu Bérard 2008-08-05 14:40:43 UTC
Indeed this bug is now fixed in latest git.

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.