From mboxrd@z Thu Jan 1 00:00:00 1970 From: John Baldwin Subject: Re: [PATCH RFC 12/13] mca: disable cmc enable on Xen PV Date: Sat, 08 Feb 2014 17:02:47 -0500 Message-ID: <2452208.OksOsMWhSU__22925.9222899147$1392323186$gmane$org@ralph.baldwin.cx> References: <1387884062-41154-1-git-send-email-roger.pau@citrix.com> <1387884062-41154-13-git-send-email-roger.pau@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta4.messagelabs.com ([85.158.143.247]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1WE2q7-0006aL-VP for xen-devel@lists.xenproject.org; Thu, 13 Feb 2014 20:24:56 +0000 In-Reply-To: <1387884062-41154-13-git-send-email-roger.pau@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Roger Pau Monne Cc: julien.grall@citrix.com, freebsd-xen@freebsd.org, freebsd-current@freebsd.org, kib@freebsd.org, xen-devel@lists.xenproject.org, gibbs@freebsd.org List-Id: xen-devel@lists.xenproject.org On Tuesday, December 24, 2013 12:21:01 PM Roger Pau Monne wrote: > Xen PV guests doesn't have a lapic, so disable the lapic call in mca > initialization. I think this is fine, but I wonder if it wouldn't be cleaner to have lapic_enable_cmc() do the check instead. Where else do you check lapic_disabled? > --- > sys/x86/x86/mca.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/sys/x86/x86/mca.c b/sys/x86/x86/mca.c > index f1369cd..e9d2c1d 100644 > --- a/sys/x86/x86/mca.c > +++ b/sys/x86/x86/mca.c > @@ -897,7 +897,7 @@ _mca_init(int boot) > } > > #ifdef DEV_APIC > - if (PCPU_GET(cmci_mask) != 0 && boot) > + if (PCPU_GET(cmci_mask) != 0 && boot && !lapic_disabled) > lapic_enable_cmc(); > #endif > } -- John Baldwin