Re: [RFC PATCH 2/3] drm/ipvr: drm driver for vxd392

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux