Hi Daniel, On 29-Feb-2016 9:18 pm, "Daniel Vetter" <<a href="mailto:daniel@ffwll.ch">daniel@ffwll.ch</a>> wrote: > > On Tue, Feb 23, 2016 at 11:04:57AM +0900, Michel Dänzer wrote: > > On 23.02.2016 04:32, Alex Deucher wrote: > > > From: Christian König <<a href="mailto:christian.koenig@amd.com">christian.koenig@amd.com</a>> > > > > > > A fence is never later than itself. This caused a bunch of overhead for AMDGPU. > > > > > > v2: simplify check as suggested by Michel. > > > > > > Signed-off-by: Christian König <<a href="mailto:christian.koenig@amd.com">christian.koenig@amd.com</a>> > > > Reviewed-by: Michel Dänzer <<a href="mailto:michel.daenzer@amd.com">michel.daenzer@amd.com</a>> > > > Reviewed-by: Alex Deucher <<a href="mailto:alexander.deucher@amd.com">alexander.deucher@amd.com</a>> > > > Signed-off-by: Alex Deucher <<a href="mailto:alexander.deucher@amd.com">alexander.deucher@amd.com</a>> > > > --- > > > include/linux/fence.h | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/include/linux/fence.h b/include/linux/fence.h > > > index bb52201..5aa95eb 100644 > > > --- a/include/linux/fence.h > > > +++ b/include/linux/fence.h > > > @@ -292,7 +292,7 @@ static inline bool fence_is_later(struct fence *f1, struct fence *f2) > > > if (WARN_ON(f1->context != f2->context)) > > > return false; > > > > > > - return f1->seqno - f2->seqno < INT_MAX; > > > + return (int)(f1->seqno - f2->seqno) > 0; > > > } > > > > > > /** > > > > > > > According to > > > > scripts/<a href="http://get_maintainer.pl">get_maintainer.pl</a> include/linux/fence.h > > > > this patch should be sent to: > > > > Sumit Semwal <<a href="mailto:sumit.semwal@linaro.org">sumit.semwal@linaro.org</a>> (maintainer:DMA BUFFER SHARING FRAMEWORK) > > <a href="mailto:linux-media@vger.kernel.org">linux-media@vger.kernel.org</a> (open list:DMA BUFFER SHARING FRAMEWORK) > > <a href="mailto:dri-devel@lists.freedesktop.org">dri-devel@lists.freedesktop.org</a> (open list:DMA BUFFER SHARING FRAMEWORK) > > <a href="mailto:linaro-mm-sig@lists.linaro.org">linaro-mm-sig@lists.linaro.org</a> (moderated list:DMA BUFFER SHARING FRAMEWORK) > > <a href="mailto:linux-kernel@vger.kernel.org">linux-kernel@vger.kernel.org</a> (open list) > > > > I'd add at least Sumit and the linaro-mm-sig list. > > Plus Maarten Lankhorst please, and Gustavo Padovan maybe. Not sure Sumit > is still all that active on dma-buf/fence maintainership ... Am here very much, though I do apologize I haven't actively provided review comments on the fence related series. I'll improve on that for sure! Yes, it is still a good idea to add Maarten for fence stuff in the least. > -Daniel > -- > Daniel Vetter > Software Engineer, Intel Corporation > <a href="http://blog.ffwll.ch">http://blog.ffwll.ch</a> > _______________________________________________ > dri-devel mailing list > <a href="mailto:dri-devel@lists.freedesktop.org">dri-devel@lists.freedesktop.org</a> > <a href="https://lists.freedesktop.org/mailman/listinfo/dri-devel">https://lists.freedesktop.org/mailman/listinfo/dri-devel</a>