[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [Xen-devel] Re: [patch 0/3]Enable CMCI (Corrected Machine Check Error Interrupt) for Intel CPUs


  • To: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
  • From: "Ke, Liping" <liping.ke@xxxxxxxxx>
  • Date: Tue, 23 Dec 2008 20:32:21 +0800
  • Accept-language: en-US
  • Acceptlanguage: en-US
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Tue, 23 Dec 2008 04:33:30 -0800
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: Aclj6QtzedLMaThJSm+cYEhexWDApwARLffQACOX8KAAB3yQjQAAs9tlAAdA1fA=
  • Thread-topic: [Xen-devel] Re: [patch 0/3]Enable CMCI (Corrected Machine Check Error Interrupt) for Intel CPUs

> 
> One thing -- if you want to reduce the window between release of a
> band by its old owner and acquisition by a new owner, we could do the
> whole lot before stop_machine_run()? Maybe cmci_cpu_down(cpu) which
> would IPI 'cpu' to clear its CMCI state and then IPI all other CPUs
> to pick up the released banks. This would be neatly hooked off
> CPU_DOWN_PREPARE or similar in Linux, but Xen doesn't have cpu
> notifiers. :-) You'd have to call cmci_cpu_down() explicitly in
> cpu_down(). Or perhaps we should have cpu notifier chains in Xen
> too... 
Hi, Keir
When we wrote the patch, yunhong also mentioned similar thoughts, 
I will have some discussion with him tomorrow.
Thanks a lot!
Criping
> 
> If we do the above I don't think we need to re-introduce your rollback
> logic. If you think about it, there's no reason to prefer the old
> owner over the new owner, so no reason to roll back. I believe?
> 
>  -- Keir
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.