From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-14.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 129C8C433DF for ; Mon, 24 Aug 2020 17:16:51 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id CE3E32067C for ; Mon, 24 Aug 2020 17:16:50 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727019AbgHXRQr (ORCPT ); Mon, 24 Aug 2020 13:16:47 -0400 Received: from relay12.mail.gandi.net ([217.70.178.232]:60797 "EHLO relay12.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728289AbgHXQgk (ORCPT ); Mon, 24 Aug 2020 12:36:40 -0400 Received: from uno.localdomain (93-34-118-233.ip49.fastwebnet.it [93.34.118.233]) (Authenticated sender: jacopo@jmondi.org) by relay12.mail.gandi.net (Postfix) with ESMTPSA id 77EE820000A; Mon, 24 Aug 2020 16:36:06 +0000 (UTC) Date: Mon, 24 Aug 2020 18:39:52 +0200 From: Jacopo Mondi To: Dave Stevenson Cc: Nicolas Saenz Julienne , Laurent Pinchart , Linux Media Mailing List , Dave Stevenson , Kieran Bingham , Niklas =?utf-8?Q?S=C3=B6derlund?= , Naushir Patuck Subject: Re: [PATCH v2 06/34] staging: vc04_services: Add new vc-sm-cma driver Message-ID: <20200824163952.t7hfri4ml4gjnddq@uno.localdomain> References: <20200504092611.9798-1-laurent.pinchart@ideasonboard.com> <20200504092611.9798-7-laurent.pinchart@ideasonboard.com> <4e42a593f16614e913613150a90e851acbfcaa8c.camel@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org Hi Dave, Nicolas, Laurent, On Wed, May 06, 2020 at 08:24:38PM +0100, Dave Stevenson wrote: > Hi Nicolas > > On Wed, 6 May 2020 at 19:04, Nicolas Saenz Julienne > wrote: > > > > Hi Laurent, Dave, > > > > On Mon, 2020-05-04 at 12:25 +0300, Laurent Pinchart wrote: > > > From: Dave Stevenson > > > > > > 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. As months have passed, libcamera moved to allocate lens shading tables using dma-buf heaps and the only user I can name of the vc-sm-cma driver is the actual ISP, that needs to import the dmabuf pointing to the lens shading maps with vc_sm_cma_import_dmabuf(). Upstreaming the whole vc-sm-cma driver as it is for this single kAPI seems a bit a no-go. Dave, what would you prefer here ? Should I provide a minimal vc-sm-cam driver that only performs buffer importing to support the ISP driver ? Is the buffer importing into VPU there to stay or is its usage transitional and can be kept out of the next submission of this series ? Thanks j > > 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 > > > Signed-off-by: Dave Stevenson > > > Signed-off-by: Jacopo Mondi > > > --- > > > 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 /* Needed for standard types */ > > > +#else > > > +#include > > > +#endif > > > + > > > +#include > > > + > > > +/* ---- 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 > > > + * > > > + * 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 > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > + > > > +#include "vchiq_connected.h" > > > +#include "vc_sm_cma_vchi.h" > > > + > > > +#include "vc_sm.h" > > > +#include "vc_sm_knl.h" > > > +#include > > > + > > > +/* ---- 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 > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > + > > > +#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 > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > + > > > +#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__ */ > >