Re: [PATCH] drm/todo: Fallout from v3d review

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

 



Daniel Vetter <daniel.vetter@xxxxxxxx> writes:

> Bunch of ideas from Eric and me on what we could do to make gem gpu
> rendering drivers a notch simpler to type.
>
> Cc: Eric Anholt <eric@xxxxxxxxxx>
> Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx>
> diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
> index f4d0b3476d9c..d1fe865e6d2e 100644
> --- a/Documentation/gpu/todo.rst
> +++ b/Documentation/gpu/todo.rst
> @@ -212,6 +212,24 @@ probably use drm_fb_helper_fbdev_teardown().
>  
>  Contact: Maintainer of the driver you plan to convert
>  
> +Clean up mmap forwarding
> +------------------------
> +
> +A lot of drivers forward gem mmap calls to dma-buf mmap for imported buffers.
> +And also a lot of them forward dma-buf mmap to the gem mmap implemenations.
> +Would be great to refactor this all into a set of small common helpers.

"implementations"

Reviewed-by: Eric Anholt <eric@xxxxxxxxxx>

> +
> +Contact: Daniel Vetter
> +
> +Put a reservation_object into drm_gem_object
> +--------------------------------------------
> +
> +This would remove the need for the ->gem_prime_res_obj callback. It would also
> +allow us to implement generic helpers for waiting for a bo, allowing for quite a
> +bit of refactoring in the various wait ioctl implemenations.
> +
> +Contact: Daniel Vetter
> +
>  idr_init_base()
>  ---------------
>  
> -- 
> 2.16.2

Attachment: signature.asc
Description: PGP signature

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux