From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from vsmx009.vodafonemail.xion.oxcs.net ([153.92.174.87]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jLiwP-0007vj-Ay for barebox@lists.infradead.org; Tue, 07 Apr 2020 07:46:39 +0000 References: <2179bad5-0ec6-a2ae-683f-f2f24e3aabfb@arcor.de> <56816c6f-86ac-e350-3d23-bf0ff1fe87d2@pengutronix.de> <6679e805-8f1e-0c02-cc73-e2d2ab4911c1@arcor.de> <63e3eb58-9f5b-54d6-e8cf-b7672fe55a4a@pengutronix.de> <9bd7f507-8f59-98ee-18af-48be3dba9372@arcor.de> <3c69c3fb-d8bd-b39d-a0ea-704c8ad99dfc@arcor.de> <612deb15-d03a-e393-3e37-efed52c9c3ed@pengutronix.de> From: Giorgio Message-ID: Date: Tue, 7 Apr 2020 09:46:25 +0200 MIME-Version: 1.0 In-Reply-To: <612deb15-d03a-e393-3e37-efed52c9c3ed@pengutronix.de> Content-Language: en-US List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: imx7d dual core boot To: Ahmad Fatoum Cc: barebox@lists.infradead.org Hi, OK, my current fix follows also the same logic as for the values of 'vbar' and 'ttb' in armv7_secure_monitor_install(). I've also found the 'set_domain(v)' in mmu.h, don't need to write a new one. I'll send a patch for this. What do you mean with the 'other i.MX7 patches' ? I've also noticed that some asm() statements in sm.c lack the 'volatile' attribute. giorgio On 4/6/20 8:44 PM, Ahmad Fatoum wrote: > Hello Giorgio, > > On 4/6/20 5:15 PM, Giorgio wrote: >> Searching in the Armv7-A ref. man. here: >> >> https://static.docs.arm.com/ddi0406/cd/DDI0406C_d_armv7ar_arm.pdf >> >> at B4.1.43 (page B4 - 1554) I found that the reset value of this cp15 reg. >> is UNKNOWN and I can also verify this. Moreover the reg. must also be >> banked: there is a copy in secure mode and one in nonsecure mode. >> My test consists in reading the register just before and just after >> the call to arm_smccc_smc(0, 0, 0, 0, 0, 0, 0, 0, &res): >> >> static u32 read_dacr(void) >> { >> unsigned int reg; >> >> asm volatile ("mrc p15, 0, %0, c3, c0, 0\n" : "=r"(reg)); >> return reg; >> } >> >> ... >> >> int __armv7_secure_monitor_install(void) >> { >> ... >> printf("%s: 1) dacr: 0x%08x\n",__func__,read_dacr()); >> >> /* Initialize the secure monitor */ >> arm_smccc_smc(0, 0, 0, 0, 0, 0, 0, 0, &res); >> >> /* We're in nonsecure mode now */ >> >> printf("%s: 2) dacr: 0x%08x\n",__func__,read_dacr()); >> >> return 0; >> } >> >> and here is what I get: >> >> samx7: / smc -n >> __armv7_secure_monitor_install: 1) dacr: 0x00000001 >> __armv7_secure_monitor_install: 2) dacr: 0xdbf7afaa >> >> If I write a 0x1 or a 0x3 to DACR just before enabling the MMU in >> nonsecure mode then barebox does not hang. > > Oh! seems i have been lucky so far: > > __armv7_secure_monitor_install: 1) dacr: 0x00000001 > __armv7_secure_monitor_install: 2) dacr: 0xa133ad17 > > Yes, we will want to call set_domain(DOMAIN_CLIENT) > somewhere, maybe just before set_ttbr? > > Do you want to send a patch? > (Your other i.MX7 patches are welcome as well :-) > > Cheers and thanks for debugging this > Ahmad > >> >> giorgio >> >> On 2020-04-06 08:16, Ahmad Fatoum wrote: >>> Hello Giorgio, >>> >>> On 4/3/20 3:47 PM, Giorgio wrote: >>>> Hi Ahmad, >>>> >>>> thank you for the detailed explanations, I'll have a look >>>> at the armv7 ref. manual for more background. >>>> >>>> I wanted just to note, the problem is specifically linked >>>> to enabling the MMU: >>>> >>>> in arch/arm/cpu/cache-armv7.S: >>>> >>>> >>>> orrne r0, r0, #1 @ MMU enabled >>>> ... >>>> mcr p15, 0, r0, c1, c0, 0 @ load control register >>>> >>>> without the 'orrne ...' the imx7 does not hang. >>> >>> I can't reproduce this exact problem. My setup: >>> >>> - i.MX7D sabresd board, >>> - imx_v7_defconfig >>> - check out of upstream/next, commit 5931fe40 ("Merge branch 'for-next/zii' into next") >>> - revert of commit 8b2104d ("driver: Call of_clk_set_defaults for each probed device") >>> - nv bootm.secure_state=nonsecure >>> >>> With this, I didn't observe any barebox hangs[1] while preparing a Linux net boot. >>> What's your setup? >>> >>> [1]: Linux still hangs due to what I assume to be a psci issue, kernel log says >>> "unsupported enable-method property: psci" before getting stuck durcing SDHCI probe >>> >> > _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox