Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <20200109123651.18520-1-jens.wiklander@linaro.org>
In-Reply-To: <20200109123651.18520-1-jens.wiklander@linaro.org>
From:   Jens Wiklander <jens.wiklander@linaro.org>
Date:   Thu, 23 Jan 2020 11:21:48 +0100
Message-ID: <CAHUa44H0YL5Ke-QgzACYF8dS1C9PE97kjF9CZZQUiSE4RN0+mA@mail.gmail.com>
Subject: Re: [PATCH 0/5] tee: shared memory cleanup
To:     "tee-dev @ lists . linaro . org" <tee-dev@lists.linaro.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Cc:     Jerome Forissier <jerome@forissier.org>,
        Sumit Garg <sumit.garg@linaro.org>,
        Etienne Carriere <etienne.carriere@linaro.org>,
        Rijo Thomas <Rijo-john.Thomas@amd.com>,
        Devaraj Rangasamy <Devaraj.Rangasamy@amd.com>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hi,

On Thu, Jan 9, 2020 at 1:37 PM Jens Wiklander <jens.wiklander@linaro.org> wrote:
>
> Hi,
>
> Here's a group of patches to the tee subsystem with cleanups and not so
> urgent fixes related to tee shared memory.
>
> - Unused parts of the shared memory object (struct tee_shm) are removed.
> - Shared memory ids usable from user space are not assigned to driver
>   private shared memory objects
> - The TEE_SHM_USER_MAPPED is used instead of TEE_SHM_REGISTER to accurately
>   tell if a shared memory object originates from another user space mapping.
>
> Only unused "features" should be removed with this patch set, there should
> be no change in behaviour or breakage with other code.

I'll pick up these as they are in a few days if there's no further comments.

Thanks,
Jens

>
> Thanks,
> Jens
>
> Jens Wiklander (5):
>   tee: remove linked list of struct tee_shm
>   tee: remove unused tee_shm_priv_alloc()
>   tee: don't assign shm id for private shms
>   tee: remove redundant teedev in struct tee_shm
>   tee: tee_shm_op_mmap(): use TEE_SHM_USER_MAPPED
>
>  drivers/tee/tee_core.c    |  1 -
>  drivers/tee/tee_private.h |  3 +-
>  drivers/tee/tee_shm.c     | 85 +++++++++++----------------------------
>  include/linux/tee_drv.h   | 19 +--------
>  4 files changed, 27 insertions(+), 81 deletions(-)
>
> --
> 2.17.1
>