From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754962AbcBHQp7 (ORCPT ); Mon, 8 Feb 2016 11:45:59 -0500 Received: from mail.skyhub.de ([78.46.96.112]:37140 "EHLO mail.skyhub.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754532AbcBHQp5 (ORCPT ); Mon, 8 Feb 2016 11:45:57 -0500 Date: Mon, 8 Feb 2016 17:45:45 +0100 From: Borislav Petkov To: Andrew Cooper Cc: Boris Ostrovsky , X86 ML , david.e.box@intel.com, Andrey Ryabinin , Lv Zheng , Jan Beulich , "H. Peter Anvin" , qiuxishi@huawei.com, cocci@systeme.lip6.fr, Xen Devel , Joerg Roedel , "Rafael J. Wysocki" , Robert Moore , Ingo Molnar , Andrey Ryabinin , Mauro Carvalho Chehab , "Luis R. Rodriguez" , Rusty Russell , Thomas Gleixner , mcb30@ipxe.org, Juergen Gross , "linux-kernel@vger.kernel.org" , Andy Lutomirski , "Luis R. Rodriguez" , long.wanglong@huawei.com, Fengguang Wu Subject: Re: [Xen-devel] [PATCH v2 3/3] paravirt: rename paravirt_enabled to paravirt_legacy Message-ID: <20160208164545.GJ28980@pd.tnic> References: <20160206220437.GA4435@pd.tnic> <56B8B6BF.6030007@oracle.com> <20160208155507.GF28980@pd.tnic> <56B8BCDB.9040701@citrix.com> <56B8BE58.6070003@oracle.com> <56B8C1BC.9040603@citrix.com> <56B8C2C8.8050406@oracle.com> <20160208163526.GI28980@pd.tnic> <56B8C490.1060004@citrix.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <56B8C490.1060004@citrix.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Feb 08, 2016 at 04:38:40PM +0000, Andrew Cooper wrote: > Does the early loader have extable support? If so, this is fairly easy > to fix. If not, we have a problem. It doesn't and regardless, you want to have this CPUID querying as simple as possible. No special handling, no special prefixes as it should be able to run on other hypervisors too. If one can't execute a simple CPUID(0x4...) on a xen guest and get the results back, then for early, we will have to do what we've done until now and simply emulate the MSR accesses. Later code can use then xen_cpuid() and all is fine. We should still get rid of paravirt_enabled() though. -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply.