Received: by 2002:ac0:a5a6:0:0:0:0:0 with SMTP id m35-v6csp1397432imm; Wed, 26 Sep 2018 17:35:04 -0700 (PDT) X-Google-Smtp-Source: ACcGV60HRUKzwWTP8qmUwBnjk1UskS1DuvYVsO/hRM0hIPpQnb1Xy+YriJ2Rc2tEjqv/6LJtzk72 X-Received: by 2002:a62:90d2:: with SMTP id q79-v6mr8282737pfk.37.1538008504754; Wed, 26 Sep 2018 17:35:04 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1538008504; cv=none; d=google.com; s=arc-20160816; b=T8YeXAtOOt+wqHvbaOOPwAY4KBMEUAXySA79+tsfzDlHMkucuL/21qMphDGusFJcdp CEZ+mqZpAa6v3sZjiySvjlvAmrfQ/Rh68Djnl3ijx7kIQgpyynQ5VQs6lvaFFPuCe79P Kt8fbjEmvkVOrlBCrtoB0j5OO1v2K5VJ1sFUu+VP9BxcUqw5Ce6qIK/YFqKaMfHAWjvh xDlj08lZPatU7uwBrLgPouEp+JeK71oTp/PkP93emCsa6BTKxml+ilDWOR3YR6fmv7ai C0zfAgw6NZQxZsdaPH/tUnqpzdDu6EYfF6c6vRCl3C8kR1yOTaQaLGu1+Ce7Mn7LY7ce WAYA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=aCL+Y3Elp9js0tskQ42GEDjYhI0l5SAZZjTRQ2rGzW4=; b=MZm9mhSjIaT4TNjNc6MvjQ4XBd64QyANWL+RqboWJ9KNIOIF1S6SZ662rlKOEarfLy LKdfBMxBn+c1KZXLwe2Km2GKV5RTFR9hQyOekXJlNnis0PBWlAkT6CTqKYvpqhDIh0x9 JCMn2OOqlxuQhurwzAa5fWA6pQZLYtQzaLszYPIY+TM57WkBRO15QY2XDVLkdP3lwQn7 sr8mQJJefpOcwgkCDUH52xf8NbhITTg+d2be4zXjPDtePi4o7W5qhBLjNCQ0/8Y59+nm Cm+c1ixBNvOqWrLiB2RUWYqfL6GKpbvWReIY74mPpgpANreevaIRrU/miMf5W9sEfVlw AeHQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=gWktOBfb; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id s1-v6si452312pgj.405.2018.09.26.17.34.49; Wed, 26 Sep 2018 17:35:04 -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; dkim=pass header.i=@gmail.com header.s=20161025 header.b=gWktOBfb; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726705AbeI0GuR (ORCPT + 99 others); Thu, 27 Sep 2018 02:50:17 -0400 Received: from mail-it1-f193.google.com ([209.85.166.193]:52120 "EHLO mail-it1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726298AbeI0GuR (ORCPT ); Thu, 27 Sep 2018 02:50:17 -0400 Received: by mail-it1-f193.google.com with SMTP id 74-v6so3902378itw.1; Wed, 26 Sep 2018 17:34:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=aCL+Y3Elp9js0tskQ42GEDjYhI0l5SAZZjTRQ2rGzW4=; b=gWktOBfbdrBncdviL20J5F04FWoaz6YrOuZLdcCDBv1a1+de7firBMdo56D5siKbuP knUUB/mvtfUFAgKRLK6W3vSmF/mpPs+gnm6yp5S+r9Ha9FPHQr2nPuRnqXV2poXWKJJ3 HIOfbb4fDeKHAeNv1BJcfuUtpb1jGZYjKdkWtTH9UaLxMDvX8nuhR5WKVgZkkOz9BpEd CI7OaIPDPdJciw65l552y88iAqeLkftXzSYBxY3FdNga0rrtQBxT8YIewUygbcNpeekX N1FJ36m7Pb9UOF1rdrmYpWB/Tlp8yZqialQsEs2ivoPFIrl6lxYgw/zD3yJUoC9cbydD y/kA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=aCL+Y3Elp9js0tskQ42GEDjYhI0l5SAZZjTRQ2rGzW4=; b=bOk2SQIDQa9qg2j0oTCXzJcVx08eoORVxGGHKVN1Cabg9OpQ53YulLZB5TUlws2oPh H0ZGCDfALRwXzvGKulmeW1WYZJypZ6RcuVYGkr/MmdyHnd5QD0QRSNOEKLBWzeRW0Gof XaIOuktmh+J1PhpkWpFij4/66BNSY4QG2aZ2BvS4e4v7gF38Fs/4vLdzf3yXiSDwKNoO j5X1s4wILLQo1bUuxjLn0L0kVujX4GKATy6GaMrrWp5dBvzQ0ymHKQBdz7OJYIiBRpVW nFK+hUdHfodStPW9rTCQBC5aaFsUxQeQmNIfjU3MNCtbjpWQqGlTQBImXg3WzM+E/4jO +KGw== X-Gm-Message-State: ABuFfojG1t/W3MrfOM7rpFN6XUWovS/NsciycGaWSP4iVSV9rr1r6Inj qt+t/ZpYtrStzf4H/91sDCq/6cb6M2rckgEEh68= X-Received: by 2002:a02:7a12:: with SMTP id a18-v6mr7812760jac.42.1538008483880; Wed, 26 Sep 2018 17:34:43 -0700 (PDT) MIME-Version: 1.0 References: <1536927045-23536-1-git-send-email-rppt@linux.vnet.ibm.com> <1536927045-23536-4-git-send-email-rppt@linux.vnet.ibm.com> <20180926183152.GA4597@rapoport-lnx> In-Reply-To: <20180926183152.GA4597@rapoport-lnx> From: Alexander Duyck Date: Wed, 26 Sep 2018 17:34:32 -0700 Message-ID: Subject: Re: [PATCH 03/30] mm: remove CONFIG_HAVE_MEMBLOCK To: rppt@linux.vnet.ibm.com Cc: linux-mm , Andrew Morton , Catalin Marinas , chris@zankel.net, David Miller , Geert Uytterhoeven , green.hu@gmail.com, Greg KH , gxt@pku.edu.cn, Ingo Molnar , jejb@parisc-linux.org, jonas@southpole.se, Jonathan Corbet , lftan@altera.com, msalter@redhat.com, Martin Schwidefsky , mattst88@gmail.com, mpe@ellerman.id.au, Michal Hocko , monstr@monstr.eu, palmer@sifive.com, paul.burton@mips.com, rkuo@codeaurora.org, richard@nod.at, dalias@libc.org, Russell King - ARM Linux , fancer.lancer@gmail.com, Thomas Gleixner , Tony Luck , vgupta@synopsys.com, Yoshinori Sato , linux-alpha@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-c6x-dev@linux-c6x.org, linux-hexagon@vger.kernel.org, linux-ia64@vger.kernel.org, LKML , linux-m68k@lists.linux-m68k.org, linux-mips@linux-mips.org, linux-parisc@vger.kernel.org, "open list:LINUX FOR POWERPC (32-BIT AND 64-BIT)" , linux-riscv@lists.infradead.org, linux-s390@vger.kernel.org, linux-sh@vger.kernel.org, linux-snps-arc@lists.infradead.org, linux-um@lists.infradead.org, nios2-dev@lists.rocketboards.org, openrisc@lists.librecores.org, sparclinux@vger.kernel.org, uclinux-h8-devel@lists.sourceforge.jp Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Sep 26, 2018 at 11:32 AM Mike Rapoport wrote: > > On Wed, Sep 26, 2018 at 09:58:41AM -0700, Alexander Duyck wrote: > > On Fri, Sep 14, 2018 at 5:11 AM 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 > > > > > > > > > diff --git a/include/linux/memblock.h b/include/linux/memblock.h > > > index 5169205..4ae91fc 100644 > > > --- a/include/linux/memblock.h > > > +++ b/include/linux/memblock.h > > > @@ -2,7 +2,6 @@ > > > #define _LINUX_MEMBLOCK_H > > > #ifdef __KERNEL__ > > > > > > -#ifdef CONFIG_HAVE_MEMBLOCK > > > /* > > > * Logical memory blocks. > > > * > > > @@ -460,7 +459,6 @@ static inline phys_addr_t memblock_alloc(phys_addr_t size, phys_addr_t align) > > > { > > > return 0; > > > } > > > -#endif /* CONFIG_HAVE_MEMBLOCK */ > > > > > > #endif /* __KERNEL__ */ > > > > There was an #else above this section and I believe it and the code > > after it needs to be stripped as well. > > Right, I've already sent the fix [1] and it's in mmots. > > [1] https://lkml.org/lkml/2018/9/19/416 > Are you sure? The patch you reference appears to be for drivers/of/fdt.c, and the bit I pointed out here is in include/linux/memblock.h. - Alex