On Wed, Jan 04, 2017 at 09:23:34PM +0530, Sumit Semwal wrote: > Hi Chris, > > Thanks for your patches! > > On 4 January 2017 at 20:40, Daniel Vetter <dan...@ffwll.ch> wrote: > > On Wed, Jan 04, 2017 at 02:12:20PM +0000, Chris Wilson wrote: > >> As the fence->status is an optional field that may be set before > >> dma_fence_signal() is called to convey that the fence completed with an > >> error, we have to ensure that it is always set to zero on initialisation > >> so that the typical use (i.e. unset) always flags a successful completion. > >> > >> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk> > >> Reviewed-by: Tvrtko Ursulin <tvrtko.ursu...@intel.com> > > > > Yeah, this looks all pretty. On the series: > > > > Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> > > > FWIW, please feel free, for this series, to apply my > > Reviewed-by: Sumit Semwal <sumit.sem...@linaro.org> > > > I'll defer to Gustavo for another pass over it and merging it to drm-misc.
It would be nice to have this in the next round of backmerges. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
