From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keir Fraser Subject: Re: Re: [patch 0/3]Enable CMCI (Corrected Machine Check Error Interrupt) for Intel CPUs Date: Wed, 24 Dec 2008 08:01:39 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: "Jiang, Yunhong" , "Ke, Liping" Cc: "xen-devel@lists.xensource.com" List-Id: xen-devel@lists.xenproject.org On 24/12/2008 02:57, "Jiang, Yunhong" wrote: > As for your proposal of do it before the stop_machine_run(), it may reduce the > window, but still can't eliminate the window of lost CMCI interrupt, unless we > do similar thing in the cmci_cpu_down() (i.e. all CPU is irq_disabled before > update the CMCI status). It is the same if we pull the notifier chain to Xen. > How is your idea? You only need hardirqs disabled, so rendezvous within an on_each_cpu() callback is sufficient for you. You don't need to get intimate with stop_machine_run()! -- Keir