On Thu, Oct 16, 2014 at 03:47:08PM +0200, David Herrmann wrote: > Hi > > On Thu, Oct 16, 2014 at 3:39 PM, Cheng, Yao <yao.cheng@xxxxxxxxx> wrote: > > Accepted :) I will update the patch to implement the mmap interface and remove the legacy MMAP_IOCTL. > > BTW I didn't see a field to get mmap_offset in struct drm_gem_open, I guess something like a new "GET_MMAP_OFFSET_IOCTL" need be added to support mapping flinked/primed BO, is it? > > Yes. There is currently no generic interface to get mmap offsets for > any gem bo (because some of them may not be mapped from user-space or > require extra parameters). > > So, either support dumb-buffers (if you don't need any advanced > parameters like tiling or cache constraints), or add an > MMAP_OFFSET_IOCTL like you proposed (and most drivers do so). Dave Airlie _really_ doesn't like it if userspace abuses the dumb buffer interface for rendering. That really should only be used for sw rendering for boot splash and similar. Some armsoc drivers don't really follow that though. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
