diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2017-11-22 23:58:21 +0000 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2017-11-23 07:43:44 +0000 |
| commit | 6ea85cc7dcff94cf2067d92b452672cf8109fa29 (patch) | |
| tree | 6454535ccf6ee43d67ff5f8353c61aae00a6b112 /tests | |
| parent | 374216eecdf05cfe0aa04b58cf54e675b4b8b795 (diff) | |
igt/perf_pmu: Bump batch_duration for legacy sampling inaccuracy
Since the legacy ringbuffer uses a sampling technique, it is limited to
an accuracy based on a 200Hz timer, or 5ms. We assert that measurements
are within 5%, so with a 100ms duration that gives us no room for the
systematic error in our sampling. Bump the duration to 500ms to give us
plenty of safety margin (back of the paper estimate is that we need
300ms to provide the accuracy we request for legacy submission), if it
then fails, it should not be due to the sampling!
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/perf_pmu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/perf_pmu.c b/tests/perf_pmu.c index 382b77d2..ef5de658 100644 --- a/tests/perf_pmu.c +++ b/tests/perf_pmu.c @@ -44,7 +44,7 @@ IGT_TEST_DESCRIPTION("Test the i915 pmu perf interface"); const double tolerance = 0.05f; -const unsigned long batch_duration_ns = 100e6; +const unsigned long batch_duration_ns = 500e6; static int open_pmu(uint64_t config) { |