diff options
author | Tvrtko Ursulin <tvrtko.ursulin@intel.com> | 2020-12-18 08:43:31 +0000 |
---|---|---|
committer | Tvrtko Ursulin <tvrtko.ursulin@intel.com> | 2020-12-18 17:17:27 +0000 |
commit | d4b3383dcec6c3585b62d693c3e40b30b029bc0e (patch) | |
tree | ae1a96718ca302a0008906f852d546cd448af2ea | |
parent | 3d6caf71a3e988cd125eb9efdd0a7cdcd0451673 (diff) |
lib/igt_perf: Request CLOCK_MONOTONIC when opening events
Ask for CLOCK_MONOTONIC which is more stable than the default perf clock.
(Ability to select a clock has been available since kernel version 4.1.)
The change should not have any significant impact on the IGT as whole
apart from maybe improving the occasional jitter in tests/tools which use
nanosleep(2) and use time slept together with perf reported time delta
either in direct or indirect calculations.
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
-rw-r--r-- | lib/igt_perf.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/lib/igt_perf.c b/lib/igt_perf.c index 418c1c18..b743859f 100644 --- a/lib/igt_perf.c +++ b/lib/igt_perf.c @@ -1,12 +1,13 @@ -#include <stdint.h> +#include <errno.h> #include <fcntl.h> -#include <unistd.h> +#include <stdint.h> #include <stdlib.h> #include <string.h> -#include <errno.h> #include <sys/stat.h> #include <sys/sysinfo.h> #include <sys/sysmacros.h> +#include <time.h> +#include <unistd.h> #include "igt_perf.h" @@ -111,6 +112,8 @@ _perf_open(uint64_t type, uint64_t config, int group, uint64_t format) attr.read_format = format; attr.config = config; + attr.use_clockid = 1; + attr.clockid = CLOCK_MONOTONIC; do { ret = perf_event_open(&attr, -1, cpu++, group, 0); |