From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: References: <20200129222456.1503424-1-chris@chris-wilson.co.uk> From: Antonio Argenziano Message-ID: <1205063d-ae84-11c3-a5ad-f50a9f5adcc3@intel.com> Date: Wed, 29 Jan 2020 14:28:21 -0800 MIME-Version: 1.0 In-Reply-To: <20200129222456.1503424-1-chris@chris-wilson.co.uk> Content-Language: en-US Subject: Re: [igt-dev] [PATCH i-g-t] i915/gem_exec_reloc: Add SIGINT injection List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" To: Chris Wilson , intel-gfx@lists.freedesktop.org Cc: igt-dev@lists.freedesktop.org List-ID: On 29/01/20 14:24, Chris Wilson wrote: > Do a pass over gem_exec_reloc where we inject lots of SIGINTs. > > Signed-off-by: Chris Wilson > Cc: Antonio Argenziano LGTM. Reviewed-by: Antonio Argenziano > --- > tests/i915/gem_exec_reloc.c | 13 +++++++++---- > 1 file changed, 9 insertions(+), 4 deletions(-) > > diff --git a/tests/i915/gem_exec_reloc.c b/tests/i915/gem_exec_reloc.c > index bc904a0ae..1aa03fba3 100644 > --- a/tests/i915/gem_exec_reloc.c > +++ b/tests/i915/gem_exec_reloc.c > @@ -379,7 +379,8 @@ static bool has_64b_reloc(int fd) > > #define NORELOC 1 > #define ACTIVE 2 > -#define HANG 4 > +#define INTERRUPTIBLE 4 > +#define HANG 8 > static void basic_reloc(int fd, unsigned before, unsigned after, unsigned flags) > { > #define OBJSZ 8192 > @@ -735,6 +736,7 @@ igt_main > { "", 0 , true}, > { "-noreloc", NORELOC, true }, > { "-active", ACTIVE, true }, > + { "-interruptible", ACTIVE | INTERRUPTIBLE }, > { "-hang", ACTIVE | HANG }, > { }, > }, *f; > @@ -762,14 +764,17 @@ igt_main > f->name) { > if ((m->before | m->after) & I915_GEM_DOMAIN_WC) > igt_require(gem_mmap__has_wc(fd)); > - basic_reloc(fd, m->before, m->after, f->flags); > + igt_while_interruptible(f->flags & INTERRUPTIBLE) > + basic_reloc(fd, m->before, m->after, f->flags); > } > } > > if (!(f->flags & NORELOC)) { > igt_subtest_f("%srange%s", > - f->basic ? "basic-" : "", f->name) > - basic_range(fd, f->flags); > + f->basic ? "basic-" : "", f->name) { > + igt_while_interruptible(f->flags & INTERRUPTIBLE) > + basic_range(fd, f->flags); > + } > } > > igt_fixture { > _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev