From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Kirill A. Shutemov" Subject: Re: [PATCH 0/5] Last bits for initial 5-level paging enabling Date: Fri, 23 Jun 2017 17:49:15 +0300 Message-ID: <20170623144915.4d6esghvicnczuaj@black.fi.intel.com> References: <20170622122608.80435-1-kirill.shutemov@linux.intel.com> <20170623090601.njsmucxdy4rev6zw@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mga04.intel.com ([192.55.52.120]:33091 "EHLO mga04.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751999AbdFWOtu (ORCPT ); Fri, 23 Jun 2017 10:49:50 -0400 Content-Disposition: inline In-Reply-To: <20170623090601.njsmucxdy4rev6zw@gmail.com> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Ingo Molnar Cc: Linus Torvalds , Andrew Morton , x86@kernel.org, Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , Andi Kleen , Dave Hansen , Andy Lutomirski , linux-arch@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org On Fri, Jun 23, 2017 at 11:06:01AM +0200, Ingo Molnar wrote: > > * Kirill A. Shutemov wrote: > > > As Ingo requested I've split and updated last two patches for my previous > > patchset. > > > > Please review and consider applying. > > > > Kirill A. Shutemov (5): > > x86: Enable 5-level paging support > > x86/mm: Rename tasksize_32bit/64bit to task_size_32bit/64bit > > x86/mpx: Do not allow MPX if we have mappings above 47-bit > > x86/mm: Prepare to expose larger address space to userspace > > x86/mm: Allow userspace have mapping above 47-bit > > Ok, looks pretty neat now. > > Can I apply them in this order cleanly, without breaking bisection: > > > x86/mm: Rename tasksize_32bit/64bit to task_size_32bit/64bit > > x86/mpx: Do not allow MPX if we have mappings above 47-bit > > x86/mm: Prepare to expose larger address space to userspace > > x86/mm: Allow userspace have mapping above 47-bit > > x86: Enable 5-level paging support > > ? > > I.e. I'd like to move the first patch last. > > The reason is that we should first get all quirks and assumptions fixed, all > facilities implemented - and only then enable 5-level paging as a final step which > produces a well working kernel. > > (This should also make it slightly easier to analyze any potential regressions in > earlier patches.) Just checked bisectability with this order on allmodconfig -- works fine. -- Kirill A. Shutemov