From patchwork Tue Sep 5 23:32:42 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Joonsoo Kim X-Patchwork-Id: 9939799 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 44A1660350 for ; Tue, 5 Sep 2017 23:32:35 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 32F8628A24 for ; Tue, 5 Sep 2017 23:32:35 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 2753428A26; Tue, 5 Sep 2017 23:32:35 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,RCVD_IN_DNSWL_MED autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 0D34328A25 for ; Tue, 5 Sep 2017 23:32:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=eDhONxoLe0l//RXwYnOXHJJzX69k5bBrtjj8OdNrhXw=; b=p9wyv2TluZoOtv wBmjBOm60Beh64ubcc3peUInvD5XiLFgzGXQstyOkXUfQpvyrViKyxUq/FzHTJcJ4YW26y55iwD5s vqReXGMeVN6nyJ2W197eh70g5bFxzzhdFmxpNwU1Mcqo2qgGdC3loESXNVXTSiFx0p+pur6/WZEJo JyY5U3FxKHUlf8/obwrZeeGFbm8xixICTqkZ82N8HE26ZOGwsHGLNz+e17Xmmyi0Z7qyv4i+6cdgi X41QhQH6NUQKZyFrlWrpVX0XPPhozfPxO8Q/2TNfomf53BdHoZnMPWNGkm/9tppHVICcHUXJk05uS UYKyyTA8hA/JECdnGuhA==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1dpNK8-0002t2-Cf; Tue, 05 Sep 2017 23:32:04 +0000 Received: from lgeamrelo12.lge.com ([156.147.23.52]) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1dpNK4-0002ok-Iq for linux-arm-kernel@lists.infradead.org; Tue, 05 Sep 2017 23:32:03 +0000 Received: from unknown (HELO lgeamrelo02.lge.com) (156.147.1.126) by 156.147.23.52 with ESMTP; 6 Sep 2017 08:31:32 +0900 X-Original-SENDERIP: 156.147.1.126 X-Original-MAILFROM: iamjoonsoo.kim@lge.com Received: from unknown (HELO localhost) (10.177.222.138) by 156.147.1.126 with ESMTP; 6 Sep 2017 08:31:32 +0900 X-Original-SENDERIP: 10.177.222.138 X-Original-MAILFROM: iamjoonsoo.kim@lge.com Date: Wed, 6 Sep 2017 08:32:42 +0900 From: Joonsoo Kim To: Tony Lindgren Subject: Re: n900 in next-20170901 Message-ID: <20170905233241.GA19231@js1304-P5Q-DELUXE> References: <20170903203737.GA12475@amd> <20170905201314.GE5024@atomide.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20170905201314.GE5024@atomide.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170905_163200_918599_4A224829 X-CRM114-Status: GOOD ( 19.77 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Stephen Rothwell , ivo.g.dimitrov.75@gmail.com, khilman@kernel.org, Vlastimil Babka , aaro.koskinen@iki.fi, kernel list , sre@kernel.org, abcloriens@gmail.com, "Aneesh Kumar K.V" , Pavel Machek , pali.rohar@gmail.com, Russell King , Andrew Morton , linux-omap@vger.kernel.org, patrikbachan@gmail.com, linux-arm-kernel , serge@hallyn.com Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP On Tue, Sep 05, 2017 at 01:13:15PM -0700, Tony Lindgren wrote: > * Pavel Machek [170903 13:38]: > > Hi! > > > > It compiles ok, but it hangs on boot; black screen, so sometime before > > display is initialized. > > Thanks for reporting it. Based on git bisect, the regression causing > commit is 9caf25f996e8 ("mm/cma: manage the memory of the CMA area > by using the ZONE_MOVABLE"). With this path applied, booting hangs > with an error in omap3_save_secure_ram_context() after a call to > _omap_save_secure_sram() that calls the related assembly code > save_secure_ram_context. > > However, looks like there is also some other commit causing issue. > > Just reverting 9caf25f996e8 on Linux next causes the oops below. > > Anybody got ideas why this now happens? > > Regards, > > Tony > > 8< -------------------- > Unable to handle kernel paging request at virtual address ce800000 > pgd = c0004000 [ce800000] *pgd=00000000 > Internal error: Oops: 805 [#1] SMP ARM > Modules linked in: > CPU: 0 PID: 1 Comm: swapper/0 Not tainted 4.13.0-next-20170905+ #662 > Hardware name: Nokia RX-51 board > task: ce0a0040 task.stack: ce0a4000 > PC is at __memzero+0x24/0x7c > LR is at 0x0 > pc : [] lr : [<00000000>] psr: 20000013 > sp : ce0a5e84 ip : 00000000 fp : c0c005a8 > r10: 00040000 r9 : cfc95000 r8 : 00000247 > r7 : ce0a5ef4 r6 : 00000000 r5 : 00000001 r4 : ce800000 > r3 : 00000000 r2 : 00000000 r1 : 0003ffc0 r0 : ce800000 > Flags: nzCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment none > Control: 10c5387d Table: 80004019 DAC: 00000051 > Process swapper/0 (pid: 1, stack limit = 0xce0a4218) > Stack: (0xce0a5e84 to 0xce0a6000) > 5e80: c0116718 00000040 00000006 c01a1470 00040000 00000001 00000040 > 5ea0: ce0a5ef4 00000247 cfc95000 00000000 c0c005a8 c0116844 c0dce000 c01a1484 > 5ec0: 00000247 c0d0e2e0 c0dce29c c0b5cbbc c0dce000 00000003 c0c5389c c0c06c54 > 5ee0: c0c06bd8 00000001 00000000 014000c0 00000000 c0b5cbbc ffffe000 c0c06bd8 > 5f00: 00000000 c0101ef8 000000aa 00000000 cfdfcbdb cfdfcbe7 c0b5d904 000000aa > 5f20: 000000aa c015cf54 c0b5cbbc 00000000 00000002 00000002 cfdfcbe7 cfdfcbec > 5f40: c0c6c59c 00000002 c0dce000 c0c53880 c0c6c66c c0dce000 c0c53884 c0dce000 > 5f60: 00000003 c0c00ecc 00000002 00000002 00000000 c0c005a8 c0864f3c 000000aa > 5f80: 00000000 00000000 c0864f3c 00000000 00000000 00000000 00000000 00000000 > 5fa0: 00000000 c0864f44 00000000 c0107e98 00000000 00000000 00000000 00000000 > 5fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 > 5fe0: 00000000 00000000 00000000 00000000 00000013 00000000 ffffffff ffffffff > [] (__memzero) from [] (__dma_clear_buffer+0x140/0x154) > [] (__dma_clear_buffer) from [] (__alloc_from_contiguous+0x50/0xdc) > [] (__alloc_from_contiguous) from [] (atomic_pool_init+0x7c/0x178) > [] (atomic_pool_init) from [] (do_one_initcall+0x3c/0x170) > [] (do_one_initcall) from [] (kernel_init_freeable+0x1fc/0x2c4) > [] (kernel_init_freeable) from [] (kernel_init+0x8/0x110) > [] (kernel_init) from [] (ret_from_fork+0x14/0x3c) > Code: e52de004 e1a0c002 e1a0e002 e2511040 (a8a0500c) Hello, I think that I made a mistake for configuration CONFIG_HIGHMEM=y and CONFIG_HAVE_MEMBLOCK_NODE_MAP=y. In this case, the MOVABLE_ZONE can be *!highmem*. Could you check that your configuration have above options? And, could you check that following patch works for you? Thanks. ------------>8----------------- diff --git a/arch/arm/mm/dma-mapping.c b/arch/arm/mm/dma-mapping.c index 38f0fde..4c39c92 100644 --- a/arch/arm/mm/dma-mapping.c +++ b/arch/arm/mm/dma-mapping.c @@ -518,7 +518,7 @@ void __init dma_contiguous_remap(void) * considered as highmem even if it's physical address belong * to lowmem. Therefore, re-mapping isn't required. */ - if (!IS_ENABLED(CONFIG_HIGHMEM)) + if (!is_highmem_idx(ZONE_MOVABLE)) iotable_init(&map, 1); } }