Re: [PATCH] drm/i915: Restore waitboost credit to the synchronous waiter

From: Eero Tamminen <eero.t.tamminen@intel.com>
To: Jesse Barnes <jesse.barnes@intel.com>,
	Chris Wilson <chris@chris-wilson.co.uk>,
	intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH] drm/i915: Restore waitboost credit to the synchronous waiter
Date: Tue, 08 Dec 2015 10:35:56 +0200	[thread overview]
Message-ID: <5666966C.5060101@intel.com> (raw)
In-Reply-To: <5665C423.6000907@intel.com>

Hi,

On 12/07/2015 07:38 PM, Jesse Barnes wrote:
[...]
> Still wishing we had a good way to benchmark these types of changes
> across a range of workloads.  Eero, have you guys looked at turbo stuff
> at all yet?

Except for this, not really:
	https://jira01.devtools.intel.com/browse/LCK-2271

To make results comparable other platforms, I'm using sustained 
performance numbers by discarding results for first round(s) that were 
run with turbo.

For some of performance tracks, we've also planned to disable power 
management completely (e.g. by using fixed freqs for CPU & GPU) because 
it causes way too much variance to performance results (more between 
runs, than within single run).

	- Eero

> Also, is the boost logic only documented in misc commit messages?  Or do
> we have a nice block of text somewhere describing the intent (which may
> not match our implementation!) and how we try to achieve it?
>
> Those are both new requests though, so no need to block this patch:
> Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org>

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx