From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Date: Wed, 07 Nov 2018 20:38:38 +0000 Subject: Re: [PATCH 2/2] mm/sparse: add common helper to mark all memblocks present Message-Id: <20181107123838.1b7234c98a87dec5a2b25e67@linux-foundation.org> List-Id: 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> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit 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 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?