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 ... -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch
