Re: [Intel-gfx] [PATCH 7/7] Revert "drm/i915: Enable semaphores on BDW"

2014-08-08 Thread Chris Wilson
On Fri, Aug 08, 2014 at 09:13:06AM +0200, Daniel Vetter wrote: > On Mon, Aug 04, 2014 at 11:15:19AM -0700, Rodrigo Vivi wrote: > > Besides that, no real performance gain was found on our tests with different > > and even multiple workloads. That either means you didn't try hard enough, or that the

Re: [Intel-gfx] [PATCH 7/7] Revert "drm/i915: Enable semaphores on BDW"

2014-08-08 Thread Daniel Vetter
On Mon, Aug 04, 2014 at 11:15:19AM -0700, Rodrigo Vivi wrote: > 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. A

Re: [Intel-gfx] [PATCH 7/7] Revert "drm/i915: Enable semaphores on BDW"

2014-08-08 Thread Daniel Vetter
On Thu, Aug 07, 2014 at 01:05:52PM -0700, Rodrigo Vivi wrote: > The rest of the series was a reference for the records of what I had and > let semaphores on bdw a bit more stable. But even with them we still get > hungs so please consider only to get the revert for now. Thanks for the reminder, pi

Re: [Intel-gfx] [PATCH 7/7] Revert "drm/i915: Enable semaphores on BDW"

2014-08-07 Thread Rodrigo Vivi
The rest of the series was a reference for the records of what I had and let semaphores on bdw a bit more stable. But even with them we still get hungs so please consider only to get the revert for now. On Mon, Aug 4, 2014 at 11:15 AM, Rodrigo Vivi wrote: > This reverts commit 521e62e49a42661a4

[Intel-gfx] [PATCH 7/7] Revert "drm/i915: Enable semaphores on BDW"

2014-08-04 Thread Rodrigo Vivi
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 apparent