[igt-dev] [PATCH i-g-t] tests/i915_query: fix cache type iteration

Nirmoy Das nirmoy.das at intel.com
Mon Jul 11 12:07:50 UTC 2022


Iterate over each cache types otherwise this will get
stuck in infinite loop if the value starts with 0 1st bit.

Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/6249
Suggested-by: Chris Wilson <chris.p.wilson at intel.com>
Signed-off-by: Nirmoy Das <nirmoy.das at intel.com>
---
 tests/i915/i915_query.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/tests/i915/i915_query.c b/tests/i915/i915_query.c
index b33ffecff..ca500d2be 100644
--- a/tests/i915/i915_query.c
+++ b/tests/i915/i915_query.c
@@ -1348,8 +1348,7 @@ static void query_parse_and_validate_hwconfig_table(int i915)
 			if (!value)
 				igt_info("-\n");
 
-			j = 0;
-			while (value) {
+			for (j = 0; value; j++) {
 				if (value & BIT(j)) {
 					value &= ~BIT(j);
 					igt_assert(j < __INTEL_HWCONFIG_CACHE_TYPE_LIMIT);
-- 
2.35.1

Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de>
Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva  
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928



More information about the igt-dev mailing list