lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAHUa44EjN6H-Nb7Jzfu3O16GXsMajua_n2MaABBPFZS7MGzfAg@mail.gmail.com>
Date: Thu, 9 Jan 2025 08:17:48 +0100
From: Jens Wiklander <jens.wiklander@...aro.org>
To: Jens Wiklander <jens.wiklander@...aro.org>, linux-kernel@...r.kernel.org, 
	linux-media@...r.kernel.org, dri-devel@...ts.freedesktop.org, 
	linaro-mm-sig@...ts.linaro.org, op-tee@...ts.trustedfirmware.org, 
	linux-arm-kernel@...ts.infradead.org, Olivier Masse <olivier.masse@....com>, 
	Thierry Reding <thierry.reding@...il.com>, Yong Wu <yong.wu@...iatek.com>, 
	Sumit Semwal <sumit.semwal@...aro.org>, 
	Benjamin Gaignard <benjamin.gaignard@...labora.com>, Brian Starkey <Brian.Starkey@....com>, 
	John Stultz <jstultz@...gle.com>, "T . J . Mercier" <tjmercier@...gle.com>, 
	Christian König <christian.koenig@....com>, 
	Sumit Garg <sumit.garg@...aro.org>, Matthias Brugger <matthias.bgg@...il.com>, 
	AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>, azarrabi@....qualcomm.com
Subject: Re: [PATCH v4 1/6] tee: add restricted memory allocation

On Wed, Jan 8, 2025 at 5:54 PM Simona Vetter <simona.vetter@...ll.ch> wrote:
>
> On Tue, Dec 17, 2024 at 11:07:37AM +0100, Jens Wiklander wrote:
> > Add restricted memory allocation to the TEE subsystem.
> >
> > Restricted memory refers to memory buffers behind a hardware enforced
> > firewall. It is not accessible to the kernel during normal circumstances
> > but rather only accessible to certain hardware IPs or CPUs executing in
> > higher privileged mode than the kernel itself. This interface allows to
> > allocate and manage such restricted memory buffers via interaction with
> > a TEE implementation.
> >
> > A new ioctl TEE_IOC_RSTMEM_ALLOC is added to allocate these restricted
> > memory buffers.
> >
> > The restricted memory is allocated for a specific use-case, like Secure
> > Video Playback, Trusted UI, or Secure Video Recording where certain
> > hardware devices can access the memory.
> >
> > More use-cases can be added in userspace ABI, but it's up to the backend
> > drivers to provide the implementation.
> >
> > Signed-off-by: Jens Wiklander <jens.wiklander@...aro.org>
> > ---
> >  drivers/tee/Makefile       |   1 +
> >  drivers/tee/tee_core.c     |  38 ++++++-
> >  drivers/tee/tee_private.h  |   2 +
> >  drivers/tee/tee_rstmem.c   | 201 +++++++++++++++++++++++++++++++++++++
> >  drivers/tee/tee_shm.c      |   2 +
> >  drivers/tee/tee_shm_pool.c |  69 ++++++++++++-
> >  include/linux/tee_core.h   |  15 +++
> >  include/linux/tee_drv.h    |   2 +
> >  include/uapi/linux/tee.h   |  44 +++++++-
> >  9 files changed, 370 insertions(+), 4 deletions(-)
> >  create mode 100644 drivers/tee/tee_rstmem.c
> >
> > diff --git a/drivers/tee/Makefile b/drivers/tee/Makefile
> > index 5488cba30bd2..a4c6b55444b9 100644
> > --- a/drivers/tee/Makefile
> > +++ b/drivers/tee/Makefile
> > @@ -3,6 +3,7 @@ obj-$(CONFIG_TEE) += tee.o
> >  tee-objs += tee_core.o
> >  tee-objs += tee_shm.o
> >  tee-objs += tee_shm_pool.o
> > +tee-objs += tee_rstmem.o
> >  obj-$(CONFIG_OPTEE) += optee/
> >  obj-$(CONFIG_AMDTEE) += amdtee/
> >  obj-$(CONFIG_ARM_TSTEE) += tstee/
> > diff --git a/drivers/tee/tee_core.c b/drivers/tee/tee_core.c
> > index d113679b1e2d..f4a45b77753b 100644
> > --- a/drivers/tee/tee_core.c
> > +++ b/drivers/tee/tee_core.c
> > @@ -1,12 +1,13 @@
> >  // SPDX-License-Identifier: GPL-2.0-only
> >  /*
> > - * Copyright (c) 2015-2016, Linaro Limited
> > + * Copyright (c) 2015-2022, 2024, Linaro Limited
> >   */
> >
> >  #define pr_fmt(fmt) "%s: " fmt, __func__
> >
> >  #include <linux/cdev.h>
> >  #include <linux/cred.h>
> > +#include <linux/dma-buf.h>
> >  #include <linux/fs.h>
> >  #include <linux/idr.h>
> >  #include <linux/module.h>
> > @@ -815,6 +816,38 @@ static int tee_ioctl_supp_send(struct tee_context *ctx,
> >       return rc;
> >  }
> >
> > +static int
> > +tee_ioctl_rstmem_alloc(struct tee_context *ctx,
> > +                    struct tee_ioctl_rstmem_alloc_data __user *udata)
> > +{
> > +     struct tee_ioctl_rstmem_alloc_data data;
> > +     struct dma_buf *dmabuf;
> > +     int id;
> > +     int fd;
> > +
> > +     if (copy_from_user(&data, udata, sizeof(data)))
> > +             return -EFAULT;
> > +
> > +     if (data.use_case == TEE_IOC_UC_RESERVED)
> > +             return -EINVAL;
> > +
> > +     dmabuf = tee_rstmem_alloc(ctx, data.flags, data.use_case, data.size,
> > +                               &id);
> > +     if (IS_ERR(dmabuf))
> > +             return PTR_ERR(dmabuf);
> > +     if (put_user(id, &udata->id)) {
> > +             fd = -EFAULT;
> > +             goto err;
> > +     }
> > +     fd = dma_buf_fd(dmabuf, O_CLOEXEC);
> > +     if (fd < 0)
> > +             goto err;
> > +     return fd;
> > +err:
> > +     dma_buf_put(dmabuf);
> > +     return fd;
> > +}
> > +
> >  static long tee_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
> >  {
> >       struct tee_context *ctx = filp->private_data;
> > @@ -839,6 +872,8 @@ static long tee_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
> >               return tee_ioctl_supp_recv(ctx, uarg);
> >       case TEE_IOC_SUPPL_SEND:
> >               return tee_ioctl_supp_send(ctx, uarg);
> > +     case TEE_IOC_RSTMEM_ALLOC:
> > +             return tee_ioctl_rstmem_alloc(ctx, uarg);
> >       default:
> >               return -EINVAL;
> >       }
> > @@ -1286,3 +1321,4 @@ MODULE_AUTHOR("Linaro");
> >  MODULE_DESCRIPTION("TEE Driver");
> >  MODULE_VERSION("1.0");
> >  MODULE_LICENSE("GPL v2");
> > +MODULE_IMPORT_NS("DMA_BUF");
> > diff --git a/drivers/tee/tee_private.h b/drivers/tee/tee_private.h
> > index 9bc50605227c..bf97796909c0 100644
> > --- a/drivers/tee/tee_private.h
> > +++ b/drivers/tee/tee_private.h
> > @@ -23,5 +23,7 @@ void teedev_ctx_put(struct tee_context *ctx);
> >  struct tee_shm *tee_shm_alloc_user_buf(struct tee_context *ctx, size_t size);
> >  struct tee_shm *tee_shm_register_user_buf(struct tee_context *ctx,
> >                                         unsigned long addr, size_t length);
> > +struct dma_buf *tee_rstmem_alloc(struct tee_context *ctx, u32 flags,
> > +                              u32 use_case, size_t size, int *shm_id);
> >
> >  #endif /*TEE_PRIVATE_H*/
> > diff --git a/drivers/tee/tee_rstmem.c b/drivers/tee/tee_rstmem.c
> > new file mode 100644
> > index 000000000000..536bca2901e2
> > --- /dev/null
> > +++ b/drivers/tee/tee_rstmem.c
> > @@ -0,0 +1,201 @@
> > +// SPDX-License-Identifier: GPL-2.0-only
> > +/*
> > + * Copyright (c) 2024 Linaro Limited
> > + */
> > +#include <linux/device.h>
> > +#include <linux/dma-buf.h>
> > +#include <linux/genalloc.h>
> > +#include <linux/scatterlist.h>
> > +#include <linux/slab.h>
> > +#include <linux/tee_core.h>
> > +#include "tee_private.h"
> > +
> > +struct tee_rstmem_attachment {
> > +     struct sg_table table;
> > +     struct device *dev;
> > +};
> > +
> > +static int rstmem_dma_attach(struct dma_buf *dmabuf,
> > +                          struct dma_buf_attachment *attachment)
> > +{
> > +     struct tee_shm *shm = dmabuf->priv;
> > +     struct tee_rstmem_attachment *a;
> > +     int rc;
> > +
> > +     a = kzalloc(sizeof(*a), GFP_KERNEL);
> > +     if (!a)
> > +             return -ENOMEM;
> > +
> > +     if (shm->pages) {
> > +             rc = sg_alloc_table_from_pages(&a->table, shm->pages,
> > +                                            shm->num_pages, 0,
> > +                                            shm->num_pages * PAGE_SIZE,
> > +                                            GFP_KERNEL);
> > +             if (rc)
> > +                     goto err;
> > +     } else {
> > +             rc = sg_alloc_table(&a->table, 1, GFP_KERNEL);
> > +             if (rc)
> > +                     goto err;
> > +             sg_set_page(a->table.sgl, phys_to_page(shm->paddr), shm->size,
> > +                         0);
> > +     }
> > +
> > +     a->dev = attachment->dev;
> > +     attachment->priv = a;
> > +
> > +     return 0;
> > +err:
> > +     kfree(a);
> > +     return rc;
> > +}
> > +
> > +static void rstmem_dma_detach(struct dma_buf *dmabuf,
> > +                           struct dma_buf_attachment *attachment)
> > +{
> > +     struct tee_rstmem_attachment *a = attachment->priv;
> > +
> > +     sg_free_table(&a->table);
> > +     kfree(a);
> > +}
> > +
> > +static struct sg_table *
> > +rstmem_dma_map_dma_buf(struct dma_buf_attachment *attachment,
> > +                    enum dma_data_direction direction)
> > +{
> > +     struct tee_rstmem_attachment *a = attachment->priv;
> > +     int ret;
> > +
> > +     ret = dma_map_sgtable(attachment->dev, &a->table, direction,
> > +                           DMA_ATTR_SKIP_CPU_SYNC);
> > +     if (ret)
> > +             return ERR_PTR(ret);
> > +
> > +     return &a->table;
> > +}
> > +
> > +static void rstmem_dma_unmap_dma_buf(struct dma_buf_attachment *attachment,
> > +                                  struct sg_table *table,
> > +                                  enum dma_data_direction direction)
> > +{
> > +     struct tee_rstmem_attachment *a = attachment->priv;
> > +
> > +     WARN_ON(&a->table != table);
> > +
> > +     dma_unmap_sgtable(attachment->dev, table, direction,
> > +                       DMA_ATTR_SKIP_CPU_SYNC);
> > +}
> > +
> > +static int rstmem_dma_buf_begin_cpu_access(struct dma_buf *dmabuf,
> > +                                        enum dma_data_direction direction)
> > +{
> > +     return -EPERM;
> > +}
> > +
> > +static int rstmem_dma_buf_end_cpu_access(struct dma_buf *dmabuf,
> > +                                      enum dma_data_direction direction)
> > +{
> > +     return -EPERM;
> > +}
> > +
> > +static int rstmem_dma_buf_mmap(struct dma_buf *dmabuf,
> > +                            struct vm_area_struct *vma)
> > +{
> > +     return -EPERM;
>
> Just an aside, you should just not implment mmap if it's not possible.
> Then you also don't need the cpu access functions. Both userspace mmap and
> in-kernel vmap support is intentionally optional for dma-buf.

Got it, thanks. I'll remove those in the next version.

Cheers,
Jens

> -Sima
>
> > +}
> > +
> > +static void rstmem_dma_buf_free(struct dma_buf *dmabuf)
> > +{
> > +     struct tee_shm *shm = dmabuf->priv;
> > +
> > +     tee_shm_put(shm);
> > +}
> > +
> > +static const struct dma_buf_ops rstmem_generic_buf_ops = {
> > +     .attach = rstmem_dma_attach,
> > +     .detach = rstmem_dma_detach,
> > +     .map_dma_buf = rstmem_dma_map_dma_buf,
> > +     .unmap_dma_buf = rstmem_dma_unmap_dma_buf,
> > +     .begin_cpu_access = rstmem_dma_buf_begin_cpu_access,
> > +     .end_cpu_access = rstmem_dma_buf_end_cpu_access,
> > +     .mmap = rstmem_dma_buf_mmap,
> > +     .release = rstmem_dma_buf_free,
> > +};
> > +
> > +struct dma_buf *tee_rstmem_alloc(struct tee_context *ctx, u32 flags,
> > +                              u32 use_case, size_t size, int *shm_id)
> > +{
> > +     struct tee_device *teedev = ctx->teedev;
> > +     DEFINE_DMA_BUF_EXPORT_INFO(exp_info);
> > +     struct dma_buf *dmabuf;
> > +     struct tee_shm *shm;
> > +     void *ret;
> > +     int rc;
> > +
> > +     if (!tee_device_get(teedev))
> > +             return ERR_PTR(-EINVAL);
> > +
> > +     if (!teedev->desc->ops->rstmem_alloc ||
> > +         !teedev->desc->ops->rstmem_free) {
> > +             dmabuf = ERR_PTR(-EINVAL);
> > +             goto err;
> > +     }
> > +
> > +     shm = kzalloc(sizeof(*shm), GFP_KERNEL);
> > +     if (!shm) {
> > +             dmabuf = ERR_PTR(-ENOMEM);
> > +             goto err;
> > +     }
> > +
> > +     refcount_set(&shm->refcount, 1);
> > +     shm->flags = TEE_SHM_RESTRICTED;
> > +     shm->use_case = use_case;
> > +     shm->ctx = ctx;
> > +
> > +     mutex_lock(&teedev->mutex);
> > +     shm->id = idr_alloc(&teedev->idr, NULL, 1, 0, GFP_KERNEL);
> > +     mutex_unlock(&teedev->mutex);
> > +     if (shm->id < 0) {
> > +             dmabuf = ERR_PTR(shm->id);
> > +             goto err_kfree;
> > +     }
> > +
> > +     rc = teedev->desc->ops->rstmem_alloc(ctx, shm, flags, use_case, size);
> > +     if (rc) {
> > +             dmabuf = ERR_PTR(rc);
> > +             goto err_idr_remove;
> > +     }
> > +
> > +     mutex_lock(&teedev->mutex);
> > +     ret = idr_replace(&teedev->idr, shm, shm->id);
> > +     mutex_unlock(&teedev->mutex);
> > +     if (IS_ERR(ret)) {
> > +             dmabuf = ret;
> > +             goto err_rstmem_free;
> > +     }
> > +     teedev_ctx_get(ctx);
> > +
> > +     exp_info.ops = &rstmem_generic_buf_ops;
> > +     exp_info.size = shm->size;
> > +     exp_info.priv = shm;
> > +     dmabuf = dma_buf_export(&exp_info);
> > +     if (IS_ERR(dmabuf)) {
> > +             tee_shm_put(shm);
> > +             return dmabuf;
> > +     }
> > +
> > +     *shm_id = shm->id;
> > +     return dmabuf;
> > +
> > +err_rstmem_free:
> > +     teedev->desc->ops->rstmem_free(ctx, shm);
> > +err_idr_remove:
> > +     mutex_lock(&teedev->mutex);
> > +     idr_remove(&teedev->idr, shm->id);
> > +     mutex_unlock(&teedev->mutex);
> > +err_kfree:
> > +     kfree(shm);
> > +err:
> > +     tee_device_put(teedev);
> > +     return dmabuf;
> > +}
> > diff --git a/drivers/tee/tee_shm.c b/drivers/tee/tee_shm.c
> > index daf6e5cfd59a..416f7f25d885 100644
> > --- a/drivers/tee/tee_shm.c
> > +++ b/drivers/tee/tee_shm.c
> > @@ -55,6 +55,8 @@ static void tee_shm_release(struct tee_device *teedev, struct tee_shm *shm)
> >                               "unregister shm %p failed: %d", shm, rc);
> >
> >               release_registered_pages(shm);
> > +     } else if (shm->flags & TEE_SHM_RESTRICTED) {
> > +             teedev->desc->ops->rstmem_free(shm->ctx, shm);
> >       }
> >
> >       teedev_ctx_put(shm->ctx);
> > diff --git a/drivers/tee/tee_shm_pool.c b/drivers/tee/tee_shm_pool.c
> > index 80004b55628d..ee57ef157a77 100644
> > --- a/drivers/tee/tee_shm_pool.c
> > +++ b/drivers/tee/tee_shm_pool.c
> > @@ -1,9 +1,8 @@
> >  // SPDX-License-Identifier: GPL-2.0-only
> >  /*
> > - * Copyright (c) 2015, 2017, 2022 Linaro Limited
> > + * Copyright (c) 2015, 2017, 2022, 2024 Linaro Limited
> >   */
> >  #include <linux/device.h>
> > -#include <linux/dma-buf.h>
> >  #include <linux/genalloc.h>
> >  #include <linux/slab.h>
> >  #include <linux/tee_core.h>
> > @@ -90,3 +89,69 @@ struct tee_shm_pool *tee_shm_pool_alloc_res_mem(unsigned long vaddr,
> >       return ERR_PTR(rc);
> >  }
> >  EXPORT_SYMBOL_GPL(tee_shm_pool_alloc_res_mem);
> > +
> > +static int rstmem_pool_op_gen_alloc(struct tee_shm_pool *pool,
> > +                                 struct tee_shm *shm, size_t size,
> > +                                 size_t align)
> > +{
> > +     size_t sz = ALIGN(size, PAGE_SIZE);
> > +     phys_addr_t pa;
> > +
> > +     pa = gen_pool_alloc(pool->private_data, sz);
> > +     if (!pa)
> > +             return -ENOMEM;
> > +
> > +     shm->size = sz;
> > +     shm->paddr = pa;
> > +
> > +     return 0;
> > +}
> > +
> > +static void rstmem_pool_op_gen_free(struct tee_shm_pool *pool,
> > +                                 struct tee_shm *shm)
> > +{
> > +     gen_pool_free(pool->private_data, shm->paddr, shm->size);
> > +     shm->paddr = 0;
> > +}
> > +
> > +static struct tee_shm_pool_ops rstmem_pool_ops_generic = {
> > +     .alloc = rstmem_pool_op_gen_alloc,
> > +     .free = rstmem_pool_op_gen_free,
> > +     .destroy_pool = pool_op_gen_destroy_pool,
> > +};
> > +
> > +struct tee_shm_pool *tee_rstmem_gen_pool_alloc(phys_addr_t paddr, size_t size)
> > +{
> > +     const size_t page_mask = PAGE_SIZE - 1;
> > +     struct tee_shm_pool *pool;
> > +     int rc;
> > +
> > +     /* Check it's page aligned */
> > +     if ((paddr | size) & page_mask)
> > +             return ERR_PTR(-EINVAL);
> > +
> > +     pool = kzalloc(sizeof(*pool), GFP_KERNEL);
> > +     if (!pool)
> > +             return ERR_PTR(-ENOMEM);
> > +
> > +     pool->private_data = gen_pool_create(PAGE_SHIFT, -1);
> > +     if (!pool->private_data) {
> > +             rc = -ENOMEM;
> > +             goto err_free;
> > +     }
> > +
> > +     rc = gen_pool_add(pool->private_data, paddr, size, -1);
> > +     if (rc)
> > +             goto err_free_pool;
> > +
> > +     pool->ops = &rstmem_pool_ops_generic;
> > +     return pool;
> > +
> > +err_free_pool:
> > +     gen_pool_destroy(pool->private_data);
> > +err_free:
> > +     kfree(pool);
> > +
> > +     return ERR_PTR(rc);
> > +}
> > +EXPORT_SYMBOL_GPL(tee_rstmem_gen_pool_alloc);
> > diff --git a/include/linux/tee_core.h b/include/linux/tee_core.h
> > index a38494d6b5f4..608302f494fe 100644
> > --- a/include/linux/tee_core.h
> > +++ b/include/linux/tee_core.h
> > @@ -26,6 +26,7 @@
> >  #define TEE_SHM_USER_MAPPED  BIT(1)  /* Memory mapped in user space */
> >  #define TEE_SHM_POOL         BIT(2)  /* Memory allocated from pool */
> >  #define TEE_SHM_PRIV         BIT(3)  /* Memory private to TEE driver */
> > +#define TEE_SHM_RESTRICTED   BIT(4)  /* Restricted memory */
> >
> >  #define TEE_DEVICE_FLAG_REGISTERED   0x1
> >  #define TEE_MAX_DEV_NAME_LEN         32
> > @@ -76,6 +77,8 @@ struct tee_device {
> >   * @supp_send:               called for supplicant to send a response
> >   * @shm_register:    register shared memory buffer in TEE
> >   * @shm_unregister:  unregister shared memory buffer in TEE
> > + * @rstmem_alloc:    allocate restricted memory
> > + * @rstmem_free:     free restricted memory
> >   */
> >  struct tee_driver_ops {
> >       void (*get_version)(struct tee_device *teedev,
> > @@ -99,6 +102,9 @@ struct tee_driver_ops {
> >                           struct page **pages, size_t num_pages,
> >                           unsigned long start);
> >       int (*shm_unregister)(struct tee_context *ctx, struct tee_shm *shm);
> > +     int (*rstmem_alloc)(struct tee_context *ctx, struct tee_shm *shm,
> > +                         u32 flags, u32 use_case, size_t size);
> > +     void (*rstmem_free)(struct tee_context *ctx, struct tee_shm *shm);
> >  };
> >
> >  /**
> > @@ -229,6 +235,15 @@ static inline void tee_shm_pool_free(struct tee_shm_pool *pool)
> >       pool->ops->destroy_pool(pool);
> >  }
> >
> > +/**
> > + * tee_rstmem_gen_pool_alloc() - Create a restricted memory manager
> > + * @paddr:   Physical address of start of pool
> > + * @size:    Size in bytes of the pool
> > + *
> > + * @returns pointer to a 'struct tee_shm_pool' or an ERR_PTR on failure.
> > + */
> > +struct tee_shm_pool *tee_rstmem_gen_pool_alloc(phys_addr_t paddr, size_t size);
> > +
> >  /**
> >   * tee_get_drvdata() - Return driver_data pointer
> >   * @returns the driver_data pointer supplied to tee_register().
> > diff --git a/include/linux/tee_drv.h b/include/linux/tee_drv.h
> > index a54c203000ed..cba067715d14 100644
> > --- a/include/linux/tee_drv.h
> > +++ b/include/linux/tee_drv.h
> > @@ -55,6 +55,7 @@ struct tee_context {
> >   * @pages:   locked pages from userspace
> >   * @num_pages:       number of locked pages
> >   * @refcount:        reference counter
> > + * @use_case:        defined by TEE_IOC_UC_* in tee.h
> >   * @flags:   defined by TEE_SHM_* in tee_core.h
> >   * @id:              unique id of a shared memory object on this device, shared
> >   *           with user space
> > @@ -71,6 +72,7 @@ struct tee_shm {
> >       struct page **pages;
> >       size_t num_pages;
> >       refcount_t refcount;
> > +     u32 use_case;
> >       u32 flags;
> >       int id;
> >       u64 sec_world_id;
> > diff --git a/include/uapi/linux/tee.h b/include/uapi/linux/tee.h
> > index d0430bee8292..88834448debb 100644
> > --- a/include/uapi/linux/tee.h
> > +++ b/include/uapi/linux/tee.h
> > @@ -1,5 +1,5 @@
> >  /*
> > - * Copyright (c) 2015-2016, Linaro Limited
> > + * Copyright (c) 2015-2017, 2020, 2024, Linaro Limited
> >   * All rights reserved.
> >   *
> >   * Redistribution and use in source and binary forms, with or without
> > @@ -48,6 +48,7 @@
> >  #define TEE_GEN_CAP_PRIVILEGED       (1 << 1)/* Privileged device (for supplicant) */
> >  #define TEE_GEN_CAP_REG_MEM  (1 << 2)/* Supports registering shared memory */
> >  #define TEE_GEN_CAP_MEMREF_NULL      (1 << 3)/* NULL MemRef support */
> > +#define TEE_GEN_CAP_RSTMEM   (1 << 4)/* Supports restricted memory */
> >
> >  #define TEE_MEMREF_NULL              (__u64)(-1) /* NULL MemRef Buffer */
> >
> > @@ -389,6 +390,47 @@ struct tee_ioctl_shm_register_data {
> >   */
> >  #define TEE_IOC_SHM_REGISTER   _IOWR(TEE_IOC_MAGIC, TEE_IOC_BASE + 9, \
> >                                    struct tee_ioctl_shm_register_data)
> > +
> > +#define TEE_IOC_UC_RESERVED          0
> > +#define TEE_IOC_UC_SECURE_VIDEO_PLAY 1
> > +#define TEE_IOC_UC_TRUSTED_UI                2
> > +#define TEE_IOC_UC_SECURE_VIDEO_RECORD       3
> > +
> > +/**
> > + * struct tee_ioctl_rstmem_alloc_data - Restricted memory allocate argument
> > + * @size:    [in/out] Size of restricted memory to allocate
> > + * @flags:   [in/out] Flags to/from allocate
> > + * @use_case [in] Restricted memory use case, TEE_IOC_UC_*
> > + * @id:              [out] Identifier of the restricted memory
> > + */
> > +struct tee_ioctl_rstmem_alloc_data {
> > +     __u64 size;
> > +     __u32 flags;
> > +     __u32 use_case;
> > +     __s32 id;
> > +};
> > +
> > +/**
> > + * TEE_IOC_RSTMEM_ALLOC - allocate restricted memory
> > + *
> > + * Allocates restricted physically memory normally not accessible by the
> > + * kernel.
> > + *
> > + * Restricted memory refers to memory buffers behind a hardware enforced
> > + * firewall. It is not accessible to the kernel during normal circumstances
> > + * but rather only accessible to certain hardware IPs or CPUs executing in
> > + * higher privileged mode than the kernel itself. This interface allows to
> > + * allocate and manage such restricted memory buffers via interaction with
> > + * a TEE implementation.
> > + *
> > + * Returns a file descriptor on success or < 0 on failure
> > + *
> > + * The returned file descriptor is a dma-buf that can be attached and
> > + * mapped for device with permission to access the physical memory.
> > + */
> > +#define TEE_IOC_RSTMEM_ALLOC     _IOWR(TEE_IOC_MAGIC, TEE_IOC_BASE + 10, \
> > +                                    struct tee_ioctl_rstmem_alloc_data)
> > +
> >  /*
> >   * Five syscalls are used when communicating with the TEE driver.
> >   * open(): opens the device associated with the driver
> > --
> > 2.43.0
> >
>
> --
> Simona Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ