On Wed, Jan 29, 2014 at 10:10:47PM +0000, Chris Wilson wrote:
> On Wed, Jan 29, 2014 at 01:58:29PM -0800, bradley.d.volkin@xxxxxxxxx wrote:
> > From: Brad Volkin <bradley.d.volkin@xxxxxxxxx>
> >
> > Signed-off-by: Brad Volkin <bradley.d.volkin@xxxxxxxxx>
> > ---
> > tests/gem_exec_parse.c | 9 +++++++++
> > 1 file changed, 9 insertions(+)
> >
> > diff --git a/tests/gem_exec_parse.c b/tests/gem_exec_parse.c
> > index 9e90408..004c3bf 100644
> > --- a/tests/gem_exec_parse.c
> > +++ b/tests/gem_exec_parse.c
> > @@ -257,6 +257,15 @@ igt_main
> > -EINVAL));
> > }
> >
> > + igt_subtest("batch-without-end") {
> > + uint32_t noop[1024] = { 0 };
> > + igt_assert(
> > + exec_batch(fd, handle,
> > + noop, sizeof(noop),
> > + I915_EXEC_RENDER,
> > + -EINVAL));
>
> Cheekier would be
> uint32_t empty[] = { MI_NOOP, MI_NOOP, MI_BATCH_BUFFER_END, 0 };
> for_each_ring() {
> igt_assert(exec_batch(fd, handle, empty, sizeof(empty), ring, 0));
> igt_assert(exec_batch(fd, handle, empty, 8, ring, -EINVAL));
> }
On this subject, it should be
{ INVALID, INVALID, NOOP, NOOP, END, 0}
assert(exec(0, 4) == -EINVAL);
assert(exec(0, 8) == -EINVAL);
assert(exec(0, 12) == -EINVAL);
assert(exec(4, 8) == -EINVAL);
assert(exec(4, 12) == 0);
assert(exec(8, 12) == 0);
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
