Re: [konrad.wilk@xxxxxxxxxx: [Xen-devel] [PATCH 1 of 1] Intel VT-D: Don't turn x2APIC if there is a missing DRHD entry for the IOAPIC]

[Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>]

On Wed, Mar 10, 2010 at 04:42:19PM +0800, Zhang, Xiantao wrote:
> Jan Beulich wrote:
> >>>> Konrad Rzeszutek Wilk  03/09/10 6:34 PM >>>
> >> 
> >> What are you guys feeling about this patch?
> > 
> > My main question is whether just disabling x2apic in this case is
> > sufficient (i.e. whether instead we shouldn't disable iommu, or at
> > least interrupt remapping, altogether).  
> 
> iommu_enable is called after this point, and it also has sanity-check to 
> determine whether enable or disable iommu and its related features. This 
> early check is only used to determine whether to enable x2apic because it 
> must  be enabled at very early time if supported.

Exactly. So the x2APIC is turned off here and then later on the
Interrupt Remaping is turned off as well.

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