[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