From: Jonathan Cavitt <jonathan.cavitt@intel.com>
To: igt-dev@lists.freedesktop.org
Cc: jonathan.cavitt@intel.com
Subject: [igt-dev] [PATCH i-g-t v2 2/2] tests/i915_query.c : Add query-l3-bank-count subtest
Date: Fri, 22 Sep 2023 07:11:18 -0700 [thread overview]
Message-ID: <20230922141118.3821111-3-jonathan.cavitt@intel.com> (raw)
In-Reply-To: <20230922141118.3821111-1-jonathan.cavitt@intel.com>
From: Janga Rahul Kumar <janga.rahul.kumar@intel.com>
Add a subtest to query L3 Bank count for each engine.
Signed-off-by: Janga Rahul Kumar <janga.rahul.kumar@intel.com>
Signed-off-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
---
tests/intel/i915_query.c | 135 +++++++++++++++++++++++++++++++++++++++
1 file changed, 135 insertions(+)
diff --git a/tests/intel/i915_query.c b/tests/intel/i915_query.c
index f97379b83b..4b9a217979 100644
--- a/tests/intel/i915_query.c
+++ b/tests/intel/i915_query.c
@@ -25,6 +25,7 @@
#include "intel_hwconfig_types.h"
#include "i915/gem.h"
#include "i915/gem_create.h"
+#include "igt_sysfs.h"
#include <limits.h>
/**
@@ -121,6 +122,18 @@
* SUBTEST: test-query-geometry-subslices
* Description: Test DRM_I915_QUERY_GEOMETRY_SUBSLICES query
* Feature: gem_core
+ *
+ * SUBTEST: query-l3-bank-count
+ * Description: Test DRM_I915_QUERY_L3BANK_COUNT query
+ * Feature: gem_core
+ *
+ * SUBTEST: query-l3-bank-count-invalid
+ * Description: Test DRM_I915_QUERY_L3BANK_COUNT query fails when an invalid engine is passed
+ * Feature: gem_core
+ *
+ * SUBTEST: query-l3-bank-count-support
+ * Description: Test DRM_I915_QUERY_L3BANK_COUNT query fails when it is not supported
+ * Feature: gem_core
*/
IGT_TEST_DESCRIPTION("Testing the i915 query uAPI.");
@@ -1474,6 +1487,107 @@ static void query_parse_and_validate_hwconfig_table(int i915)
free(data);
}
+static int __get_l3_count(int i915, const struct i915_engine_class_instance engine, uint64_t *out)
+{
+ struct drm_i915_query_item item;
+
+ memset(&item, 0, sizeof(item));
+ item.query_id = DRM_I915_QUERY_L3BANK_COUNT;
+ item.flags = ((engine.engine_instance << 8) | engine.engine_class);
+ item.data_ptr = to_user_pointer(out);
+
+ i915_query_items(i915, &item, 1);
+ return item.length < 0 ? item.length : 0;
+}
+
+static bool query_l3_count_supported(int i915)
+{
+ struct i915_engine_class_instance *engines;
+ unsigned int engines_count;
+ uint32_t class;
+ uint64_t val;
+ int ret;
+
+ class = 0xFF;
+ engines = gem_list_engines(i915, 1u << 0, class, &engines_count);
+ if (!engines) {
+ igt_warn("Failed to get the engines list per tile\n");
+ return false;
+ }
+
+ ret = __get_l3_count(i915, engines[0], &val);
+ free(engines);
+
+ return ret == 0;
+}
+
+static void test_l3_count_support(int i915)
+{
+ int devid = intel_get_drm_devid(i915);
+ bool supported = intel_graphics_ver(devid) >= IP_VER(12, 0);
+
+ igt_assert(supported == query_l3_count_supported(i915));
+}
+
+static void test_l3_count_invalid(int i915)
+{
+ struct i915_engine_class_instance engine = {
+ .engine_class = 0xDEAD,
+ .engine_instance = 0xBEEF
+ };
+ uint64_t val;
+
+ igt_assert_eq(__get_l3_count(i915, engine, &val), -EINVAL);
+}
+
+static void test_l3_count(int i915)
+{
+ struct i915_engine_class_instance *engines;
+ uint32_t class;
+ unsigned int engines_count;
+ uint64_t expected_bank_count;
+ int num_gts;
+
+ class = 0xFF; // checking for all 8 classes
+ expected_bank_count = 0;
+
+ num_gts = igt_sysfs_get_num_gt(i915);
+
+ for (int gt = 0; gt < num_gts; gt++) {
+ engines = gem_list_engines(i915, 1u << gt, class, &engines_count);
+
+ igt_fail_on_f(!engines, "Failed to get the engines list per tile");
+
+ for (int i = 0 ; i < engines_count ; i++) {
+ uint64_t l3_count;
+
+ igt_assert_eq(__get_l3_count(i915, engines[i], &l3_count), 0);
+
+ if (IS_METEORLAKE(i915)) // ARROWLAKE (ARL) to be added
+ igt_assert((l3_count%2) == 0);
+ else if (IS_PONTEVECCHIO(i915))
+ igt_assert((l3_count%4) == 0);
+ else if (IS_DG2(i915))
+ igt_assert((l3_count%8) == 0);
+
+ igt_debug("class=%u instance=%u l3 bank count=%lu\n",
+ engines[i].engine_class,
+ engines[i].engine_instance,
+ l3_count);
+
+ if (i == 0) {
+ expected_bank_count = l3_count;
+ continue;
+ }
+
+ igt_assert_eq(l3_count, expected_bank_count);
+
+ }
+
+ free(engines);
+ }
+}
+
igt_main
{
int fd = -1;
@@ -1574,4 +1688,25 @@ igt_main
igt_fixture {
drm_close_driver(fd);
}
+
+ igt_describe("Test to check DRM_I915_QUERY_L3BANK_COUNT support.");
+ igt_subtest("query-l3-bank-count-support") {
+ test_l3_count_support(fd);
+ }
+
+ igt_subtest_group {
+ igt_fixture {
+ igt_require(query_l3_count_supported(fd));
+ }
+
+ igt_describe("Basic test to query l3 bank count");
+ igt_subtest("query-l3-bank-count") {
+ test_l3_count(fd);
+ }
+
+ igt_describe("Negative test for DRM_I915_QUERY_L3BANK_COUNT");
+ igt_subtest("query-l3-bank-count-invalid") {
+ test_l3_count_invalid(fd);
+ }
+ }
}
--
2.25.1
next prev parent reply other threads:[~2023-09-22 14:22 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-09-22 14:11 [igt-dev] [PATCH i-g-t v2 0/2] intel/i915_query: Add query-l3-bank-count subtest Jonathan Cavitt
2023-09-22 14:11 ` [igt-dev] [PATCH i-g-t v2 1/2] drm-uapi: Add DRM_I915_QUERY_L3BANK_COUNT query Jonathan Cavitt
2023-09-25 8:34 ` Kamil Konieczny
2023-09-22 14:11 ` Jonathan Cavitt [this message]
2023-09-22 16:49 ` [igt-dev] ✓ CI.xeBAT: success for intel/i915_query: Add query-l3-bank-count subtest Patchwork
2023-09-22 16:54 ` [igt-dev] ✓ Fi.CI.BAT: " Patchwork
2023-09-23 18:26 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230922141118.3821111-3-jonathan.cavitt@intel.com \
--to=jonathan.cavitt@intel.com \
--cc=igt-dev@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox