From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pl0-f72.google.com (mail-pl0-f72.google.com [209.85.160.72]) by kanga.kvack.org (Postfix) with ESMTP id 976646B0003 for ; Tue, 22 May 2018 20:03:00 -0400 (EDT) Received: by mail-pl0-f72.google.com with SMTP id 89-v6so13158304plc.1 for ; Tue, 22 May 2018 17:03:00 -0700 (PDT) Received: from mail.linuxfoundation.org (mail.linuxfoundation.org. [140.211.169.12]) by mx.google.com with ESMTPS id j84-v6si17563583pfk.203.2018.05.22.17.02.58 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 22 May 2018 17:02:58 -0700 (PDT) Date: Tue, 22 May 2018 17:02:57 -0700 From: Andrew Morton Subject: Re: [PATCH] Revert "mm/cma: manage the memory of the CMA area by using the ZONE_MOVABLE" Message-Id: <20180522170257.ae5387d8717e81def908a53b@linux-foundation.org> In-Reply-To: <20180521061631.GA26882@js1304-desktop> References: <20180517125959.8095-1-ville.syrjala@linux.intel.com> <20180517132109.GU12670@dhcp22.suse.cz> <20180517133629.GH23723@intel.com> <20180517135832.GI23723@intel.com> <20180517164947.GV12670@dhcp22.suse.cz> <20180517170816.GW12670@dhcp22.suse.cz> <20180518040104.GA17433@js1304-desktop> <20180519144632.GE23723@intel.com> <20180521061631.GA26882@js1304-desktop> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Joonsoo Kim Cc: Ville =?ISO-8859-1?Q?Syrj=E4l=E4?= , Laura Abbott , Michal Hocko , "Aneesh Kumar K . V" , Tony Lindgren , Vlastimil Babka , Johannes Weiner , Laura Abbott , Marek Szyprowski , Mel Gorman , Michal Nazarewicz , Minchan Kim , Rik van Riel , Russell King , Will Deacon , Linus Torvalds , linux-mm@kvack.org, linux-kernel@vger.kernel.org On Mon, 21 May 2018 15:16:33 +0900 Joonsoo Kim wrote: > > (gdb) list *(dma_direct_alloc+0x22f) > > 0x573fbf is in dma_direct_alloc (../lib/dma-direct.c:104). > > 94 > > 95 if (!page) > > 96 return NULL; > > 97 ret = page_address(page); > > 98 if (force_dma_unencrypted()) { > > 99 set_memory_decrypted((unsigned long)ret, 1 << page_order); > > 100 *dma_handle = __phys_to_dma(dev, page_to_phys(page)); > > 101 } else { > > 102 *dma_handle = phys_to_dma(dev, page_to_phys(page)); > > 103 } > > 104 memset(ret, 0, size); > > 105 return ret; > > 106 } > > > > Okay. I find the reason about this error. It's getting rather late and we don't seem to have a final set of fixes yet. Perhaps the best approach here is to revert and try again for 4.18?