From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751880Ab2AYTBR (ORCPT ); Wed, 25 Jan 2012 14:01:17 -0500 Received: from mga01.intel.com ([192.55.52.88]:37213 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750842Ab2AYTBQ (ORCPT ); Wed, 25 Jan 2012 14:01:16 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.71,315,1320652800"; d="scan'208";a="110709550" From: "Yu, Fenghua" To: "H. Peter Anvin" , Brian Gerst CC: Ingo Molnar , Thomas Gleixner , Linus Torvalds , Andrew Morton , "Mallick, Asit K" , "Luck, Tony" , "Siddha, Suresh B" , "Brown, Len" , Randy Dunlap , "Srivatsa S. Bhat" , Konrad Rzeszutek Wilk , Peter Zijlstra , Chen Gong , linux-kernel , "Van De Ven, Arjan" , linux-pm , x86 Subject: RE: [PATCH v5 10/12] x86/mtrr/main.c: Ask the first online CPU to save mtrr Thread-Topic: [PATCH v5 10/12] x86/mtrr/main.c: Ask the first online CPU to save mtrr Thread-Index: AQHM0IXtchbg6t0dpEylW7Itt3XhjJYJMbWAgAV5GACADsrlsA== Date: Wed, 25 Jan 2012 19:01:02 +0000 Message-ID: <3E5A0FA7E9CA944F9D5414FEC6C712200349E1@ORSMSX105.amr.corp.intel.com> References: <1326301493-28760-1-git-send-email-fenghua.yu@intel.com> <1326301493-28760-11-git-send-email-fenghua.yu@intel.com> <4F136A50.80203@zytor.com> In-Reply-To: <4F136A50.80203@zytor.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.22.254.140] Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by nfs id q0PJ1QGZ005723 On 01/12/2012 04:33 AM, Brian Gerst wrote: > On Wed, Jan 11, 2012 at 12:04 PM, Fenghua Yu wrote: >> From: Fenghua Yu >> >> Ask the first online CPU to save mtrr instead of asking BSP. BSP could be >> offline when mtrr_save_state() is called. > > If you can use any non-boot cpu to save the MTRRs why not just use the > current cpu? They should all be in sync anyways. The current cpu can not be used to save the MTRRs because its MTRRs are not initialized yet at this point. Later on, set_mtrr() will be called to initialize MTRRs on the current CPU (and all booted CPUs). Thanks. -Fenghua {.n++%ݶw{.n+{G{ayʇڙ,jfhz_(階ݢj"mG?&~iOzv^m ?I