From: "Pali Rohár" <pali.rohar@gmail.com>
To: Matthijs van Duin <matthijsvanduin@gmail.com>
Cc: Joel Fernandes <joelf@ti.com>, Tony Lindgren <tony@atomide.com>,
linux-omap@vger.kernel.org, Aaro Koskinen <aaro.koskinen@iki.fi>,
Nishanth Menon <nm@ti.com>, Pavel Machek <pavel@ucw.cz>,
Sebastian Reichel <sre@ring0.de>
Subject: Re: runtime check for omap-aes bus access permission (was: Re: 3.13-rc3 (commit 7ce93f3) breaks Nokia N900 DT boot)
Date: Sat, 31 Jan 2015 20:09:58 +0100 [thread overview]
Message-ID: <201501312009.58483@pali> (raw)
In-Reply-To: <CAALWOA92XmRoJwg5vcL+bLQU-O+6Guxgq4cq+76kYRbLOL+i6Q@mail.gmail.com>
[-- Attachment #1: Type: Text/Plain, Size: 2688 bytes --]
On Saturday 31 January 2015 15:38:28 Matthijs van Duin wrote:
> On Sunday 08 December 2013 00:22:06 Tony Lindgren wrote:
> > * Sebastian Reichel <sre@ring0.de> [131207 15:04]:
> > > On Sat, Dec 07, 2013 at 01:11:37PM -0800, Tony Lindgren
> > > wrote:
> > > > > I asked Pali to send me his copy of the updated NOLO
> > > > > bootloader, so that I can test this. I just checked
> > > > > the omap documentation (I only have access to the
> > > > > public one) and crypto related stuff is not
> > > > > documented for the L3_PM_READ_PERMISSION register.
> > > > > There are a couple of reserved bits, which may be
> > > > > used for this, though.
> > > > >
> > > > > I also CC'd Joel Fernandes, since he worked on the
> > > > > driver before and may have access to the
> > > > > documentation.
> > > >
> > > > Looks like at least the 36xx public version referenced
> > > > here has them:
> > > >
> > > > http://www.spinics.net/lists/linux-omap/msg21857.html
> > > >
> > > > I'd assume the registers are the same for 34xx since we
> > > > don't have them defined separately in the kernel.
> > >
> > > I can't find it in the omap36xx documentation either.
> > > Maybe I search at the wrong position.
>
> I've checked a few of the oldest (in case it was later
> removed) and newest (in case it was later added) omap3-series
> public TRMs I have, none of them list the aes module or
> associated interconnect info. The region is either "reserved"
> or just silently skipped over. The practice of pretending
> something doesn't exist in the TRM while simultaneously
> releasing a linux driver continues to puzzle me.
>
> > > I tried to find something crypto related in
> > >
> > > Table 9-89. L3_PM_READ_PERMISSION_i
> >
> > Hmm maybe it's done based on the address in
> > L3_PM_ADDR_MATCH_k?
>
> According to the address (aes@480c5000) it's attached to the
> L4-Core interconnect, so why would an L3 firewall be
> involved? Its access control would be configured in the
> L4-Core AP (2KB @ 0x48040000), and since they have an
> integrated memory map you'd automatically know which entry is
> responsible, assuming you can access the AP at all.
Do you have idea if it is possible to write such check in kernel
if address (aes@480c5000) is readable or not?
I have configured two testing N900 devices. One with signed
bootloader which enable omap aes support and one device with
signed bootloader which does not enable omap aes support.
So I can run any code/kernel patch and compare results/dumps
between those two devices.
Just I do not know what to do, or what to test...
--
Pali Rohár
pali.rohar@gmail.com
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]
next prev parent reply other threads:[~2015-01-31 19:10 UTC|newest]
Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-06 21:36 3.13-rc3 (commit 7ce93f3) breaks Nokia N900 DT boot Sebastian Reichel
2013-12-06 22:27 ` Tony Lindgren
2013-12-07 0:00 ` Sebastian Reichel
2013-12-07 0:38 ` Tony Lindgren
2013-12-07 8:18 ` Pali Rohár
2013-12-07 13:48 ` Sebastian Reichel
2013-12-07 13:57 ` Pali Rohár
2013-12-07 16:51 ` Tony Lindgren
2013-12-07 17:53 ` Tony Lindgren
2013-12-07 18:49 ` runtime check for omap-aes bus access permission (was: Re: 3.13-rc3 (commit 7ce93f3) breaks Nokia N900 DT boot) Sebastian Reichel
2013-12-07 21:11 ` Tony Lindgren
2013-12-07 23:03 ` Sebastian Reichel
2013-12-07 23:22 ` Tony Lindgren
2014-09-08 23:45 ` Pali Rohár
2014-11-25 21:08 ` Pali Rohár
2014-11-25 21:31 ` Pali Rohár
2014-11-26 17:54 ` Tony Lindgren
2015-01-17 9:18 ` Pali Rohár
2015-01-17 17:04 ` Tony Lindgren
2015-01-17 17:29 ` Pali Rohár
2015-01-17 17:41 ` Tony Lindgren
2015-01-31 11:34 ` Pali Rohár
2015-01-31 15:13 ` Matthijs van Duin
2015-01-31 19:06 ` Pali Rohár
2015-02-11 12:39 ` Pali Rohár
2015-02-11 15:22 ` Matthijs van Duin
2015-02-11 20:28 ` Pali Rohár
2015-02-11 20:33 ` Tony Lindgren
2015-02-11 20:40 ` Nishanth Menon
2015-02-18 21:14 ` Pali Rohár
2015-05-28 7:37 ` Pali Rohár
2015-05-28 16:01 ` Tony Lindgren
2015-05-28 20:26 ` Matthijs van Duin
2015-05-28 22:24 ` Tony Lindgren
2015-05-28 22:27 ` Pali Rohár
2015-05-29 0:15 ` Tony Lindgren
2015-05-29 0:58 ` Matthijs van Duin
2015-05-29 1:35 ` Matthijs van Duin
2015-05-29 15:50 ` Tony Lindgren
2015-05-29 18:16 ` Tony Lindgren
2015-05-30 15:22 ` Matthijs van Duin
2015-06-01 17:58 ` Tony Lindgren
2015-06-01 20:32 ` Matthijs van Duin
2015-06-01 20:52 ` Tony Lindgren
2015-06-02 4:21 ` Matthijs van Duin
2015-02-19 18:20 ` Pali Rohár
2015-02-19 20:25 ` Matthijs van Duin
2015-02-19 21:10 ` Aaro Koskinen
2015-01-24 10:40 ` Pali Rohár
2015-01-31 14:38 ` Matthijs van Duin
2015-01-31 19:09 ` Pali Rohár [this message]
2015-02-01 1:36 ` Matthijs van Duin
2015-02-01 8:56 ` Pali Rohár
2015-02-11 20:43 ` Pavel Machek
2015-02-11 21:14 ` Pali Rohár
2015-02-09 11:55 ` 3.13-rc3 (commit 7ce93f3) breaks Nokia N900 DT boot Pali Rohár
2013-12-08 14:13 ` Aaro Koskinen
2013-12-08 16:40 ` Tony Lindgren
2013-12-08 17:10 ` Sebastian Reichel
2013-12-08 17:43 ` Tony Lindgren
2013-12-08 17:59 ` Aaro Koskinen
2013-12-08 18:09 ` Sebastian Reichel
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=201501312009.58483@pali \
--to=pali.rohar@gmail.com \
--cc=aaro.koskinen@iki.fi \
--cc=joelf@ti.com \
--cc=linux-omap@vger.kernel.org \
--cc=matthijsvanduin@gmail.com \
--cc=nm@ti.com \
--cc=pavel@ucw.cz \
--cc=sre@ring0.de \
--cc=tony@atomide.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).