• Chris Wilson's avatar
    drm/i915: Deny wrapping an userptr into a framebuffer · 6f6e0838
    Chris Wilson authored
    commit cc917ab4 upstream.
    
    Pinning a userptr onto the hardware raises interesting questions about
    the lifetime of such a surface as the framebuffer extends that life
    beyond the client's address space. That is the hardware will need to
    keep scanning out from the backing storage even after the client wants
    to remap its address space. As the hardware pins the backing storage,
    the userptr becomes invalid and this raises a WARN when the clients
    tries to unmap its address space. The situation can be even more
    complicated when the buffer is passed between processes, between a
    client and display server, where the lifetime and hardware access is
    even more confusing. Deny it.
    Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
    Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
    Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
    Cc: Michał Winiarski <michal.winiarski@intel.com>
    Reviewed-by: default avatarTvrtko Ursulin <tvrtko.ursulin@intel.com>
    Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
    Signed-off-by: default avatarLuis Henriques <luis.henriques@canonical.com>
    6f6e0838
i915_gem_userptr.c 19.4 KB