From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753521AbcBIG7c (ORCPT ); Tue, 9 Feb 2016 01:59:32 -0500 Received: from mx2.suse.de ([195.135.220.15]:53255 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752034AbcBIG7b (ORCPT ); Tue, 9 Feb 2016 01:59:31 -0500 Date: Tue, 9 Feb 2016 07:59:27 +0100 From: "Luis R. Rodriguez" To: Borislav Petkov Cc: Boris Ostrovsky , Andy Lutomirski , "Luis R. Rodriguez" , cocci@systeme.lip6.fr, Juergen Gross , mcb30@ipxe.org, Thomas Gleixner , Andrey Ryabinin , Joerg Roedel , Robert Moore , Mauro Carvalho Chehab , "Rafael J. Wysocki" , Xen Devel , "H. Peter Anvin" , Rusty Russell , Jan Beulich , Lv Zheng , "linux-kernel@vger.kernel.org" , long.wanglong@huawei.com, Fengguang Wu , qiuxishi@huawei.com, Andrey Ryabinin , Konrad Rzeszutek Wilk , david.e.box@intel.com, X86 ML , Ingo Molnar Subject: Re: [PATCH v2 3/3] paravirt: rename paravirt_enabled to paravirt_legacy Message-ID: <20160209065927.GJ25240@wotan.suse.de> References: <1454733014-15237-1-git-send-email-mcgrof@kernel.org> <1454733014-15237-4-git-send-email-mcgrof@kernel.org> <20160206085930.GF25240@wotan.suse.de> <56B8B4D8.7050208@oracle.com> <20160208154608.GE28980@pd.tnic> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160208154608.GE28980@pd.tnic> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Feb 08, 2016 at 04:46:08PM +0100, Borislav Petkov wrote: > On Mon, Feb 08, 2016 at 10:31:36AM -0500, Boris Ostrovsky wrote: > > This range is reserved for hypervisors but the only hypervisor that uses it > > is Xen PV (lguest doesn't run in 64-bit mode). > > Yeah, this is mentioned in arch/x86/include/asm/page_64_types.h: > > /* > * Set __PAGE_OFFSET to the most negative possible address + > * PGDIR_SIZE*16 (pgd slot 272). The gap is to allow a space for a > * hypervisor to fit. Choosing 16 slots here is arbitrary, but it's > * what Xen requires. > */ > #define __PAGE_OFFSET _AC(0xffff880000000000, UL) > > It is also in Documentation/x86/x86_64/mm.txt: > > ffff800000000000 - ffff87ffffffffff (=43 bits) guard hole, reserved for hypervisor > > albeit a bit laconic. It doesn't sound like we have a replacement strategy for this yet? If not, for PV we can rely on the subarch, but only once its set, I can take this on as part of my work if agreeable given I'm adding the use of the Xen subarch. If we want this in before linker table stuff gets fleshed out this I can just split out setting the subarch on xen_start_kernel() and replacing paravirt_enabled() where applicable. But -- what about HVMLite for this, given as-is currently HVMLite is to use PC subarch? Luis From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Luis R. Rodriguez" Subject: Re: [PATCH v2 3/3] paravirt: rename paravirt_enabled to paravirt_legacy Date: Tue, 9 Feb 2016 07:59:27 +0100 Message-ID: <20160209065927.GJ25240@wotan.suse.de> References: <1454733014-15237-1-git-send-email-mcgrof@kernel.org> <1454733014-15237-4-git-send-email-mcgrof@kernel.org> <20160206085930.GF25240@wotan.suse.de> <56B8B4D8.7050208@oracle.com> <20160208154608.GE28980@pd.tnic> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20160208154608.GE28980@pd.tnic> Sender: linux-kernel-owner@vger.kernel.org To: Borislav Petkov Cc: Boris Ostrovsky , Andy Lutomirski , "Luis R. Rodriguez" , cocci@systeme.lip6.fr, Juergen Gross , mcb30@ipxe.org, Thomas Gleixner , Andrey Ryabinin , Joerg Roedel , Robert Moore , Mauro Carvalho Chehab , "Rafael J. Wysocki" , Xen Devel , "H. Peter Anvin" , Rusty Russell , Jan Beulich , Lv Zheng , "linux-kernel@vger.kernel.org" , long.wanglong@huawei.com, Fengguang Wu , qiuxishi@huawei.com, Andrey Ryabinin , Konrad Rzeszutek Wilk , david.e.bo List-Id: xen-devel@lists.xenproject.org On Mon, Feb 08, 2016 at 04:46:08PM +0100, Borislav Petkov wrote: > On Mon, Feb 08, 2016 at 10:31:36AM -0500, Boris Ostrovsky wrote: > > This range is reserved for hypervisors but the only hypervisor that uses it > > is Xen PV (lguest doesn't run in 64-bit mode). > > Yeah, this is mentioned in arch/x86/include/asm/page_64_types.h: > > /* > * Set __PAGE_OFFSET to the most negative possible address + > * PGDIR_SIZE*16 (pgd slot 272). The gap is to allow a space for a > * hypervisor to fit. Choosing 16 slots here is arbitrary, but it's > * what Xen requires. > */ > #define __PAGE_OFFSET _AC(0xffff880000000000, UL) > > It is also in Documentation/x86/x86_64/mm.txt: > > ffff800000000000 - ffff87ffffffffff (=43 bits) guard hole, reserved for hypervisor > > albeit a bit laconic. It doesn't sound like we have a replacement strategy for this yet? If not, for PV we can rely on the subarch, but only once its set, I can take this on as part of my work if agreeable given I'm adding the use of the Xen subarch. If we want this in before linker table stuff gets fleshed out this I can just split out setting the subarch on xen_start_kernel() and replacing paravirt_enabled() where applicable. But -- what about HVMLite for this, given as-is currently HVMLite is to use PC subarch? Luis