From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753526AbeEWAJH (ORCPT ); Tue, 22 May 2018 20:09:07 -0400 Received: from mail-wr0-f196.google.com ([209.85.128.196]:45437 "EHLO mail-wr0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753467AbeEWAJF (ORCPT ); Tue, 22 May 2018 20:09:05 -0400 X-Google-Smtp-Source: AB8JxZp67vxI0vz3WtwCGavL0m1lSmMoBT+qGSExe41jI2+WgAucMBN7pqOyqDaMsrbiF6w34rI9D+sRyqokGvMfjto= MIME-Version: 1.0 In-Reply-To: <20180522170257.ae5387d8717e81def908a53b@linux-foundation.org> 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> <20180522170257.ae5387d8717e81def908a53b@linux-foundation.org> From: Joonsoo Kim Date: Wed, 23 May 2018 09:09:04 +0900 Message-ID: Subject: Re: [PATCH] Revert "mm/cma: manage the memory of the CMA area by using the ZONE_MOVABLE" To: Andrew Morton Cc: =?UTF-8?B?VmlsbGUgU3lyasOkbMOk?= , 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 Memory Management List , LKML Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 2018-05-23 9:02 GMT+09:00 Andrew Morton : > 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? Yes. Reverting seems to be a right decision at this moment. Could you apply original revert patch from Ville? Thanks.