From patchwork Mon Oct 16 09:05:15 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Micha=C5=82_Winiarski?= X-Patchwork-Id: 10007891 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 6331160235 for ; Mon, 16 Oct 2017 09:05:37 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 531F7283D9 for ; Mon, 16 Oct 2017 09:05:37 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 47F8F283F2; Mon, 16 Oct 2017 09:05:37 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=2.0 tests=BAYES_00, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 37086283D9 for ; Mon, 16 Oct 2017 09:05:36 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id BF6426E349; Mon, 16 Oct 2017 09:05:35 +0000 (UTC) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by gabe.freedesktop.org (Postfix) with ESMTPS id F2A096E348 for ; Mon, 16 Oct 2017 09:05:33 +0000 (UTC) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 16 Oct 2017 02:05:33 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.43,386,1503385200"; d="scan'208";a="146919146" Received: from irsmsx102.ger.corp.intel.com ([163.33.3.155]) by orsmga002.jf.intel.com with ESMTP; 16 Oct 2017 02:05:32 -0700 Received: from localhost (172.28.171.152) by IRSMSX102.ger.corp.intel.com (163.33.3.155) with Microsoft SMTP Server (TLS) id 14.3.319.2; Mon, 16 Oct 2017 10:05:32 +0100 From: =?UTF-8?q?Micha=C5=82=20Winiarski?= To: Date: Mon, 16 Oct 2017 11:05:15 +0200 Message-ID: <20171016090517.11128-2-michal.winiarski@intel.com> X-Mailer: git-send-email 2.13.5 In-Reply-To: <20171016090517.11128-1-michal.winiarski@intel.com> References: <20171016090517.11128-1-michal.winiarski@intel.com> MIME-Version: 1.0 X-Originating-IP: [172.28.171.152] Subject: [Intel-gfx] [PATCH i-g-t v2 2/4] lib/i915: Extract helpers for determining scheduler capabilities X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" X-Virus-Scanned: ClamAV using ClamSMTP Couple of tests are using either determining scheduler capabilities or pretty printing. Let's move those to helpers in lib. We can also keep the value obtained from getparam static. v2: Break the trend of expanding ioctl_wrappers Signed-off-by: Michał Winiarski Cc: Arkadiusz Hiler Cc: Chris Wilson Cc: Katarzyna Dec Cc: Petri Latvala Reviewed-by: Chris Wilson Reviewed-by: Katarzyna Dec Acked-by: Arkadiusz Hiler --- .../intel-gpu-tools/intel-gpu-tools-docs.xml | 5 + lib/Makefile.sources | 2 + lib/i915/gem_scheduler.c | 125 +++++++++++++++++++++ lib/i915/gem_scheduler.h | 37 ++++++ lib/ioctl_wrappers.h | 2 + lib/meson.build | 10 +- tests/gem_exec_nop.c | 34 +----- tests/gem_exec_schedule.c | 47 ++------ tests/gem_exec_whisper.c | 22 +--- tests/gem_sync.c | 34 +----- 10 files changed, 189 insertions(+), 129 deletions(-) create mode 100644 lib/i915/gem_scheduler.c create mode 100644 lib/i915/gem_scheduler.h diff --git a/docs/reference/intel-gpu-tools/intel-gpu-tools-docs.xml b/docs/reference/intel-gpu-tools/intel-gpu-tools-docs.xml index 04a0485c..8c14bd07 100644 --- a/docs/reference/intel-gpu-tools/intel-gpu-tools-docs.xml +++ b/docs/reference/intel-gpu-tools/intel-gpu-tools-docs.xml @@ -44,6 +44,11 @@ + + + + igt/i915 API Reference + diff --git a/lib/Makefile.sources b/lib/Makefile.sources index 965e230e..c33723f9 100644 --- a/lib/Makefile.sources +++ b/lib/Makefile.sources @@ -3,6 +3,8 @@ lib_source_list = \ drmtest.c \ drmtest.h \ i830_reg.h \ + i915/gem_scheduler.c \ + i915/gem_scheduler.h \ i915_3d.h \ i915_reg.h \ i915_pciids.h \ diff --git a/lib/i915/gem_scheduler.c b/lib/i915/gem_scheduler.c new file mode 100644 index 00000000..ad156306 --- /dev/null +++ b/lib/i915/gem_scheduler.c @@ -0,0 +1,125 @@ +/* + * Copyright © 2017 Intel Corporation + * + * Permission is hereby granted, free of charge, to any person obtaining a + * copy of this software and associated documentation files (the "Software"), + * to deal in the Software without restriction, including without limitation + * the rights to use, copy, modify, merge, publish, distribute, sublicense, + * and/or sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice (including the next + * paragraph) shall be included in all copies or substantial portions of the + * Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS + * IN THE SOFTWARE. + */ + +#include +#include +#include + +#include "igt_core.h" +#include "ioctl_wrappers.h" + +#include "i915/gem_scheduler.h" + +#define LOCAL_I915_PARAM_HAS_SCHEDULER 41 + +/** + * SECTION:gem_scheduler + * @short_description: Helpers for querying scheduler capabilities + * @title: GEM Scheduler + * + * This helper library contains functions used for getting information on + * currently used scheduling model. + */ + +/** + * gem_scheduler_capability: + * @fd: open i915 drm file descriptor + * + * Returns: Scheduler capability bitmap. + */ +unsigned gem_scheduler_capability(int fd) +{ + static int caps = -1; + + if (caps < 0) { + struct drm_i915_getparam gp; + + memset(&gp, 0, sizeof(gp)); + gp.param = LOCAL_I915_PARAM_HAS_SCHEDULER; + gp.value = ∩︀ + + caps = 0; + ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp, sizeof(gp)); + errno = 0; + } + + return caps; +} + +/** + * gem_has_scheduler: + * @fd: open i915 drm file descriptor + * + * Feature test macro to query whether the driver has scheduling capability. + */ +bool gem_scheduler_enabled(int fd) +{ + return gem_scheduler_capability(fd) & + LOCAL_I915_SCHEDULER_CAP_ENABLED; +} + +/** + * gem_has_ctx_priority: + * @fd: open i915 drm file descriptor + * + * Feature test macro to query whether the driver supports assigning custom + * priorities to contexts from userspace. + */ +bool gem_scheduler_has_ctx_priority(int fd) +{ + return gem_scheduler_capability(fd) & + LOCAL_I915_SCHEDULER_CAP_PRIORITY; +} + +/** + * gem_has_preemption: + * @fd: open i915 drm file descriptor + * + * Feature test macro to query whether the driver supports preempting active + * (currently executing on HW) workloads. + */ +bool gem_scheduler_has_preemption(int fd) +{ + return gem_scheduler_capability(fd) & + LOCAL_I915_SCHEDULER_CAP_PREEMPTION; +} + +/** + * gem_scheduler_print_capability: + * @fd: open i915 drm file descriptor + * + * Helper for pretty-printing scheduler capability. + */ +void gem_scheduler_print_capability(int fd) +{ + unsigned caps = gem_scheduler_capability(fd); + + if (!caps) + return; + + igt_info("Has kernel scheduler\n"); + if (caps & LOCAL_I915_SCHEDULER_CAP_PRIORITY) + igt_info(" - With priority sorting\n"); + if (caps & LOCAL_I915_SCHEDULER_CAP_PREEMPTION) + igt_info(" - With preemption enabled\n"); +} diff --git a/lib/i915/gem_scheduler.h b/lib/i915/gem_scheduler.h new file mode 100644 index 00000000..9fcb0266 --- /dev/null +++ b/lib/i915/gem_scheduler.h @@ -0,0 +1,37 @@ +/* + * Copyright © 2017 Intel Corporation + * + * Permission is hereby granted, free of charge, to any person obtaining a + * copy of this software and associated documentation files (the "Software"), + * to deal in the Software without restriction, including without limitation + * the rights to use, copy, modify, merge, publish, distribute, sublicense, + * and/or sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice (including the next + * paragraph) shall be included in all copies or substantial portions of the + * Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS + * IN THE SOFTWARE. + */ + +#ifndef GEM_SCHEDULER_H +#define GEM_SCHEDULER_H + +#define LOCAL_I915_SCHEDULER_CAP_ENABLED (1 << 0) +#define LOCAL_I915_SCHEDULER_CAP_PRIORITY (1 << 1) +#define LOCAL_I915_SCHEDULER_CAP_PREEMPTION (1 << 2) + +unsigned gem_scheduler_capability(int fd); +bool gem_scheduler_enabled(int fd); +bool gem_scheduler_has_ctx_priority(int fd); +bool gem_scheduler_has_preemption(int fd); +void gem_scheduler_print_capability(int fd); + +#endif /* GEM_SCHEDULER_H */ diff --git a/lib/ioctl_wrappers.h b/lib/ioctl_wrappers.h index 1663b7f8..76a4e80d 100644 --- a/lib/ioctl_wrappers.h +++ b/lib/ioctl_wrappers.h @@ -36,6 +36,8 @@ #include #include +#include "i915/gem_scheduler.h" + /** * igt_ioctl: * @fd: file descriptor diff --git a/lib/meson.build b/lib/meson.build index 89ed2723..22811743 100644 --- a/lib/meson.build +++ b/lib/meson.build @@ -2,6 +2,7 @@ lib_headers = [ 'debug.h', 'drmtest.h', 'i830_reg.h', + 'i915/gem_scheduler.h', 'i915_3d.h', 'i915_reg.h', 'i915_pciids.h', @@ -48,6 +49,7 @@ lib_headers = [ lib_sources = [ 'drmtest.c', + 'i915/gem_scheduler.c', 'igt_debugfs.c', 'igt_aux.c', 'igt_gt.c', @@ -152,13 +154,7 @@ lib_version = vcs_tag(input : 'version.h.in', output : 'version.h', lib_intermediates = [] foreach f: lib_sources - # No / in the target name - if f.contains('uwildmat') - name = 'uwildmat' - else - name = f - endif - + name = f.underscorify(f) lib = static_library('igt-' + name, [ f, lib_version ], include_directories: inc, diff --git a/tests/gem_exec_nop.c b/tests/gem_exec_nop.c index feb89ee3..8fd31e83 100644 --- a/tests/gem_exec_nop.c +++ b/tests/gem_exec_nop.c @@ -44,8 +44,6 @@ #include #include "drm.h" -#define BIT(x) (1ul << (x)) - #define LOCAL_I915_EXEC_NO_RELOC (1<<11) #define LOCAL_I915_EXEC_HANDLE_LUT (1<<12) @@ -54,10 +52,6 @@ #define ENGINE_FLAGS (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK) -#define LOCAL_PARAM_HAS_SCHEDULER 41 -#define HAS_SCHEDULER BIT(0) -#define HAS_PRIORITY BIT(1) -#define HAS_PREEMPTION BIT(2) #define LOCAL_CONTEXT_PARAM_PRIORITY 6 #define MAX_PRIO 1023 #define MIN_PRIO -1023 @@ -665,31 +659,9 @@ static void preempt(int fd, uint32_t handle, ring_name, count, elapsed(&start, &now)*1e6 / count); } -static unsigned int has_scheduler(int fd) -{ - drm_i915_getparam_t gp; - unsigned int caps = 0; - - gp.param = LOCAL_PARAM_HAS_SCHEDULER; - gp.value = (int *)∩︀ - drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp); - - if (!caps) - return 0; - - igt_info("Has kernel scheduler\n"); - if (caps & HAS_PRIORITY) - igt_info(" - With priority sorting\n"); - if (caps & HAS_PREEMPTION) - igt_info(" - With preemption enabled\n"); - - return caps; -} - igt_main { const struct intel_execution_engine *e; - unsigned int sched_caps = 0; uint32_t handle = 0; int device = -1; @@ -699,7 +671,7 @@ igt_main device = drm_open_driver(DRIVER_INTEL); igt_require_gem(device); gem_show_submission_method(device); - sched_caps = has_scheduler(device); + gem_scheduler_print_capability(device); handle = gem_create(device, 4096); gem_write(device, handle, 0, &bbe, sizeof(bbe)); @@ -746,8 +718,8 @@ igt_main igt_subtest_group { igt_fixture { - igt_require(sched_caps & HAS_PRIORITY); - igt_require(sched_caps & HAS_PREEMPTION); + igt_require(gem_scheduler_has_ctx_priority(device)); + igt_require(gem_scheduler_has_preemption(device)); } for (e = intel_execution_engines; e->name; e++) { diff --git a/tests/gem_exec_schedule.c b/tests/gem_exec_schedule.c index f5c849d0..66e24244 100644 --- a/tests/gem_exec_schedule.c +++ b/tests/gem_exec_schedule.c @@ -33,12 +33,6 @@ #include "igt_rand.h" #include "igt_sysfs.h" -#define BIT(x) (1ul << (x)) - -#define LOCAL_PARAM_HAS_SCHEDULER 41 -#define HAS_SCHEDULER BIT(0) -#define HAS_PRIORITY BIT(1) -#define HAS_PREEMPTION BIT(2) #define LOCAL_CONTEXT_PARAM_PRIORITY 6 #define LO 0 @@ -70,11 +64,6 @@ static void ctx_set_priority(int fd, uint32_t ctx, int prio) igt_assert_eq(__ctx_set_priority(fd, ctx, prio), 0); } -static void ctx_has_priority(int fd) -{ - igt_require(__ctx_set_priority(fd, 0, MAX_PRIO) == 0); -} - static void store_dword(int fd, uint32_t ctx, unsigned ring, uint32_t target, uint32_t offset, uint32_t value, uint32_t cork, unsigned write_domain) @@ -979,31 +968,9 @@ static void test_pi_ringfull(int fd, unsigned int engine) munmap(result, 4096); } -static unsigned int has_scheduler(int fd) -{ - drm_i915_getparam_t gp; - unsigned int caps = 0; - - gp.param = LOCAL_PARAM_HAS_SCHEDULER; - gp.value = (int *)∩︀ - drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp); - - if (!caps) - return 0; - - igt_info("Has kernel scheduler\n"); - if (caps & HAS_PRIORITY) - igt_info(" - With priority sorting\n"); - if (caps & HAS_PREEMPTION) - igt_info(" - With preemption enabled\n"); - - return caps; -} - igt_main { const struct intel_execution_engine *e; - unsigned int sched_caps = 0; int fd = -1; igt_skip_on_simulation(); @@ -1011,7 +978,7 @@ igt_main igt_fixture { fd = drm_open_driver_master(DRIVER_INTEL); gem_show_submission_method(fd); - sched_caps = has_scheduler(fd); + gem_scheduler_print_capability(fd); igt_require_gem(fd); gem_require_mmap_wc(fd); igt_fork_hang_detector(fd); @@ -1033,8 +1000,8 @@ igt_main igt_subtest_group { igt_fixture { - igt_require(sched_caps & HAS_SCHEDULER); - ctx_has_priority(fd); + igt_require(gem_scheduler_enabled(fd)); + igt_require(gem_scheduler_has_ctx_priority(fd)); } igt_subtest("smoketest-all") @@ -1062,7 +1029,7 @@ igt_main igt_subtest_group { igt_fixture { - igt_require(sched_caps & HAS_PREEMPTION); + igt_require(gem_scheduler_has_preemption(fd)); } igt_subtest_f("preempt-%s", e->name) @@ -1095,8 +1062,8 @@ igt_main igt_subtest_group { igt_fixture { - igt_require(sched_caps & HAS_SCHEDULER); - ctx_has_priority(fd); + igt_require(gem_scheduler_enabled(fd)); + igt_require(gem_scheduler_has_ctx_priority(fd)); /* need separate rings */ igt_require(gem_has_execlists(fd)); @@ -1106,7 +1073,7 @@ igt_main igt_subtest_group { igt_fixture { gem_require_ring(fd, e->exec_id | e->flags); - igt_require(sched_caps & HAS_PREEMPTION); + igt_require(gem_scheduler_has_preemption(fd)); } igt_subtest_f("pi-ringfull-%s", e->name) diff --git a/tests/gem_exec_whisper.c b/tests/gem_exec_whisper.c index e7f7b1f1..5ff8f396 100644 --- a/tests/gem_exec_whisper.c +++ b/tests/gem_exec_whisper.c @@ -191,21 +191,8 @@ static void fini_hang(struct hang *h) close(h->fd); } -#define LOCAL_PARAM_HAS_SCHEDULER 41 #define LOCAL_CONTEXT_PARAM_PRIORITY 6 -static bool __has_scheduler(int fd) -{ - drm_i915_getparam_t gp; - int has = -1; - - gp.param = LOCAL_PARAM_HAS_SCHEDULER; - gp.value = &has; - drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp); - - return has > 0; -} - static int __ctx_set_priority(int fd, uint32_t ctx, int prio) { struct local_i915_gem_context_param param; @@ -250,13 +237,8 @@ static void whisper(int fd, unsigned engine, unsigned flags) int debugfs; if (flags & PRIORITY) { - int __fd = drm_open_driver(DRIVER_INTEL); - bool has_scheduler = __has_scheduler(__fd); - bool ctx_has_priority = - __ctx_set_priority(__fd, 0, 1) == 0; - close(__fd); - - igt_require(has_scheduler && ctx_has_priority); + igt_require(gem_scheduler_enabled(fd)); + igt_require(gem_scheduler_has_ctx_priority(fd)); } debugfs = igt_debugfs_dir(fd); diff --git a/tests/gem_sync.c b/tests/gem_sync.c index f0c5d76f..fda70344 100644 --- a/tests/gem_sync.c +++ b/tests/gem_sync.c @@ -27,18 +27,12 @@ #include "igt.h" #include "igt_sysfs.h" -#define BIT(x) (1ul << (x)) - #define LOCAL_I915_EXEC_NO_RELOC (1<<11) #define LOCAL_I915_EXEC_HANDLE_LUT (1<<12) #define LOCAL_I915_EXEC_BSD_SHIFT (13) #define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT) -#define LOCAL_PARAM_HAS_SCHEDULER 41 -#define HAS_SCHEDULER BIT(0) -#define HAS_PRIORITY BIT(1) -#define HAS_PREEMPTION BIT(2) #define LOCAL_CONTEXT_PARAM_PRIORITY 6 #define MAX_PRIO 1023 #define MIN_PRIO -1023 @@ -807,32 +801,10 @@ preempt(int fd, unsigned ring, int num_children, int timeout) gem_context_destroy(fd, ctx[0]); } -static unsigned int has_scheduler(int fd) -{ - drm_i915_getparam_t gp; - unsigned int caps = 0; - - gp.param = LOCAL_PARAM_HAS_SCHEDULER; - gp.value = (int *)∩︀ - drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp); - - if (!caps) - return 0; - - igt_info("Has kernel scheduler\n"); - if (caps & HAS_PRIORITY) - igt_info(" - With priority sorting\n"); - if (caps & HAS_PREEMPTION) - igt_info(" - With preemption enabled\n"); - - return caps; -} - igt_main { const struct intel_execution_engine *e; const int ncpus = sysconf(_SC_NPROCESSORS_ONLN); - unsigned int sched_caps = 0; int fd = -1; igt_skip_on_simulation(); @@ -841,7 +813,7 @@ igt_main fd = drm_open_driver(DRIVER_INTEL); igt_require_gem(fd); gem_show_submission_method(fd); - sched_caps = has_scheduler(fd); + gem_scheduler_print_capability(fd); igt_fork_hang_detector(fd); } @@ -886,8 +858,8 @@ igt_main igt_subtest_group { igt_fixture { - igt_require(sched_caps & HAS_PRIORITY); - igt_require(sched_caps & HAS_PREEMPTION); + igt_require(gem_scheduler_has_ctx_priority(fd)); + igt_require(gem_scheduler_has_preemption(fd)); } igt_subtest("preempt-all")