From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH 2/2] mm/sparse: add common helper to mark all memblocks present Date: Wed, 7 Nov 2018 12:38:38 -0800 Message-ID: <20181107123838.1b7234c98a87dec5a2b25e67@linux-foundation.org> References: <20181107173859.24096-1-logang@deltatee.com> <20181107173859.24096-3-logang@deltatee.com> <20181107121207.62cb37cf58484b7cc80a8fd8@linux-foundation.org> <724be9bb-59b6-33f3-7b59-3ca644d59bf7@deltatee.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Logan Gunthorpe Cc: Thomas Gleixner , linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-arch@vger.kernel.org, linux-riscv@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-sh@vger.kernel.org, Stephen Bates , Palmer Dabbelt , Albert Ou , Christoph Hellwig , Arnd Bergmann , Michal Hocko , Vlastimil Babka , Oscar Salvador List-Id: linux-arch.vger.kernel.org On Wed, 7 Nov 2018 13:36:34 -0700 Logan Gunthorpe wrote: > > Actually if both names suck, then there also is the option to rename both > > instead of adding a comment to explain the suckage. > > Ok, well, I wasn't expecting to take on a big rename like that as it > would create a patch touching a bunch of arches and mm files... But if > we can come to some agreement on a better name and someone is willing to > take that patch without significant delay then I'd be happy to create > the patch and add it to the start of my series. Some other time ;) Let's just get the missing documentation added for now? From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.linuxfoundation.org ([140.211.169.12]:36244 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726480AbeKHGKm (ORCPT ); Thu, 8 Nov 2018 01:10:42 -0500 Date: Wed, 7 Nov 2018 12:38:38 -0800 From: Andrew Morton Subject: Re: [PATCH 2/2] mm/sparse: add common helper to mark all memblocks present Message-ID: <20181107123838.1b7234c98a87dec5a2b25e67@linux-foundation.org> In-Reply-To: References: <20181107173859.24096-1-logang@deltatee.com> <20181107173859.24096-3-logang@deltatee.com> <20181107121207.62cb37cf58484b7cc80a8fd8@linux-foundation.org> <724be9bb-59b6-33f3-7b59-3ca644d59bf7@deltatee.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-arch-owner@vger.kernel.org List-ID: To: Logan Gunthorpe Cc: Thomas Gleixner , linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-arch@vger.kernel.org, linux-riscv@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-sh@vger.kernel.org, Stephen Bates , Palmer Dabbelt , Albert Ou , Christoph Hellwig , Arnd Bergmann , Michal Hocko , Vlastimil Babka , Oscar Salvador Message-ID: <20181107203838.ne6toMvj0yC5HsznkEPB3AcT6PgQPDNZgaTN2psMAak@z> On Wed, 7 Nov 2018 13:36:34 -0700 Logan Gunthorpe wrote: > > Actually if both names suck, then there also is the option to rename both > > instead of adding a comment to explain the suckage. > > Ok, well, I wasn't expecting to take on a big rename like that as it > would create a patch touching a bunch of arches and mm files... But if > we can come to some agreement on a better name and someone is willing to > take that patch without significant delay then I'd be happy to create > the patch and add it to the start of my series. Some other time ;) Let's just get the missing documentation added for now?