From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by gabe.freedesktop.org (Postfix) with ESMTPS id E799F6E31C for ; Fri, 17 Jul 2020 01:54:20 +0000 (UTC) From: Umesh Nerlige Ramappa Date: Thu, 16 Jul 2020 18:54:19 -0700 Message-Id: <20200717015419.37439-1-umesh.nerlige.ramappa@intel.com> MIME-Version: 1.0 Subject: [igt-dev] [PATCH] i915/perf: Sanity check reports in mapped OA buffer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" To: Lionel G Landwerlin , igt-dev@lists.freedesktop.org List-ID: For applications that need a faster way to access reports in the OA buffer, i915 now provides a way to map the OA buffer to privileged user space. Add a test to sanity check reports in the mapped OA buffer. Signed-off-by: Umesh Nerlige Ramappa --- include/drm-uapi/i915_drm.h | 32 ++++++++++ tests/i915/perf.c | 118 ++++++++++++++++++++++++++++++++++++ 2 files changed, 150 insertions(+) diff --git a/include/drm-uapi/i915_drm.h b/include/drm-uapi/i915_drm.h index 2b55af13..f7523d55 100644 --- a/include/drm-uapi/i915_drm.h +++ b/include/drm-uapi/i915_drm.h @@ -2048,6 +2048,38 @@ struct drm_i915_perf_open_param { */ #define I915_PERF_IOCTL_CONFIG _IO('i', 0x2) +/** + * Returns OA buffer properties to be used with mmap. + * + * This ioctl is available in perf revision 6. + */ +#define I915_PERF_IOCTL_GET_OA_BUFFER_INFO _IO('i', 0x3) + +/** + * OA buffer size and offset. + */ +struct drm_i915_perf_oa_buffer_info { + __u32 size; + __u32 offset; + __u64 reserved[4]; +}; + +/** + * Returns current position of OA buffer head and tail. + * + * This ioctl is available in perf revision 6. + */ +#define I915_PERF_IOCTL_GET_OA_BUFFER_HEAD_TAIL _IO('i', 0x4) + +/** + * OA buffer head and tail. + */ +struct drm_i915_perf_oa_buffer_head_tail { + __u32 head; + __u32 tail; + __u64 reserved[4]; +}; + /** * Common to all i915 perf records */ diff --git a/tests/i915/perf.c b/tests/i915/perf.c index 92edc9f1..36e98ffa 100644 --- a/tests/i915/perf.c +++ b/tests/i915/perf.c @@ -204,6 +204,7 @@ static struct intel_perf *intel_perf = NULL; static struct intel_perf_metric_set *test_set = NULL; static bool *undefined_a_counters; static uint64_t oa_exp_1_millisec; +struct intel_mmio_data mmio_data; static igt_render_copyfunc_t render_copy = NULL; static uint32_t (*read_report_ticks)(uint32_t *report, @@ -4768,6 +4769,108 @@ test_whitelisted_registers_userspace_config(void) i915_perf_remove_config(drm_fd, config_id); } +#define OA_BUFFER_DATA(tail, head, oa_buffer_size) \ + (((tail) - (head)) & ((oa_buffer_size) - 1)) + +static uint32_t oa_status_reg(void) +{ + if (IS_HASWELL(devid)) + return intel_register_read(&mmio_data, 0x2346) & 0x7; + else if (IS_GEN12(devid)) + return intel_register_read(&mmio_data, 0xdafc) & 0x7; + else + return intel_register_read(&mmio_data, 0x2b08) & 0xf; +} + +static void check_reports_from_mapped_buffer(enum drm_i915_oa_format fmt, + int oa_exponent) +{ + struct drm_i915_perf_oa_buffer_info oa_buffer; + struct drm_i915_perf_oa_buffer_head_tail oa_ht; + struct oa_format format = get_oa_format(fmt); + size_t report_size = format.size; + uint8_t *reports; + uint32_t *report0, *report1; + uint32_t num_reports, timer_reports = 0; + uint32_t period_us = oa_exponent_to_ns(oa_exponent) / 1000; + void *oa_vaddr; + int i; + + do_ioctl(stream_fd, I915_PERF_IOCTL_GET_OA_BUFFER_INFO, &oa_buffer); + + igt_debug("size = %d\n", oa_buffer.size); + igt_debug("offset = %x\n", oa_buffer.offset); + + igt_assert_eq(oa_buffer.size & (oa_buffer.size - 1), 0); + igt_assert_eq(oa_status_reg(), 0); + + /* try a couple invalid mmaps */ + /* bad offsets */ + igt_assert(mmap(0, oa_buffer.size, PROT_READ, MAP_PRIVATE, + stream_fd, 0) == (void *) -1); + igt_assert(mmap(0, oa_buffer.size, PROT_READ, MAP_PRIVATE, + stream_fd, 8192) == (void *) -1); + igt_assert(mmap(0, oa_buffer.size, PROT_READ, MAP_PRIVATE, + stream_fd, 11) == (void *) -1); + + /* bad size */ + igt_assert(mmap(0, oa_buffer.size + 4096, PROT_READ, MAP_PRIVATE, + stream_fd, oa_buffer.offset) == (void *) -1); + + /* do the right thing */ + oa_vaddr = mmap(0, oa_buffer.size, PROT_READ, MAP_PRIVATE, stream_fd, oa_buffer.offset); + + /* wait for approx 100 reports */ + usleep(100 * period_us); + + do_ioctl(stream_fd, I915_PERF_IOCTL_GET_OA_BUFFER_HEAD_TAIL, &oa_ht); + + igt_debug("head = %x\n", oa_ht.head); + igt_debug("tail = %x\n", oa_ht.tail); + + reports = (uint8_t *) (oa_vaddr + oa_ht.head); + + num_reports = OA_BUFFER_DATA(oa_ht.tail, + oa_ht.head, + oa_buffer.size) / report_size; + + for (i = 0; i < num_reports; i++) { + report1 = (uint32_t *)(reports + (i * report_size)); + if (!oa_report_is_periodic(oa_exponent, report1)) + continue; + + timer_reports++; + if (timer_reports >= 2) + sanity_check_reports(report0, report1, fmt); + + report0 = report1; + } + + munmap(oa_vaddr, oa_buffer.size); +} + +static void test_mapped_oa_buffer(void) +{ + int oa_exponent = max_oa_exponent_for_period_lte(1000000); + enum drm_i915_oa_format fmt = test_set->perf_oa_format; + uint64_t properties[] = { + DRM_I915_PERF_PROP_SAMPLE_OA, true, + DRM_I915_PERF_PROP_OA_METRICS_SET, test_set->perf_oa_metrics_set, + DRM_I915_PERF_PROP_OA_FORMAT, fmt, + DRM_I915_PERF_PROP_OA_EXPONENT, oa_exponent, + + }; + struct drm_i915_perf_open_param param = { + .flags = I915_PERF_FLAG_FD_CLOEXEC, + .num_properties = sizeof(properties) / 16, + .properties_ptr = to_user_pointer(properties), + }; + + stream_fd = __perf_open(drm_fd, ¶m, false); + check_reports_from_mapped_buffer(fmt, oa_exponent); + __perf_close(stream_fd); +} + static unsigned read_i915_module_ref(void) { @@ -4936,6 +5039,9 @@ igt_main render_copy = igt_get_render_copyfunc(devid); igt_require_f(render_copy, "no render-copy function\n"); + + intel_register_access_init(&mmio_data, intel_get_pci_device(), + 0, drm_fd); } igt_subtest("non-system-wide-paranoid") @@ -5096,6 +5202,17 @@ igt_main igt_subtest("whitelisted-registers-userspace-config") test_whitelisted_registers_userspace_config(); + + igt_subtest_group { + igt_fixture { + igt_require(i915_perf_revision(drm_fd) >= 6); + } + + igt_describe("Verify mapping of oa buffer"); + igt_subtest("mapped-oa-buffer") + test_mapped_oa_buffer(); + } + igt_fixture { /* leave sysctl options in their default state... */ write_u64_file("/proc/sys/dev/i915/oa_max_sample_rate", 100000); @@ -5104,6 +5221,7 @@ igt_main if (intel_perf) intel_perf_free(intel_perf); + intel_register_access_fini(&mmio_data); close(drm_fd); } } -- 2.20.1 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev