Bug 810161 - Regression: change to 16 bpp by default in virt breaks KDE Plasma desktop
Summary: Regression: change to 16 bpp by default in virt breaks KDE Plasma desktop
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: xorg-x11-drv-cirrus
Version: 17
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: X/OpenGL Maintenance List
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard: RejectedBlocker AcceptedNTH
Depends On: 821268
Blocks: F17-accepted, F17FinalFreezeExcept F17Target-kde
TreeView+ depends on / blocked
 
Reported: 2012-04-05 09:58 UTC by Jaroslav Reznik
Modified: 2013-10-14 12:02 UTC (History)
10 users (show)

Fixed In Version: kde-settings-4.8-13.fc17
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-05-19 07:11:37 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Screenshot (195.72 KB, image/png)
2012-04-05 09:58 UTC, Jaroslav Reznik
no flags Details


Links
System ID Private Priority Status Summary Last Updated
KDE Software Compilation 297588 0 NOR RESOLVED Oxygen glitches while using raster and 16 bpp depth 2020-08-17 12:42:02 UTC
Qt Bug Tracker QTBUG-25198 0 None None None Never
Red Hat Bugzilla 809052 0 unspecified CLOSED Gnome - Corrupted windows displaying 2021-02-22 00:41:40 UTC

Internal Links: 809052

Description Jaroslav Reznik 2012-04-05 09:58:52 UTC
Created attachment 575348 [details]
Screenshot

Description of problem: recent change between 1.3.2-18 and 1.3.2-19 (so between rc2 and rc3) breaks KDE Plasma desktop in virt. environment.


Version-Release number of selected component (if applicable): 1.3.2-19


How reproducible: everytime


Steps to Reproduce:
1. boot Fedora 17 Beta rc3 to KDE desktop
2. run for example System Settings
  
Actual results:
Background has visible glitches, same for default Oxygen style (for example scrollbars)

Expected results:
No breakage, 24 bpp in virt. environment for cirrus as default.

Additional info:
Can't find the reason for the switch to 16 bpp...

Comment 1 Jaroslav Reznik 2012-04-05 11:48:20 UTC
Proposing as F17Beta blocker.

Alpha crit. 17 - working graphical env. - it's quite unusable in this state, Beta crit. 15 - running in virt. environment in previous version of virt. manager.

I found the bug that causes regression - #809052.

Comment 2 Martin Krizek 2012-04-05 12:03:18 UTC
Reproduced here. Another screenshot showing more of this: http://mkrizek.fedorapeople.org/kde-purple.png

Comment 3 Adam Williamson 2012-04-05 14:59:13 UTC
I already saw and reported this in the relevant bodhi page and to ajax (and in #fedora-kde), don't recall whether I opened a bug though.

It only affects cirrus/vnc, which is the Old and Ugly, and it's really not show-stopping, it's just glitches. qxl/spice is fine.

The reason for the change to 16bpp is https://bugzilla.redhat.com/show_bug.cgi?id=809052 , and more generally, ajax considers it a much more solid/tested path than 24bpp. apparently, all the other 'fallback' drivers use 16bpp.

the update that causes it is https://admin.fedoraproject.org/updates/xorg-x11-drv-cirrus-1.3.2-19.fc17,mesa-8.0.2-2.fc17 .

we basically have a choice between #809052 or this bug, and #809052 is far more serious.



-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

Comment 4 Rex Dieter 2012-04-05 15:25:00 UTC
I'd tend to agree that the minor corruption is definitely a lesser of two evils.  My gut is unsure whether this is particularly blocker-worthy either, though I'll have to double check the blocker criteria to say for sure.

We do have some hackish workarounds available too, fwiw:
* don't use oxygen widget style
* set env QT_GRAPHICSSYSTEM=native

Comment 5 Jaroslav Reznik 2012-04-05 20:06:13 UTC
For Beta I think it's enough to document it in Common bugs - with the solution to use qxl/spice. For final it's the must as the cirrus/vnc combo is default at least in F16.

Comment 6 Adam Williamson 2012-04-06 04:12:49 UTC
Discussed at 2012-04-05 go/no-go meeting - http://meetbot.fedoraproject.org/fedora-meeting-1/2012-04-05/gono-go_continuation_f17_beta_rc3_part_two_or_three.2012-04-05-15.00.html . We agreed that as the impact of this bug is only cosmetic - all apps affected by it remain usable - and there are 'workarounds' (switching to qxl/spice, or those cited by Rex in comment #4), the bug is rejected as a beta blocker. It is accepted as NTH, however, and we will be doing an RC4 spin as there are other blockers, so if a fix for this is made available quickly, it should land in RC4.



-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

Comment 7 Jaroslav Reznik 2012-04-06 08:10:37 UTC
(In reply to comment #4)
> * set env QT_GRAPHICSSYSTEM=native

Checked, native works well, fixes both Plasma Desktop and Oxygen style issues.

Comment 8 Kevin Kofler 2012-04-06 11:43:13 UTC
Maybe a bug in Qt's raster graphics system then? Has anybody looked yet at the 16 bpp codepath in the place we last fixed 24 bpp in?

Comment 9 Jaroslav Reznik 2012-04-06 13:09:54 UTC
(In reply to comment #8)
> Maybe a bug in Qt's raster graphics system then? Has anybody looked yet at the
> 16 bpp codepath in the place we last fixed 24 bpp in?

Yep, looks like the bug in Qt's raster graphics engine using 16 bpp. The 24 bpp fixes were in qpixmap_x11, so unrelated to raster breakage.

For background issues - ksplash renders directly using X and it's a little bit degraded, but ok, same for native -> raster issue, taking look, I'll file Qt bug.

For Oxygen style - reported upstream (it's still raster engine problem but if we can get workaround in Oxygen).

=> temporary switch to native for rc4?

Comment 10 Kevin Kofler 2012-04-06 20:36:21 UTC
> The 24 bpp fixes were in qpixmap_x11, so unrelated to raster breakage.

The latest 24 bpp fix was for a crash with raster, see bug #749647.


And I don't think switching to native is a good idea.

Comment 11 Jaroslav Reznik 2012-04-09 11:45:36 UTC
(In reply to comment #10)
> > The 24 bpp fixes were in qpixmap_x11, so unrelated to raster breakage.
> 
> The latest 24 bpp fix was for a crash with raster, see bug #749647.

Ah, sorry, this one.

> And I don't think switching to native is a good idea.

It's indeed not a good idea for final but hopefully only temporary solution for beta. I'm trying to find someone upstream to help with 16 bpp path...

Comment 12 Than Ngo 2012-04-10 08:57:28 UTC
(In reply to comment #10)
> > The 24 bpp fixes were in qpixmap_x11, so unrelated to raster breakage.
> 
> The latest 24 bpp fix was for a crash with raster, see bug #749647.
> 
> 
> And I don't think switching to native is a good idea.

we switch to native only by using KVM with Cirrus Logic grahic card. IMO it's the best workaround atm.

Just add following 3 lines in /etc/kde/env/env.sh (kde-settings)

if ( lspci | grep -qi "Cirrus Logic" ) ; then
   export QT_GRAPHICSSYSTEM=native
fi

Comment 13 Jaroslav Reznik 2012-04-10 09:14:53 UTC
(In reply to comment #12)
> (In reply to comment #10)
> > > The 24 bpp fixes were in qpixmap_x11, so unrelated to raster breakage.
> > 
> > The latest 24 bpp fix was for a crash with raster, see bug #749647.
> > 
> > 
> > And I don't think switching to native is a good idea.
> 
> we switch to native only by using KVM with Cirrus Logic grahic card. IMO it's
> the best workaround atm.
> 
> Just add following 3 lines in /etc/kde/env/env.sh (kde-settings)
> 
> if ( lspci | grep -qi "Cirrus Logic" ) ; then
>    export QT_GRAPHICSSYSTEM=native
> fi

Seem reasonable for me.

The background issues is probably caused by Qt raster not supporting dithering. Checking the purple effect.

Comment 14 Kamil Páral 2012-04-12 12:10:56 UTC
Beta is out, moving to F17 Final NTH

Comment 15 Kevin Kofler 2012-04-12 12:20:59 UTC
Proposing as Final Blocker.

Comment 16 Adam Williamson 2012-04-20 21:57:34 UTC
Discussed at 2012-04-02 blocker review meeting: http://meetbot.fedoraproject.org/fedora-bugzappers/2012-04-20/fedora-bugzappers.2012-04-20-17.01.log.txt . Everyone felt somewhat conflicted about it and bad about rejecting it, but in the end there was a consensus that the bug just isn't severe enough to constitute a breakage of any criteria: its impact is entirely cosmetic, and it only happens in one particular configuration of one particular virtualization setup (albeit our officially supported one). So, reluctantly, this is rejected as a release blocker. It is accepted as NTH.

Excuse me, I have to go and make my secret base resistant against KoflerCorp ballistic missiles.

Comment 17 Kevin Kofler 2012-04-21 21:07:44 UTC
You keep claiming this is "entirely cosmetic". Have you actually tried using it? Jaroslav Řezník has and he says this severely impact usability.

I see only one reasonable outcome, and that's reverting cirrus to 24 bpp and fixing the issues with that one. Qt upstream does not support 16 bpp at all.

Comment 18 Kevin Kofler 2012-04-21 21:09:51 UTC
(Also because in addition to causing this bug, the change to 16 bpp also significantly degrades color quality and as such is at best a temporary workaround, not something we want to ship in a stable release.)

Comment 19 Adam Williamson 2012-04-23 11:30:55 UTC
Yes, I've tried using it. I don't see any reduction in usability. Various widgets have pink backgrounds; that's the entire extent of the bug so far as I can see. It doesn't render any of them any more difficult to use so far as I could see.

If there are more serious effects, please provide a specific explanation of them. General assertions that such exist, without any details, aren't a lot of use.

Comment 20 Than Ngo 2012-05-08 15:45:45 UTC
i think this workaround is safe

if ( lspci | grep -qi "VGA compatible controller: Cirrus Logic GD 5446" ) ; then
    export QT_GRAPHICSSYSTEM=native
fi

Comment 21 Fedora Update System 2012-05-08 16:23:28 UTC
kde-settings-4.8-8.fc17.1 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/kde-settings-4.8-8.fc17.1

Comment 22 Jaroslav Reznik 2012-05-09 13:09:04 UTC
The update fixes the most visible glitches while using cirrus gfx in 16 bpp. It's still not the optimal one - KDM uses raster engine, so there's no dithering and does not look very well. Also using native engine and 16 bpp with dithering leads to not a good looking user experience. There's still noticiable performance loss using 16 bpp and desktop effects.

I'm not sure I'm happy with the resolution of blocker bug but at least - the desktop is somehow usable. So it should stay documented in CommonBugs to make sure users are using Plasma Workspace with QXL/Spice combo.

Comment 23 Jaroslav Reznik 2012-05-09 14:32:47 UTC
We need the same patch in prefdm script too for KDM.

Comment 24 Kevin Kofler 2012-05-10 07:26:08 UTC
Actually, I think the right place is to do this in the qt package in /etc/profile.d! The issue also affects Qt apps in non-KDE environments.

Comment 25 Fedora Update System 2012-05-10 20:08:24 UTC
kde-settings-4.8-10.fc17,qt-4.8.1-10.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/kde-settings-4.8-10.fc17,qt-4.8.1-10.fc17

Comment 26 Fedora Update System 2012-05-11 21:51:47 UTC
Package kde-settings-4.8-10.fc17, qt-4.8.1-10.fc17:
* should fix your issue,
* was pushed to the Fedora 17 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing kde-settings-4.8-10.fc17 qt-4.8.1-10.fc17'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-7713/kde-settings-4.8-10.fc17,qt-4.8.1-10.fc17
then log in and leave karma (feedback).

Comment 27 Fedora Update System 2012-05-12 16:22:09 UTC
kde-settings-4.8-8.fc17.1 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 28 Kevin Kofler 2012-05-12 17:15:39 UTC
Ugh, why did this get pushed? It's superseded by FEDORA-2012-7713 which also fixes the problem for KDM and for Qt apps outside of KDE Plasma sessions.

Comment 29 Rex Dieter 2012-05-12 19:23:29 UTC
it was queue'd for stable prior to FEDORA-2012-7713 landing

Comment 30 Rex Dieter 2012-05-15 13:09:01 UTC
OK, current hack/workaround isn't going to help kdm, /etc/profile isn't parsed for non-login shells (like prefdm).  On to plan B (or C).

Comment 31 Petr Schindler 2012-05-15 13:37:02 UTC
I can confirm that kdm still has broken graphic.

Comment 32 Jaroslav Reznik 2012-05-17 08:55:49 UTC
Also Plastique style fix only the pink issue but the dithering one is still there.

Comment 33 Kamil Páral 2012-05-18 10:10:36 UTC
I tried clean install of F17 RC1 KDE Live i686. After that I updated to kde-settings-4.8-10.fc17 (qt-4.8.1-10.fc17 was already present on RC1). I don't see any difference between desktop or kdm before and after update. See for yourself:

http://imgur.com/a/YPUp3

(first image is from clean rc1, second image is after applying updates).

Comment 34 Rex Dieter 2012-05-18 11:57:36 UTC
The primary symptom fixed in the latest kde-settings-4.8-13.fc17 update is corrupted/pink scrollbars in kdm, which you don't see in kdm until you click on some menu items (see some of the attached screenshots for examples).

kde-settings-4.8-10.fc17 included a workaround for user sessions only.

Comment 35 Kamil Páral 2012-05-18 12:15:52 UTC
In that case yes, kdm menu items are no longer corrupted.

Comment 36 Fedora Update System 2012-05-19 07:11:37 UTC
kde-settings-4.8-13.fc17, qt-4.8.1-10.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 37 Jaroslav Reznik 2012-05-21 08:49:53 UTC
There's still visible corruption of the background but in KDM it's more visual annoyance than usability - so I think the release notes/common bugs recommending Qxl in virt machine are ok.

Comment 38 John Ellson 2012-06-23 16:11:32 UTC
The workaround from Comment #20, as installed in /etc/profile.d/qt-graphicssystem.sh, is causing problems.

Apparently the default PATH for ssh, when executed from cron on a different host, does not include /usr/sbin, so lspci isn't found and the cron job fails.

A workaround that works-for-me is to use the full path to lspci:

  
    if ( /usr/sbin/lspci | grep -qi "VGA compatible ....

Comment 39 Kevin Kofler 2012-06-23 17:05:38 UTC
The lspci path issue is already filed as bug #827440, it's fixed in
kde-settings-4.8-15.fc17 (update FEDORA-2012-9374), currently in updates-testing.

Comment 40 Red Hat Bugzilla 2013-10-04 00:21:55 UTC
Removing external tracker bug with the id '25198' as it is not valid for this tracker


Note You need to log in before you can comment on or make changes to this bug.