On Thu, Nov 02, 2017 at 10:03:34PM +0200, Ville Syrjala wrote: > From: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > 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.vetter@xxxxxxxx> > the incoming fence object as being RCU protected and not the pointer to > the object. > > Cc: Dave Airlie <airlied@xxxxxxxxxx> > Cc: Jason Ekstrand <jason@xxxxxxxxxxxxxx> > Cc: linaro-mm-sig@xxxxxxxxxxxxxxxx > Cc: linux-media@xxxxxxxxxxxxxxx > Cc: Alex Deucher <alexander.deucher@xxxxxxx> > Cc: Christian König <christian.koenig@xxxxxxx> > Cc: Sumit Semwal <sumit.semwal@xxxxxxxxxx> > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > --- > 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-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch