Hi
On Tue, Sep 23, 2014 at 3:46 PM, Daniel Vetter
wrote:
> The only user I could dig out was i915 back when ums+gem was still a
> thing. But we've just very much killed that, and even when someone
> screams about that we should resurrect that with a special hack
> (wrapping drm_gem_mmap) in i915, not in the core code.
Agreed! Drivers should just add the hacks themselves if there is
user-space that needs it. Same as for patch #1:
Reviewed-by: David Herrmann
Thanks
David
> So good riddance to another entry point of the legacy buffer mapping
> code.
>
> Signed-off-by: Daniel Vetter
> ---
> drivers/gpu/drm/drm_gem.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/drm_gem.c b/drivers/gpu/drm/drm_gem.c
> index eb5dd67153e4..f9af78541505 100644
> --- a/drivers/gpu/drm/drm_gem.c
> +++ b/drivers/gpu/drm/drm_gem.c
> @@ -888,7 +888,7 @@ int drm_gem_mmap(struct file *filp, struct vm_area_struct
> *vma)
>vma_pages(vma));
> if (!node) {
> mutex_unlock(>struct_mutex);
> - return drm_mmap(filp, vma);
> + return -EINVAL;
> } else if (!drm_vma_node_is_allowed(node, filp)) {
> mutex_unlock(>struct_mutex);
> return -EACCES;
> --
> 2.1.0
>
> ___
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel