[Intel-gfx] [PATCH i-g-t] i915/gem_exec_parallel: Reopen the existing device

[Intel-gfx] [PATCH i-g-t] i915/gem_exec_parallel: Reopen the existing device

[Chris Wilson]

Avoid any unnecessary filtering inside drm_open_driver() by explicitly
reopening the same device.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 tests/i915/gem_exec_parallel.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/i915/gem_exec_parallel.c b/tests/i915/gem_exec_parallel.c
index bdb8e3e90..1a988b957 100644
--- a/tests/i915/gem_exec_parallel.c
+++ b/tests/i915/gem_exec_parallel.c
@@ -77,7 +77,7 @@ static void *thread(void *data)
 	pthread_mutex_unlock(t->mutex);
 
 	if (t->flags & FDS) {
-		fd = drm_open_driver(DRIVER_INTEL);
+		fd = gem_reopen_driver(t->fd);
 		gem_context_copy_engines(t->fd, 0, fd, 0);
 	} else {
 		fd = t->fd;
-- 
2.29.2

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [Intel-gfx] [PATCH i-g-t] i915/gem_exec_parallel: Reopen the existing device

[Dixit, Ashutosh]

On Wed, 04 Nov 2020 14:23:21 -0800, Chris Wilson wrote:
>
> Avoid any unnecessary filtering inside drm_open_driver() by explicitly
> reopening the same device.
>
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> ---
>  tests/i915/gem_exec_parallel.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/i915/gem_exec_parallel.c b/tests/i915/gem_exec_parallel.c
> index bdb8e3e90..1a988b957 100644
> --- a/tests/i915/gem_exec_parallel.c
> +++ b/tests/i915/gem_exec_parallel.c
> @@ -77,7 +77,7 @@ static void *thread(void *data)
>	pthread_mutex_unlock(t->mutex);
>
>	if (t->flags & FDS) {
> -		fd = drm_open_driver(DRIVER_INTEL);
> +		fd = gem_reopen_driver(t->fd);

Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx