Re: [PATCH 0/7] Consider G2D hardware restrictions.

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

 



Btw, what is the hw response to invalid input (ie. bottom>top, invalid
size, etc)?

Ie. if it will just ignore the blit or raise an error irq which can be
handled sanely, it could be ok to avoid the overhead of the cmdstream
checking in the kernel.  The kernel part really just needs to ensure
that userspace can't cause security problems (read/write access to
non-gfx-buffers, or lock up the system, that sort of thing).  It
doesn't need to guarantee that the results are sensible.

BR,
-R

On Wed, Mar 13, 2013 at 5:03 AM, Inki Dae <inki.dae@xxxxxxxxxxx> wrote:
> This patch set checks the contents of g2d command list from user
> is valid or not according to G2D hardware restrictions. For now,
> G2D driver wasn't considered for them properly.
>
> For this, this patch set includes relevant code cleaups, fixups
> and adds a new function to get buffer size to the gem to be
> accessed by G2D dma.
>
> Inki Dae (1):
>   drm/exynos: Add a new function to get gem buffer size
>
> YoungJun Cho (6):
>   drm/exynos: Fix error routine to getting dma addr.
>   drm/exynos: clear node object type at gem unmap
>   drm/exynos: Fix G2D core mulfunctioning issue
>   drm/exynos: Clean up some G2D codes for readability
>   drm/exynos: Deal with g2d buffer info more efficiently
>   drm/exynos: Check g2d cmd list for g2d restrictions
>
>  drivers/gpu/drm/exynos/exynos_drm_g2d.c |  381 ++++++++++++++++++++++++++-----
>  drivers/gpu/drm/exynos/exynos_drm_gem.c |   21 ++
>  drivers/gpu/drm/exynos/exynos_drm_gem.h |    5 +
>  3 files changed, 349 insertions(+), 58 deletions(-)
>
> --
> 1.7.4.1
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://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