Sorry for the copy error. In fact, I was going to send out patches which fix it. Thanks -----Original Message----- From: Ben Widawsky [mailto:benjamin.widawsky at intel.com] Sent: Sunday, April 28, 2013 2:46 AM To: Li, Zhong Cc: intel-gfx at lists.freedesktop.org; Xiang, Haihao; Vetter, Daniel Subject: Re: [PATCH 5/8] gem_exec_nop.c: add vebox test case On Tue, Apr 23, 2013 at 03:06:45PM +0800, Zhong Li wrote: > Signed-off-by: Zhong Li <zhong.li at intel.com> > --- > tests/gem_exec_nop.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/tests/gem_exec_nop.c b/tests/gem_exec_nop.c index > 8608592..d0fa5d5 100644 > --- a/tests/gem_exec_nop.c > +++ b/tests/gem_exec_nop.c > @@ -133,7 +133,10 @@ int main(int argc, char **argv) > if (drmtest_run_subtest("blt")) > if (HAS_BLT_RING(devid)) > loop(fd, handle, I915_EXEC_BLT, "blt"); > - > + > + if (drmtest_run_subtest("vebox")) > + if (HAS_BLT_RING(devid)) > + loop(fd, handle, I915_EXEC_VEBOX, "vebox"); > > gem_close(fd, handle); > > -- > 1.7.9.5 > Patch has the same problems as the previous ones. Fixed locally. I don't mind so much that you made copy/paste mistakes since I do that ALL the time too. The whitespace mistakes however should be part of your editor setup. Please try to correct that ASAP. -- Ben Widawsky, Intel Open Source Technology Center
