From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Kirill A. Shutemov" Subject: Re: [RFC, PATCHv1 00/28] 5-level paging Date: Thu, 8 Dec 2016 22:20:45 +0300 Message-ID: <20161208192045.GA30380@node.shutemov.name> References: <20161208162150.148763-1-kirill.shutemov@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org To: Linus Torvalds Cc: "Kirill A. Shutemov" , Andrew Morton , the arch/x86 maintainers , Thomas Gleixner , Ingo Molnar , Arnd Bergmann , "H. Peter Anvin" , Andi Kleen , Dave Hansen , Andy Lutomirski , "linux-arch@vger.kernel.org" , linux-mm , Linux Kernel Mailing List List-Id: linux-arch.vger.kernel.org On Thu, Dec 08, 2016 at 10:16:07AM -0800, Linus Torvalds wrote: > On Thu, Dec 8, 2016 at 8:21 AM, Kirill A. Shutemov > wrote: > > > > This patchset is still very early. There are a number of things missing > > that we have to do before asking anyone to merge it (listed below). > > It would be great if folks can start testing applications now (in QEMU) to > > look for breakage. > > Any early comments on the design or the patches would be appreciated as > > well. > > Looks ok to me. Starting off with a compile-time config option seems fine. > > I do think that the x86 cpuid part should (patch 15) should be the > first patch, so that we see "la57" as a capability in /proc/cpuinfo > whether it's being enabled or not? We should merge that part > regardless of any mm patches, I think. Okay, I'll split up the CPUID part into separate patch and move it beginning for the patchset REQUIRED_MASK portion will stay where it is. -- Kirill A. Shutemov -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wj0-f196.google.com ([209.85.210.196]:35176 "EHLO mail-wj0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752906AbcLHTUt (ORCPT ); Thu, 8 Dec 2016 14:20:49 -0500 Received: by mail-wj0-f196.google.com with SMTP id he10so41653975wjc.2 for ; Thu, 08 Dec 2016 11:20:49 -0800 (PST) Date: Thu, 8 Dec 2016 22:20:45 +0300 From: "Kirill A. Shutemov" Subject: Re: [RFC, PATCHv1 00/28] 5-level paging Message-ID: <20161208192045.GA30380@node.shutemov.name> References: <20161208162150.148763-1-kirill.shutemov@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-arch-owner@vger.kernel.org List-ID: To: Linus Torvalds Cc: "Kirill A. Shutemov" , Andrew Morton , the arch/x86 maintainers , Thomas Gleixner , Ingo Molnar , Arnd Bergmann , "H. Peter Anvin" , Andi Kleen , Dave Hansen , Andy Lutomirski , "linux-arch@vger.kernel.org" , linux-mm , Linux Kernel Mailing List Message-ID: <20161208192045.fV9V75YuvFSC7PVsp6lMCzJnHTmFOXusj5b53Y1DnDQ@z> On Thu, Dec 08, 2016 at 10:16:07AM -0800, Linus Torvalds wrote: > On Thu, Dec 8, 2016 at 8:21 AM, Kirill A. Shutemov > wrote: > > > > This patchset is still very early. There are a number of things missing > > that we have to do before asking anyone to merge it (listed below). > > It would be great if folks can start testing applications now (in QEMU) to > > look for breakage. > > Any early comments on the design or the patches would be appreciated as > > well. > > Looks ok to me. Starting off with a compile-time config option seems fine. > > I do think that the x86 cpuid part should (patch 15) should be the > first patch, so that we see "la57" as a capability in /proc/cpuinfo > whether it's being enabled or not? We should merge that part > regardless of any mm patches, I think. Okay, I'll split up the CPUID part into separate patch and move it beginning for the patchset REQUIRED_MASK portion will stay where it is. -- Kirill A. Shutemov