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 6590C10EB87 for ; Thu, 23 Mar 2023 22:55:44 +0000 (UTC) From: Umesh Nerlige Ramappa To: igt-dev@lists.freedesktop.org Date: Thu, 23 Mar 2023 15:55:29 -0700 Message-Id: <20230323225534.3739835-26-umesh.nerlige.ramappa@intel.com> In-Reply-To: <20230323225534.3739835-1-umesh.nerlige.ramappa@intel.com> References: <20230323225534.3739835-1-umesh.nerlige.ramappa@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [igt-dev] [PATCH i-g-t v5 25/30] i915/perf: Make sanity check failures descriptive List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: Make the asserts in the sanity check descriptive. Signed-off-by: Umesh Nerlige Ramappa Reviewed-by: Kamil Konieczny --- tests/i915/perf.c | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/tests/i915/perf.c b/tests/i915/perf.c index 02d0f232..ce58a6b8 100644 --- a/tests/i915/perf.c +++ b/tests/i915/perf.c @@ -1017,7 +1017,9 @@ gen8_sanity_check_test_oa_reports(const uint32_t *oa_report0, continue; igt_debug("A40_%d: delta = %"PRIu64"\n", j, delta); - igt_assert(delta <= max_delta); + igt_assert_f(delta <= max_delta, + "A40_%d: delta = %"PRIu64", max_delta = %"PRIu64"\n", + j, delta, max_delta); } for (int j = 0; j < format.n_a64; j++) { @@ -1029,7 +1031,9 @@ gen8_sanity_check_test_oa_reports(const uint32_t *oa_report0, continue; igt_debug("A64_%d: delta = %"PRIu64"\n", format.first_a + j, delta); - igt_assert(delta <= max_delta); + igt_assert_f(delta <= max_delta, + "A64_%d: delta = %"PRIu64", max_delta = %"PRIu64"\n", + format.first_a + j, delta, max_delta); } for (int j = 0; j < format.n_a; j++) { @@ -1044,7 +1048,9 @@ gen8_sanity_check_test_oa_reports(const uint32_t *oa_report0, continue; igt_debug("A%d: delta = %"PRIu32"\n", a_id, delta); - igt_assert(delta <= max_delta); + igt_assert_f(delta <= max_delta, + "A%d: delta = %"PRIu32", max_delta = %"PRIu64"\n", + a_id, delta, max_delta); } /* The TestOa metric set defines all B counters to be a @@ -1106,7 +1112,9 @@ gen8_sanity_check_test_oa_reports(const uint32_t *oa_report0, igt_debug("C%d: delta = %"PRIu32", max_delta=%"PRIu64"\n", j, delta, max_delta); - igt_assert(delta <= max_delta); + igt_assert_f(delta <= max_delta, + "C%d: delta = %"PRIu32", max_delta = %"PRIu64"\n", + j, delta, max_delta); } } -- 2.36.1