aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRodrigo Vivi <rodrigo.vivi@intel.com>2014-08-04 14:15:19 -0400
committerDaniel Vetter <daniel.vetter@ffwll.ch>2014-08-08 10:22:19 -0400
commitbe71eabebaf9f142612d34d42292b454e984dcb5 (patch)
treedbac6e46e8a02c31d64167d7c25f73b9e825d3b9
parentece4a17d237a79f63fbfaf3f724a12b6d500555c (diff)
Revert "drm/i915: Enable semaphores on BDW"
This reverts commit 521e62e49a42661a4ee0102644517dbe2f100a23. Although POST_SYNC brought a bit of stability to Semaphores on BDW it didn't solved all issues and some hungs can still occour when semaphores are enabled on BDW. Also some sloweness can be found on some igt tests, althoguth it apparently doesn't affect real workloads. Besides that, no real performance gain was found on our tests with different and even multiple workloads. Let's disable it again for now. At least until we are sure it is safe to re-enable it. Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
-rw-r--r--drivers/gpu/drm/i915/i915_drv.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
index 6c4b25ce8bb0..ec96f9a9724c 100644
--- a/drivers/gpu/drm/i915/i915_drv.c
+++ b/drivers/gpu/drm/i915/i915_drv.c
@@ -481,6 +481,10 @@ bool i915_semaphore_is_enabled(struct drm_device *dev)
481 if (i915.semaphores >= 0) 481 if (i915.semaphores >= 0)
482 return i915.semaphores; 482 return i915.semaphores;
483 483
484 /* Until we get further testing... */
485 if (IS_GEN8(dev))
486 return false;
487
484#ifdef CONFIG_INTEL_IOMMU 488#ifdef CONFIG_INTEL_IOMMU
485 /* Enable semaphores on SNB when IO remapping is off */ 489 /* Enable semaphores on SNB when IO remapping is off */
486 if (INTEL_INFO(dev)->gen == 6 && intel_iommu_gfx_mapped) 490 if (INTEL_INFO(dev)->gen == 6 && intel_iommu_gfx_mapped)