broadwell bugs with fglrx in xserver-xorg-video-intel

Bug #1359105 reported by Maarten Lankhorst on 2014-08-20
20
This bug affects 2 people
Affects Status Importance Assigned to Milestone
HWE Next
Undecided
Unassigned
xserver-xorg-video-intel (Ubuntu)
Undecided
Unassigned
Precise
Undecided
Unassigned
Trusty
Undecided
Unassigned
xserver-xorg-video-intel-lts-trusty (Ubuntu)
Undecided
Unassigned
Precise
Undecided
Unassigned
Trusty
Undecided
Unassigned

Bug Description

When using the fglrx driver gpu-manager will fallback to UXA. UXA is not implemented for broadwell in the 910 driver, but is required when using fglrx for offloading. When using FGLRX in the IGPU (integrated gpu) mode fglrx unloads itself and makes intel drive the screen. This crashes with SNA, because fglrx steals the driver privates.

Upstream has some commits to fix this, but they're missing in the 910 driver from trusty.

Maarten Lankhorst (mlankhorst) wrote :

[Impact]
* Fixes crash when using SNA with fglrx IGPU mode.
* Adds UXA support for broadwell.

[Test Case]
* On an affected system:
* Run aticonfig --px-igpu to force intel driver being used.
* Disable gpu-manager, set accelmethod in /etc/X11/xorg.conf to UXA or SNA
* SNA: Observe a crash in freescreen, while a normal xserver with full acceleration is expected.
* UXA (default): xserver starts normally, but llvmpipe is used and no acceleration is used in intel at all.

[Regression Potential]
* Adding pci-id and names for broadwell and crv should always be safe, no regressions expected there.
* The uxa patches seem to be a noop for < gen8, which means I don't expect any regressions either.
* The sna-freescreen patch can not make things worse, because if sna is NULL the next statement would already crash, so in the worst case intel would crash elsewhere.
* I'll watch for regressions in -proposed, but I don't expect any from them because those patches are in utopic too. :-)

Hello Maarten, or anyone else affected,

Accepted xserver-xorg-video-intel into trusty-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/xserver-xorg-video-intel/2:2.99.910-0ubuntu1.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in xserver-xorg-video-intel (Ubuntu Trusty):
status: New → Fix Committed
tags: added: verification-needed
Maarten Lankhorst (mlankhorst) wrote :

Tested by woodrow.

tags: added: verification-done
removed: verification-needed

The verification of the Stable Release Update for xserver-xorg-video-intel has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package xserver-xorg-video-intel - 2:2.99.910-0ubuntu1.1

---------------
xserver-xorg-video-intel (2:2.99.910-0ubuntu1.1) trusty; urgency=medium

  * Backport fixes from upstream driver for fglrx and broadwell. (LP: #1359105)
  * Add more pci-ids for broadwell.
    - add-pci-ids-crv.diff
    - add-pci-ids-brw.diff
    - add-names-brw.diff
  * Enable basic UXA blit acceleration for broadwell.
    - uxa-bdw-0.diff
    - uxa-bdw-1.diff
    - uxa-bdw-2.diff
  * Fix a crash in FreeScreen with sna when used with fglrx.
    - sna-freescreen.diff
 -- Maarten Lankhorst <email address hidden> Wed, 20 Aug 2014 10:48:06 +0200

Changed in xserver-xorg-video-intel (Ubuntu Trusty):
status: Fix Committed → Fix Released
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package xserver-xorg-video-intel - 2:2.99.914-1~exp1ubuntu3

---------------
xserver-xorg-video-intel (2:2.99.914-1~exp1ubuntu3) utopic; urgency=medium

  * Add patch to enable SNA with fglrx. (LP: #1359105)
 -- Maarten Lankhorst <email address hidden> Mon, 22 Sep 2014 15:29:05 +0200

Changed in xserver-xorg-video-intel (Ubuntu):
status: New → Fix Released
Changed in hwe-next:
status: New → Fix Released

Hello Maarten, or anyone else affected,

Accepted xserver-xorg-video-intel-lts-trusty into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/xserver-xorg-video-intel-lts-trusty/2:2.99.910-0ubuntu1.3~precise1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

tags: removed: verification-done
tags: added: verification-needed
Timo Aaltonen (tjaalton) on 2014-12-05
tags: added: verification-done
removed: verification-needed
Changed in xserver-xorg-video-intel-lts-trusty (Ubuntu Trusty):
status: New → Invalid
Mathew Hodson (mathew-hodson) wrote :

This bug was fixed in the package xserver-xorg-video-intel-lts-trusty - 2:2.99.910-0ubuntu1.3~precise1

Changed in xserver-xorg-video-intel-lts-trusty (Ubuntu Precise):
status: New → Fix Released
Changed in xserver-xorg-video-intel-lts-trusty (Ubuntu):
status: New → Invalid
Changed in xserver-xorg-video-intel (Ubuntu Precise):
status: New → Invalid
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers