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

Matthew Auld matthew.auld at intel.com
Mon Jul 11 13:53:33 UTC 2022


On 11/07/2022 13:07, Nirmoy Das wrote:
> 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>

Acked-by: Matthew Auld <matthew.auld 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);


More information about the igt-dev mailing list