[Bug 412211] New: xorg-x11-driver-video-7.3-138.3.i586 enables EXA on intel , which is broken
https://bugzilla.novell.com/show_bug.cgi?id=412211 User gnorton@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c400788 Summary: xorg-x11-driver-video-7.3-138.3.i586 enables EXA on intel, which is broken Product: openSUSE 11.0 Version: Final Platform: All OS/Version: Other Status: NEW Severity: Major Priority: P5 - None Component: X.Org AssignedTo: sndirsch@novell.com ReportedBy: gnorton@novell.com QAContact: xorg-maintainer-bugs@forge.provo.novell.com Depends on: 400788 Found By: --- The latest "update" xorg-x11-driver-video-7.3-138.3.i586 blindly enabled EXA acceleration mode on intel video cards. This causes application crashes in Firefox and Moonlight (at the very least) as we try to create pixmaps > 8188 bytes wide. I've attached a patch which fixes the issue of the application crashes, however the greater issue is how such a major change could even be contemplated for an update release and wasn't pushed to 11.1. Changing the AccelMode of a driver is not a trivial change as it forces applications to go thru new, untested codepaths (as seen), and will have possibly dramatic performance impacts or application crash Bug #400788 is the relevent reference bug for the decision process to enable EXA mid-release. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User gnorton@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c1 --- Comment #1 from Geoff Norton <gnorton@novell.com> 2008-07-25 08:04:15 MDT --- Created an attachment (id=230044) --> (https://bugzilla.novell.com/attachment.cgi?id=230044) patch -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #230044|text/x-patch |text/plain mime type| | Attachment #230044|0 |1 is patch| | -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c4 --- Comment #4 from Stefan Dirsch <sndirsch@novell.com> 2008-07-25 08:47:43 MDT --- Created an attachment (id=230058) --> (https://bugzilla.novell.com/attachment.cgi?id=230058) upstream.patch Upstream patch. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c5 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #230058|0 |1 is obsolete| | --- Comment #5 from Stefan Dirsch <sndirsch@novell.com> 2008-07-25 08:52:35 MDT --- Created an attachment (id=230060) --> (https://bugzilla.novell.com/attachment.cgi?id=230060) upstream.patch -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c7 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|Major |Critical Priority|P5 - None |P2 - High --- Comment #7 from Stefan Dirsch <sndirsch@novell.com> 2008-07-25 09:02:45 MDT --- I think it's appropriate to handle this as critical. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User gnorton@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c8 --- Comment #8 from Geoff Norton <gnorton@novell.com> 2008-07-25 09:30:10 MDT --- Stefan, More problems with EXA enabling. If you want yet another seperate bug feel free to bust this out. With the recent change to EXA as default I'm taking a 10% performance hit on my compilation times with 1 fullscreen gnome-terminal compared to XAA. Here is the timing logs of a time make in our source tree (all from make clean): Run #1: real 1m35.285s user 1m1.912s sys 0m11.825s real: 95.285s Run #2: real 1m33.286s user 1m1.744s sys 0m12.149s real: 93.286s Run #3: real 1m33.575s user 1m1.760s sys 0m12.053s real: 93.575s XAA: Run #1: real 1m25.114s user 1m2.448s sys 0m12.289s real: 85.114s Run #2: real 1m24.098s user 1m2.352s sys 0m12.189s real: 84.098s Run #3: real 1m24.414s user 1m2.328s sys 0m12.349s real: 84.414s EXA AVG: (95.285+93.286+93.575)/3 == 94.049 XAA AVG: (85.114+84.098+84.414)/3 == 84.542 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User gnorton@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c9 --- Comment #9 from Geoff Norton <gnorton@novell.com> 2008-07-25 09:41:32 MDT --- results of x11perf -aa10text XAA: ~379000.0/sec EXA: ~46000.0/sec -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c10 --- Comment #10 from Stefan Dirsch <sndirsch@novell.com> 2008-07-25 10:25:51 MDT --- Please report EXA performance issues upstream on bugs.freedesktop.org (product xorg, component Driver/intel). Intel promised us to address them. Add me to Cc (sndirsch@suse.de), so I can add some Intel developers, which are going to push this to the upstream Intel developers. Thanks. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User gnorton@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c11 --- Comment #11 from Geoff Norton <gnorton@novell.com> 2008-07-25 10:40:26 MDT --- Stefan, These perforamnce issues are known upstream (talking to cworth and keithp). The issue here is the jarring performance regression for openSUSE users from when they installed oS11 to when they apply the update. From further testing it doesn't seem to apply only to text primitives either, it appears that we're taking a fairly noticable hit on all accelerated operations resulting my desktop going from performant and snappy a few days ago to sluggish and slow today. The question is wether inflicting this pain on our users is acceptible. I would vote no, but I'm not the maintainer of this package. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c12 --- Comment #12 from Stefan Dirsch <sndirsch@novell.com> 2008-07-25 11:25:13 MDT --- Geoff, reverting a change, which caused some regression, has always been an option and will also remain in the future. Up to now I've seen two reports about the EXA performance issue since the update has been released. I think it was this monday. See Bugs #411183/#411315. I don't read openSUSE ML and/or forums (since I'm swamped by Bugzilla *all the time*), so I might have missed reports on these. Reverting to XAA would also mean that we see these Xserver crashes again, which require a complete reboot of your machine. These crashes can happen at any time. Please report these EXA performance issues upstream. As long as everybody thinks it is a known issue, you can't expect Intel will ever change the situation. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User gnorton@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c13 --- Comment #13 from Geoff Norton <gnorton@novell.com> 2008-07-25 11:54:39 MDT --- Stefan, cworth and keithp are intel driver developers for X. They informed me these bugs are already upstream. At this point I've officially spent too much time on this issue. I'm of the opinion that such a serious performance regression of our users desktops is unacceptible to change mid-release. We will work around this on our team by forcing XAA and suggesting our uses to do the same. Pushing something on the future hope of possible performance increases that so dramatically degrades the desktop experience is just baffling to me. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c14 --- Comment #14 from Stefan Dirsch <sndirsch@novell.com> 2008-07-25 12:43:43 MDT --- Not sure what you mean with "our team" and "our users". Is there a difference between "your users" and a common openSUSE user? Also I'm wondering that a Xserver crash with the need to reboot your machine is no issue to you at all. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User gnorton@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c15 --- Comment #15 from Geoff Norton <gnorton@novell.com> 2008-07-25 13:01:42 MDT --- Our team == Moonlight hackers Our users == Moonlight users I've never experienced a X lockup with XAA, so for our use case we have the following: 1) X changed mid stream 2) The change broke rendering large pixmaps in Moonlight and Firefox (at least) 3) My computer operates 10% of its original speed with relation to text rendering 4) My computer operates 80-90% of its original speed for other primitive rendering As I said in my last comment. I've officially spent too much time on this issue. I don't agree with the change, its your package proceed as you see fit. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User gnorton@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c16 --- Comment #16 from Geoff Norton <gnorton@novell.com> 2008-07-25 13:06:03 MDT --- Additionally: https://bugzilla.novell.com/show_bug.cgi?id=400788#c26 I'm on a 82G33/G31 which according to your comment wasn't affected by the crashing bug anyways, so my experience before the "fix" was no crashes, after the "fix" no crashes and performance takes a nose dive. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User toshok@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c17 Chris Toshok <toshok@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |toshok@novell.com --- Comment #17 from Chris Toshok <toshok@novell.com> 2008-07-25 13:07:00 MDT --- So is your stance that since you fixed the X server crash, we should be thankful and not complain about the 90+% performance drop (and crashing applications)? Given that the driver authors know about the performance problems, and aren't going to have them fixed ANYTIME soon (by their own words), it seems logical to switch back to XAA and put some maintenance cycles into it ourselves. i.e. fix the crashes, and not expect ours (and your) users to suffer through performance problems and an unstable desktop. Of course X server crashes are important issues, but I don't see how this default driver switch decision was made. Who decided it? And how can they reconcile the pain they're putting *everyone* through to fix crashes experienced by a smaller subset of the community? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c18 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEEDINFO Info Provider| |ast@novell.com Summary|xorg-x11-driver-video-7.3-138.3.i586 enables EXA|Application crashes due to overly-restrictive |on intel, which is broken |integer overflow check in EXA --- Comment #18 from Stefan Dirsch <sndirsch@novell.com> 2008-07-25 15:15:20 MDT --- Package with patch applied and patchinfo submitted to /work/src/done/11.0/xorg-x11-server/ /work/src/done/PATCHINFO/xorg-x11-server.patch.box Anja, could you create a SWAMPID for this issue? Thanks. (A possible switch back to XAA can be discussed later. This would be in a different package (xorg-x11-driver-video) anyway.) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c19 --- Comment #19 from Stefan Dirsch <sndirsch@novell.com> 2008-07-26 09:26:41 MDT --- Packages are available for testing. http://beta.suse.com/private/sndirsch/bug412211 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c21 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED --- Comment #21 from Stefan Dirsch <sndirsch@novell.com> 2008-07-28 07:01:48 MDT --- Thanks, Anja. SWAMPID is now filled in. Closing as fixed. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 Bug 412211 depends on bug 400788, which changed state. Bug 400788 Summary: intel/i965_dri: Various OpenGL apps crash X-server (965G/965GM/IGD) https://bugzilla.novell.com/show_bug.cgi?id=400788 What |Old Value |New Value ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|FIXED | -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 Bug 412211 depends on bug 400788, which changed state. Bug 400788 Summary: intel/i965_dri: Various OpenGL apps crash X-server (965G/965GM/IGD) https://bugzilla.novell.com/show_bug.cgi?id=400788 What |Old Value |New Value ---------------------------------------------------------------------------- Status|REOPENED |RESOLVED Resolution| |WONTFIX -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=412211 User ast@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=412211#c22 --- Comment #22 from Anja Stock <ast@novell.com> 2008-08-01 09:42:15 MDT --- released -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com