Re: [PATCH 1/3] dma-fence: Clear fence->status during dma_fence_init()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 <daniel@xxxxxxxx> 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 <chris@xxxxxxxxxxxxxxxxxx>
> >> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
> >
> > Yeah, this looks all pretty. On the series:
> >
> > Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
> >
> FWIW, please feel free, for this series, to apply my
> 
> Reviewed-by: Sumit Semwal <sumit.semwal@xxxxxxxxxx>
> 
> > 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@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux