[PATCH i-g-t v4 3/5] lib/gem_engine_topology: Fix premature break from primary find loop
Kamil Konieczny
kamil.konieczny at linux.intel.com
Thu Jul 18 12:11:31 UTC 2024
Hi Janusz,
On 2024-07-18 at 10:55:14 +0200, Janusz Krzysztofik wrote:
> When looking for a primary counterpart of a render device, we try to open
> each primary minor. When one fails, we should continue rather then break,
> since contiguity of minor numbers assigned to devices is not guaranteed.
>
> Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik at linux.intel.com>
> ---
> lib/i915/gem_engine_topology.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/i915/gem_engine_topology.c b/lib/i915/gem_engine_topology.c
> index c878e95f08..8589af5c1a 100644
> --- a/lib/i915/gem_engine_topology.c
> +++ b/lib/i915/gem_engine_topology.c
> @@ -479,7 +479,7 @@ static int __open_primary(int dir)
> sprintf(buf, "/sys/dev/char/%d:%d", major, minor);
> dir = openat(fd, buf, O_RDONLY);
> if (dir < 0)
> - break;
> + continue;
Good catch!
Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
>
> if (readlinkat(dir, "device", device, sizeof(device)) > 0 &&
> !strcmp(device, target))
> --
> 2.45.2
>
More information about the igt-dev
mailing list