From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Mon, 17 Oct 2022 15:43:47 +0200 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1okQPE-00Bf0C-JT for lore@lore.pengutronix.de; Mon, 17 Oct 2022 15:43:47 +0200 Received: from bombadil.infradead.org ([2607:7c80:54:3::133]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1okQPC-0000Pt-Ew for lore@pengutronix.de; Mon, 17 Oct 2022 15:43:47 +0200 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Cc:Content-ID:Content-Description:Resent-Date:Resent-From :Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=a9WqZT5pwA5h1MmG1T/dKaNid3UDX233wVmnzXZCLYk=; b=ufrBp/AdsZ8Qzkfp3Wk7K0AkeL pCfpLikwg84bAkESe5ij8YNxRdFZNW7cHxmQEO2CHp1PRYYlCRC4g516Ia5lwMhKPgL/nvawpxotq +En2iT5O6qHNm+pxxfA+PhXS5uOoETQ1lgzcpnpi0+QkO4oiGSubFqUdNkrTp91tHh5EjNKtu+aqE 7PRvlq2oD8mvb82hKkdS6kaeCS5ZAeZYXzR3FfPk743WDgvjG3OCASCB4a27ffT4blmxhBJcwdOIx 4qTD35BLv+uUpdWK1KyYPSms4fkw5qukCsfJabiPl7Z6et+CAOyfW4rf/AWXfXftRp5Nd/ItUZd01 U+1rJJKQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1okQN4-00CBWC-F6; Mon, 17 Oct 2022 13:41:34 +0000 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1okQMy-00CBTS-3x for barebox@lists.infradead.org; Mon, 17 Oct 2022 13:41:31 +0000 Received: from ptz.office.stw.pengutronix.de ([2a0a:edc0:0:900:1d::77] helo=[127.0.0.1]) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1okQMw-0000JQ-Ta; Mon, 17 Oct 2022 15:41:26 +0200 Message-ID: <962d3f49-3125-1c02-3cc3-5e811ceca7a1@pengutronix.de> Date: Mon, 17 Oct 2022 15:41:26 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.12.0 Content-Language: en-US To: Ian Abbott , Barebox List References: <98fb1314-e0c9-c75c-c89f-7bcc36a2b9c0@mev.co.uk> <15b5c180-09f0-0a6e-da52-4c21051ec403@mev.co.uk> From: Ahmad Fatoum In-Reply-To: <15b5c180-09f0-0a6e-da52-4c21051ec403@mev.co.uk> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221017_064128_180343_84B1D2CB X-CRM114-Status: GOOD ( 17.24 ) X-BeenThere: barebox@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "barebox" X-SA-Exim-Connect-IP: 2607:7c80:54:3::133 X-SA-Exim-Mail-From: barebox-bounces+lore=pengutronix.de@lists.infradead.org X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on metis.ext.pengutronix.de X-Spam-Level: X-Spam-Status: No, score=-4.7 required=4.0 tests=AWL,BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,NICE_REPLY_A,RCVD_IN_DNSWL_MED,SPF_HELO_NONE, SPF_NONE autolearn=unavailable autolearn_force=no version=3.4.2 Subject: Re: [v2022.10.0] initcall of of_probe_memory failed (EBUSY) X-SA-Exim-Version: 4.2.1 (built Wed, 08 May 2019 21:11:16 +0000) X-SA-Exim-Scanned: Yes (on metis.ext.pengutronix.de) Hello Ian, On 17.10.22 15:10, Ian Abbott wrote: > On 17/10/2022 13:24, Ahmad Fatoum wrote: >> On 17.10.22 14:03, Ian Abbott wrote: >>> Barebox v2022.10.0 seems to work for me, but I now get this (harmless?) error during initialization: >>> >>> initcall of_probe_memory+0x1/0x34 failed: Device or resource busy >>> >>> (This is on a 32-bit ARM SoCFPGA/CycloneV based system.) >>> >>> git bisect is blaming commit d0b5f6bde15b ("of: reserved-mem: reserve regions prior to mmu_initcall()"). >> >> Can you define #define DEBUG at the top of common/resource.c and paste >> the output? > > Here is what I get for a Terasic DE0-Nano-SoC.  (I was using a custom board for the original report, but the symptoms are the same.) Thanks for the debug output. > Board: Terasic DE-0(Atlas) > __request_region ok: 0x00000000:0x3fffffff flags=0x0 Here socfpga_detect_sdram() will register ram0 after reading it from hardware. > __request_region ok: 0xffd04000:0xffd04fff flags=0x0 > __request_region ok: 0xffd05000:0xffd05fff flags=0x0 > __request_region ok: 0xffc02000:0xffc02fff flags=0x0 > __request_region ok: 0xffc03000:0xffc03fff flags=0x0 > __request_region: 0x00000000:0x3fffffff (ram0) conflicts with 0x00000000:0x3fffffff (ram0) And then when device-tree is parsed, of_probe_memory() will register the same memory bank again. The error reports this unexpected situation, but as you noted should not break the boot. The correct solution would be removing the memory@0 node from the device tree. After all, if barebox has read it from hardware, why hardcode it in the device tree? Still, barebox was supposed to have logic to fuse identical and overlapping memory banks. Can you try the patch I just sent out? Thanks! Ahmad > initcall of_probe_memory+0x1/0x34 failed: Device or resource busy > __request_region ok: 0x00000000:0x00000fff flags=0x80000200 > __request_region ok: 0x3ffe4000:0x3ffe7fff flags=0x200 > __request_region: 0x00000000:0x00000fff (zero page) conflicts with 0x00000000:0x00000fff (fdt-memreserve-0) > __request_region ok: 0x1fefd960:0x3fdfb2bf flags=0x200 > __request_region ok: 0x3fe00000:0x3fe71e0b flags=0x200 > __request_region ok: 0x3fe71e0c:0x3fe8272f flags=0x200 > __request_region ok: 0x3fe82730:0x3fe85d1f flags=0x200 > __request_region ok: 0xffc04000:0xffc04fff flags=0x0 > __request_region ok: 0xfffec600:0xfffec6ff flags=0x0 > __request_region ok: 0x3ffe8000:0x3ffeffff flags=0x200 > __request_region ok: 0xff702000:0xff703fff flags=0x0 > socfpga_designware_eth ff702000.ethernet@ff702000.of: user ID: 0x10, Synopsys ID: 0x37 > mdio_bus: miibus0: probed > __request_region ok: 0xff704000:0xff704fff flags=0x0 > dw_mmc ff704000.dwmmc0@ff704000.of: registered as mmc0 > __request_region ok: 0xffd02000:0xffd02fff flags=0x0 > __request_region ok: 0xff706000:0xff706fff flags=0x0 > __request_region ok: 0xffb90000:0xffb90003 flags=0x0 > __request_region ok: 0x3fdfb2c0:0x3fdfffc9 flags=0x200 > malloc space: 0x1fefd960 -> 0x3fdfb2bf (size 511 MiB) > mmc0: detected SD card version 2.0 > mmc0: registered mmc0 > > Hit any to stop autoboot:    3 > > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |