On Wed, Aug 03, 2016 at 08:08:19PM +0100, Chris Wilson wrote: > On Wed, Aug 03, 2016 at 08:04:29PM +0200, David Herrmann wrote: > > Rather than using "struct file*", use "struct drm_file*" as tag VM tag for > > BOs. This will pave the way for "struct drm_file*" without any "struct > > file*" back-pointer. > > > > Signed-off-by: David Herrmann <dh.herrmann@xxxxxxxxx> > > Ok, the danger of untyped is having to check each and trying to spot any > missed conversions. Hm yeah ... any reason we can't just switch to struct drm_file instead? At least I can't come up with any other use case where we might supply something else, and it's always nice to enlist the compiler for a little bit of help. -Daniel > > Couldn't find a mistake or omission, > Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > -Chris > > -- > Chris Wilson, Intel Open Source Technology Centre -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
