Hi Daniel, On 29-Feb-2016 9:18 pm, "Daniel Vetter" <daniel at ffwll.ch> 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 <christian.koenig at amd.com> > > > > > > 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 <christian.koenig at amd.com> > > > Reviewed-by: Michel Dänzer <michel.daenzer at amd.com> > > > Reviewed-by: Alex Deucher <alexander.deucher at amd.com> > > > Signed-off-by: Alex Deucher <alexander.deucher at amd.com> > > > --- > > > 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/get_maintainer.pl include/linux/fence.h > > > > this patch should be sent to: > > > > Sumit Semwal <sumit.semwal at linaro.org> (maintainer:DMA BUFFER SHARING FRAMEWORK) > > linux-media at vger.kernel.org (open list:DMA BUFFER SHARING FRAMEWORK) > > dri-devel at lists.freedesktop.org (open list:DMA BUFFER SHARING FRAMEWORK) > > linaro-mm-sig at lists.linaro.org (moderated list:DMA BUFFER SHARING FRAMEWORK) > > linux-kernel at vger.kernel.org (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 > http://blog.ffwll.ch > _______________________________________________ > dri-devel mailing list > dri-devel at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -------------- next part -------------- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160302/b909f7b7/attachment.html>
