From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keir Fraser Subject: Re: [PATCH 5/5] VT-d: adjust IOMMU interrupt affinities when all CPUs are online Date: Wed, 21 Nov 2012 11:10:36 +0000 Message-ID: References: <50ACB8CE02000078000AA4B0@nat28.tlf.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <50ACB8CE02000078000AA4B0@nat28.tlf.novell.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: Jan Beulich , xen-devel Cc: Wei Huang , Wei Wang , xiantao.zhang@intel.com, Gang Wei List-Id: xen-devel@lists.xenproject.org On 21/11/2012 10:19, "Jan Beulich" wrote: > Since these interrupts get setup before APs get brought online, their > affinities naturally could only ever point to CPU 0 alone so far. > Adjust this to include potentially multiple CPUs in the target mask > (when running in one of the cluster modes), and take into account NUMA > information (to handle the interrupts on a CPU on the node where the > respective IOMMU is). > > Signed-off-by: Jan Beulich > > --- a/xen/drivers/passthrough/vtd/dmar.c > +++ b/xen/drivers/passthrough/vtd/dmar.c > @@ -839,6 +839,7 @@ void acpi_dmar_reinstate(void) > > void acpi_dmar_zap(void) > { > + adjust_vtd_irq_affinities(); Is this just a handy place to hook? Does it logically make sense? -- Keir