raspi/linux
1
0
Fork 1
mirror of https://github.com/raspberrypi/linux.git synced 2025-12-06 01:49:46 +00:00
Code Issues Projects Releases Wiki Activity

drm: vc4: Block swiotlb bounce buffers being imported as dmabuf

Browse Source 
The dmabuf import already checks that the backing buffer is contiguous
and rejects it if it isn't. vc4 also requires that the buffer is
in the bottom 1GB of RAM, and this is all correctly defined via
dma-ranges.

However the kernel silently uses swiotlb to bounce dma buffers
around if they are in the wrong region. This relies on dma sync
functions to be called in order to copy the data to/from the
bounce buffer.

DRM is based on all memory allocations being coherent with the
GPU so that any updates to a framebuffer will be acted on without
the need for any additional update. This is fairly fundamentally
incompatible with needing to call dma_sync_ to handle the bounce
buffer copies, and therefore we have to detect and reject mappings
that use bounce buffers.

Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>
This commit is contained in:
Dave Stevenson
2023-11-22 19:17:44 +00:00
committed by Dom Cobley
parent d3fd7de91e
commit 810b31ee8a
1 changed files with 18 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
drivers/gpu/drm/vc4/vc4_drv.c
View File

@@ -30,6 +30,7 @@
#include <linux/of_device.h>
#include <linux/platform_device.h>
#include <linux/pm_runtime.h>
#include <linux/dma-direct.h>
#include <drm/clients/drm_client_setup.h>
#include <drm/drm_atomic_helper.h>
@@ -176,6 +177,19 @@ static void vc4_close(struct drm_device *dev, struct drm_file *file)
kfree(vc4file);
}
static struct drm_gem_object *
vc4_prime_import_sg_table(struct drm_device *dev,
struct dma_buf_attachment *attach,
struct sg_table *sgt)
{
phys_addr_t phys = dma_to_phys(dev->dev, sg_dma_address(sgt->sgl));
if (swiotlb_find_pool(dev->dev, phys))
return ERR_PTR(-EINVAL);
return drm_gem_dma_prime_import_sg_table(dev, attach, sgt);
}
DEFINE_DRM_GEM_FOPS(vc4_drm_fops);
static const struct drm_ioctl_desc vc4_drm_ioctls[] = {
@@ -212,8 +226,9 @@ const struct drm_driver vc4_drm_driver = {
.gem_create_object = vc4_create_object,
DRM_GEM_DMA_DRIVER_OPS_WITH_DUMB_CREATE(vc4_bo_dumb_create),
DRM_FBDEV_DMA_DRIVER_OPS,
.dumb_create = vc4_bo_dumb_create,
.gem_prime_import_sg_table = vc4_prime_import_sg_table,
.ioctls = vc4_drm_ioctls,
.num_ioctls = ARRAY_SIZE(vc4_drm_ioctls),
@@ -235,8 +250,9 @@ const struct drm_driver vc5_drm_driver = {
.debugfs_init = vc4_debugfs_init,
#endif
DRM_GEM_DMA_DRIVER_OPS_WITH_DUMB_CREATE(vc5_dumb_create),
DRM_FBDEV_DMA_DRIVER_OPS,
.dumb_create = vc5_dumb_create,
.gem_prime_import_sg_table = vc4_prime_import_sg_table,
.fops = &vc4_drm_fops,