Re: [PATCH] DRI2: By default, throttle after two pending swaps.

2014-05-14 Thread Chris Wilson
On Tue, May 13, 2014 at 09:20:45AM -0700, Jamey Sharp wrote: On May 13, 2014 12:57 AM, Chris Wilson [1]ch...@chris-wilson.co.uk wrote: =0 ickle:/usr/src/piglit$ ./bin/glx-oml-sync-control-getmscrate PIGLIT: {'result': 'pass' } =0 ickle:/usr/src/piglit$

Re: [PATCH] DRI2: By default, throttle after two pending swaps.

2014-05-14 Thread Jamey Sharp
On Wed, May 14, 2014 at 01:31:14PM +0100, Chris Wilson wrote: On Tue, May 13, 2014 at 09:20:45AM -0700, Jamey Sharp wrote: Yeah, we have new tests that haven't quite been merged yet as we're addressing Eric's review comments. Our current version, revised yesterday, is available

Re: [PATCH] DRI2: By default, throttle after two pending swaps.

2014-05-14 Thread Chris Wilson
On Wed, May 14, 2014 at 06:49:10AM -0700, Jamey Sharp wrote: On Wed, May 14, 2014 at 01:31:14PM +0100, Chris Wilson wrote: On Tue, May 13, 2014 at 09:20:45AM -0700, Jamey Sharp wrote: Yeah, we have new tests that haven't quite been merged yet as we're addressing Eric's review

Re: [PATCH] DRI2: By default, throttle after two pending swaps.

2014-05-13 Thread Jamey Sharp
On Tue, May 13, 2014 at 12:01 AM, Chris Wilson ch...@chris-wilson.co.uk wrote: On Mon, May 12, 2014 at 06:30:47PM -0700, Jamey Sharp wrote: According to Chris Wilson, the Intel driver has a horrifying kludge to get triple-buffering despite the swap_limit being set to only allow

Re: [PATCH] DRI2: By default, throttle after two pending swaps.

2014-05-13 Thread Chris Wilson
On Mon, May 12, 2014 at 06:30:47PM -0700, Jamey Sharp wrote: According to Chris Wilson, the Intel driver has a horrifying kludge to get triple-buffering despite the swap_limit being set to only allow double-buffering by default. This kludge makes OML_sync_control unusable if the Intel driver

Re: [PATCH] DRI2: By default, throttle after two pending swaps.

2014-05-13 Thread Chris Wilson
On Tue, May 13, 2014 at 12:18:49AM -0700, Jamey Sharp wrote: On Tue, May 13, 2014 at 12:01 AM, Chris Wilson ch...@chris-wilson.co.uk wrote: On Mon, May 12, 2014 at 06:30:47PM -0700, Jamey Sharp wrote: According to Chris Wilson, the Intel driver has a horrifying kludge to get

Re: [PATCH] DRI2: By default, throttle after two pending swaps.

2014-05-13 Thread Jamey Sharp
On May 13, 2014 12:57 AM, Chris Wilson ch...@chris-wilson.co.uk wrote: =0 ickle:/usr/src/piglit$ ./bin/glx-oml-sync-control-getmscrate PIGLIT: {'result': 'pass' } =0 ickle:/usr/src/piglit$ ./bin/glx-oml-sync-control-waitformsc PIGLIT: {'result': 'pass' } =0 ickle:/usr/src/piglit$

[PATCH] DRI2: By default, throttle after two pending swaps.

2014-05-12 Thread Jamey Sharp
According to Chris Wilson, the Intel driver has a horrifying kludge to get triple-buffering despite the swap_limit being set to only allow double-buffering by default. This kludge makes OML_sync_control unusable if the Intel driver goes into page-flipping mode. Since triple-buffering is preferred