From: Nicolas Dufresne <nicolas@ndufresne.ca> To: Olivier Masse <olivier.masse@nxp.com>, "brian.starkey@arm.com" <brian.starkey@arm.com> Cc: "sumit.semwal@linaro.org" <sumit.semwal@linaro.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "linaro-mm-sig@lists.linaro.org" <linaro-mm-sig@lists.linaro.org>, "christian.koenig@amd.com" <christian.koenig@amd.com>, "linux-media@vger.kernel.org" <linux-media@vger.kernel.org>, "nd@arm.com" <nd@arm.com>, "Clément Faure" <clement.faure@nxp.com>, "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>, "benjamin.gaignard@collabora.com" <benjamin.gaignard@collabora.com> Subject: Re: [EXT] Re: [PATCH 1/3] dma-buf: heaps: add Linaro secure dmabuf heap support Date: Wed, 17 Aug 2022 10:29:22 -0400 [thread overview] Message-ID: <eef2fc413695cb96a5071627bfe44830f80cfe9e.camel@ndufresne.ca> (raw) In-Reply-To: <de46324d8fa8fb6a8dda4641e531d30842410744.camel@nxp.com> Hi Folks, Le mardi 16 août 2022 à 11:20 +0000, Olivier Masse a écrit : > Hi Brian, > > > On ven., 2022-08-12 at 17:39 +0100, Brian Starkey wrote: > > Caution: EXT Ema > > [...] > > > > Interesting, that's not how the devices I've worked on operated. > > > > Are you saying that you have to have a display controller driver > > running in the TEE to display one of these buffers? > > In fact the display controller is managing 3 plans : UI, PiP and > video. The video plan is protected in secure as you can see on slide > 11: > https://static.linaro.org/connect/san19/presentations/san19-107.pdf just wanted to highlight that all the WPE/GStreamer bit in this presentation is based on NXP Vendor Media CODEC design, which rely on their own i.MX VPU API. I don't see any effort to extend this to a wider audience. It is not explaining how this can work with a mainline kernel with v4l2 stateful or stateless drivers and generic GStreamer/FFMPEG/Chromium support. I'm raising this, since I'm worried that no one cares of solving that high level problem from a generic point of view. In that context, any additions to the mainline Linux kernel can only be flawed and will only serves specific vendors and not the larger audience. Another aspect, is that this design might be bound to a specific (NXP ?) security design. I've learn recently that newer HW is going to use multiple level of MMU (like virtual machines do) to protect the memory rather then marking pages. Will all this work for that too ? regards, Nicolas
WARNING: multiple messages have this Message-ID (diff)
From: Nicolas Dufresne <nicolas@ndufresne.ca> To: Olivier Masse <olivier.masse@nxp.com>, "brian.starkey@arm.com" <brian.starkey@arm.com> Cc: "Clément Faure" <clement.faure@nxp.com>, "benjamin.gaignard@collabora.com" <benjamin.gaignard@collabora.com>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>, "sumit.semwal@linaro.org" <sumit.semwal@linaro.org>, "linaro-mm-sig@lists.linaro.org" <linaro-mm-sig@lists.linaro.org>, "nd@arm.com" <nd@arm.com>, "christian.koenig@amd.com" <christian.koenig@amd.com>, "linux-media@vger.kernel.org" <linux-media@vger.kernel.org> Subject: Re: [EXT] Re: [PATCH 1/3] dma-buf: heaps: add Linaro secure dmabuf heap support Date: Wed, 17 Aug 2022 10:29:22 -0400 [thread overview] Message-ID: <eef2fc413695cb96a5071627bfe44830f80cfe9e.camel@ndufresne.ca> (raw) In-Reply-To: <de46324d8fa8fb6a8dda4641e531d30842410744.camel@nxp.com> Hi Folks, Le mardi 16 août 2022 à 11:20 +0000, Olivier Masse a écrit : > Hi Brian, > > > On ven., 2022-08-12 at 17:39 +0100, Brian Starkey wrote: > > Caution: EXT Ema > > [...] > > > > Interesting, that's not how the devices I've worked on operated. > > > > Are you saying that you have to have a display controller driver > > running in the TEE to display one of these buffers? > > In fact the display controller is managing 3 plans : UI, PiP and > video. The video plan is protected in secure as you can see on slide > 11: > https://static.linaro.org/connect/san19/presentations/san19-107.pdf just wanted to highlight that all the WPE/GStreamer bit in this presentation is based on NXP Vendor Media CODEC design, which rely on their own i.MX VPU API. I don't see any effort to extend this to a wider audience. It is not explaining how this can work with a mainline kernel with v4l2 stateful or stateless drivers and generic GStreamer/FFMPEG/Chromium support. I'm raising this, since I'm worried that no one cares of solving that high level problem from a generic point of view. In that context, any additions to the mainline Linux kernel can only be flawed and will only serves specific vendors and not the larger audience. Another aspect, is that this design might be bound to a specific (NXP ?) security design. I've learn recently that newer HW is going to use multiple level of MMU (like virtual machines do) to protect the memory rather then marking pages. Will all this work for that too ? regards, Nicolas
next prev parent reply other threads:[~2022-08-17 14:29 UTC|newest] Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-08-05 13:53 [PATCH 0/3] Add dma-buf secure-heap Olivier Masse 2022-08-05 13:53 ` [PATCH 1/3] dma-buf: heaps: add Linaro secure dmabuf heap support Olivier Masse 2022-08-05 15:41 ` Brian Starkey 2022-08-05 15:41 ` Brian Starkey 2022-08-08 14:39 ` [EXT] " Olivier Masse 2022-08-08 14:39 ` Olivier Masse 2022-08-12 16:39 ` Brian Starkey 2022-08-12 16:39 ` Brian Starkey 2022-08-16 11:20 ` Olivier Masse 2022-08-16 11:20 ` Olivier Masse 2022-08-17 13:57 ` Brian Starkey 2022-08-17 13:57 ` Brian Starkey 2022-08-17 14:29 ` Nicolas Dufresne [this message] 2022-08-17 14:29 ` Nicolas Dufresne 2022-08-17 14:52 ` Olivier Masse 2022-08-17 14:52 ` Olivier Masse 2022-08-18 5:25 ` Cyrille Fleury 2022-08-18 5:25 ` Cyrille Fleury 2022-08-19 15:13 ` Nicolas Dufresne 2022-08-19 15:13 ` Nicolas Dufresne 2022-08-23 15:58 ` Cyrille Fleury 2022-08-23 15:58 ` Cyrille Fleury 2022-08-17 16:12 ` Lucas Stach 2022-08-17 16:12 ` Lucas Stach 2022-08-09 13:38 ` Olivier Masse 2022-08-09 13:38 ` Olivier Masse 2022-08-10 9:43 ` Christian König 2022-08-10 13:31 ` [EXT] " Olivier Masse 2022-08-05 13:53 ` [PATCH 2/3] dt-bindings: reserved-memory: add linaro,secure-heap Olivier Masse 2022-08-05 15:46 ` Brian Starkey 2022-08-05 15:46 ` Brian Starkey 2022-08-09 7:01 ` [EXT] " Olivier Masse 2022-08-09 7:01 ` Olivier Masse 2022-08-05 13:53 ` [PATCH 3/3] plat-hikey: Add linaro,secure-heap compatible Olivier Masse
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=eef2fc413695cb96a5071627bfe44830f80cfe9e.camel@ndufresne.ca \ --to=nicolas@ndufresne.ca \ --cc=benjamin.gaignard@collabora.com \ --cc=brian.starkey@arm.com \ --cc=christian.koenig@amd.com \ --cc=clement.faure@nxp.com \ --cc=dri-devel@lists.freedesktop.org \ --cc=linaro-mm-sig@lists.linaro.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-media@vger.kernel.org \ --cc=nd@arm.com \ --cc=olivier.masse@nxp.com \ --cc=sumit.semwal@linaro.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.