drm: omapdrm: Do no allocate non-scanout GEMs through DMM/TILER
authorIvaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>
Wed, 19 Jan 2022 10:23:11 +0000 (12:23 +0200)
committerTomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Mon, 28 Mar 2022 09:39:52 +0000 (12:39 +0300)
commit3ccd59f93004c369cc4c0f0939f80cb069f0fc2d
tree5ce3eca7d913e4bbc1fab6988f74f8919a828c5b
parent1948d28db6215d055930f0cbb6e300622ede5931
drm: omapdrm: Do no allocate non-scanout GEMs through DMM/TILER

On devices with DMM, all allocations are done through either DMM or TILER.
DMM/TILER being a limited resource means that such allocations will start
to fail before actual free memory is exhausted. What is even worse is that
with time DMM/TILER space gets fragmented to the point that even if we have
enough free DMM/TILER space and free memory, allocation fails because there
is no big enough free block in DMM/TILER space.

Such failures can be easily observed with OMAP xorg DDX, for example -
starting few GUI applications (so buffers for their windows are allocated)
and then rotating landscape<->portrait while closing and opening new
windows soon results in allocation failures.

Fix that by mapping buffers through DMM/TILER only when really needed,
like, for scanout buffers.

Signed-off-by: Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Link: https://patchwork.freedesktop.org/patch/msgid/1642587791-13222-4-git-send-email-ivo.g.dimitrov.75@gmail.com
drivers/gpu/drm/omapdrm/omap_gem.c