From: Dave Stevenson <dave.stevenson@raspberrypi.com>
To: Nicolas Saenz Julienne <nsaenzjulienne@suse.de>
Cc: "Laurent Pinchart" <laurent.pinchart@ideasonboard.com>,
"Linux Media Mailing List" <linux-media@vger.kernel.org>,
"Dave Stevenson" <dave.stevenson@raspberrypi.org>,
"Kieran Bingham" <kieran.bingham@ideasonboard.com>,
"Jacopo Mondi" <jacopo@jmondi.org>,
"Niklas Söderlund" <niklas.soderlund@ragnatech.se>,
"Naushir Patuck" <naush@raspberrypi.com>
Subject: Re: [PATCH v2 06/34] staging: vc04_services: Add new vc-sm-cma driver
Date: Wed, 6 May 2020 20:24:38 +0100 [thread overview]
Message-ID: <CAPY8ntB3RCwzCzj+v0QEQp19f-X7tgfhFZ_DK3-LUiWXGK0eNA@mail.gmail.com> (raw)
In-Reply-To: <4e42a593f16614e913613150a90e851acbfcaa8c.camel@suse.de>
Hi Nicolas
On Wed, 6 May 2020 at 19:04, Nicolas Saenz Julienne
<nsaenzjulienne@suse.de> wrote:
>
> Hi Laurent, Dave,
>
> On Mon, 2020-05-04 at 12:25 +0300, Laurent Pinchart wrote:
> > From: Dave Stevenson <dave.stevenson@raspberrypi.org>
> >
> > Add Broadcom VideoCore Shared Memory support.
> >
> > This new driver allows contiguous memory blocks to be imported
> > into the VideoCore VPU memory map, and manages the lifetime of
> > those objects, only releasing the source dmabuf once the VPU has
> > confirmed it has finished with it.
> >
>
> I'm still digesting all this, but a question came up, who is using the
> ioctls?
We have a userspace library that uses it [1].
It is used by things like MMAL to share buffers between the VPU and
ARM, rather than having to get VCHI to copy all the data between
mirrored buffers.
I think what has happened here is that Laurent has picked up the
version of the driver from the top of our downstream kernel tree.
For libcamera and the ISP driver, we need a significantly smaller
feature set, basically import of dmabufs only, no allocations or cache
management. For the ISP driver it's mainly dmabuf import from
videobuf2 for the image buffers, but there's also a need to pass in
lens shading tables which are relatively large. With a small amount of
rework in libcamera, we can make it so that we use dma-buf heaps to do
the allocation, and pass in a dmabuf fd to the ISP driver to then map
onto the VPU. That removes all the ioctls handling from this driver.
Downstream we do have other use cases that want to be able to do other
functions on shared memory, but that too should be reworkable into
using dma-buf heaps for allocations, and vcsm only handles importing
dmabufs via an ioctl. All that can be hidden away in the vcsm library,
so applications don't care.
We've also got some legacy code kicking around, as there was
originally a version of the driver that mapped the VPU's memory blocks
to the ARM. That's why the vcsm library has two code paths through
almost every function - one for each driver.
Laurent: What's your view? Halt the review this particular patch for
now and rework, or try and get this all integrated?
Mainline obviously already has dma-buf heaps merged, whilst I have a
PR cherry-picking it back into our downstream 5.4. The main reason it
hasn't been merged is that I haven't had a test case to prove it
works. The rework should be relatively simple, but will need small
updates to both libcamera and ISP driver.
Dave
[1] https://github.com/raspberrypi/userland/tree/master/host_applications/linux/libs/sm
> Regards,
> Nicolas
>
> > Driver upported from the RaspberryPi BSP at revision:
> > 890691d1c996 ("staging: vc04_services: Fix vcsm overflow bug when
> > counting transactions")
> > forward ported to recent mainline kernel version.
> >
> > Signed-off-by: Naushir Patuck <naush@raspberrypi.com>
> > Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.org>
> > Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
> > ---
> > drivers/staging/vc04_services/Kconfig | 2 +
> > drivers/staging/vc04_services/Makefile | 1 +
> > .../include/linux/broadcom/vc_sm_cma_ioctl.h | 114 ++
> > .../staging/vc04_services/vc-sm-cma/Kconfig | 10 +
> > .../staging/vc04_services/vc-sm-cma/Makefile | 13 +
> > drivers/staging/vc04_services/vc-sm-cma/TODO | 1 +
> > .../staging/vc04_services/vc-sm-cma/vc_sm.c | 1732
> > +++++++++++++++++
> > .../staging/vc04_services/vc-sm-cma/vc_sm.h | 84 +
> > .../vc04_services/vc-sm-cma/vc_sm_cma_vchi.c | 505 +++++
> > .../vc04_services/vc-sm-cma/vc_sm_cma_vchi.h | 63 +
> > .../vc04_services/vc-sm-cma/vc_sm_defs.h | 300 +++
> > .../vc04_services/vc-sm-cma/vc_sm_knl.h | 28 +
> > 12 files changed, 2853 insertions(+)
> > create mode 100644
> > drivers/staging/vc04_services/include/linux/broadcom/vc_sm_cma_ioctl.
> > h
> > create mode 100644 drivers/staging/vc04_services/vc-sm-cma/Kconfig
> > create mode 100644 drivers/staging/vc04_services/vc-sm-cma/Makefile
> > create mode 100644 drivers/staging/vc04_services/vc-sm-cma/TODO
> > create mode 100644 drivers/staging/vc04_services/vc-sm-cma/vc_sm.c
> > create mode 100644 drivers/staging/vc04_services/vc-sm-cma/vc_sm.h
> > create mode 100644 drivers/staging/vc04_services/vc-sm-
> > cma/vc_sm_cma_vchi.c
> > create mode 100644 drivers/staging/vc04_services/vc-sm-
> > cma/vc_sm_cma_vchi.h
> > create mode 100644 drivers/staging/vc04_services/vc-sm-
> > cma/vc_sm_defs.h
> > create mode 100644 drivers/staging/vc04_services/vc-sm-
> > cma/vc_sm_knl.h
> >
> > diff --git a/drivers/staging/vc04_services/Kconfig
> > b/drivers/staging/vc04_services/Kconfig
> > index 6baf9dd57f1f..b9f1f019ebd8 100644
> > --- a/drivers/staging/vc04_services/Kconfig
> > +++ b/drivers/staging/vc04_services/Kconfig
> > @@ -23,5 +23,7 @@ source "drivers/staging/vc04_services/bcm2835-
> > audio/Kconfig"
> >
> > source "drivers/staging/vc04_services/bcm2835-camera/Kconfig"
> >
> > +source "drivers/staging/vc04_services/vc-sm-cma/Kconfig"
> > +
> > endif
> >
> > diff --git a/drivers/staging/vc04_services/Makefile
> > b/drivers/staging/vc04_services/Makefile
> > index 54d9e2f31916..6e1abf494c1a 100644
> > --- a/drivers/staging/vc04_services/Makefile
> > +++ b/drivers/staging/vc04_services/Makefile
> > @@ -12,6 +12,7 @@ vchiq-objs := \
> >
> > obj-$(CONFIG_SND_BCM2835) += bcm2835-audio/
> > obj-$(CONFIG_VIDEO_BCM2835) += bcm2835-camera/
> > +obj-$(CONFIG_BCM_VC_SM_CMA) += vc-sm-cma/
> >
> > ccflags-y += -D__VCCOREVER__=0x04000000
> >
> > diff --git
> > a/drivers/staging/vc04_services/include/linux/broadcom/vc_sm_cma_ioct
> > l.h
> > b/drivers/staging/vc04_services/include/linux/broadcom/vc_sm_cma_ioct
> > l.h
> > new file mode 100644
> > index 000000000000..107460ad1be3
> > --- /dev/null
> > +++
> > b/drivers/staging/vc04_services/include/linux/broadcom/vc_sm_cma_ioct
> > l.h
> > @@ -0,0 +1,114 @@
> > +/* SPDX-License-Identifier: GPL-2.0 */
> > +
> > +/*
> > + * Copyright 2019 Raspberry Pi (Trading) Ltd. All rights reserved.
> > + *
> > + * Based on vmcs_sm_ioctl.h Copyright Broadcom Corporation.
> > + */
> > +
> > +#ifndef __VC_SM_CMA_IOCTL_H
> > +#define __VC_SM_CMA_IOCTL_H
> > +
> > +/* ---- Include Files ----------------------------------------------
> > ------ */
> > +
> > +#if defined(__KERNEL__)
> > +#include <linux/types.h> /* Needed for standard types */
> > +#else
> > +#include <stdint.h>
> > +#endif
> > +
> > +#include <linux/ioctl.h>
> > +
> > +/* ---- Constants and Types ----------------------------------------
> > ------ */
> > +
> > +#define VC_SM_CMA_RESOURCE_NAME 32
> > +#define VC_SM_CMA_RESOURCE_NAME_DEFAULT "sm-host-resource"
> > +
> > +/* Type define used to create unique IOCTL number */
> > +#define VC_SM_CMA_MAGIC_TYPE 'J'
> > +
> > +/* IOCTL commands on /dev/vc-sm-cma */
> > +enum vc_sm_cma_cmd_e {
> > + VC_SM_CMA_CMD_ALLOC = 0x5A, /* Start at 0x5A arbitrarily
> > */
> > +
> > + VC_SM_CMA_CMD_IMPORT_DMABUF,
> > +
> > + VC_SM_CMA_CMD_CLEAN_INVALID2,
> > +
> > + VC_SM_CMA_CMD_LAST /* Do not delete */
> > +};
> > +
> > +/* Cache type supported, conveniently matches the user space
> > definition in
> > + * user-vcsm.h.
> > + */
> > +enum vc_sm_cma_cache_e {
> > + VC_SM_CMA_CACHE_NONE,
> > + VC_SM_CMA_CACHE_HOST,
> > + VC_SM_CMA_CACHE_VC,
> > + VC_SM_CMA_CACHE_BOTH,
> > +};
> > +
> > +/* IOCTL Data structures */
> > +struct vc_sm_cma_ioctl_alloc {
> > + /* user -> kernel */
> > + __u32 size;
> > + __u32 num;
> > + __u32 cached; /* enum vc_sm_cma_cache_e */
> > + __u32 pad;
> > + __u8 name[VC_SM_CMA_RESOURCE_NAME];
> > +
> > + /* kernel -> user */
> > + __s32 handle;
> > + __u32 vc_handle;
> > + __u64 dma_addr;
> > +};
> > +
> > +struct vc_sm_cma_ioctl_import_dmabuf {
> > + /* user -> kernel */
> > + __s32 dmabuf_fd;
> > + __u32 cached; /* enum vc_sm_cma_cache_e */
> > + __u8 name[VC_SM_CMA_RESOURCE_NAME];
> > +
> > + /* kernel -> user */
> > + __s32 handle;
> > + __u32 vc_handle;
> > + __u32 size;
> > + __u32 pad;
> > + __u64 dma_addr;
> > +};
> > +
> > +/*
> > + * Cache functions to be set to struct
> > vc_sm_cma_ioctl_clean_invalid2
> > + * invalidate_mode.
> > + */
> > +#define VC_SM_CACHE_OP_NOP 0x00
> > +#define VC_SM_CACHE_OP_INV 0x01
> > +#define VC_SM_CACHE_OP_CLEAN 0x02
> > +#define VC_SM_CACHE_OP_FLUSH 0x03
> > +
> > +struct vc_sm_cma_ioctl_clean_invalid2 {
> > + __u32 op_count;
> > + __u32 pad;
> > + struct vc_sm_cma_ioctl_clean_invalid_block {
> > + __u32 invalidate_mode;
> > + __u32 block_count;
> > + void * __user start_address;
> > + __u32 block_size;
> > + __u32 inter_block_stride;
> > + } s[0];
> > +};
> > +
> > +/* IOCTL numbers */
> > +#define VC_SM_CMA_IOCTL_MEM_ALLOC\
> > + _IOR(VC_SM_CMA_MAGIC_TYPE, VC_SM_CMA_CMD_ALLOC,\
> > + struct vc_sm_cma_ioctl_alloc)
> > +
> > +#define VC_SM_CMA_IOCTL_MEM_IMPORT_DMABUF\
> > + _IOR(VC_SM_CMA_MAGIC_TYPE, VC_SM_CMA_CMD_IMPORT_DMABUF,\
> > + struct vc_sm_cma_ioctl_import_dmabuf)
> > +
> > +#define VC_SM_CMA_IOCTL_MEM_CLEAN_INVALID2\
> > + _IOR(VC_SM_CMA_MAGIC_TYPE, VC_SM_CMA_CMD_CLEAN_INVALID2,\
> > + struct vc_sm_cma_ioctl_clean_invalid2)
> > +
> > +#endif /* __VC_SM_CMA_IOCTL_H */
> > diff --git a/drivers/staging/vc04_services/vc-sm-cma/Kconfig
> > b/drivers/staging/vc04_services/vc-sm-cma/Kconfig
> > new file mode 100644
> > index 000000000000..5ac115da6b49
> > --- /dev/null
> > +++ b/drivers/staging/vc04_services/vc-sm-cma/Kconfig
> > @@ -0,0 +1,10 @@
> > +config BCM_VC_SM_CMA
> > + bool "VideoCore Shared Memory (CMA) driver"
> > + select BCM2835_VCHIQ
> > + select RBTREE
> > + select DMA_SHARED_BUFFER
> > + help
> > + Say Y here to enable the shared memory interface that
> > + supports sharing dmabufs with VideoCore.
> > + This operates over the VCHIQ interface to a service
> > + running on VideoCore.
> > diff --git a/drivers/staging/vc04_services/vc-sm-cma/Makefile
> > b/drivers/staging/vc04_services/vc-sm-cma/Makefile
> > new file mode 100644
> > index 000000000000..77d173694fbf
> > --- /dev/null
> > +++ b/drivers/staging/vc04_services/vc-sm-cma/Makefile
> > @@ -0,0 +1,13 @@
> > +ccflags-y += \
> > + -I$(srctree)/$(src)/../ \
> > + -I$(srctree)/$(src)/../interface/vchi \
> > + -I$(srctree)/$(src)/../interface/vchiq_arm\
> > + -I$(srctree)/$(src)/../include
> > +
> > +ccflags-y += \
> > + -D__VCCOREVER__=0
> > +
> > +vc-sm-cma-$(CONFIG_BCM_VC_SM_CMA) := \
> > + vc_sm.o vc_sm_cma_vchi.o
> > +
> > +obj-$(CONFIG_BCM_VC_SM_CMA) += vc-sm-cma.o
> > diff --git a/drivers/staging/vc04_services/vc-sm-cma/TODO
> > b/drivers/staging/vc04_services/vc-sm-cma/TODO
> > new file mode 100644
> > index 000000000000..ac9b5f8a7389
> > --- /dev/null
> > +++ b/drivers/staging/vc04_services/vc-sm-cma/TODO
> > @@ -0,0 +1 @@
> > +No currently outstanding tasks except some clean-up.
> > diff --git a/drivers/staging/vc04_services/vc-sm-cma/vc_sm.c
> > b/drivers/staging/vc04_services/vc-sm-cma/vc_sm.c
> > new file mode 100644
> > index 000000000000..cd5fb561debb
> > --- /dev/null
> > +++ b/drivers/staging/vc04_services/vc-sm-cma/vc_sm.c
> > @@ -0,0 +1,1732 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * VideoCore Shared Memory driver using CMA.
> > + *
> > + * Copyright: 2018, Raspberry Pi (Trading) Ltd
> > + * Dave Stevenson <dave.stevenson@raspberrypi.org>
> > + *
> > + * Based on vmcs_sm driver from Broadcom Corporation for some API,
> > + * and taking some code for buffer allocation and dmabuf handling
> > from
> > + * videobuf2.
> > + *
> > + *
> > + * This driver has 3 main uses:
> > + * 1) Allocating buffers for the kernel or userspace that can be
> > shared with the
> > + * VPU.
> > + * 2) Importing dmabufs from elsewhere for sharing with the VPU.
> > + * 3) Allocating buffers for use by the VPU.
> > + *
> > + * In the first and second cases the native handle is a dmabuf.
> > Releasing the
> > + * resource inherently comes from releasing the dmabuf, and this
> > will trigger
> > + * unmapping on the VPU. The underlying allocation and our buffer
> > structure are
> > + * retained until the VPU has confirmed that it has finished with
> > it.
> > + *
> > + * For the VPU allocations the VPU is responsible for triggering the
> > release,
> > + * and therefore the released message decrements the dma_buf
> > refcount (with the
> > + * VPU mapping having already been marked as released).
> > + */
> > +
> > +/* ---- Include Files ----------------------------------------------
> > ------- */
> > +#include <linux/cdev.h>
> > +#include <linux/device.h>
> > +#include <linux/debugfs.h>
> > +#include <linux/dma-mapping.h>
> > +#include <linux/dma-buf.h>
> > +#include <linux/errno.h>
> > +#include <linux/fs.h>
> > +#include <linux/kernel.h>
> > +#include <linux/list.h>
> > +#include <linux/miscdevice.h>
> > +#include <linux/module.h>
> > +#include <linux/mm.h>
> > +#include <linux/of_device.h>
> > +#include <linux/platform_device.h>
> > +#include <linux/proc_fs.h>
> > +#include <linux/slab.h>
> > +#include <linux/seq_file.h>
> > +#include <linux/syscalls.h>
> > +#include <linux/types.h>
> > +#include <asm/cacheflush.h>
> > +
> > +#include "vchiq_connected.h"
> > +#include "vc_sm_cma_vchi.h"
> > +
> > +#include "vc_sm.h"
> > +#include "vc_sm_knl.h"
> > +#include <linux/broadcom/vc_sm_cma_ioctl.h>
> > +
> > +/* ---- Private Constants and Types --------------------------------
> > ------- */
> > +
> > +#define DEVICE_NAME "vcsm-cma"
> > +#define DEVICE_MINOR 0
> > +
> > +#define VC_SM_RESOURCE_NAME_DEFAULT "sm-host-resource"
> > +
> > +#define VC_SM_DIR_ROOT_NAME "vcsm-cma"
> > +#define VC_SM_STATE "state"
> > +
> > +/* Private file data associated with each opened device. */
> > +struct vc_sm_privdata_t {
> > + pid_t pid; /* PID of creator. */
> > +
> > + int restart_sys; /* Tracks restart on interrupt. */
> > + enum vc_sm_msg_type int_action; /* Interrupted action. */
> > + u32 int_trans_id; /* Interrupted transaction. */
> > +};
> > +
> > +typedef int (*VC_SM_SHOW) (struct seq_file *s, void *v);
> > +struct sm_pde_t {
> > + VC_SM_SHOW show; /* Debug fs function hookup. */
> > + struct dentry *dir_entry; /* Debug fs directory entry. */
> > + void *priv_data; /* Private data */
> > +};
> > +
> > +/* Global state information. */
> > +struct sm_state_t {
> > + struct platform_device *pdev;
> > +
> > + struct miscdevice misc_dev;
> > +
> > + struct sm_instance *sm_handle; /* Handle for videocore
> > service. */
> > +
> > + spinlock_t kernelid_map_lock; /* Spinlock protecting
> > kernelid_map */
> > + struct idr kernelid_map;
> > +
> > + struct mutex map_lock; /* Global map lock. */
> > + struct list_head buffer_list; /* List of buffer. */
> > +
> > + struct vc_sm_privdata_t *data_knl; /* Kernel internal data
> > tracking. */
> > + struct vc_sm_privdata_t *vpu_allocs; /* All allocations from
> > the VPU */
> > + struct dentry *dir_root; /* Debug fs entries root. */
> > + struct sm_pde_t dir_state; /* Debug fs entries state
> > sub-tree. */
> > +
> > + bool require_released_callback; /* VPU will send a released
> > msg when it
> > + * has finished with a
> > resource.
> > + */
> > + u32 int_trans_id; /* Interrupted transaction. */
> > +};
> > +
> > +struct vc_sm_dma_buf_attachment {
> > + struct device *dev;
> > + struct sg_table sg_table;
> > + struct list_head list;
> > + enum dma_data_direction dma_dir;
> > +};
> > +
> > +/* ---- Private Variables ------------------------------------------
> > ----- */
> > +
> > +static struct sm_state_t *sm_state;
> > +static int sm_inited;
> > +
> > +/* ---- Private Function Prototypes --------------------------------
> > ------ */
> > +
> > +/* ---- Private Functions ------------------------------------------
> > ------ */
> > +
> > +static int get_kernel_id(struct vc_sm_buffer *buffer)
> > +{
> > + int handle;
> > +
> > + spin_lock(&sm_state->kernelid_map_lock);
> > + handle = idr_alloc(&sm_state->kernelid_map, buffer, 0, 0,
> > GFP_KERNEL);
> > + spin_unlock(&sm_state->kernelid_map_lock);
> > +
> > + return handle;
> > +}
> > +
> > +static struct vc_sm_buffer *lookup_kernel_id(int handle)
> > +{
> > + return idr_find(&sm_state->kernelid_map, handle);
> > +}
> > +
> > +static void free_kernel_id(int handle)
> > +{
> > + spin_lock(&sm_state->kernelid_map_lock);
> > + idr_remove(&sm_state->kernelid_map, handle);
> > + spin_unlock(&sm_state->kernelid_map_lock);
> > +}
> > +
> > +static int vc_sm_cma_seq_file_show(struct seq_file *s, void *v)
> > +{
> > + struct sm_pde_t *sm_pde;
> > +
> > + sm_pde = (struct sm_pde_t *)(s->private);
> > +
> > + if (sm_pde && sm_pde->show)
> > + sm_pde->show(s, v);
> > +
> > + return 0;
> > +}
> > +
> > +static int vc_sm_cma_single_open(struct inode *inode, struct file
> > *file)
> > +{
> > + return single_open(file, vc_sm_cma_seq_file_show, inode-
> > >i_private);
> > +}
> > +
> > +static const struct file_operations vc_sm_cma_debug_fs_fops = {
> > + .open = vc_sm_cma_single_open,
> > + .read = seq_read,
> > + .llseek = seq_lseek,
> > + .release = single_release,
> > +};
> > +
> > +static int vc_sm_cma_global_state_show(struct seq_file *s, void *v)
> > +{
> > + struct vc_sm_buffer *resource = NULL;
> > + int resource_count = 0;
> > +
> > + if (!sm_state)
> > + return 0;
> > +
> > + seq_printf(s, "\nVC-ServiceHandle %p\n", sm_state-
> > >sm_handle);
> > +
> > + /* Log all applicable mapping(s). */
> > +
> > + mutex_lock(&sm_state->map_lock);
> > + seq_puts(s, "\nResources\n");
> > + if (!list_empty(&sm_state->buffer_list)) {
> > + list_for_each_entry(resource, &sm_state->buffer_list,
> > + global_buffer_list) {
> > + resource_count++;
> > +
> > + seq_printf(s, "\nResource %p\n",
> > + resource);
> > + seq_printf(s, " NAME %s\n",
> > + resource->name);
> > + seq_printf(s, " SIZE %zu\n",
> > + resource->size);
> > + seq_printf(s, " DMABUF %p\n",
> > + resource->dma_buf);
> > + if (resource->imported) {
> > + seq_printf(s,
> > " ATTACH %p\n",
> > + resource->import.attach);
> > + seq_printf(s,
> > " SGT %p\n",
> > + resource->import.sgt);
> > + } else {
> > + seq_printf(s,
> > " SGT %p\n",
> > + resource->alloc.sg_table);
> > + }
> > + seq_printf(s, " DMA_ADDR %pad\n",
> > + &resource->dma_addr);
> > + seq_printf(s,
> > " VC_HANDLE %08x\n",
> > + resource->vc_handle);
> > + seq_printf(s, " VC_MAPPING %d\n",
> > + resource->vpu_state);
> > + }
> > + }
> > + seq_printf(s, "\n\nTotal resource count: %d\n\n",
> > resource_count);
> > +
> > + mutex_unlock(&sm_state->map_lock);
> > +
> > + return 0;
> > +}
> > +
> > +/*
> > + * Adds a buffer to the private data list which tracks all the
> > allocated
> > + * data.
> > + */
> > +static void vc_sm_add_resource(struct vc_sm_privdata_t *privdata,
> > + struct vc_sm_buffer *buffer)
> > +{
> > + mutex_lock(&sm_state->map_lock);
> > + list_add(&buffer->global_buffer_list, &sm_state->buffer_list);
> > + mutex_unlock(&sm_state->map_lock);
> > +
> > + pr_debug("[%s]: added buffer %p (name %s, size %zu)\n",
> > + __func__, buffer, buffer->name, buffer->size);
> > +}
> > +
> > +/*
> > + * Cleans up imported dmabuf.
> > + */
> > +static void vc_sm_clean_up_dmabuf(struct vc_sm_buffer *buffer)
> > +{
> > + if (!buffer->imported)
> > + return;
> > +
> > + /* Handle cleaning up imported dmabufs */
> > + mutex_lock(&buffer->lock);
> > + if (buffer->import.sgt) {
> > + dma_buf_unmap_attachment(buffer->import.attach,
> > + buffer->import.sgt,
> > + DMA_BIDIRECTIONAL);
> > + buffer->import.sgt = NULL;
> > + }
> > + if (buffer->import.attach) {
> > + dma_buf_detach(buffer->dma_buf, buffer->import.attach);
> > + buffer->import.attach = NULL;
> > + }
> > + mutex_unlock(&buffer->lock);
> > +}
> > +
> > +/*
> > + * Instructs VPU to decrement the refcount on a buffer.
> > + */
> > +static void vc_sm_vpu_free(struct vc_sm_buffer *buffer)
> > +{
> > + if (buffer->vc_handle && buffer->vpu_state == VPU_MAPPED) {
> > + struct vc_sm_free_t free = { buffer->vc_handle, 0 };
> > + int status = vc_sm_cma_vchi_free(sm_state->sm_handle,
> > &free,
> > + &sm_state->int_trans_id);
> > + if (status != 0 && status != -EINTR) {
> > + pr_err("[%s]: failed to free memory on
> > videocore (status: %u, trans_id: %u)\n",
> > + __func__, status, sm_state-
> > >int_trans_id);
> > + }
> > +
> > + if (sm_state->require_released_callback) {
> > + /* Need to wait for the VPU to confirm the
> > free. */
> > +
> > + /* Retain a reference on this until the VPU has
> > + * released it
> > + */
> > + buffer->vpu_state = VPU_UNMAPPING;
> > + } else {
> > + buffer->vpu_state = VPU_NOT_MAPPED;
> > + buffer->vc_handle = 0;
> > + }
> > + }
> > +}
> > +
> > +/*
> > + * Release an allocation.
> > + * All refcounting is done via the dma buf object.
> > + *
> > + * Must be called with the mutex held. The function will either
> > release the
> > + * mutex (if defering the release) or destroy it. The caller must
> > therefore not
> > + * reuse the buffer on return.
> > + */
> > +static void vc_sm_release_resource(struct vc_sm_buffer *buffer)
> > +{
> > + pr_debug("[%s]: buffer %p (name %s, size %zu), imported %u\n",
> > + __func__, buffer, buffer->name, buffer->size,
> > + buffer->imported);
> > +
> > + if (buffer->vc_handle) {
> > + /* We've sent the unmap request but not had the
> > response. */
> > + pr_debug("[%s]: Waiting for VPU unmap response on
> > %p\n",
> > + __func__, buffer);
> > + goto defer;
> > + }
> > + if (buffer->in_use) {
> > + /* dmabuf still in use - we await the release */
> > + pr_debug("[%s]: buffer %p is still in use\n", __func__,
> > buffer);
> > + goto defer;
> > + }
> > +
> > + /* Release the allocation (whether imported dmabuf or CMA
> > allocation) */
> > + if (buffer->imported) {
> > + if (buffer->import.dma_buf)
> > + dma_buf_put(buffer->import.dma_buf);
> > + else
> > + pr_err("%s: Imported dmabuf already been put
> > for buf %p\n",
> > + __func__, buffer);
> > + buffer->import.dma_buf = NULL;
> > + } else {
> > + dma_free_coherent(&sm_state->pdev->dev, buffer->size,
> > + buffer->cookie, buffer->dma_addr);
> > + }
> > +
> > + /* Free our buffer. Start by removing it from the list */
> > + mutex_lock(&sm_state->map_lock);
> > + list_del(&buffer->global_buffer_list);
> > + mutex_unlock(&sm_state->map_lock);
> > +
> > + pr_debug("%s: Release our allocation - done\n", __func__);
> > + mutex_unlock(&buffer->lock);
> > +
> > + mutex_destroy(&buffer->lock);
> > +
> > + kfree(buffer);
> > + return;
> > +
> > +defer:
> > + mutex_unlock(&buffer->lock);
> > +}
> > +
> > +/* Create support for private data tracking. */
> > +static struct vc_sm_privdata_t *vc_sm_cma_create_priv_data(pid_t id)
> > +{
> > + char alloc_name[32];
> > + struct vc_sm_privdata_t *file_data = NULL;
> > +
> > + /* Allocate private structure. */
> > + file_data = kzalloc(sizeof(*file_data), GFP_KERNEL);
> > +
> > + if (!file_data)
> > + return NULL;
> > +
> > + snprintf(alloc_name, sizeof(alloc_name), "%d", id);
> > +
> > + file_data->pid = id;
> > +
> > + return file_data;
> > +}
> > +
> > +/* Dma buf operations for use with our own allocations */
> > +
> > +static int vc_sm_dma_buf_attach(struct dma_buf *dmabuf,
> > + struct dma_buf_attachment *attachment)
> > +
> > +{
> > + struct vc_sm_dma_buf_attachment *a;
> > + struct sg_table *sgt;
> > + struct vc_sm_buffer *buf = dmabuf->priv;
> > + struct scatterlist *rd, *wr;
> > + int ret, i;
> > +
> > + a = kzalloc(sizeof(*a), GFP_KERNEL);
> > + if (!a)
> > + return -ENOMEM;
> > +
> > + pr_debug("%s dmabuf %p attachment %p\n", __func__, dmabuf,
> > attachment);
> > +
> > + mutex_lock(&buf->lock);
> > +
> > + INIT_LIST_HEAD(&a->list);
> > +
> > + sgt = &a->sg_table;
> > +
> > + /* Copy the buf->base_sgt scatter list to the attachment, as we
> > can't
> > + * map the same scatter list to multiple attachments at the
> > same time.
> > + */
> > + ret = sg_alloc_table(sgt, buf->alloc.sg_table->orig_nents,
> > GFP_KERNEL);
> > + if (ret) {
> > + kfree(a);
> > + return -ENOMEM;
> > + }
> > +
> > + rd = buf->alloc.sg_table->sgl;
> > + wr = sgt->sgl;
> > + for (i = 0; i < sgt->orig_nents; ++i) {
> > + sg_set_page(wr, sg_page(rd), rd->length, rd->offset);
> > + rd = sg_next(rd);
> > + wr = sg_next(wr);
> > + }
> > +
> > + a->dma_dir = DMA_NONE;
> > + attachment->priv = a;
> > +
> > + list_add(&a->list, &buf->attachments);
> > + mutex_unlock(&buf->lock);
> > +
> > + return 0;
> > +}
> > +
> > +static void vc_sm_dma_buf_detach(struct dma_buf *dmabuf,
> > + struct dma_buf_attachment *attachment)
> > +{
> > + struct vc_sm_dma_buf_attachment *a = attachment->priv;
> > + struct vc_sm_buffer *buf = dmabuf->priv;
> > + struct sg_table *sgt;
> > +
> > + pr_debug("%s dmabuf %p attachment %p\n", __func__, dmabuf,
> > attachment);
> > + if (!a)
> > + return;
> > +
> > + sgt = &a->sg_table;
> > +
> > + /* release the scatterlist cache */
> > + if (a->dma_dir != DMA_NONE)
> > + dma_unmap_sg(attachment->dev, sgt->sgl, sgt-
> > >orig_nents,
> > + a->dma_dir);
> > + sg_free_table(sgt);
> > +
> > + mutex_lock(&buf->lock);
> > + list_del(&a->list);
> > + mutex_unlock(&buf->lock);
> > +
> > + kfree(a);
> > +}
> > +
> > +static struct sg_table *vc_sm_map_dma_buf(struct dma_buf_attachment
> > *attachment,
> > + enum dma_data_direction
> > direction)
> > +{
> > + struct vc_sm_dma_buf_attachment *a = attachment->priv;
> > + /* stealing dmabuf mutex to serialize map/unmap operations */
> > + struct mutex *lock = &attachment->dmabuf->lock;
> > + struct sg_table *table;
> > +
> > + mutex_lock(lock);
> > + pr_debug("%s attachment %p\n", __func__, attachment);
> > + table = &a->sg_table;
> > +
> > + /* return previously mapped sg table */
> > + if (a->dma_dir == direction) {
> > + mutex_unlock(lock);
> > + return table;
> > + }
> > +
> > + /* release any previous cache */
> > + if (a->dma_dir != DMA_NONE) {
> > + dma_unmap_sg(attachment->dev, table->sgl, table-
> > >orig_nents,
> > + a->dma_dir);
> > + a->dma_dir = DMA_NONE;
> > + }
> > +
> > + /* mapping to the client with new direction */
> > + table->nents = dma_map_sg(attachment->dev, table->sgl,
> > + table->orig_nents, direction);
> > + if (!table->nents) {
> > + pr_err("failed to map scatterlist\n");
> > + mutex_unlock(lock);
> > + return ERR_PTR(-EIO);
> > + }
> > +
> > + a->dma_dir = direction;
> > + mutex_unlock(lock);
> > +
> > + pr_debug("%s attachment %p\n", __func__, attachment);
> > + return table;
> > +}
> > +
> > +static void vc_sm_unmap_dma_buf(struct dma_buf_attachment
> > *attachment,
> > + struct sg_table *table,
> > + enum dma_data_direction direction)
> > +{
> > + pr_debug("%s attachment %p\n", __func__, attachment);
> > + dma_unmap_sg(attachment->dev, table->sgl, table->nents,
> > direction);
> > +}
> > +
> > +static int vc_sm_dmabuf_mmap(struct dma_buf *dmabuf, struct
> > vm_area_struct *vma)
> > +{
> > + struct vc_sm_buffer *buf = dmabuf->priv;
> > + int ret;
> > +
> > + pr_debug("%s dmabuf %p, buf %p, vm_start %08lX\n", __func__,
> > dmabuf,
> > + buf, vma->vm_start);
> > +
> > + mutex_lock(&buf->lock);
> > +
> > + /* now map it to userspace */
> > + vma->vm_pgoff = 0;
> > +
> > + ret = dma_mmap_coherent(&sm_state->pdev->dev, vma, buf->cookie,
> > + buf->dma_addr, buf->size);
> > +
> > + if (ret) {
> > + pr_err("Remapping memory failed, error: %d\n", ret);
> > + return ret;
> > + }
> > +
> > + vma->vm_flags |= VM_DONTEXPAND | VM_DONTDUMP;
> > +
> > + mutex_unlock(&buf->lock);
> > +
> > + if (ret)
> > + pr_err("%s: failure mapping buffer to userspace\n",
> > + __func__);
> > +
> > + return ret;
> > +}
> > +
> > +static void vc_sm_dma_buf_release(struct dma_buf *dmabuf)
> > +{
> > + struct vc_sm_buffer *buffer;
> > +
> > + if (!dmabuf)
> > + return;
> > +
> > + buffer = (struct vc_sm_buffer *)dmabuf->priv;
> > +
> > + mutex_lock(&buffer->lock);
> > +
> > + pr_debug("%s dmabuf %p, buffer %p\n", __func__, dmabuf,
> > buffer);
> > +
> > + buffer->in_use = 0;
> > +
> > + /* Unmap on the VPU */
> > + vc_sm_vpu_free(buffer);
> > + pr_debug("%s vpu_free done\n", __func__);
> > +
> > + /* Unmap our dma_buf object (the vc_sm_buffer remains until
> > released
> > + * on the VPU).
> > + */
> > + vc_sm_clean_up_dmabuf(buffer);
> > + pr_debug("%s clean_up dmabuf done\n", __func__);
> > +
> > + /* buffer->lock will be destroyed by vc_sm_release_resource if
> > finished
> > + * with, otherwise unlocked. Do NOT unlock here.
> > + */
> > + vc_sm_release_resource(buffer);
> > + pr_debug("%s done\n", __func__);
> > +}
> > +
> > +static int vc_sm_dma_buf_begin_cpu_access(struct dma_buf *dmabuf,
> > + enum dma_data_direction
> > direction)
> > +{
> > + struct vc_sm_buffer *buf;
> > + struct vc_sm_dma_buf_attachment *a;
> > +
> > + if (!dmabuf)
> > + return -EFAULT;
> > +
> > + buf = dmabuf->priv;
> > + if (!buf)
> > + return -EFAULT;
> > +
> > + mutex_lock(&buf->lock);
> > +
> > + list_for_each_entry(a, &buf->attachments, list) {
> > + dma_sync_sg_for_cpu(a->dev, a->sg_table.sgl,
> > + a->sg_table.nents, direction);
> > + }
> > + mutex_unlock(&buf->lock);
> > +
> > + return 0;
> > +}
> > +
> > +static int vc_sm_dma_buf_end_cpu_access(struct dma_buf *dmabuf,
> > + enum dma_data_direction
> > direction)
> > +{
> > + struct vc_sm_buffer *buf;
> > + struct vc_sm_dma_buf_attachment *a;
> > +
> > + if (!dmabuf)
> > + return -EFAULT;
> > + buf = dmabuf->priv;
> > + if (!buf)
> > + return -EFAULT;
> > +
> > + mutex_lock(&buf->lock);
> > +
> > + list_for_each_entry(a, &buf->attachments, list) {
> > + dma_sync_sg_for_device(a->dev, a->sg_table.sgl,
> > + a->sg_table.nents, direction);
> > + }
> > + mutex_unlock(&buf->lock);
> > +
> > + return 0;
> > +}
> > +
> > +static const struct dma_buf_ops dma_buf_ops = {
> > + .map_dma_buf = vc_sm_map_dma_buf,
> > + .unmap_dma_buf = vc_sm_unmap_dma_buf,
> > + .mmap = vc_sm_dmabuf_mmap,
> > + .release = vc_sm_dma_buf_release,
> > + .attach = vc_sm_dma_buf_attach,
> > + .detach = vc_sm_dma_buf_detach,
> > + .begin_cpu_access = vc_sm_dma_buf_begin_cpu_access,
> > + .end_cpu_access = vc_sm_dma_buf_end_cpu_access,
> > +};
> > +
> > +/* Dma_buf operations for chaining through to an imported dma_buf */
> > +
> > +static
> > +int vc_sm_import_dma_buf_attach(struct dma_buf *dmabuf,
> > + struct dma_buf_attachment *attachment)
> > +{
> > + struct vc_sm_buffer *buf = dmabuf->priv;
> > +
> > + if (!buf->imported)
> > + return -EINVAL;
> > + return buf->import.dma_buf->ops->attach(buf->import.dma_buf,
> > + attachment);
> > +}
> > +
> > +static
> > +void vc_sm_import_dma_buf_detatch(struct dma_buf *dmabuf,
> > + struct dma_buf_attachment
> > *attachment)
> > +{
> > + struct vc_sm_buffer *buf = dmabuf->priv;
> > +
> > + if (!buf->imported)
> > + return;
> > + buf->import.dma_buf->ops->detach(buf->import.dma_buf,
> > attachment);
> > +}
> > +
> > +static
> > +struct sg_table *vc_sm_import_map_dma_buf(struct dma_buf_attachment
> > *attachment,
> > + enum dma_data_direction
> > direction)
> > +{
> > + struct vc_sm_buffer *buf = attachment->dmabuf->priv;
> > +
> > + if (!buf->imported)
> > + return NULL;
> > + return buf->import.dma_buf->ops->map_dma_buf(attachment,
> > + direction);
> > +}
> > +
> > +static
> > +void vc_sm_import_unmap_dma_buf(struct dma_buf_attachment
> > *attachment,
> > + struct sg_table *table,
> > + enum dma_data_direction direction)
> > +{
> > + struct vc_sm_buffer *buf = attachment->dmabuf->priv;
> > +
> > + if (!buf->imported)
> > + return;
> > + buf->import.dma_buf->ops->unmap_dma_buf(attachment, table,
> > direction);
> > +}
> > +
> > +static
> > +int vc_sm_import_dmabuf_mmap(struct dma_buf *dmabuf, struct
> > vm_area_struct *vma)
> > +{
> > + struct vc_sm_buffer *buf = dmabuf->priv;
> > +
> > + pr_debug("%s: mmap dma_buf %p, buf %p, imported db %p\n",
> > __func__,
> > + dmabuf, buf, buf->import.dma_buf);
> > + if (!buf->imported) {
> > + pr_err("%s: mmap dma_buf %p- not an imported buffer\n",
> > + __func__, dmabuf);
> > + return -EINVAL;
> > + }
> > + return buf->import.dma_buf->ops->mmap(buf->import.dma_buf,
> > vma);
> > +}
> > +
> > +static
> > +void vc_sm_import_dma_buf_release(struct dma_buf *dmabuf)
> > +{
> > + struct vc_sm_buffer *buf = dmabuf->priv;
> > +
> > + pr_debug("%s: Relasing dma_buf %p\n", __func__, dmabuf);
> > + mutex_lock(&buf->lock);
> > + if (!buf->imported)
> > + return;
> > +
> > + buf->in_use = 0;
> > +
> > + vc_sm_vpu_free(buf);
> > +
> > + vc_sm_release_resource(buf);
> > +}
> > +
> > +static
> > +int vc_sm_import_dma_buf_begin_cpu_access(struct dma_buf *dmabuf,
> > + enum dma_data_direction
> > direction)
> > +{
> > + struct vc_sm_buffer *buf = dmabuf->priv;
> > +
> > + if (!buf->imported)
> > + return -EINVAL;
> > + return buf->import.dma_buf->ops->begin_cpu_access(buf-
> > >import.dma_buf,
> > + direction);
> > +}
> > +
> > +static
> > +int vc_sm_import_dma_buf_end_cpu_access(struct dma_buf *dmabuf,
> > + enum dma_data_direction
> > direction)
> > +{
> > + struct vc_sm_buffer *buf = dmabuf->priv;
> > +
> > + if (!buf->imported)
> > + return -EINVAL;
> > + return buf->import.dma_buf->ops->end_cpu_access(buf-
> > >import.dma_buf,
> > + direction);
> > +}
> > +
> > +static const struct dma_buf_ops dma_buf_import_ops = {
> > + .map_dma_buf = vc_sm_import_map_dma_buf,
> > + .unmap_dma_buf = vc_sm_import_unmap_dma_buf,
> > + .mmap = vc_sm_import_dmabuf_mmap,
> > + .release = vc_sm_import_dma_buf_release,
> > + .attach = vc_sm_import_dma_buf_attach,
> > + .detach = vc_sm_import_dma_buf_detatch,
> > + .begin_cpu_access = vc_sm_import_dma_buf_begin_cpu_access,
> > + .end_cpu_access = vc_sm_import_dma_buf_end_cpu_access,
> > +};
> > +
> > +/* Import a dma_buf to be shared with VC. */
> > +int
> > +vc_sm_cma_import_dmabuf_internal(struct vc_sm_privdata_t *private,
> > + struct dma_buf *dma_buf,
> > + int fd,
> > + struct dma_buf **imported_buf)
> > +{
> > + DEFINE_DMA_BUF_EXPORT_INFO(exp_info);
> > + struct vc_sm_buffer *buffer = NULL;
> > + struct vc_sm_import import = { };
> > + struct vc_sm_import_result result = { };
> > + struct dma_buf_attachment *attach = NULL;
> > + struct sg_table *sgt = NULL;
> > + dma_addr_t dma_addr;
> > + int ret = 0;
> > + int status;
> > +
> > + /* Setup our allocation parameters */
> > + pr_debug("%s: importing dma_buf %p/fd %d\n", __func__, dma_buf,
> > fd);
> > +
> > + if (fd < 0)
> > + get_dma_buf(dma_buf);
> > + else
> > + dma_buf = dma_buf_get(fd);
> > +
> > + if (!dma_buf)
> > + return -EINVAL;
> > +
> > + attach = dma_buf_attach(dma_buf, &sm_state->pdev->dev);
> > + if (IS_ERR(attach)) {
> > + ret = PTR_ERR(attach);
> > + goto error;
> > + }
> > +
> > + sgt = dma_buf_map_attachment(attach, DMA_BIDIRECTIONAL);
> > + if (IS_ERR(sgt)) {
> > + ret = PTR_ERR(sgt);
> > + goto error;
> > + }
> > +
> > + /* Verify that the address block is contiguous */
> > + if (sgt->nents != 1) {
> > + ret = -ENOMEM;
> > + goto error;
> > + }
> > +
> > + /* Allocate local buffer to track this allocation. */
> > + buffer = kzalloc(sizeof(*buffer), GFP_KERNEL);
> > + if (!buffer) {
> > + ret = -ENOMEM;
> > + goto error;
> > + }
> > +
> > + import.type = VC_SM_ALLOC_NON_CACHED;
> > + dma_addr = sg_dma_address(sgt->sgl);
> > + import.addr = (u32)dma_addr;
> > + if ((import.addr & 0xC0000000) != 0xC0000000) {
> > + pr_err("%s: Expecting an uncached alias for dma_addr
> > %pad\n",
> > + __func__, &dma_addr);
> > + import.addr |= 0xC0000000;
> > + }
> > + import.size = sg_dma_len(sgt->sgl);
> > + import.allocator = current->tgid;
> > + import.kernel_id = get_kernel_id(buffer);
> > +
> > + memcpy(import.name, VC_SM_RESOURCE_NAME_DEFAULT,
> > + sizeof(VC_SM_RESOURCE_NAME_DEFAULT));
> > +
> > + pr_debug("[%s]: attempt to import \"%s\" data - type %u, addr
> > %pad, size %u.\n",
> > + __func__, import.name, import.type, &dma_addr,
> > import.size);
> > +
> > + /* Allocate the videocore buffer. */
> > + status = vc_sm_cma_vchi_import(sm_state->sm_handle, &import,
> > &result,
> > + &sm_state->int_trans_id);
> > + if (status == -EINTR) {
> > + pr_debug("[%s]: requesting import memory action restart
> > (trans_id: %u)\n",
> > + __func__, sm_state->int_trans_id);
> > + ret = -ERESTARTSYS;
> > + private->restart_sys = -EINTR;
> > + private->int_action = VC_SM_MSG_TYPE_IMPORT;
> > + goto error;
> > + } else if (status || !result.res_handle) {
> > + pr_debug("[%s]: failed to import memory on videocore
> > (status: %u, trans_id: %u)\n",
> > + __func__, status, sm_state->int_trans_id);
> > + ret = -ENOMEM;
> > + goto error;
> > + }
> > +
> > + mutex_init(&buffer->lock);
> > + INIT_LIST_HEAD(&buffer->attachments);
> > + memcpy(buffer->name, import.name,
> > + min(sizeof(buffer->name), sizeof(import.name) - 1));
> > +
> > + /* Keep track of the buffer we created. */
> > + buffer->private = private;
> > + buffer->vc_handle = result.res_handle;
> > + buffer->size = import.size;
> > + buffer->vpu_state = VPU_MAPPED;
> > +
> > + buffer->imported = 1;
> > + buffer->import.dma_buf = dma_buf;
> > +
> > + buffer->import.attach = attach;
> > + buffer->import.sgt = sgt;
> > + buffer->dma_addr = dma_addr;
> > + buffer->in_use = 1;
> > + buffer->kernel_id = import.kernel_id;
> > +
> > + /*
> > + * We're done - we need to export a new dmabuf chaining through
> > most
> > + * functions, but enabling us to release our own internal
> > references
> > + * here.
> > + */
> > + exp_info.ops = &dma_buf_import_ops;
> > + exp_info.size = import.size;
> > + exp_info.flags = O_RDWR;
> > + exp_info.priv = buffer;
> > +
> > + buffer->dma_buf = dma_buf_export(&exp_info);
> > + if (IS_ERR(buffer->dma_buf)) {
> > + ret = PTR_ERR(buffer->dma_buf);
> > + goto error;
> > + }
> > +
> > + vc_sm_add_resource(private, buffer);
> > +
> > + *imported_buf = buffer->dma_buf;
> > +
> > + return 0;
> > +
> > +error:
> > + if (result.res_handle) {
> > + struct vc_sm_free_t free = { result.res_handle, 0 };
> > +
> > + vc_sm_cma_vchi_free(sm_state->sm_handle, &free,
> > + &sm_state->int_trans_id);
> > + }
> > + free_kernel_id(import.kernel_id);
> > + kfree(buffer);
> > + if (sgt)
> > + dma_buf_unmap_attachment(attach, sgt,
> > DMA_BIDIRECTIONAL);
> > + if (attach)
> > + dma_buf_detach(dma_buf, attach);
> > + dma_buf_put(dma_buf);
> > + return ret;
> > +}
> > +
> > +static int vc_sm_cma_vpu_alloc(u32 size, u32 align, const char
> > *name,
> > + u32 mem_handle, struct vc_sm_buffer
> > **ret_buffer)
> > +{
> > + DEFINE_DMA_BUF_EXPORT_INFO(exp_info);
> > + struct vc_sm_buffer *buffer = NULL;
> > + struct sg_table *sgt;
> > + int aligned_size;
> > + int ret = 0;
> > +
> > + /* Align to the user requested align */
> > + aligned_size = ALIGN(size, align);
> > + /* and then to a page boundary */
> > + aligned_size = PAGE_ALIGN(aligned_size);
> > +
> > + if (!aligned_size)
> > + return -EINVAL;
> > +
> > + /* Allocate local buffer to track this allocation. */
> > + buffer = kzalloc(sizeof(*buffer), GFP_KERNEL);
> > + if (!buffer)
> > + return -ENOMEM;
> > +
> > + mutex_init(&buffer->lock);
> > + /* Acquire the mutex as vc_sm_release_resource will release it
> > in the
> > + * error path.
> > + */
> > + mutex_lock(&buffer->lock);
> > +
> > + buffer->cookie = dma_alloc_coherent(&sm_state->pdev->dev,
> > + aligned_size, &buffer-
> > >dma_addr,
> > + GFP_KERNEL);
> > + if (!buffer->cookie) {
> > + pr_err("[%s]: dma_alloc_coherent alloc of %d bytes
> > failed\n",
> > + __func__, aligned_size);
> > + ret = -ENOMEM;
> > + goto error;
> > + }
> > +
> > + pr_debug("[%s]: alloc of %d bytes success\n",
> > + __func__, aligned_size);
> > +
> > + sgt = kmalloc(sizeof(*sgt), GFP_KERNEL);
> > + if (!sgt) {
> > + ret = -ENOMEM;
> > + goto error;
> > + }
> > +
> > + ret = dma_get_sgtable(&sm_state->pdev->dev, sgt, buffer-
> > >cookie,
> > + buffer->dma_addr, buffer->size);
> > + if (ret < 0) {
> > + pr_err("failed to get scatterlist from DMA API\n");
> > + kfree(sgt);
> > + ret = -ENOMEM;
> > + goto error;
> > + }
> > + buffer->alloc.sg_table = sgt;
> > +
> > + INIT_LIST_HEAD(&buffer->attachments);
> > +
> > + memcpy(buffer->name, name,
> > + min(sizeof(buffer->name), strlen(name)));
> > +
> > + exp_info.ops = &dma_buf_ops;
> > + exp_info.size = aligned_size;
> > + exp_info.flags = O_RDWR;
> > + exp_info.priv = buffer;
> > +
> > + buffer->dma_buf = dma_buf_export(&exp_info);
> > + if (IS_ERR(buffer->dma_buf)) {
> > + ret = PTR_ERR(buffer->dma_buf);
> > + goto error;
> > + }
> > + buffer->dma_addr = (u32)sg_dma_address(buffer->alloc.sg_table-
> > >sgl);
> > + if ((buffer->dma_addr & 0xC0000000) != 0xC0000000) {
> > + pr_warn_once("%s: Expecting an uncached alias for
> > dma_addr %pad\n",
> > + __func__, &buffer->dma_addr);
> > + buffer->dma_addr |= 0xC0000000;
> > + }
> > + buffer->private = sm_state->vpu_allocs;
> > +
> > + buffer->vc_handle = mem_handle;
> > + buffer->vpu_state = VPU_MAPPED;
> > + buffer->vpu_allocated = 1;
> > + buffer->size = size;
> > + /*
> > + * Create an ID that will be passed along with our message so
> > + * that when we service the release reply, we can look up which
> > + * resource is being released.
> > + */
> > + buffer->kernel_id = get_kernel_id(buffer);
> > +
> > + vc_sm_add_resource(sm_state->vpu_allocs, buffer);
> > +
> > + mutex_unlock(&buffer->lock);
> > +
> > + *ret_buffer = buffer;
> > + return 0;
> > +error:
> > + if (buffer)
> > + vc_sm_release_resource(buffer);
> > + return ret;
> > +}
> > +
> > +static void
> > +vc_sm_vpu_event(struct sm_instance *instance, struct vc_sm_result_t
> > *reply,
> > + int reply_len)
> > +{
> > + switch (reply->trans_id & ~0x80000000) {
> > + case VC_SM_MSG_TYPE_CLIENT_VERSION:
> > + {
> > + /* Acknowledge that the firmware supports the version
> > command */
> > + pr_debug("%s: firmware acked version msg. Require
> > release cb\n",
> > + __func__);
> > + sm_state->require_released_callback = true;
> > + }
> > + break;
> > + case VC_SM_MSG_TYPE_RELEASED:
> > + {
> > + struct vc_sm_released *release = (struct vc_sm_released
> > *)reply;
> > + struct vc_sm_buffer *buffer =
> > + lookup_kernel_id(release-
> > >kernel_id);
> > + if (!buffer) {
> > + pr_err("%s: VC released a buffer that is
> > already released, kernel_id %d\n",
> > + __func__, release->kernel_id);
> > + break;
> > + }
> > + mutex_lock(&buffer->lock);
> > +
> > + pr_debug("%s: Released addr %08x, size %u, id %08x,
> > mem_handle %08x\n",
> > + __func__, release->addr, release->size,
> > + release->kernel_id, release->vc_handle);
> > +
> > + buffer->vc_handle = 0;
> > + buffer->vpu_state = VPU_NOT_MAPPED;
> > + free_kernel_id(release->kernel_id);
> > +
> > + if (buffer->vpu_allocated) {
> > + /* VPU allocation, so release the dmabuf which
> > will
> > + * trigger the clean up.
> > + */
> > + mutex_unlock(&buffer->lock);
> > + dma_buf_put(buffer->dma_buf);
> > + } else {
> > + vc_sm_release_resource(buffer);
> > + }
> > + }
> > + break;
> > + case VC_SM_MSG_TYPE_VC_MEM_REQUEST:
> > + {
> > + struct vc_sm_buffer *buffer = NULL;
> > + struct vc_sm_vc_mem_request *req =
> > + (struct vc_sm_vc_mem_request
> > *)reply;
> > + struct vc_sm_vc_mem_request_result reply;
> > + int ret;
> > +
> > + pr_debug("%s: Request %u bytes of memory, align %d name
> > %s, trans_id %08x\n",
> > + __func__, req->size, req->align, req->name,
> > + req->trans_id);
> > + ret = vc_sm_cma_vpu_alloc(req->size, req->align, req-
> > >name,
> > + req->vc_handle, &buffer);
> > +
> > + reply.trans_id = req->trans_id;
> > + if (!ret) {
> > + reply.addr = buffer->dma_addr;
> > + reply.kernel_id = buffer->kernel_id;
> > + pr_debug("%s: Allocated resource buffer %p,
> > addr %pad\n",
> > + __func__, buffer, &buffer->dma_addr);
> > + } else {
> > + pr_err("%s: Allocation failed size %u, name %s,
> > vc_handle %u\n",
> > + __func__, req->size, req->name, req-
> > >vc_handle);
> > + reply.addr = 0;
> > + reply.kernel_id = 0;
> > + }
> > + vc_sm_vchi_client_vc_mem_req_reply(sm_state->sm_handle,
> > &reply,
> > + &sm_state-
> > >int_trans_id);
> > + break;
> > + }
> > + break;
> > + default:
> > + pr_err("%s: Unknown vpu cmd %x\n", __func__, reply-
> > >trans_id);
> > + break;
> > + }
> > +}
> > +
> > +/* Userspace handling */
> > +/*
> > + * Open the device. Creates a private state to help track all
> > allocation
> > + * associated with this device.
> > + */
> > +static int vc_sm_cma_open(struct inode *inode, struct file *file)
> > +{
> > + /* Make sure the device was started properly. */
> > + if (!sm_state) {
> > + pr_err("[%s]: invalid device\n", __func__);
> > + return -EPERM;
> > + }
> > +
> > + file->private_data = vc_sm_cma_create_priv_data(current->tgid);
> > + if (!file->private_data) {
> > + pr_err("[%s]: failed to create data tracker\n",
> > __func__);
> > +
> > + return -ENOMEM;
> > + }
> > +
> > + return 0;
> > +}
> > +
> > +/*
> > + * Close the vcsm-cma device.
> > + * All allocations are file descriptors to the dmabuf objects, so we
> > will get
> > + * the clean up request on those as those are cleaned up.
> > + */
> > +static int vc_sm_cma_release(struct inode *inode, struct file *file)
> > +{
> > + struct vc_sm_privdata_t *file_data =
> > + (struct vc_sm_privdata_t *)file->private_data;
> > + int ret = 0;
> > +
> > + /* Make sure the device was started properly. */
> > + if (!sm_state || !file_data) {
> > + pr_err("[%s]: invalid device\n", __func__);
> > + ret = -EPERM;
> > + goto out;
> > + }
> > +
> > + pr_debug("[%s]: using private data %p\n", __func__, file_data);
> > +
> > + /* Terminate the private data. */
> > + kfree(file_data);
> > +
> > +out:
> > + return ret;
> > +}
> > +
> > +/*
> > + * Allocate a shared memory handle and block.
> > + * Allocation is from CMA, and then imported into the VPU mappings.
> > + */
> > +int vc_sm_cma_ioctl_alloc(struct vc_sm_privdata_t *private,
> > + struct vc_sm_cma_ioctl_alloc *ioparam)
> > +{
> > + DEFINE_DMA_BUF_EXPORT_INFO(exp_info);
> > + struct vc_sm_buffer *buffer = NULL;
> > + struct vc_sm_import import = { 0 };
> > + struct vc_sm_import_result result = { 0 };
> > + struct dma_buf *dmabuf = NULL;
> > + struct sg_table *sgt;
> > + int aligned_size;
> > + int ret = 0;
> > + int status;
> > + int fd = -1;
> > +
> > + aligned_size = PAGE_ALIGN(ioparam->size);
> > +
> > + if (!aligned_size)
> > + return -EINVAL;
> > +
> > + /* Allocate local buffer to track this allocation. */
> > + buffer = kzalloc(sizeof(*buffer), GFP_KERNEL);
> > + if (!buffer) {
> > + ret = -ENOMEM;
> > + goto error;
> > + }
> > +
> > + buffer->cookie = dma_alloc_coherent(&sm_state->pdev->dev,
> > + aligned_size,
> > + &buffer->dma_addr,
> > + GFP_KERNEL);
> > + if (!buffer->cookie) {
> > + pr_err("[%s]: dma_alloc_coherent alloc of %d bytes
> > failed\n",
> > + __func__, aligned_size);
> > + ret = -ENOMEM;
> > + goto error;
> > + }
> > +
> > + import.type = VC_SM_ALLOC_NON_CACHED;
> > + import.allocator = current->tgid;
> > +
> > + if (*ioparam->name)
> > + memcpy(import.name, ioparam->name, sizeof(import.name)
> > - 1);
> > + else
> > + memcpy(import.name, VC_SM_RESOURCE_NAME_DEFAULT,
> > + sizeof(VC_SM_RESOURCE_NAME_DEFAULT));
> > +
> > + mutex_init(&buffer->lock);
> > + INIT_LIST_HEAD(&buffer->attachments);
> > + memcpy(buffer->name, import.name,
> > + min(sizeof(buffer->name), sizeof(import.name) - 1));
> > +
> > + exp_info.ops = &dma_buf_ops;
> > + exp_info.size = aligned_size;
> > + exp_info.flags = O_RDWR;
> > + exp_info.priv = buffer;
> > +
> > + dmabuf = dma_buf_export(&exp_info);
> > + if (IS_ERR(dmabuf)) {
> > + ret = PTR_ERR(dmabuf);
> > + goto error;
> > + }
> > + buffer->dma_buf = dmabuf;
> > +
> > + import.addr = buffer->dma_addr;
> > + import.size = aligned_size;
> > + import.kernel_id = get_kernel_id(buffer);
> > +
> > + /* Wrap it into a videocore buffer. */
> > + status = vc_sm_cma_vchi_import(sm_state->sm_handle, &import,
> > &result,
> > + &sm_state->int_trans_id);
> > + if (status == -EINTR) {
> > + pr_debug("[%s]: requesting import memory action restart
> > (trans_id: %u)\n",
> > + __func__, sm_state->int_trans_id);
> > + ret = -ERESTARTSYS;
> > + private->restart_sys = -EINTR;
> > + private->int_action = VC_SM_MSG_TYPE_IMPORT;
> > + goto error;
> > + } else if (status || !result.res_handle) {
> > + pr_err("[%s]: failed to import memory on videocore
> > (status: %u, trans_id: %u)\n",
> > + __func__, status, sm_state->int_trans_id);
> > + ret = -ENOMEM;
> > + goto error;
> > + }
> > +
> > + /* Keep track of the buffer we created. */
> > + buffer->private = private;
> > + buffer->vc_handle = result.res_handle;
> > + buffer->size = import.size;
> > + buffer->vpu_state = VPU_MAPPED;
> > + buffer->kernel_id = import.kernel_id;
> > +
> > + sgt = kmalloc(sizeof(*sgt), GFP_KERNEL);
> > + if (!sgt) {
> > + ret = -ENOMEM;
> > + goto error;
> > + }
> > +
> > + ret = dma_get_sgtable(&sm_state->pdev->dev, sgt, buffer-
> > >cookie,
> > + buffer->dma_addr, buffer->size);
> > + if (ret < 0) {
> > + /* FIXME: error handling */
> > + pr_err("failed to get scatterlist from DMA API\n");
> > + kfree(sgt);
> > + ret = -ENOMEM;
> > + goto error;
> > + }
> > + buffer->alloc.sg_table = sgt;
> > +
> > + fd = dma_buf_fd(dmabuf, O_CLOEXEC);
> > + if (fd < 0)
> > + goto error;
> > +
> > + vc_sm_add_resource(private, buffer);
> > +
> > + pr_debug("[%s]: Added resource as fd %d, buffer %p, private %p,
> > dma_addr %pad\n",
> > + __func__, fd, buffer, private, &buffer->dma_addr);
> > +
> > + /* We're done */
> > + ioparam->handle = fd;
> > + ioparam->vc_handle = buffer->vc_handle;
> > + ioparam->dma_addr = buffer->dma_addr;
> > + return 0;
> > +
> > +error:
> > + pr_err("[%s]: something failed - cleanup. ret %d\n", __func__,
> > ret);
> > +
> > + if (dmabuf) {
> > + /* dmabuf has been exported, therefore allow dmabuf
> > cleanup to
> > + * deal with this
> > + */
> > + dma_buf_put(dmabuf);
> > + } else {
> > + /* No dmabuf, therefore just free the buffer here */
> > + if (buffer->cookie)
> > + dma_free_coherent(&sm_state->pdev->dev, buffer-
> > >size,
> > + buffer->cookie, buffer-
> > >dma_addr);
> > + kfree(buffer);
> > + }
> > + return ret;
> > +}
> > +
> > +#ifndef CONFIG_ARM64
> > +/* Converts VCSM_CACHE_OP_* to an operating function. */
> > +static void (*cache_op_to_func(const unsigned int cache_op))
> > + (const void*, const
> > void*)
> > +{
> > + switch (cache_op) {
> > + case VC_SM_CACHE_OP_NOP:
> > + return NULL;
> > +
> > + case VC_SM_CACHE_OP_INV:
> > + case VC_SM_CACHE_OP_CLEAN:
> > + case VC_SM_CACHE_OP_FLUSH:
> > + return dmac_flush_range;
> > +
> > + default:
> > + pr_err("[%s]: Invalid cache_op: 0x%08x\n", __func__,
> > cache_op);
> > + return NULL;
> > + }
> > +}
> > +
> > +/*
> > + * Clean/invalid/flush cache of which buffer is already pinned (i.e.
> > accessed).
> > + */
> > +static int clean_invalid_contig_2d(const void __user *addr,
> > + const size_t block_count,
> > + const size_t block_size,
> > + const size_t stride,
> > + const unsigned int cache_op)
> > +{
> > + size_t i;
> > + void (*op_fn)(const void *start, const void *end);
> > +
> > + if (!block_size) {
> > + pr_err("[%s]: size cannot be 0\n", __func__);
> > + return -EINVAL;
> > + }
> > +
> > + op_fn = cache_op_to_func(cache_op);
> > + if (!op_fn)
> > + return -EINVAL;
> > +
> > + for (i = 0; i < block_count; i ++, addr += stride)
> > + op_fn(addr, addr + block_size);
> > +
> > + return 0;
> > +}
> > +
> > +static int vc_sm_cma_clean_invalid2(unsigned int cmdnr, unsigned
> > long arg)
> > +{
> > + struct vc_sm_cma_ioctl_clean_invalid2 ioparam;
> > + struct vc_sm_cma_ioctl_clean_invalid_block *block = NULL;
> > + int i, ret = 0;
> > +
> > + /* Get parameter data. */
> > + if (copy_from_user(&ioparam, (void *)arg, sizeof(ioparam))) {
> > + pr_err("[%s]: failed to copy-from-user header for cmd
> > %x\n",
> > + __func__, cmdnr);
> > + return -EFAULT;
> > + }
> > + block = kmalloc(ioparam.op_count * sizeof(*block), GFP_KERNEL);
> > + if (!block)
> > + return -EFAULT;
> > +
> > + if (copy_from_user(block, (void *)(arg + sizeof(ioparam)),
> > + ioparam.op_count * sizeof(*block)) != 0) {
> > + pr_err("[%s]: failed to copy-from-user payload for cmd
> > %x\n",
> > + __func__, cmdnr);
> > + ret = -EFAULT;
> > + goto out;
> > + }
> > +
> > + for (i = 0; i < ioparam.op_count; i++) {
> > + const struct vc_sm_cma_ioctl_clean_invalid_block *
> > const op =
> > + block +
> > i;
> > +
> > + if (op->invalidate_mode == VC_SM_CACHE_OP_NOP)
> > + continue;
> > +
> > + ret = clean_invalid_contig_2d((void __user *)op-
> > >start_address,
> > + op->block_count, op-
> > >block_size,
> > + op->inter_block_stride,
> > + op->invalidate_mode);
> > + if (ret)
> > + break;
> > + }
> > +out:
> > + kfree(block);
> > +
> > + return ret;
> > +}
> > +#endif
> > +
> > +static long vc_sm_cma_ioctl(struct file *file, unsigned int cmd,
> > + unsigned long arg)
> > +{
> > + int ret = 0;
> > + unsigned int cmdnr = _IOC_NR(cmd);
> > + struct vc_sm_privdata_t *file_data =
> > + (struct vc_sm_privdata_t *)file->private_data;
> > +
> > + /* Validate we can work with this device. */
> > + if (!sm_state || !file_data) {
> > + pr_err("[%s]: invalid device\n", __func__);
> > + return -EPERM;
> > + }
> > +
> > + /* Action is a re-post of a previously interrupted action? */
> > + if (file_data->restart_sys == -EINTR) {
> > + struct vc_sm_action_clean_t action_clean;
> > +
> > + pr_debug("[%s]: clean up of action %u (trans_id: %u)
> > following EINTR\n",
> > + __func__, file_data->int_action,
> > + file_data->int_trans_id);
> > +
> > + action_clean.res_action = file_data->int_action;
> > + action_clean.action_trans_id = file_data->int_trans_id;
> > +
> > + file_data->restart_sys = 0;
> > + }
> > +
> > + /* Now process the command. */
> > + switch (cmdnr) {
> > + /* New memory allocation.
> > + */
> > + case VC_SM_CMA_CMD_ALLOC:
> > + {
> > + struct vc_sm_cma_ioctl_alloc ioparam;
> > +
> > + /* Get the parameter data. */
> > + if (copy_from_user
> > + (&ioparam, (void *)arg, sizeof(ioparam)) != 0) {
> > + pr_err("[%s]: failed to copy-from-user for cmd
> > %x\n",
> > + __func__, cmdnr);
> > + ret = -EFAULT;
> > + break;
> > + }
> > +
> > + ret = vc_sm_cma_ioctl_alloc(file_data, &ioparam);
> > + if (!ret &&
> > + (copy_to_user((void *)arg, &ioparam,
> > + sizeof(ioparam)) != 0)) {
> > + /* FIXME: Release allocation */
> > + pr_err("[%s]: failed to copy-to-user for cmd
> > %x\n",
> > + __func__, cmdnr);
> > + ret = -EFAULT;
> > + }
> > + break;
> > + }
> > +
> > + case VC_SM_CMA_CMD_IMPORT_DMABUF:
> > + {
> > + struct vc_sm_cma_ioctl_import_dmabuf ioparam;
> > + struct dma_buf *new_dmabuf;
> > +
> > + /* Get the parameter data. */
> > + if (copy_from_user
> > + (&ioparam, (void *)arg, sizeof(ioparam)) != 0) {
> > + pr_err("[%s]: failed to copy-from-user for cmd
> > %x\n",
> > + __func__, cmdnr);
> > + ret = -EFAULT;
> > + break;
> > + }
> > +
> > + ret = vc_sm_cma_import_dmabuf_internal(file_data,
> > + NULL,
> > + ioparam.dmabuf_f
> > d,
> > + &new_dmabuf);
> > +
> > + if (!ret) {
> > + struct vc_sm_buffer *buf = new_dmabuf->priv;
> > +
> > + ioparam.size = buf->size;
> > + ioparam.handle = dma_buf_fd(new_dmabuf,
> > + O_CLOEXEC);
> > + ioparam.vc_handle = buf->vc_handle;
> > + ioparam.dma_addr = buf->dma_addr;
> > +
> > + if (ioparam.handle < 0 ||
> > + (copy_to_user((void *)arg, &ioparam,
> > + sizeof(ioparam)) != 0)) {
> > + dma_buf_put(new_dmabuf);
> > + /* FIXME: Release allocation */
> > + ret = -EFAULT;
> > + }
> > + }
> > + break;
> > + }
> > +
> > +#ifndef CONFIG_ARM64
> > + /*
> > + * Flush/Invalidate the cache for a given mapping.
> > + * Blocks must be pinned (i.e. accessed) before this call.
> > + */
> > + case VC_SM_CMA_CMD_CLEAN_INVALID2:
> > + ret = vc_sm_cma_clean_invalid2(cmdnr, arg);
> > + break;
> > +#endif
> > +
> > + default:
> > + pr_debug("[%s]: cmd %x tgid %u, owner %u\n", __func__,
> > cmdnr,
> > + current->tgid, file_data->pid);
> > +
> > + ret = -EINVAL;
> > + break;
> > + }
> > +
> > + return ret;
> > +}
> > +
> > +#ifdef CONFIG_COMPAT
> > +struct vc_sm_cma_ioctl_clean_invalid2_32 {
> > + u32 op_count;
> > + struct vc_sm_cma_ioctl_clean_invalid_block_32 {
> > + u16 invalidate_mode;
> > + u16 block_count;
> > + compat_uptr_t start_address;
> > + u32 block_size;
> > + u32 inter_block_stride;
> > + } s[0];
> > +};
> > +
> > +#define VC_SM_CMA_CMD_CLEAN_INVALID2_32\
> > + _IOR(VC_SM_CMA_MAGIC_TYPE, VC_SM_CMA_CMD_CLEAN_INVALID2,\
> > + struct vc_sm_cma_ioctl_clean_invalid2_32)
> > +
> > +static long vc_sm_cma_compat_ioctl(struct file *file, unsigned int
> > cmd,
> > + unsigned long arg)
> > +{
> > + switch (cmd) {
> > + case VC_SM_CMA_CMD_CLEAN_INVALID2_32:
> > + /* FIXME */
> > + return -EINVAL;
> > +
> > + default:
> > + return vc_sm_cma_ioctl(file, cmd, arg);
> > + }
> > +}
> > +#endif
> > +
> > +/* Device operations that we managed in this driver. */
> > +static const struct file_operations vc_sm_ops = {
> > + .owner = THIS_MODULE,
> > + .unlocked_ioctl = vc_sm_cma_ioctl,
> > +#ifdef CONFIG_COMPAT
> > + .compat_ioctl = vc_sm_cma_compat_ioctl,
> > +#endif
> > + .open = vc_sm_cma_open,
> > + .release = vc_sm_cma_release,
> > +};
> > +
> > +/* Driver load/unload functions */
> > +/* Videocore connected. */
> > +static void vc_sm_connected_init(void)
> > +{
> > + int ret;
> > + struct vchi_instance_handle *vchi_instance;
> > + struct vc_sm_version version;
> > + struct vc_sm_result_t version_result;
> > +
> > + pr_info("[%s]: start\n", __func__);
> > +
> > + /*
> > + * Initialize and create a VCHI connection for the shared
> > memory service
> > + * running on videocore.
> > + */
> > + ret = vchi_initialise(&vchi_instance);
> > + if (ret) {
> > + pr_err("[%s]: failed to initialise VCHI instance
> > (ret=%d)\n",
> > + __func__, ret);
> > +
> > + return;
> > + }
> > +
> > + ret = vchi_connect(vchi_instance);
> > + if (ret) {
> > + pr_err("[%s]: failed to connect VCHI instance
> > (ret=%d)\n",
> > + __func__, ret);
> > +
> > + return;
> > + }
> > +
> > + /* Initialize an instance of the shared memory service. */
> > + sm_state->sm_handle = vc_sm_cma_vchi_init(vchi_instance, 1,
> > + vc_sm_vpu_event);
> > + if (!sm_state->sm_handle) {
> > + pr_err("[%s]: failed to initialize shared memory
> > service\n",
> > + __func__);
> > +
> > + return;
> > + }
> > +
> > + /* Create a debug fs directory entry (root). */
> > + sm_state->dir_root = debugfs_create_dir(VC_SM_DIR_ROOT_NAME,
> > NULL);
> > +
> > + sm_state->dir_state.show = &vc_sm_cma_global_state_show;
> > + sm_state->dir_state.dir_entry =
> > + debugfs_create_file(VC_SM_STATE, 0444, sm_state-
> > >dir_root,
> > + &sm_state->dir_state,
> > + &vc_sm_cma_debug_fs_fops);
> > +
> > + INIT_LIST_HEAD(&sm_state->buffer_list);
> > +
> > + /* Create a shared memory device. */
> > + sm_state->misc_dev.minor = MISC_DYNAMIC_MINOR;
> > + sm_state->misc_dev.name = DEVICE_NAME;
> > + sm_state->misc_dev.fops = &vc_sm_ops;
> > + sm_state->misc_dev.parent = NULL;
> > + /* Temporarily set as 666 until udev rules have been sorted */
> > + sm_state->misc_dev.mode = 0666;
> > + ret = misc_register(&sm_state->misc_dev);
> > + if (ret) {
> > + pr_err("vcsm-cma: failed to register misc device.\n");
> > + goto err_remove_debugfs;
> > + }
> > +
> > + sm_state->data_knl = vc_sm_cma_create_priv_data(0);
> > + if (!sm_state->data_knl) {
> > + pr_err("[%s]: failed to create kernel private data
> > tracker\n",
> > + __func__);
> > + goto err_remove_misc_dev;
> > + }
> > +
> > + version.version = 2;
> > + ret = vc_sm_cma_vchi_client_version(sm_state->sm_handle,
> > &version,
> > + &version_result,
> > + &sm_state->int_trans_id);
> > + if (ret) {
> > + pr_err("[%s]: Failed to send version request %d\n",
> > __func__,
> > + ret);
> > + }
> > +
> > + /* Done! */
> > + sm_inited = 1;
> > + pr_info("[%s]: installed successfully\n", __func__);
> > + return;
> > +
> > +err_remove_misc_dev:
> > + misc_deregister(&sm_state->misc_dev);
> > +err_remove_debugfs:
> > + debugfs_remove_recursive(sm_state->dir_root);
> > + vc_sm_cma_vchi_stop(&sm_state->sm_handle);
> > +}
> > +
> > +/* Driver loading. */
> > +static int bcm2835_vc_sm_cma_probe(struct platform_device *pdev)
> > +{
> > + pr_info("%s: Videocore shared memory driver\n", __func__);
> > +
> > + sm_state = devm_kzalloc(&pdev->dev, sizeof(*sm_state),
> > GFP_KERNEL);
> > + if (!sm_state)
> > + return -ENOMEM;
> > + sm_state->pdev = pdev;
> > + mutex_init(&sm_state->map_lock);
> > +
> > + spin_lock_init(&sm_state->kernelid_map_lock);
> > + idr_init_base(&sm_state->kernelid_map, 1);
> > +
> > + pdev->dev.dma_parms = devm_kzalloc(&pdev->dev,
> > + sizeof(*pdev-
> > >dev.dma_parms),
> > + GFP_KERNEL);
> > + /* dma_set_max_seg_size checks if dma_parms is NULL. */
> > + dma_set_max_seg_size(&pdev->dev, 0x3FFFFFFF);
> > +
> > + vchiq_add_connected_callback(vc_sm_connected_init);
> > + return 0;
> > +}
> > +
> > +/* Driver unloading. */
> > +static int bcm2835_vc_sm_cma_remove(struct platform_device *pdev)
> > +{
> > + pr_debug("[%s]: start\n", __func__);
> > + if (sm_inited) {
> > + misc_deregister(&sm_state->misc_dev);
> > +
> > + /* Remove all proc entries. */
> > + debugfs_remove_recursive(sm_state->dir_root);
> > +
> > + /* Stop the videocore shared memory service. */
> > + vc_sm_cma_vchi_stop(&sm_state->sm_handle);
> > + }
> > +
> > + if (sm_state) {
> > + idr_destroy(&sm_state->kernelid_map);
> > +
> > + /* Free the memory for the state structure. */
> > + mutex_destroy(&sm_state->map_lock);
> > + }
> > +
> > + pr_debug("[%s]: end\n", __func__);
> > + return 0;
> > +}
> > +
> > +/* Kernel API calls */
> > +/* Get an internal resource handle mapped from the external one. */
> > +int vc_sm_cma_int_handle(void *handle)
> > +{
> > + struct dma_buf *dma_buf = (struct dma_buf *)handle;
> > + struct vc_sm_buffer *buf;
> > +
> > + /* Validate we can work with this device. */
> > + if (!sm_state || !handle) {
> > + pr_err("[%s]: invalid input\n", __func__);
> > + return 0;
> > + }
> > +
> > + buf = (struct vc_sm_buffer *)dma_buf->priv;
> > + return buf->vc_handle;
> > +}
> > +EXPORT_SYMBOL_GPL(vc_sm_cma_int_handle);
> > +
> > +/* Free a previously allocated shared memory handle and block. */
> > +int vc_sm_cma_free(void *handle)
> > +{
> > + struct dma_buf *dma_buf = (struct dma_buf *)handle;
> > +
> > + /* Validate we can work with this device. */
> > + if (!sm_state || !handle) {
> > + pr_err("[%s]: invalid input\n", __func__);
> > + return -EPERM;
> > + }
> > +
> > + pr_debug("%s: handle %p/dmabuf %p\n", __func__, handle,
> > dma_buf);
> > +
> > + dma_buf_put(dma_buf);
> > +
> > + return 0;
> > +}
> > +EXPORT_SYMBOL_GPL(vc_sm_cma_free);
> > +
> > +/* Import a dmabuf to be shared with VC. */
> > +int vc_sm_cma_import_dmabuf(struct dma_buf *src_dmabuf, void
> > **handle)
> > +{
> > + struct dma_buf *new_dma_buf;
> > + struct vc_sm_buffer *buf;
> > + int ret;
> > +
> > + /* Validate we can work with this device. */
> > + if (!sm_state || !src_dmabuf || !handle) {
> > + pr_err("[%s]: invalid input\n", __func__);
> > + return -EPERM;
> > + }
> > +
> > + ret = vc_sm_cma_import_dmabuf_internal(sm_state->data_knl,
> > src_dmabuf,
> > + -1, &new_dma_buf);
> > +
> > + if (!ret) {
> > + pr_debug("%s: imported to ptr %p\n", __func__,
> > new_dma_buf);
> > + buf = (struct vc_sm_buffer *)new_dma_buf->priv;
> > +
> > + /* Assign valid handle at this time.*/
> > + *handle = new_dma_buf;
> > + } else {
> > + /*
> > + * succeeded in importing the dma_buf, but then
> > + * failed to look it up again. How?
> > + * Release the fd again.
> > + */
> > + pr_err("%s: imported vc_sm_cma_get_buffer failed %d\n",
> > + __func__, ret);
> > + }
> > +
> > + return ret;
> > +}
> > +EXPORT_SYMBOL_GPL(vc_sm_cma_import_dmabuf);
> > +
> > +static struct platform_driver bcm2835_vcsm_cma_driver = {
> > + .probe = bcm2835_vc_sm_cma_probe,
> > + .remove = bcm2835_vc_sm_cma_remove,
> > + .driver = {
> > + .name = DEVICE_NAME,
> > + .owner = THIS_MODULE,
> > + },
> > +};
> > +
> > +module_platform_driver(bcm2835_vcsm_cma_driver);
> > +
> > +MODULE_AUTHOR("Dave Stevenson");
> > +MODULE_DESCRIPTION("VideoCore CMA Shared Memory Driver");
> > +MODULE_LICENSE("GPL v2");
> > +MODULE_ALIAS("platform:vcsm-cma");
> > diff --git a/drivers/staging/vc04_services/vc-sm-cma/vc_sm.h
> > b/drivers/staging/vc04_services/vc-sm-cma/vc_sm.h
> > new file mode 100644
> > index 000000000000..f1c7b95b14ce
> > --- /dev/null
> > +++ b/drivers/staging/vc04_services/vc-sm-cma/vc_sm.h
> > @@ -0,0 +1,84 @@
> > +/* SPDX-License-Identifier: GPL-2.0 */
> > +
> > +/*
> > + * VideoCore Shared Memory driver using CMA.
> > + *
> > + * Copyright: 2018, Raspberry Pi (Trading) Ltd
> > + *
> > + */
> > +
> > +#ifndef VC_SM_H
> > +#define VC_SM_H
> > +
> > +#include <linux/device.h>
> > +#include <linux/dma-direction.h>
> > +#include <linux/kref.h>
> > +#include <linux/mm_types.h>
> > +#include <linux/mutex.h>
> > +#include <linux/rbtree.h>
> > +#include <linux/sched.h>
> > +#include <linux/shrinker.h>
> > +#include <linux/types.h>
> > +#include <linux/miscdevice.h>
> > +
> > +#define VC_SM_MAX_NAME_LEN 32
> > +
> > +enum vc_sm_vpu_mapping_state {
> > + VPU_NOT_MAPPED,
> > + VPU_MAPPED,
> > + VPU_UNMAPPING
> > +};
> > +
> > +struct vc_sm_alloc_data {
> > + unsigned long num_pages;
> > + void *priv_virt;
> > + struct sg_table *sg_table;
> > +};
> > +
> > +struct vc_sm_imported {
> > + struct dma_buf *dma_buf;
> > + struct dma_buf_attachment *attach;
> > + struct sg_table *sgt;
> > +};
> > +
> > +struct vc_sm_buffer {
> > + struct list_head global_buffer_list; /* Global list of
> > buffers. */
> > +
> > + /* Index in the kernel_id idr so that we can find the
> > + * mmal_msg_context again when servicing the VCHI reply.
> > + */
> > + int kernel_id;
> > +
> > + size_t size;
> > +
> > + /* Lock over all the following state for this buffer */
> > + struct mutex lock;
> > + struct list_head attachments;
> > +
> > + char name[VC_SM_MAX_NAME_LEN];
> > +
> > + int in_use:1; /* Kernel is still using this resource */
> > + int imported:1; /* Imported dmabuf */
> > +
> > + enum vc_sm_vpu_mapping_state vpu_state;
> > + u32 vc_handle; /* VideoCore handle for this buffer */
> > + int vpu_allocated; /*
> > + * The VPU made this allocation.
> > Release the
> > + * local dma_buf when the VPU releases
> > the
> > + * resource.
> > + */
> > +
> > + /* DMABUF related fields */
> > + struct dma_buf *dma_buf;
> > + dma_addr_t dma_addr;
> > + void *cookie;
> > +
> > + struct vc_sm_privdata_t *private;
> > +
> > + union {
> > + struct vc_sm_alloc_data alloc;
> > + struct vc_sm_imported import;
> > + };
> > +};
> > +
> > +#endif
> > diff --git a/drivers/staging/vc04_services/vc-sm-cma/vc_sm_cma_vchi.c
> > b/drivers/staging/vc04_services/vc-sm-cma/vc_sm_cma_vchi.c
> > new file mode 100644
> > index 000000000000..6a203c60bf7f
> > --- /dev/null
> > +++ b/drivers/staging/vc04_services/vc-sm-cma/vc_sm_cma_vchi.c
> > @@ -0,0 +1,505 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * VideoCore Shared Memory CMA allocator
> > + *
> > + * Copyright: 2018, Raspberry Pi (Trading) Ltd
> > + * Copyright 2011-2012 Broadcom Corporation. All rights reserved.
> > + *
> > + * Based on vmcs_sm driver from Broadcom Corporation.
> > + *
> > + */
> > +
> > +/* ---- Include Files ----------------------------------------------
> > ------- */
> > +#include <linux/completion.h>
> > +#include <linux/kernel.h>
> > +#include <linux/kthread.h>
> > +#include <linux/list.h>
> > +#include <linux/mutex.h>
> > +#include <linux/semaphore.h>
> > +#include <linux/slab.h>
> > +#include <linux/types.h>
> > +
> > +#include "vc_sm_cma_vchi.h"
> > +
> > +#define VC_SM_VER 1
> > +#define VC_SM_MIN_VER 0
> > +
> > +/* ---- Private Constants and Types --------------------------------
> > ------ */
> > +
> > +/* Command blocks come from a pool */
> > +#define SM_MAX_NUM_CMD_RSP_BLKS 32
> > +
> > +struct sm_cmd_rsp_blk {
> > + struct list_head head; /* To create lists */
> > + /* To be signaled when the response is there */
> > + struct completion cmplt;
> > +
> > + u32 id;
> > + u16 length;
> > +
> > + u8 msg[VC_SM_MAX_MSG_LEN];
> > +
> > + uint32_t wait:1;
> > + uint32_t sent:1;
> > + uint32_t alloc:1;
> > +
> > +};
> > +
> > +struct sm_instance {
> > + u32 num_connections;
> > + struct vchi_service_handle
> > *vchi_handle[VCHI_MAX_NUM_CONNECTIONS];
> > + struct task_struct *io_thread;
> > + struct completion io_cmplt;
> > +
> > + vpu_event_cb vpu_event;
> > +
> > + /* Mutex over the following lists */
> > + struct mutex lock;
> > + u32 trans_id;
> > + struct list_head cmd_list;
> > + struct list_head rsp_list;
> > + struct list_head dead_list;
> > +
> > + struct sm_cmd_rsp_blk free_blk[SM_MAX_NUM_CMD_RSP_BLKS];
> > +
> > + /* Mutex over the free_list */
> > + struct mutex free_lock;
> > + struct list_head free_list;
> > +
> > + struct semaphore free_sema;
> > +
> > +};
> > +
> > +/* ---- Private Variables ------------------------------------------
> > ------ */
> > +
> > +/* ---- Private Function Prototypes --------------------------------
> > ------ */
> > +
> > +/* ---- Private Functions ------------------------------------------
> > ------ */
> > +static int
> > +bcm2835_vchi_msg_queue(struct vchi_service_handle *handle,
> > + void *data,
> > + unsigned int size)
> > +{
> > + return vchi_queue_kernel_message(handle,
> > + data,
> > + size);
> > +}
> > +
> > +static struct
> > +sm_cmd_rsp_blk *vc_vchi_cmd_create(struct sm_instance *instance,
> > + enum vc_sm_msg_type id, void *msg,
> > + u32 size, int wait)
> > +{
> > + struct sm_cmd_rsp_blk *blk;
> > + struct vc_sm_msg_hdr_t *hdr;
> > +
> > + if (down_interruptible(&instance->free_sema)) {
> > + blk = kmalloc(sizeof(*blk), GFP_KERNEL);
> > + if (!blk)
> > + return NULL;
> > +
> > + blk->alloc = 1;
> > + init_completion(&blk->cmplt);
> > + } else {
> > + mutex_lock(&instance->free_lock);
> > + blk =
> > + list_first_entry(&instance->free_list,
> > + struct sm_cmd_rsp_blk, head);
> > + list_del(&blk->head);
> > + mutex_unlock(&instance->free_lock);
> > + }
> > +
> > + blk->sent = 0;
> > + blk->wait = wait;
> > + blk->length = sizeof(*hdr) + size;
> > +
> > + hdr = (struct vc_sm_msg_hdr_t *)blk->msg;
> > + hdr->type = id;
> > + mutex_lock(&instance->lock);
> > + instance->trans_id++;
> > + /*
> > + * Retain the top bit for identifying asynchronous events, or
> > VPU cmds.
> > + */
> > + instance->trans_id &= ~0x80000000;
> > + hdr->trans_id = instance->trans_id;
> > + blk->id = instance->trans_id;
> > + mutex_unlock(&instance->lock);
> > +
> > + if (size)
> > + memcpy(hdr->body, msg, size);
> > +
> > + return blk;
> > +}
> > +
> > +static void
> > +vc_vchi_cmd_delete(struct sm_instance *instance, struct
> > sm_cmd_rsp_blk *blk)
> > +{
> > + if (blk->alloc) {
> > + kfree(blk);
> > + return;
> > + }
> > +
> > + mutex_lock(&instance->free_lock);
> > + list_add(&blk->head, &instance->free_list);
> > + mutex_unlock(&instance->free_lock);
> > + up(&instance->free_sema);
> > +}
> > +
> > +static void vc_sm_cma_vchi_rx_ack(struct sm_instance *instance,
> > + struct sm_cmd_rsp_blk *cmd,
> > + struct vc_sm_result_t *reply,
> > + u32 reply_len)
> > +{
> > + mutex_lock(&instance->lock);
> > + list_for_each_entry(cmd,
> > + &instance->rsp_list,
> > + head) {
> > + if (cmd->id == reply->trans_id)
> > + break;
> > + }
> > + mutex_unlock(&instance->lock);
> > +
> > + if (&cmd->head == &instance->rsp_list) {
> > + //pr_debug("%s: received response %u, throw away...",
> > + pr_err("%s: received response %u, throw away...",
> > + __func__,
> > + reply->trans_id);
> > + } else if (reply_len > sizeof(cmd->msg)) {
> > + pr_err("%s: reply too big (%u) %u, throw away...",
> > + __func__, reply_len,
> > + reply->trans_id);
> > + } else {
> > + memcpy(cmd->msg, reply,
> > + reply_len);
> > + complete(&cmd->cmplt);
> > + }
> > +}
> > +
> > +static int vc_sm_cma_vchi_videocore_io(void *arg)
> > +{
> > + struct sm_instance *instance = arg;
> > + struct sm_cmd_rsp_blk *cmd = NULL, *cmd_tmp;
> > + struct vc_sm_result_t *reply;
> > + u32 reply_len;
> > + s32 status;
> > + int svc_use = 1;
> > +
> > + while (1) {
> > + if (svc_use)
> > + vchi_service_release(instance->vchi_handle[0]);
> > + svc_use = 0;
> > +
> > + if (wait_for_completion_interruptible(&instance-
> > >io_cmplt))
> > + continue;
> > +
> > + vchi_service_use(instance->vchi_handle[0]);
> > + svc_use = 1;
> > +
> > + do {
> > + /*
> > + * Get new command and move it to response list
> > + */
> > + mutex_lock(&instance->lock);
> > + if (list_empty(&instance->cmd_list)) {
> > + /* no more commands to process */
> > + mutex_unlock(&instance->lock);
> > + break;
> > + }
> > + cmd = list_first_entry(&instance->cmd_list,
> > + struct sm_cmd_rsp_blk,
> > head);
> > + list_move(&cmd->head, &instance->rsp_list);
> > + cmd->sent = 1;
> > + mutex_unlock(&instance->lock);
> > +
> > + /* Send the command */
> > + status =
> > + bcm2835_vchi_msg_queue(instance-
> > >vchi_handle[0],
> > + cmd->msg, cmd-
> > >length);
> > + if (status) {
> > + pr_err("%s: failed to queue message
> > (%d)",
> > + __func__, status);
> > + }
> > +
> > + /* If no reply is needed then we're done */
> > + if (!cmd->wait) {
> > + mutex_lock(&instance->lock);
> > + list_del(&cmd->head);
> > + mutex_unlock(&instance->lock);
> > + vc_vchi_cmd_delete(instance, cmd);
> > + continue;
> > + }
> > +
> > + if (status) {
> > + complete(&cmd->cmplt);
> > + continue;
> > + }
> > +
> > + } while (1);
> > +
> > + while (!vchi_msg_peek(instance->vchi_handle[0], (void
> > **)&reply,
> > + &reply_len, VCHI_FLAGS_NONE)) {
> > + if (reply->trans_id & 0x80000000) {
> > + /* Async event or cmd from the VPU */
> > + if (instance->vpu_event)
> > + instance->vpu_event(instance,
> > reply,
> > + reply_len);
> > + } else {
> > + vc_sm_cma_vchi_rx_ack(instance, cmd,
> > reply,
> > + reply_len);
> > + }
> > +
> > + vchi_msg_remove(instance->vchi_handle[0]);
> > + }
> > +
> > + /* Go through the dead list and free them */
> > + mutex_lock(&instance->lock);
> > + list_for_each_entry_safe(cmd, cmd_tmp, &instance-
> > >dead_list,
> > + head) {
> > + list_del(&cmd->head);
> > + vc_vchi_cmd_delete(instance, cmd);
> > + }
> > + mutex_unlock(&instance->lock);
> > + }
> > +
> > + return 0;
> > +}
> > +
> > +static void vc_sm_cma_vchi_callback(void *param,
> > + const enum vchi_callback_reason
> > reason,
> > + void *msg_handle)
> > +{
> > + struct sm_instance *instance = param;
> > +
> > + (void)msg_handle;
> > +
> > + switch (reason) {
> > + case VCHI_CALLBACK_MSG_AVAILABLE:
> > + complete(&instance->io_cmplt);
> > + break;
> > +
> > + case VCHI_CALLBACK_SERVICE_CLOSED:
> > + pr_info("%s: service CLOSED!!", __func__);
> > + default:
> > + break;
> > + }
> > +}
> > +
> > +struct sm_instance *vc_sm_cma_vchi_init(struct vchi_instance_handle
> > *vchi_instance,
> > + unsigned int num_connections,
> > + vpu_event_cb vpu_event)
> > +{
> > + u32 i;
> > + struct sm_instance *instance;
> > + int status;
> > +
> > + pr_debug("%s: start", __func__);
> > +
> > + if (num_connections > VCHI_MAX_NUM_CONNECTIONS) {
> > + pr_err("%s: unsupported number of connections %u
> > (max=%u)",
> > + __func__, num_connections,
> > VCHI_MAX_NUM_CONNECTIONS);
> > +
> > + goto err_null;
> > + }
> > + /* Allocate memory for this instance */
> > + instance = kzalloc(sizeof(*instance), GFP_KERNEL);
> > +
> > + /* Misc initialisations */
> > + mutex_init(&instance->lock);
> > + init_completion(&instance->io_cmplt);
> > + INIT_LIST_HEAD(&instance->cmd_list);
> > + INIT_LIST_HEAD(&instance->rsp_list);
> > + INIT_LIST_HEAD(&instance->dead_list);
> > + INIT_LIST_HEAD(&instance->free_list);
> > + sema_init(&instance->free_sema, SM_MAX_NUM_CMD_RSP_BLKS);
> > + mutex_init(&instance->free_lock);
> > + for (i = 0; i < SM_MAX_NUM_CMD_RSP_BLKS; i++) {
> > + init_completion(&instance->free_blk[i].cmplt);
> > + list_add(&instance->free_blk[i].head, &instance-
> > >free_list);
> > + }
> > +
> > + /* Open the VCHI service connections */
> > + instance->num_connections = num_connections;
> > + for (i = 0; i < num_connections; i++) {
> > + struct service_creation params = {
> > + .version = VCHI_VERSION_EX(VC_SM_VER,
> > VC_SM_MIN_VER),
> > + .service_id = VC_SM_SERVER_NAME,
> > + .callback = vc_sm_cma_vchi_callback,
> > + .callback_param = instance,
> > + };
> > +
> > + status = vchi_service_open(vchi_instance,
> > + ¶ms, &instance-
> > >vchi_handle[i]);
> > + if (status) {
> > + pr_err("%s: failed to open VCHI service (%d)",
> > + __func__, status);
> > +
> > + goto err_close_services;
> > + }
> > + }
> > +
> > + /* Create the thread which takes care of all io to/from
> > videoocore. */
> > + instance->io_thread =
> > kthread_create(&vc_sm_cma_vchi_videocore_io,
> > + (void *)instance, "SMIO");
> > + if (!instance->io_thread) {
> > + pr_err("%s: failed to create SMIO thread", __func__);
> > +
> > + goto err_close_services;
> > + }
> > + instance->vpu_event = vpu_event;
> > + set_user_nice(instance->io_thread, -10);
> > + wake_up_process(instance->io_thread);
> > +
> > + pr_debug("%s: success - instance %p", __func__, instance);
> > + return instance;
> > +
> > +err_close_services:
> > + for (i = 0; i < instance->num_connections; i++) {
> > + if (instance->vchi_handle[i])
> > + vchi_service_close(instance->vchi_handle[i]);
> > + }
> > + kfree(instance);
> > +err_null:
> > + pr_debug("%s: FAILED", __func__);
> > + return NULL;
> > +}
> > +
> > +int vc_sm_cma_vchi_stop(struct sm_instance **handle)
> > +{
> > + struct sm_instance *instance;
> > + u32 i;
> > +
> > + if (!handle) {
> > + pr_err("%s: invalid pointer to handle %p", __func__,
> > handle);
> > + goto lock;
> > + }
> > +
> > + if (!*handle) {
> > + pr_err("%s: invalid handle %p", __func__, *handle);
> > + goto lock;
> > + }
> > +
> > + instance = *handle;
> > +
> > + /* Close all VCHI service connections */
> > + for (i = 0; i < instance->num_connections; i++) {
> > + s32 success;
> > +
> > + vchi_service_use(instance->vchi_handle[i]);
> > +
> > + success = vchi_service_close(instance->vchi_handle[i]);
> > + }
> > +
> > + kfree(instance);
> > +
> > + *handle = NULL;
> > + return 0;
> > +
> > +lock:
> > + return -EINVAL;
> > +}
> > +
> > +static int vc_sm_cma_vchi_send_msg(struct sm_instance *handle,
> > + enum vc_sm_msg_type msg_id, void
> > *msg,
> > + u32 msg_size, void *result, u32
> > result_size,
> > + u32 *cur_trans_id, u8 wait_reply)
> > +{
> > + int status = 0;
> > + struct sm_instance *instance = handle;
> > + struct sm_cmd_rsp_blk *cmd_blk;
> > +
> > + if (!handle) {
> > + pr_err("%s: invalid handle", __func__);
> > + return -EINVAL;
> > + }
> > + if (!msg) {
> > + pr_err("%s: invalid msg pointer", __func__);
> > + return -EINVAL;
> > + }
> > +
> > + cmd_blk =
> > + vc_vchi_cmd_create(instance, msg_id, msg, msg_size,
> > wait_reply);
> > + if (!cmd_blk) {
> > + pr_err("[%s]: failed to allocate global tracking
> > resource",
> > + __func__);
> > + return -ENOMEM;
> > + }
> > +
> > + if (cur_trans_id)
> > + *cur_trans_id = cmd_blk->id;
> > +
> > + mutex_lock(&instance->lock);
> > + list_add_tail(&cmd_blk->head, &instance->cmd_list);
> > + mutex_unlock(&instance->lock);
> > + complete(&instance->io_cmplt);
> > +
> > + if (!wait_reply)
> > + /* We're done */
> > + return 0;
> > +
> > + /* Wait for the response */
> > + if (wait_for_completion_interruptible(&cmd_blk->cmplt)) {
> > + mutex_lock(&instance->lock);
> > + if (!cmd_blk->sent) {
> > + list_del(&cmd_blk->head);
> > + mutex_unlock(&instance->lock);
> > + vc_vchi_cmd_delete(instance, cmd_blk);
> > + return -ENXIO;
> > + }
> > +
> > + list_move(&cmd_blk->head, &instance->dead_list);
> > + mutex_unlock(&instance->lock);
> > + complete(&instance->io_cmplt);
> > + return -EINTR; /* We're done */
> > + }
> > +
> > + if (result && result_size) {
> > + memcpy(result, cmd_blk->msg, result_size);
> > + } else {
> > + struct vc_sm_result_t *res =
> > + (struct vc_sm_result_t *)cmd_blk->msg;
> > + status = (res->success == 0) ? 0 : -ENXIO;
> > + }
> > +
> > + mutex_lock(&instance->lock);
> > + list_del(&cmd_blk->head);
> > + mutex_unlock(&instance->lock);
> > + vc_vchi_cmd_delete(instance, cmd_blk);
> > + return status;
> > +}
> > +
> > +int vc_sm_cma_vchi_free(struct sm_instance *handle, struct
> > vc_sm_free_t *msg,
> > + u32 *cur_trans_id)
> > +{
> > + return vc_sm_cma_vchi_send_msg(handle, VC_SM_MSG_TYPE_FREE,
> > + msg, sizeof(*msg), 0, 0,
> > cur_trans_id, 0);
> > +}
> > +
> > +int vc_sm_cma_vchi_import(struct sm_instance *handle, struct
> > vc_sm_import *msg,
> > + struct vc_sm_import_result *result, u32
> > *cur_trans_id)
> > +{
> > + return vc_sm_cma_vchi_send_msg(handle, VC_SM_MSG_TYPE_IMPORT,
> > + msg, sizeof(*msg), result,
> > sizeof(*result),
> > + cur_trans_id, 1);
> > +}
> > +
> > +int vc_sm_cma_vchi_client_version(struct sm_instance *handle,
> > + struct vc_sm_version *msg,
> > + struct vc_sm_result_t *result,
> > + u32 *cur_trans_id)
> > +{
> > + return vc_sm_cma_vchi_send_msg(handle,
> > VC_SM_MSG_TYPE_CLIENT_VERSION,
> > + //msg, sizeof(*msg), result,
> > sizeof(*result),
> > + //cur_trans_id, 1);
> > + msg, sizeof(*msg), NULL, 0,
> > + cur_trans_id, 0);
> > +}
> > +
> > +int vc_sm_vchi_client_vc_mem_req_reply(struct sm_instance *handle,
> > + struct
> > vc_sm_vc_mem_request_result *msg,
> > + uint32_t *cur_trans_id)
> > +{
> > + return vc_sm_cma_vchi_send_msg(handle,
> > + VC_SM_MSG_TYPE_VC_MEM_REQUEST_RE
> > PLY,
> > + msg, sizeof(*msg), 0, 0,
> > cur_trans_id,
> > + 0);
> > +}
> > diff --git a/drivers/staging/vc04_services/vc-sm-cma/vc_sm_cma_vchi.h
> > b/drivers/staging/vc04_services/vc-sm-cma/vc_sm_cma_vchi.h
> > new file mode 100644
> > index 000000000000..e8db34bd1e91
> > --- /dev/null
> > +++ b/drivers/staging/vc04_services/vc-sm-cma/vc_sm_cma_vchi.h
> > @@ -0,0 +1,63 @@
> > +/* SPDX-License-Identifier: GPL-2.0 */
> > +
> > +/*
> > + * VideoCore Shared Memory CMA allocator
> > + *
> > + * Copyright: 2018, Raspberry Pi (Trading) Ltd
> > + * Copyright 2011-2012 Broadcom Corporation. All rights reserved.
> > + *
> > + * Based on vmcs_sm driver from Broadcom Corporation.
> > + *
> > + */
> > +
> > +#ifndef __VC_SM_CMA_VCHI_H__INCLUDED__
> > +#define __VC_SM_CMA_VCHI_H__INCLUDED__
> > +
> > +#include "interface/vchi/vchi.h"
> > +
> > +#include "vc_sm_defs.h"
> > +
> > +/*
> > + * Forward declare.
> > + */
> > +struct sm_instance;
> > +
> > +typedef void (*vpu_event_cb)(struct sm_instance *instance,
> > + struct vc_sm_result_t *reply, int
> > reply_len);
> > +
> > +/*
> > + * Initialize the shared memory service, opens up vchi connection to
> > talk to it.
> > + */
> > +struct sm_instance *vc_sm_cma_vchi_init(struct vchi_instance_handle
> > *vchi_instance,
> > + unsigned int num_connections,
> > + vpu_event_cb vpu_event);
> > +
> > +/*
> > + * Terminates the shared memory service.
> > + */
> > +int vc_sm_cma_vchi_stop(struct sm_instance **handle);
> > +
> > +/*
> > + * Ask the shared memory service to free up some memory that was
> > previously
> > + * allocated by the vc_sm_cma_vchi_alloc function call.
> > + */
> > +int vc_sm_cma_vchi_free(struct sm_instance *handle, struct
> > vc_sm_free_t *msg,
> > + u32 *cur_trans_id);
> > +
> > +/*
> > + * Import a contiguous block of memory and wrap it in a GPU
> > MEM_HANDLE_T.
> > + */
> > +int vc_sm_cma_vchi_import(struct sm_instance *handle, struct
> > vc_sm_import *msg,
> > + struct vc_sm_import_result *result,
> > + u32 *cur_trans_id);
> > +
> > +int vc_sm_cma_vchi_client_version(struct sm_instance *handle,
> > + struct vc_sm_version *msg,
> > + struct vc_sm_result_t *result,
> > + u32 *cur_trans_id);
> > +
> > +int vc_sm_vchi_client_vc_mem_req_reply(struct sm_instance *handle,
> > + struct
> > vc_sm_vc_mem_request_result *msg,
> > + uint32_t *cur_trans_id);
> > +
> > +#endif /* __VC_SM_CMA_VCHI_H__INCLUDED__ */
> > diff --git a/drivers/staging/vc04_services/vc-sm-cma/vc_sm_defs.h
> > b/drivers/staging/vc04_services/vc-sm-cma/vc_sm_defs.h
> > new file mode 100644
> > index 000000000000..8a0d1f6dbfe8
> > --- /dev/null
> > +++ b/drivers/staging/vc04_services/vc-sm-cma/vc_sm_defs.h
> > @@ -0,0 +1,300 @@
> > +/* SPDX-License-Identifier: GPL-2.0 */
> > +
> > +/*
> > + * VideoCore Shared Memory CMA allocator
> > + *
> > + * Copyright: 2018, Raspberry Pi (Trading) Ltd
> > + *
> > + * Based on vc_sm_defs.h from the vmcs_sm driver Copyright Broadcom
> > Corporation.
> > + * All IPC messages are copied across to this file, even if the vc-
> > sm-cma
> > + * driver is not currently using them.
> > + *
> > +
> > *********************************************************************
> > *******
> > + */
> > +
> > +#ifndef __VC_SM_DEFS_H__INCLUDED__
> > +#define __VC_SM_DEFS_H__INCLUDED__
> > +
> > +/* FourCC code used for VCHI connection */
> > +#define VC_SM_SERVER_NAME MAKE_FOURCC("SMEM")
> > +
> > +/* Maximum message length */
> > +#define VC_SM_MAX_MSG_LEN (sizeof(union vc_sm_msg_union_t) + \
> > + sizeof(struct vc_sm_msg_hdr_t))
> > +#define VC_SM_MAX_RSP_LEN (sizeof(union vc_sm_msg_union_t))
> > +
> > +/* Resource name maximum size */
> > +#define VC_SM_RESOURCE_NAME 32
> > +
> > +/*
> > + * Version to be reported to the VPU
> > + * VPU assumes 0 (aka 1) which does not require the released
> > callback, nor
> > + * expect the client to handle VC_MEM_REQUESTS.
> > + * Version 2 requires the released callback, and must support
> > VC_MEM_REQUESTS.
> > + */
> > +#define VC_SM_PROTOCOL_VERSION 2
> > +
> > +enum vc_sm_msg_type {
> > + /* Message types supported for HOST->VC direction */
> > +
> > + /* Allocate shared memory block */
> > + VC_SM_MSG_TYPE_ALLOC,
> > + /* Lock allocated shared memory block */
> > + VC_SM_MSG_TYPE_LOCK,
> > + /* Unlock allocated shared memory block */
> > + VC_SM_MSG_TYPE_UNLOCK,
> > + /* Unlock allocated shared memory block, do not answer command
> > */
> > + VC_SM_MSG_TYPE_UNLOCK_NOANS,
> > + /* Free shared memory block */
> > + VC_SM_MSG_TYPE_FREE,
> > + /* Resize a shared memory block */
> > + VC_SM_MSG_TYPE_RESIZE,
> > + /* Walk the allocated shared memory block(s) */
> > + VC_SM_MSG_TYPE_WALK_ALLOC,
> > +
> > + /* A previously applied action will need to be reverted */
> > + VC_SM_MSG_TYPE_ACTION_CLEAN,
> > +
> > + /*
> > + * Import a physical address and wrap into a MEM_HANDLE_T.
> > + * Release with VC_SM_MSG_TYPE_FREE.
> > + */
> > + VC_SM_MSG_TYPE_IMPORT,
> > + /*
> > + *Tells VC the protocol version supported by this client.
> > + * 2 supports the async/cmd messages from the VPU for final
> > release
> > + * of memory, and for VC allocations.
> > + */
> > + VC_SM_MSG_TYPE_CLIENT_VERSION,
> > + /* Response to VC request for memory */
> > + VC_SM_MSG_TYPE_VC_MEM_REQUEST_REPLY,
> > +
> > + /*
> > + * Asynchronous/cmd messages supported for VC->HOST direction.
> > + * Signalled by setting the top bit in vc_sm_result_t trans_id.
> > + */
> > +
> > + /*
> > + * VC has finished with an imported memory allocation.
> > + * Release any Linux reference counts on the underlying block.
> > + */
> > + VC_SM_MSG_TYPE_RELEASED,
> > + /* VC request for memory */
> > + VC_SM_MSG_TYPE_VC_MEM_REQUEST,
> > +
> > + VC_SM_MSG_TYPE_MAX
> > +};
> > +
> > +/* Type of memory to be allocated */
> > +enum vc_sm_alloc_type_t {
> > + VC_SM_ALLOC_CACHED,
> > + VC_SM_ALLOC_NON_CACHED,
> > +};
> > +
> > +/* Message header for all messages in HOST->VC direction */
> > +struct vc_sm_msg_hdr_t {
> > + u32 type;
> > + u32 trans_id;
> > + u8 body[0];
> > +
> > +};
> > +
> > +/* Request to allocate memory (HOST->VC) */
> > +struct vc_sm_alloc_t {
> > + /* type of memory to allocate */
> > + enum vc_sm_alloc_type_t type;
> > + /* byte amount of data to allocate per unit */
> > + u32 base_unit;
> > + /* number of unit to allocate */
> > + u32 num_unit;
> > + /* alignment to be applied on allocation */
> > + u32 alignment;
> > + /* identity of who allocated this block */
> > + u32 allocator;
> > + /* resource name (for easier tracking on vc side) */
> > + char name[VC_SM_RESOURCE_NAME];
> > +
> > +};
> > +
> > +/* Result of a requested memory allocation (VC->HOST) */
> > +struct vc_sm_alloc_result_t {
> > + /* Transaction identifier */
> > + u32 trans_id;
> > +
> > + /* Resource handle */
> > + u32 res_handle;
> > + /* Pointer to resource buffer */
> > + u32 res_mem;
> > + /* Resource base size (bytes) */
> > + u32 res_base_size;
> > + /* Resource number */
> > + u32 res_num;
> > +
> > +};
> > +
> > +/* Request to free a previously allocated memory (HOST->VC) */
> > +struct vc_sm_free_t {
> > + /* Resource handle (returned from alloc) */
> > + u32 res_handle;
> > + /* Resource buffer (returned from alloc) */
> > + u32 res_mem;
> > +
> > +};
> > +
> > +/* Request to lock a previously allocated memory (HOST->VC) */
> > +struct vc_sm_lock_unlock_t {
> > + /* Resource handle (returned from alloc) */
> > + u32 res_handle;
> > + /* Resource buffer (returned from alloc) */
> > + u32 res_mem;
> > +
> > +};
> > +
> > +/* Request to resize a previously allocated memory (HOST->VC) */
> > +struct vc_sm_resize_t {
> > + /* Resource handle (returned from alloc) */
> > + u32 res_handle;
> > + /* Resource buffer (returned from alloc) */
> > + u32 res_mem;
> > + /* Resource *new* size requested (bytes) */
> > + u32 res_new_size;
> > +
> > +};
> > +
> > +/* Result of a requested memory lock (VC->HOST) */
> > +struct vc_sm_lock_result_t {
> > + /* Transaction identifier */
> > + u32 trans_id;
> > +
> > + /* Resource handle */
> > + u32 res_handle;
> > + /* Pointer to resource buffer */
> > + u32 res_mem;
> > + /*
> > + * Pointer to former resource buffer if the memory
> > + * was reallocated
> > + */
> > + u32 res_old_mem;
> > +
> > +};
> > +
> > +/* Generic result for a request (VC->HOST) */
> > +struct vc_sm_result_t {
> > + /* Transaction identifier */
> > + u32 trans_id;
> > +
> > + s32 success;
> > +
> > +};
> > +
> > +/* Request to revert a previously applied action (HOST->VC) */
> > +struct vc_sm_action_clean_t {
> > + /* Action of interest */
> > + enum vc_sm_msg_type res_action;
> > + /* Transaction identifier for the action of interest */
> > + u32 action_trans_id;
> > +
> > +};
> > +
> > +/* Request to remove all data associated with a given allocator
> > (HOST->VC) */
> > +struct vc_sm_free_all_t {
> > + /* Allocator identifier */
> > + u32 allocator;
> > +};
> > +
> > +/* Request to import memory (HOST->VC) */
> > +struct vc_sm_import {
> > + /* type of memory to allocate */
> > + enum vc_sm_alloc_type_t type;
> > + /* pointer to the VC (ie physical) address of the allocated
> > memory */
> > + u32 addr;
> > + /* size of buffer */
> > + u32 size;
> > + /* opaque handle returned in RELEASED messages */
> > + u32 kernel_id;
> > + /* Allocator identifier */
> > + u32 allocator;
> > + /* resource name (for easier tracking on vc side) */
> > + char name[VC_SM_RESOURCE_NAME];
> > +};
> > +
> > +/* Result of a requested memory import (VC->HOST) */
> > +struct vc_sm_import_result {
> > + /* Transaction identifier */
> > + u32 trans_id;
> > +
> > + /* Resource handle */
> > + u32 res_handle;
> > +};
> > +
> > +/* Notification that VC has finished with an allocation (VC->HOST)
> > */
> > +struct vc_sm_released {
> > + /* cmd type / trans_id */
> > + u32 cmd;
> > +
> > + /* pointer to the VC (ie physical) address of the allocated
> > memory */
> > + u32 addr;
> > + /* size of buffer */
> > + u32 size;
> > + /* opaque handle returned in RELEASED messages */
> > + u32 kernel_id;
> > + u32 vc_handle;
> > +};
> > +
> > +/*
> > + * Client informing VC as to the protocol version it supports.
> > + * >=2 requires the released callback, and supports VC asking for
> > memory.
> > + * Failure means that the firmware doesn't support this call, and
> > therefore the
> > + * client should either fail, or NOT rely on getting the released
> > callback.
> > + */
> > +struct vc_sm_version {
> > + u32 version;
> > +};
> > +
> > +/* Request FROM VideoCore for some memory */
> > +struct vc_sm_vc_mem_request {
> > + /* cmd type */
> > + u32 cmd;
> > +
> > + /* trans_id (from VPU) */
> > + u32 trans_id;
> > + /* size of buffer */
> > + u32 size;
> > + /* alignment of buffer */
> > + u32 align;
> > + /* resource name (for easier tracking) */
> > + char name[VC_SM_RESOURCE_NAME];
> > + /* VPU handle for the resource */
> > + u32 vc_handle;
> > +};
> > +
> > +/* Response from the kernel to provide the VPU with some memory */
> > +struct vc_sm_vc_mem_request_result {
> > + /* Transaction identifier for the VPU */
> > + u32 trans_id;
> > + /* pointer to the physical address of the allocated memory */
> > + u32 addr;
> > + /* opaque handle returned in RELEASED messages */
> > + u32 kernel_id;
> > +};
> > +
> > +/* Union of ALL messages */
> > +union vc_sm_msg_union_t {
> > + struct vc_sm_alloc_t alloc;
> > + struct vc_sm_alloc_result_t alloc_result;
> > + struct vc_sm_free_t free;
> > + struct vc_sm_lock_unlock_t lock_unlock;
> > + struct vc_sm_action_clean_t action_clean;
> > + struct vc_sm_resize_t resize;
> > + struct vc_sm_lock_result_t lock_result;
> > + struct vc_sm_result_t result;
> > + struct vc_sm_free_all_t free_all;
> > + struct vc_sm_import import;
> > + struct vc_sm_import_result import_result;
> > + struct vc_sm_version version;
> > + struct vc_sm_released released;
> > + struct vc_sm_vc_mem_request vc_request;
> > + struct vc_sm_vc_mem_request_result vc_request_result;
> > +};
> > +
> > +#endif /* __VC_SM_DEFS_H__INCLUDED__ */
> > diff --git a/drivers/staging/vc04_services/vc-sm-cma/vc_sm_knl.h
> > b/drivers/staging/vc04_services/vc-sm-cma/vc_sm_knl.h
> > new file mode 100644
> > index 000000000000..988fdd967922
> > --- /dev/null
> > +++ b/drivers/staging/vc04_services/vc-sm-cma/vc_sm_knl.h
> > @@ -0,0 +1,28 @@
> > +/* SPDX-License-Identifier: GPL-2.0 */
> > +
> > +/*
> > + * VideoCore Shared Memory CMA allocator
> > + *
> > + * Copyright: 2018, Raspberry Pi (Trading) Ltd
> > + *
> > + * Based on vc_sm_defs.h from the vmcs_sm driver Copyright Broadcom
> > Corporation.
> > + *
> > + */
> > +
> > +#ifndef __VC_SM_KNL_H__INCLUDED__
> > +#define __VC_SM_KNL_H__INCLUDED__
> > +
> > +#if !defined(__KERNEL__)
> > +#error "This interface is for kernel use only..."
> > +#endif
> > +
> > +/* Free a previously allocated or imported shared memory handle and
> > block. */
> > +int vc_sm_cma_free(void *handle);
> > +
> > +/* Get an internal resource handle mapped from the external one. */
> > +int vc_sm_cma_int_handle(void *handle);
> > +
> > +/* Import a block of memory into the GPU space. */
> > +int vc_sm_cma_import_dmabuf(struct dma_buf *dmabuf, void **handle);
> > +
> > +#endif /* __VC_SM_KNL_H__INCLUDED__ */
>
next prev parent reply other threads:[~2020-05-06 19:25 UTC|newest]
Thread overview: 104+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-04 9:25 [PATCH v2 00/34] Drivers for the BCM283x CSI-2/CCP2 receiver and ISP Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 01/34] media: uapi: v4l2-core: Add sensor ancillary data V4L2 fourcc type Laurent Pinchart
2020-05-04 13:48 ` Hans Verkuil
2020-05-04 14:39 ` Dave Stevenson
2020-05-04 15:32 ` Hans Verkuil
2020-05-04 16:08 ` Laurent Pinchart
2020-05-05 11:20 ` Dave Stevenson
2020-05-04 9:25 ` [PATCH v2 02/34] media: uapi: Add MEDIA_BUS_FMT_SENSOR_DATA media bus format Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 03/34] dt-bindings: media: Document BCM283x CSI2/CCP2 receiver Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 04/34] media: bcm2835-unicam: Driver for CCP2/CSI2 camera interface Laurent Pinchart
2020-05-04 15:21 ` Hans Verkuil
2020-05-05 1:26 ` kbuild test robot
2020-05-06 18:01 ` Nicolas Saenz Julienne
2020-08-29 11:20 ` Jacopo Mondi
2020-08-29 18:32 ` Laurent Pinchart
2020-08-31 7:38 ` Jacopo Mondi
2020-08-31 14:17 ` Laurent Pinchart
2020-08-31 14:46 ` Jacopo Mondi
2020-08-31 14:56 ` Laurent Pinchart
2020-09-01 8:41 ` Dave Stevenson
2020-09-01 10:22 ` Jacopo Mondi
2020-09-01 16:37 ` Dave Stevenson
2020-09-01 17:11 ` Laurent Pinchart
2020-09-15 7:03 ` Sakari Ailus
2020-09-15 9:32 ` Laurent Pinchart
2020-09-15 13:28 ` Dave Stevenson
2020-10-30 17:53 ` Jacopo Mondi
2020-09-15 17:30 ` Dave Stevenson
2020-05-04 9:25 ` [PATCH v2 05/34] ARM: dts: bcm2711: Add Unicam DT nodes Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 06/34] staging: vc04_services: Add new vc-sm-cma driver Laurent Pinchart
2020-05-05 2:38 ` kbuild test robot
2020-05-05 12:17 ` kbuild test robot
2020-05-06 3:05 ` kbuild test robot
2020-05-06 18:04 ` Nicolas Saenz Julienne
2020-05-06 19:24 ` Dave Stevenson [this message]
2020-05-08 0:11 ` Laurent Pinchart
2020-05-18 12:06 ` Hans Verkuil
2020-08-24 16:39 ` Jacopo Mondi
2020-08-25 17:52 ` Dave Stevenson
2020-08-27 10:38 ` Jacopo Mondi
2020-08-27 12:51 ` Dave Stevenson
2020-08-27 16:46 ` Jacopo Mondi
2020-08-27 17:19 ` Dave Stevenson
2020-05-11 18:42 ` Nicolas Saenz Julienne
2020-05-18 15:48 ` Dave Stevenson
2020-05-20 14:41 ` Nicolas Saenz Julienne
2020-05-21 11:01 ` Dave Stevenson
2020-05-04 9:25 ` [PATCH v2 07/34] staging: bcm2835: Break MMAL support out from camera Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 08/34] staging: mmal-vchiq: Allocate and free components as required Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 09/34] staging: mmal-vchiq: Avoid use of bool in structures Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 10/34] staging: mmal-vchiq: Make timeout a defined parameter Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 11/34] staging: mmal-vchiq: Make a mmal_buf struct for passing parameters Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 12/34] staging: mmal-vchiq: Add support for event callbacks Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 13/34] staging: mmal-vchiq: Support sending data to MMAL ports Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 14/34] staging: mmal-vchiq: Fixup vchiq-mmal include ordering Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 15/34] staging: mmal-vchiq: Use vc-sm-cma to support zero copy Laurent Pinchart
2020-05-04 16:30 ` Nicolas Saenz Julienne
2020-05-05 16:07 ` kbuild test robot
2020-05-11 19:15 ` Nicolas Saenz Julienne
2020-05-04 9:25 ` [PATCH v2 16/34] staging: mmal-vchiq: Fix client_component for 64 bit kernel Laurent Pinchart
2020-05-18 10:19 ` Hans Verkuil
2020-05-04 9:25 ` [PATCH v2 17/34] staging: mmal_vchiq: Add in the Bayer encoding formats Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 18/34] staging: mmal-vchiq: Always return the param size from param_get Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 19/34] staging: mmal-vchiq: If the VPU returns an error, don't negate it Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 20/34] staging: mmal-vchiq: Fix handling of VB2_MEMORY_DMABUF buffers Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 21/34] staging: mmal-vchiq: Update mmal_parameters.h with recently defined params Laurent Pinchart
2020-05-04 9:25 ` [PATCH v2 22/34] staging: mmal-vchiq: Free the event context for control ports Laurent Pinchart
2020-05-04 9:26 ` [PATCH v2 23/34] staging: mmal-vchiq: Fix memory leak in error path Laurent Pinchart
2020-05-04 9:26 ` [PATCH v2 24/34] staging: mmal-vchiq: Fix formatting errors in mmal_parameters.h Laurent Pinchart
2020-05-04 9:26 ` [PATCH v2 25/34] staging: vchiq_arm: Register vcsm-cma as a platform driver Laurent Pinchart
2020-05-04 9:26 ` [PATCH v2 26/34] staging: vchiq_arm: Set up dma ranges on child devices Laurent Pinchart
2020-05-04 16:54 ` Nicolas Saenz Julienne
2020-08-25 16:57 ` Jacopo Mondi
2020-05-04 9:26 ` [PATCH v2 27/34] staging: vchiq: Use the old dma controller for OF config on platform devices Laurent Pinchart
2020-05-04 15:44 ` Nicolas Saenz Julienne
2020-05-04 9:26 ` [PATCH v2 28/34] staging: vchiq_2835_arm: Implement a DMA pool for small bulk transfers Laurent Pinchart
2020-05-04 9:26 ` [PATCH v2 29/34] staging: vchiq: Add 36-bit address support Laurent Pinchart
2020-05-04 17:40 ` Nicolas Saenz Julienne
2020-05-04 20:46 ` Phil Elwell
2020-05-05 10:13 ` Nicolas Saenz Julienne
2020-05-05 10:57 ` Phil Elwell
2020-05-05 13:22 ` kbuild test robot
2020-05-04 9:26 ` [PATCH v2 30/34] staging: vchiq_arm: Give vchiq children DT nodes Laurent Pinchart
2020-05-04 17:12 ` Nicolas Saenz Julienne
2020-05-04 19:42 ` Phil Elwell
2020-05-05 10:37 ` Nicolas Saenz Julienne
2020-05-05 10:50 ` Phil Elwell
2020-05-04 9:26 ` [PATCH v2 31/34] staging: vchiq_arm: Add a matching unregister call Laurent Pinchart
2020-05-04 9:26 ` [PATCH v2 32/34] media: videobuf2: Allow exporting of a struct dmabuf Laurent Pinchart
2020-05-04 13:36 ` Hans Verkuil
2020-05-04 9:26 ` [PATCH v2 33/34] staging: bcm2835-isp: Add support for BC2835 ISP Laurent Pinchart
2020-05-11 19:19 ` Nicolas Saenz Julienne
2020-05-18 13:38 ` Dave Stevenson
2020-05-20 13:46 ` Nicolas Saenz Julienne
2020-05-18 12:02 ` Hans Verkuil
2020-05-18 14:36 ` Dave Stevenson
2020-05-18 15:07 ` Hans Verkuil
2020-05-19 12:47 ` Naushir Patuck
2020-05-19 13:03 ` Jacopo Mondi
2020-06-24 11:28 ` Hans Verkuil
2020-05-04 9:26 ` [PATCH v2 34/34] staging: vchiq: Load bcm2835_isp driver from vchiq Laurent Pinchart
2020-05-04 15:15 ` [PATCH v2 00/34] Drivers for the BCM283x CSI-2/CCP2 receiver and ISP Nicolas Saenz Julienne
2020-05-04 15:38 ` Laurent Pinchart
2020-05-04 16:15 ` Nicolas Saenz Julienne
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAPY8ntB3RCwzCzj+v0QEQp19f-X7tgfhFZ_DK3-LUiWXGK0eNA@mail.gmail.com \
--to=dave.stevenson@raspberrypi.com \
--cc=dave.stevenson@raspberrypi.org \
--cc=jacopo@jmondi.org \
--cc=kieran.bingham@ideasonboard.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=naush@raspberrypi.com \
--cc=niklas.soderlund@ragnatech.se \
--cc=nsaenzjulienne@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).