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

[Xen-devel] [PATCH v12 0/7] VMX: Properly handle pi descriptor and per-cpu blocking list



The current VT-d PI related code may operate incorrectly in the 
following scenarios: 
1. When VT-d PI is enabled, needn't migrate pirq which is using VT-d PI during
vCPU migration. Patch [1/7] solves this by introducing a new flag to indicate
that the pt-irq is delivered through VT-d PI.

2. Patch [2/7] is to make io.h self-contained. It avoids compile errors 
after including hvm/vmx/vmcs.h in asm-x86/msi.h and asm-x86/iommu.h. It is a
preparation for Patch [3/7].

2. msi_msg_to_remap_entry() is buggy when the live IRTE is in posted format.
It wrongly inherits the 'im' field from the live IRTE but updates all the
other fileds to remapping format. Patch [3/7] handles this.

3. [4/7] is a cleanup patch 

4. When a pCPU is unplugged, and there might be vCPUs on its 
list. Since the pCPU is offline, those vCPUs might not be woken 
up again. [5/7] addresses it. 

5. IRTE is updated through structure assigment which is unsafe in some cases.
To resolve this, Patch [6/7] provides two variants, atomic and non-atomic, to
update IRTE. And a bug is raised when we can't meet the caller's atomic
requirement.

6. We didn't change the IRTE to remapping format when pt-irq is configurated
to have multi-destination vCPUs. Patch [7/7] resolves this problem.

Chao Gao (5):
  passthrough: don't migrate pirq when it is delivered through VT-d PI
  x86/hvm: make io.h self-contained
  VT-d: Introduce new fields in msi_desc to track binding with guest
    interrupt
  VT-d: introduce update_irte to update irte safely
  passthrough/io: Fall back to remapping interrupt when we can't use
    VT-d PI

Feng Wu (2):
  VT-d: Some cleanups
  VMX: Fixup PI descriptor when cpu is offline

 xen/arch/x86/hvm/hvm.c                 |   3 +
 xen/arch/x86/hvm/vmx/vmcs.c            |   1 +
 xen/arch/x86/hvm/vmx/vmx.c             |  70 +++++++++++
 xen/arch/x86/msi.c                     |   2 +
 xen/drivers/passthrough/io.c           |  73 +++--------
 xen/drivers/passthrough/vtd/intremap.c | 220 ++++++++++++++-------------------
 xen/include/asm-x86/hvm/io.h           |   1 +
 xen/include/asm-x86/hvm/vmx/vmx.h      |   1 +
 xen/include/asm-x86/iommu.h            |   4 +-
 xen/include/asm-x86/msi.h              |   5 +
 xen/include/xen/hvm/irq.h              |   1 +
 11 files changed, 195 insertions(+), 186 deletions(-)

-- 
1.8.3.1


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

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