Re: [PATCH i-g-t 0/3] Unify slow/combinatorial test handling

David Weinehall
2015-10-26T13:55:37Z

On Fri, Oct 23, 2015 at 02:47:57PM +0200, Daniel Vetter wrote:
> On Fri, Oct 23, 2015 at 12:58:45PM +0100, Chris Wilson wrote:
> > On Fri, Oct 23, 2015 at 02:42:33PM +0300, David Weinehall wrote:
> > > Until now we've had no unified way to handle slow/combinatorial tests.
> > > Most of the time we don't want to run slow/combinatorial tests, so this
> > > should remain the default, but when we do want to run such tests,
> > > it has been handled differently in different tests.
> > >
> > > This patch adds a --with-slow-combinatorial command line option to
> > > igt_core, changes gem_concurrent_blit and kms_frontbuffer_tracking
> > > to use this instead of their own methods, and removes gem_concurrent_all
> > > in the process, since it's now unnecessary.
> >
> > I'm not going to remember the --with-slow-combinatorial option. How
> > about just --all, or --slow?
>
> Yeah, --all as a shorthand sounds good to me.

OK, will rename it to '--all'.

Kind regards, David