Quoting Mika Kuoppala (2017-10-04 14:13:59)
> Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes:
>
> > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> > ---
> > tests/gem_workarounds.c | 28 +++++++++++++++++++++++-----
> > 1 file changed, 23 insertions(+), 5 deletions(-)
> >
> > diff --git a/tests/gem_workarounds.c b/tests/gem_workarounds.c
> > index 95ec250a..c252133f 100644
> > --- a/tests/gem_workarounds.c
> > +++ b/tests/gem_workarounds.c
> > @@ -164,8 +164,14 @@ static int workaround_fail_count(int fd)
> > return fail_count;
> > }
> >
> > -static void check_workarounds(int fd, enum operation op)
> > +#define CONTEXT 0x1
> > +static void check_workarounds(int fd, enum operation op, unsigned int flags)
> > {
> > + uint32_t ctx = 0;
> > +
> > + if (flags & CONTEXT)
> > + ctx = gem_context_create(fd);
> > +
> > igt_assert_eq(workaround_fail_count(fd), 0);
>
> The batch to read them will still use the default context. I would
> have assumed that you want to read the results throught your
> newly created context?
Yes, that was stupid!
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
