On Thu, Nov 02, 2017 at 10:03:34PM +0200, Ville Syrjala wrote: > From: Chris Wilson <ch...@chris-wilson.co.uk> > > In order to silent sparse in dma_fence_get_rcu_safe(), we need to mark
s/silent/silence/ On the series (assuming sparse is indeed happy now, I didn't check that): Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> > the incoming fence object as being RCU protected and not the pointer to > the object. > > Cc: Dave Airlie <airl...@redhat.com> > Cc: Jason Ekstrand <ja...@jlekstrand.net> > Cc: linaro-mm-...@lists.linaro.org > Cc: linux-media@vger.kernel.org > Cc: Alex Deucher <alexander.deuc...@amd.com> > Cc: Christian König <christian.koe...@amd.com> > Cc: Sumit Semwal <sumit.sem...@linaro.org> > Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk> > --- > include/linux/dma-fence.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/linux/dma-fence.h b/include/linux/dma-fence.h > index efdabbb64e3c..4c008170fe65 100644 > --- a/include/linux/dma-fence.h > +++ b/include/linux/dma-fence.h > @@ -242,7 +242,7 @@ static inline struct dma_fence *dma_fence_get_rcu(struct > dma_fence *fence) > * The caller is required to hold the RCU read lock. > */ > static inline struct dma_fence * > -dma_fence_get_rcu_safe(struct dma_fence * __rcu *fencep) > +dma_fence_get_rcu_safe(struct dma_fence __rcu **fencep) > { > do { > struct dma_fence *fence; > -- > 2.13.6 > > _______________________________________________ > dri-devel mailing list > dri-de...@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch