[igt-dev] [PATCH i-g-t 3/5] tests/gem_exec_fence: Adopt to use allocator

Zbigniew Kempczyński zbigniew.kempczynski at intel.com
Wed Aug 18 03:15:08 UTC 2021


On Tue, Aug 17, 2021 at 12:32:28PM -0700, Dixit, Ashutosh wrote:
> On Mon, 16 Aug 2021 23:31:16 -0700, Zbigniew Kempczyński wrote:
> >
> > @@ -3162,6 +3262,7 @@ igt_main
> >			igt_require(exec_has_fence_array(i915));
> >			igt_assert(has_syncobj(i915));
> >			igt_fork_hang_detector(i915);
> > +			intel_allocator_multiprocess_start();
> >		}
> >
> >		igt_subtest("invalid-fence-array")
> > @@ -3195,6 +3296,7 @@ igt_main
> >			test_syncobj_channel(i915);
> >
> >		igt_fixture {
> > +			intel_allocator_multiprocess_stop();
> 
> Not all these functions bracketed by
> intel_allocator_multiprocess_start/stop are forking, but as we have
> discussed previously intel_allocator_multiprocess_start/stop probably has
> no adverse effects even when we don't fork. Therefore this is:
> 
> Reviewed-by: Ashutosh Dixit <ashutosh.dixit at intel.com>

Thanks for review and r-b. gem_exec_fence and gem_exec_schedule will land
soon when I'll ensure there's no regression on CI (imo should be clear).

--
Zbigniew


More information about the igt-dev mailing list