[PATCH i-g-t] tests/intel/xe_exec_store: Remove redundant code

Ch, Sai Gowtham sai.gowtham.ch at intel.com
Fri Jan 12 09:07:45 UTC 2024



>-----Original Message-----
>From: Kumar, Janga Rahul <janga.rahul.kumar at intel.com>
>Sent: Friday, January 12, 2024 1:19 PM
>To: igt-dev at lists.freedesktop.org; Gandi, Ramadevi <ramadevi.gandi at intel.com>;
>Kumar, Janga Rahul <janga.rahul.kumar at intel.com>; Ch, Sai Gowtham
><sai.gowtham.ch at intel.com>
>Subject: [PATCH i-g-t] tests/intel/xe_exec_store: Remove redundant code
>
>From: Janga Rahul Kumar <janga.rahul.kumar at intel.com>
>
>Remove redundant code related to exec queue creation.
>
>Signed-off-by: Janga Rahul Kumar <janga.rahul.kumar at intel.com>
>---
> tests/intel/xe_exec_store.c | 1 -
> 1 file changed, 1 deletion(-)
>
>diff --git a/tests/intel/xe_exec_store.c b/tests/intel/xe_exec_store.c index
>870394d3c..3fa8880ba 100644
>--- a/tests/intel/xe_exec_store.c
>+++ b/tests/intel/xe_exec_store.c
>@@ -148,7 +148,6 @@ static void basic_inst(int fd, int inst_type, struct
>drm_xe_engine_class_instanc
> 	else
> 		igt_assert_f(inst_type < 2, "Entered wrong inst_type.\n");
>
>-	exec_queue = xe_exec_queue_create(fd, vm, eci, 0);
> 	exec.exec_queue_id = exec_queue;
> 	exec.address = data->addr;
> 	sync.flags &= DRM_XE_SYNC_FLAG_SIGNAL;
>--
Looks good to me.
Reviewed-by: Sai Gowtham Ch <sai.gowtham.ch at intel.com>
>2.25.1



More information about the igt-dev mailing list