From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756152AbcBHVGR (ORCPT ); Mon, 8 Feb 2016 16:06:17 -0500 Received: from mail.skyhub.de ([78.46.96.112]:53136 "EHLO mail.skyhub.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755087AbcBHVGQ (ORCPT ); Mon, 8 Feb 2016 16:06:16 -0500 Date: Mon, 8 Feb 2016 22:06:03 +0100 From: Borislav Petkov To: Boris Ostrovsky Cc: Andrew Cooper , 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: <20160208210603.GP28980@pd.tnic> References: <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> <20160208164545.GJ28980@pd.tnic> <56B8C7E3.5060706@oracle.com> <56B8FE61.3050800@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <56B8FE61.3050800@oracle.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 03:45:21PM -0500, Boris Ostrovsky wrote: > So it looks like we can just simply revert a18a0f6850 because the very next > patch to microcode code (fbae4ba8c4a) makes the original problem (of using > __pa_nodebug, which we shouldn't use on PV) go away: we don't call > load_ucode_ap from resume path anymore. Even better. Care to cut a proper patch explaining why we don't need the paravirt_enabled() check anymore, and prep it ontop of http://git.kernel.org/cgit/linux/kernel/git/bp/bp.git/log/?h=tip-microcode ? I'll queue it for 4.6. Thanks! -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply.