[igt-dev] [PATCH i-g-t] tests/i915/gem_linear_blits: Added description to the subtest

[Zbigniew Kempczyński]

On Wed, Mar 09, 2022 at 03:03:13PM +0100, Kamil Konieczny wrote:
> Dnia 2022-03-08 at 10:16:01 +0530, sai.gowtham.ch at intel.com napisał(a):
> > From: Ch Sai Gowtham <sai.gowtham.ch at intel.com>
> > 
> > Added description to subtests.
> > 
> > Cc: Melkaveri Arjun <arjun.melkaveri at intel.com>
> > Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> > Signed-off-by: Ch Sai Gowtham <sai.gowtham.ch at intel.com>
> > ---
> >  tests/i915/gem_linear_blits.c | 4 ++++
> >  1 file changed, 4 insertions(+)
> > 
> > diff --git a/tests/i915/gem_linear_blits.c b/tests/i915/gem_linear_blits.c
> > index 6504c9f2..9f049863 100644
> > --- a/tests/i915/gem_linear_blits.c
> > +++ b/tests/i915/gem_linear_blits.c
> > @@ -258,9 +258,12 @@ igt_main
> >  		count = (count + ncpus - 1) / ncpus;
> >  	}
> >  
> > +	igt_describe("Basic blitter functionality check with 2 buffers");
> >  	igt_subtest("basic")
> >  		run_test(fd, 2, do_relocs);
> >  
> > +	igt_describe("The intent is to push beyond the working GTT size to force"
> > +			" the driver to rebind the buffers");
> >  	igt_subtest("normal") {
> >  		intel_allocator_multiprocess_start();
> >  		igt_fork(child, ncpus)
> > @@ -269,6 +272,7 @@ igt_main
> >  		intel_allocator_multiprocess_stop();
> >  	}
> >  
> > +	igt_describe("Test with interrupts in between the parent process");

Little vague for me. Idea is to properly handle syscalls interrupted by 
signal delivery.

--
Zbigniew

> >  	igt_subtest("interruptible") {
> >  		intel_allocator_multiprocess_start();
> >  		igt_fork_signal_helper();
> > -- 
> > 2.35.1
> > 
> Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> 
> --
> Kamil