[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