On 03/06/2015 01:58 PM, Chris Wilson wrote:
> In order to facilitate the concurrency offered by triple buffering and to
> offset the latency induced by swapping via an external process, which
> may incur extra rendering itself, only throttle to the previous frame
> and not the last. The second issue that mostly affects swap benchmarks,
> but also can incur jitter in the throttling, is that the throttle bo is
> closer to the next SwapBuffers rather than immediately after the previous
> SwapBuffers. Throttling to the previous frame doubles the maximum possible
> latency at the benefit of improving throughput and reducing jitter.
> 
> v2: Rename "first_post_swapbuffer" batches array to a plain
> throttle_batch[] as the pluralisation was contorting the name and not
> making it clear as to whether it was the first batch or first_post_swap
> batch. Not least of which was that not all throttle points are SwapBuffers.
> 
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
> Cc: Kenneth Graunke <kenn...@whitecape.org>
> Cc: Ben Widawsky <b...@bwidawsk.net>
> Cc: Kristian Høgsberg <k...@bitplanet.net>
> Cc: Chad Versace <chad.vers...@linux.intel.com>

Both patches
Reviewed-by: Chad Versace <chad.vers...@intel.com>

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to