Received: by 2002:ac0:a5a6:0:0:0:0:0 with SMTP id m35-v6csp582628imm; Wed, 19 Sep 2018 03:46:12 -0700 (PDT) X-Google-Smtp-Source: ANB0VdYxw7YZacjaN08cHpYtuPwiR7CJ0BIJQkFClZSAvbSEyqP0PuFL8+f32esBhKO6mflAfPTd X-Received: by 2002:a62:6781:: with SMTP id t1-v6mr35494371pfj.200.1537353972406; Wed, 19 Sep 2018 03:46:12 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1537353972; cv=none; d=google.com; s=arc-20160816; b=U2cqhifGIJjrKGtzy6Hs1dkiDMmlnDxcnf6wZV3BL7t7Ao3S5eXsCDqA3i4u3eC2o8 BBJvJcvqWtkIDuEiMuWMBdnhPr4d/FPgxYgL4FcqLYPjPTUmZ/jUW47kAEzunpS7h9xR /hgV/2b1gYvocWZlVJqpLNmvaWtQQy5UUVZiJuOyn1uri3TNJRz9pXzCtMVjyOVApEUp 0gPvH4m+YiOd0rW4XomEDmRdt3299HDg+6bHS/y3XIOSqc1WMc8b6Q0utcylubeRRRQg r/sRcE81IOf52ygpKs2qX8cfkLe6Rslf0m+U3wQEZSc2YfmJ8XxRk8/uL8X4HbNs2PVo Odvg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :organization:references:in-reply-to:message-id:subject:cc:to:from :date; bh=XkMLPfElqcx0OtcGYaE1CK2UN3/2GF2ozNng+aVtVjc=; b=iehUA3YTY89jTa7NoMyOmu3L9OU1ILRAb4INUCb6szzwTN1BZJjFLs/n9rHxPQm/F2 Raiew3jlAWOHTKC2ZS1RMctPq5KySvERzuNROxu3JqZhSKsSYPDh2ZneM4VoQofRXUKn rOXBuBFTXrUMQ/m0fNVDXeExb/2so1fLLAl6dAJWcgX7++5F7I/NaTpcK0ZSCKdclIGQ J10z8gvnnmXs3znCGB0xeowBpEHhxn4dANnlLgyq7jcnTuIfLmFh53UdSBOCoNY25k9I zMLYuGnAqLuAb1Z9++PXHEObz5Ld+sS+hI/NiupeWT3CulX1PZv0647KNT7m/iuqFiYa L4KA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id g14-v6si19180052pgj.685.2018.09.19.03.45.55; Wed, 19 Sep 2018 03:46:12 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731110AbeISQWv (ORCPT + 99 others); Wed, 19 Sep 2018 12:22:51 -0400 Received: from szxga04-in.huawei.com ([45.249.212.190]:12621 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1727820AbeISQWv (ORCPT ); Wed, 19 Sep 2018 12:22:51 -0400 Received: from DGGEMS410-HUB.china.huawei.com (unknown [172.30.72.58]) by Forcepoint Email with ESMTP id 7A7F3E7CBC61F; Wed, 19 Sep 2018 18:45:28 +0800 (CST) Received: from localhost (10.202.226.46) by DGGEMS410-HUB.china.huawei.com (10.3.19.210) with Microsoft SMTP Server id 14.3.399.0; Wed, 19 Sep 2018 18:45:20 +0800 Date: Wed, 19 Sep 2018 11:45:07 +0100 From: Jonathan Cameron To: Mike Rapoport CC: , Andrew Morton , "David S. Miller" , Greg Kroah-Hartman , Ingo Molnar , "Michael Ellerman" , Michal Hocko , Paul Burton , Thomas Gleixner , Tony Luck , , , , , , Subject: Re: [RFC PATCH 03/29] mm: remove CONFIG_HAVE_MEMBLOCK Message-ID: <20180919114507.000059f3@huawei.com> In-Reply-To: <20180919103457.GA20545@rapoport-lnx> References: <1536163184-26356-1-git-send-email-rppt@linux.vnet.ibm.com> <1536163184-26356-4-git-send-email-rppt@linux.vnet.ibm.com> <20180919100449.00006df9@huawei.com> <20180919103457.GA20545@rapoport-lnx> Organization: Huawei X-Mailer: Claws Mail 3.16.0 (GTK+ 2.24.32; i686-w64-mingw32) MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.202.226.46] X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 19 Sep 2018 13:34:57 +0300 Mike Rapoport wrote: > Hi Jonathan, > > On Wed, Sep 19, 2018 at 10:04:49AM +0100, Jonathan Cameron wrote: > > On Wed, 5 Sep 2018 18:59:18 +0300 > > Mike Rapoport wrote: > > > > > All architecures use memblock for early memory management. There is no need > > > for the CONFIG_HAVE_MEMBLOCK configuration option. > > > > > > Signed-off-by: Mike Rapoport > > > > Hi Mike, > > > > A minor editing issue in here that is stopping boot on arm64 platforms with latest > > version of the mm tree. > > Can you please try the following patch: > > > From 079bd5d24a01df3df9500d0a33d89cb9f7da4588 Mon Sep 17 00:00:00 2001 > From: Mike Rapoport > Date: Wed, 19 Sep 2018 13:29:27 +0300 > Subject: [PATCH] of/fdt: fixup #ifdefs after removal of HAVE_MEMBLOCK config > option > > The removal of HAVE_MEMBLOCK configuration option, mistakenly dropped the > wrong #endif. This patch restores that #endif and removes the part that > should have been actually removed, starting from #else and up to the > correct #endif > > Reported-by: Jonathan Cameron > Signed-off-by: Mike Rapoport Hi Mike, That's identical to the local patch I'm carrying to fix this so looks good to me. For what it's worth given you'll probably fold this into the larger patch. Tested-by: Jonathan Cameron Thanks for the quick reply. Jonathan > --- > drivers/of/fdt.c | 21 +-------------------- > 1 file changed, 1 insertion(+), 20 deletions(-) > > diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c > index 48314e9..bb532aa 100644 > --- a/drivers/of/fdt.c > +++ b/drivers/of/fdt.c > @@ -1119,6 +1119,7 @@ int __init early_init_dt_scan_chosen(unsigned long node, const char *uname, > #endif > #ifndef MAX_MEMBLOCK_ADDR > #define MAX_MEMBLOCK_ADDR ((phys_addr_t)~0) > +#endif > > void __init __weak early_init_dt_add_memory_arch(u64 base, u64 size) > { > @@ -1175,26 +1176,6 @@ int __init __weak early_init_dt_reserve_memory_arch(phys_addr_t base, > return memblock_reserve(base, size); > } > > -#else > -void __init __weak early_init_dt_add_memory_arch(u64 base, u64 size) > -{ > - WARN_ON(1); > -} > - > -int __init __weak early_init_dt_mark_hotplug_memory_arch(u64 base, u64 size) > -{ > - return -ENOSYS; > -} > - > -int __init __weak early_init_dt_reserve_memory_arch(phys_addr_t base, > - phys_addr_t size, bool nomap) > -{ > - pr_err("Reserved memory not supported, ignoring range %pa - %pa%s\n", > - &base, &size, nomap ? " (nomap)" : ""); > - return -ENOSYS; > -} > -#endif > - > static void * __init early_init_dt_alloc_memory_arch(u64 size, u64 align) > { > return memblock_alloc(size, align);