From patchwork Wed Oct 18 08:29:28 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Joonsoo Kim X-Patchwork-Id: 10014071 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 3862960215 for ; Wed, 18 Oct 2017 08:26:29 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 246F328AF8 for ; Wed, 18 Oct 2017 08:26:29 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 18FAC28AFC; Wed, 18 Oct 2017 08:26:29 +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 0A31628AF8 for ; Wed, 18 Oct 2017 08:26:27 +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=f5sO5niTzJV/Lbs/kuz4ZK3ypnhWZ8M6YdToTgw1MEM=; b=XTKSTErZJ1Gx55 YBmvmGpJOb/0p+TK9lhSY7GFopmkAApjQgrc/8B9Fs0OTiJCEMydwYJAA8CHcTWt9pzVmRAB+G87H 1911YwvG8Cqtz8CvuW+OSq2EyPKt2EJhC1tvizqX2y0hwQqxtXXcpyqO7l5G+Z31td/lK4cmzsj6q +S6qHdRmT23ZhkTaKTTxDRsBhw1JL8WA9iMmKRGii/a1Ws+OHJWz9M5J8yCJyPCB9G8FukynndwDw 6O0NtdlhkeIs4wc9R5eKOJ1F36WXY4am+dPYpKhGGv0lpoPHvqF2UDH50QUOwUnt1LbVW4x8YDL/o Eqre5FeICxqUy3/lHWog==; 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 1e4jgH-000373-SS; Wed, 18 Oct 2017 08:26:25 +0000 Received: from lgeamrelo13.lge.com ([156.147.23.53]) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1e4jgD-0002wv-BV for linux-arm-kernel@lists.infradead.org; Wed, 18 Oct 2017 08:26:24 +0000 Received: from unknown (HELO lgeamrelo01.lge.com) (156.147.1.125) by 156.147.23.53 with ESMTP; 18 Oct 2017 17:25:56 +0900 X-Original-SENDERIP: 156.147.1.125 X-Original-MAILFROM: iamjoonsoo.kim@lge.com Received: from unknown (HELO localhost) (10.177.222.138) by 156.147.1.125 with ESMTP; 18 Oct 2017 17:25:55 +0900 X-Original-SENDERIP: 10.177.222.138 X-Original-MAILFROM: iamjoonsoo.kim@lge.com Date: Wed, 18 Oct 2017 17:29:28 +0900 From: Joonsoo Kim To: Tony Lindgren Subject: Re: n900 in next-20170901 Message-ID: <20171018082927.GB27595@js1304-P5Q-DELUXE> References: <20170905233241.GA19231@js1304-P5Q-DELUXE> <20170906133057.GH5024@atomide.com> <20170907073038.GA2111@js1304-P5Q-DELUXE> <20170907161650.GP5024@atomide.com> <20170913075516.GA25036@js1304-P5Q-DELUXE> <20170913163126.GS5024@atomide.com> <20170915065608.GA8567@js1304-P5Q-DELUXE> <20170921172811.GB4394@atomide.com> <20170925080806.GA9837@js1304-P5Q-DELUXE> <20170925145437.GG4394@atomide.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20170925145437.GG4394@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-20171018_012621_699754_818D7E32 X-CRM114-Status: GOOD ( 28.37 ) 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 Mon, Sep 25, 2017 at 07:54:37AM -0700, Tony Lindgren wrote: > * Joonsoo Kim [170925 01:06]: > > On Thu, Sep 21, 2017 at 10:28:11AM -0700, Tony Lindgren wrote: > > > * Joonsoo Kim [170914 23:55]: > > > > On Wed, Sep 13, 2017 at 09:31:27AM -0700, Tony Lindgren wrote: > > > > > Yes I disabled CONFIG_HIGHMEM and n900 boots. To disable it, > > > > > you need to remove it from arch/arm/mach-omap2/Kconfig that > > > > > selects it if ARCH_OMAP2PLUS_TYPICAL is selected. > > > > > > > > Okay. Problem would be related to address traslation. I'd like to > > > > check address traslation more. Could you apply following patch and > > > > test it? And, please send me the dmesg log and your kernel config. > > > > Please test this with CONFIG_DEBUG_VIRTUAL = n and CONFIG_CMA_DEBUG=y and > > > > CONFIG_HIGHMEM=y and with kernel bootparam 'ignore_loglevel'. > > > > > > > > It would be really appreciate if you send me two logs for before/after > > > > commit 9caf25f996e8. > > > > > > Sorry for the delays, I finally got around testing this for you. > > > > No problem! I really appreciate your help! > > > > > Compile with your patch failed for modules with __virt_to_phys_debug > > > being undefined so I added EXPORT_SYMBOL there. I also enabled DEBUG_LL > > > and EARLYPRINTK to get output. > > > > > > Below is dmesg output for 9caf25f996e8 + your patch. I'll send you > > > the full logs separately. > > > > Hmm...there is only one caller for the CMA memory, that is, atomic_pool_init(). > > Could you test one more time with 9caf25f996e8 + following patch? I'd like to > > know the actual user for the CMA memory. > > Hmm not getting any stack with that patch after manually applying > it because of tabs to spaces mangling. Sorry for long delay. Seems like your system doesn't use any CMA memory by CMA API. Could you test one more thing? This one is to disable CMA memory allocation from the page allocator. With this, we can be sure that CMA memory isn't used at all. If there is no difference with this patch, that is, the system down, I think that some initialization step is broken. In this case, please test following patch. I make a branch in github that all these patch is applied. Feel free to use it. https://github.com/JoonsooKim/linux/tree/cma-debug4-next-20180901 Thanks. --------------->8---------------------- diff --git a/mm/page_alloc.c b/mm/page_alloc.c index 6dbc49e..1e48e67 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c @@ -1861,7 +1861,7 @@ static int fallbacks[MIGRATE_TYPES][4] = { static struct page *__rmqueue_cma_fallback(struct zone *zone, unsigned int order) { - return __rmqueue_smallest(zone, order, MIGRATE_CMA); + return NULL; } #else static inline struct page *__rmqueue_cma_fallback(struct zone *zone, ----------------->8---------------------- diff --git a/arch/arm/mm/dma-mapping.c b/arch/arm/mm/dma-mapping.c index c68f34a..c72b4c3 100644 --- a/arch/arm/mm/dma-mapping.c +++ b/arch/arm/mm/dma-mapping.c @@ -497,6 +497,9 @@ void __init dma_contiguous_remap(void) map.length = end - start; map.type = MT_MEMORY_DMA_READY; + dmac_flush_range(map.virtual, map.virtual + map.length); + outer_flush_range(start, end); + /* * Clear previous low-memory mapping to ensure that the * TLB does not see any conflicting entries, then flush @@ -510,6 +513,7 @@ void __init dma_contiguous_remap(void) addr += PMD_SIZE) pmd_clear(pmd_off_k(addr)); + flush_cache_all(); flush_tlb_kernel_range(__phys_to_virt(start), __phys_to_virt(end));