[Intel-gfx] [PATCH] igt/gem_trtt: Exercise the TRTT hardware

Chris Wilson chris at chris-wilson.co.uk
Mon Jan 11 04:37:48 PST 2016


On Mon, Jan 11, 2016 at 12:32:08PM +0000, Chris Wilson wrote:
> On Sat, Jan 09, 2016 at 05:01:30PM +0530, akash.goel at intel.com wrote:
> > +static void submit_and_sync(int fd, struct drm_i915_gem_execbuffer2 *execbuf,
> > +			    uint32_t batch_buf_handle)
> > +{
> > +	gem_execbuf(fd, execbuf);
> > +	gem_sync(fd, batch_buf_handle);
> 
> The only caller of this also does its own sync. This seems irrelevant
> and serves a bad example.

Oh, I see why. Your code is broken...

...
...
...


You didn't mark the output buffers as being written.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list